- 🇦🇺Australia klonos 90% Melbourne, Australia - 10% Larissa, Greece
I have tested this on D7.94 (and Views 7.x-3.28 FWIW), where previously I would get these 4 warnings (after for example clearing caches):
Warning: strnatcasecmp() expects parameter 1 to be string, array given in admin_menu_element_sort() (line 733 of /app/sites/all/modules/admin_menu/admin_menu.module). Warning: strnatcasecmp() expects parameter 2 to be string, array given in admin_menu_element_sort() (line 733 of /app/sites/all/modules/admin_menu/admin_menu.module). Warning: strnatcasecmp() expects parameter 1 to be string, array given in admin_menu_element_sort() (line 733 of /app/sites/all/modules/admin_menu/admin_menu.module). Notice: Array to string conversion in check_plain() (line 1908 of /app/includes/bootstrap.inc).
After applying the patch from the MR via via https://git.drupalcode.org/project/helper/-/merge_requests/1.patch all these warnings/notices go away.
The code does exactly what the maintainer of the module suggested in #5, so RTBC 👍
- 🇩🇪Germany Anybody Porta Westfalica
Any chance to get this committed and released?
The issue still exists... just ran into this again. RTBC since > 1y
- 🇩🇪Germany Anybody Porta Westfalica
@Dave Reid still running into this 5 years after the report. Any chance to merge this and tag a new release? ;D