Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Notice: Undefined offset in /var/www/magento/src/vendor/smile-sa/elasticsuite/src/module-elasticsuite-catalog/Model/Layer/Filter/Category.php on line 143 #45

Closed
romainruaud opened this issue Jun 3, 2016 · 1 comment
Closed

Notice: Undefined offset in /var/www/magento/src/vendor/smile-sa/elasticsuite/src/module-elasticsuite-catalog/Model/Layer/Filter/Category.php on line 143 #45

romainruaud opened this issue Jun 3, 2016 · 1 comment
Assignees
@romainruaud
Labels
bug
Milestone
2.1

Comments

@romainruaud
Copy link
Collaborator 

foreach ($categories as $category) {
    $productCount = $optionsFacetedData[$category->getId()]['count'];
    if ($category->getIsActive() && isset($optionsFacetedData[(int) $category->getId()]) && $productCount > 0) {

There should be a test on $optionsFacetedData[$category->getId()]['count']
@romainruaud romainruaud added the bug label Jun 3, 2016
@romainruaud romainruaud added this to the v 2.1 milestone Jun 3, 2016
@romainruaud romainruaud self-assigned this Jun 3, 2016
@romainruaud romainruaud mentioned this issue Jun 6, 2016
Merged
@afoucret
Copy link
Contributor

afoucret commented Jun 6, 2016

Fixed by PR #46

@afoucret afoucret closed this as completed Jun 6, 2016
@phronesis phronesis mentioned this issue Aug 15, 2016
Closed
@southerncomputer southerncomputer mentioned this issue Oct 17, 2016
Closed
@globallydigital globallydigital mentioned this issue Nov 26, 2016
Closed
@woutersamaey woutersamaey mentioned this issue Jun 9, 2017
Closed
botisSmile pushed a commit to botisSmile/elasticsuite that referenced this issue Sep 8, 2021
@rbayet
Merge pull request Smile-SA#45 from Elastic-Suite/fix-graphql-tests-d…
11c59e2 
…ue-to-gq-update

Fix graphql integration tests (missing header)
@ghost ghost mentioned this issue Jul 31, 2022
Closed
@menardavid menardavid mentioned this issue Nov 10, 2022
Closed
@Minirock Minirock mentioned this issue Jul 26, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@romainruaud romainruaud

Labels
bug
Projects
None yet
Milestone
2.1
Development

No branches or pull requests
2 participants
@afoucret @romainruaud