Warning: Invalid argument supplied for foreach() in _rootcandy_besturlmatch() (line 268 of
/sites/all/themes/rootcandy/template.php).
Warning: Invalid argument supplied for foreach() in rootcandy_links__rc_main_navigation() (line 142 of /sites/all/themes/rootcandy/template.php).

CommentFileSizeAuthor
#2 root_candy-empty-navigation-menu-causes-issues-1541640-2.patch766 bytesb-prod

Comments

sign’s picture

sign commented
Status: Active » Postponed (maintainer needs more info)

Can't reproduce,
can you give me some more info?
What links if any, etc...?

b-prod’s picture

b-prod commented
Title: Warning Invalid argument RC-D7-DEV » Empty navigation menu causes issues in the theme function
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new root_candy-empty-navigation-menu-causes-issues-1541640-2.patch766 bytes

In the function rootcandy_preprocess_page(), the $rootcandy_navigation variable may be an empty string, according to the stored settings.
If so, this empty string is passed to the theme links__rc_main_navigation and causes issue in the rootcandy_links__rc_main_navigation() function, that does not take care of such case and always expects an array of links.

jhr’s picture

jhr commented

I was getting the same warnings. Used #2 patched, no more warnings.

b-prod’s picture

b-prod commented
Status: Needs review » Reviewed & tested by the community

Set issue as RTBC, according to jhr.