New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #11840 #11902

Merged
merged 1 commit into from Dec 21, 2018

Conversation

Projects
None yet
3 participants
@DelecroixQuentin
Copy link
Contributor

DelecroixQuentin commented Dec 21, 2018

Questions Answers
Branch? develop
Description? Fix [PHP Warning #2] count(): Parameter must be an array or an object that implements Countable which can happen sometimes with debug mode ON
Type? bug fix
Category? WS
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #11840
How to test? NA

This change is Reviewable

@prestonBot

This comment has been minimized.

Copy link
Collaborator

prestonBot commented Dec 21, 2018

Hello @DelecroixQuentin!

This is your first pull request on the PrestaShop project. Thank you, and welcome to this Open Source community!

@mickaelandrieu

This comment has been minimized.

Copy link
Contributor

mickaelandrieu commented Dec 21, 2018

Thanks @DelecroixQuentin!

@mickaelandrieu mickaelandrieu merged commit 706ea9a into PrestaShop:develop Dec 21, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@mickaelandrieu mickaelandrieu added this to the 1.7.6.0 milestone Dec 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment