- 🇬🇧United Kingdom aaron.ferris
Can confirm this bug, does feel a little odd why theme_links would be called with no links, but I don't see any issue with a check like the patch.
Warning: count(): Parameter must be an array or an object that implements Countable in menu_css_names_links() (line 54 of /var/www/html/sites/all/modules/contrib/menu_css_names 2/menu_css_names.module).
TODO
1. Create MR - Status changed to Needs work
9 months ago 1:59pm 1 June 2024 - First commit to issue fork.
- Merge request !5Checks if it is an array before using the count method → (Merged) created by charlliequadros
- Status changed to Needs review
9 months ago 2:52pm 1 June 2024 - 🇬🇧United Kingdom aaron.ferris
Thanks for this, can confirm im not seeing any notice related to these links on index with this patch/MR.
1. Drupal 7
2. Search api solr 7.x-1.16
3. Search API 7.x-1.29
4. Entity: 7.x-1.10
5. Using the following for solr locally: https://github.com/ddev/ddev-contrib/tree/master/docker-compose-services...
6. Search api solr configset from https://git.drupalcode.org/project/search_api_solr/-/tree/4.x/jump-start...
7. Couple of content types
8. Create some nodes
9. Index= No notices, also stepped through via Xdebug, no obvious issues I can see.
Committing to 7.x.
- Status changed to Fixed
9 months ago 4:37pm 1 June 2024 - Status changed to Fixed
9 months ago 6:20pm 2 June 2024