There is this problem:

Notice: Undefined index: path in panopoly_admin_menu_link_alter() (line472 of/usr/local/var/www/panopoly/profiles/panopoly/modules/panopoly/panopoly_admin/panopoly_admin.module)

Comments

populist’s picture

populist commented
Status: Active » Fixed

This was fixed with a quick !empty() check in panopoly_admin. Not sure why $item['path'] ends up being false, but so it does:

-  if (strpos($item['path'], 'panopoly/settings/')) {
+  if (!empty($item['path']) && strpos($item['path'], 'panopoly/settings/')) {
populist’s picture

populist commented

After some more digging, $item['path'] is empty because defaultcontent.module injects content that way. Since only Panopoly Demo has this trouble, I updated that module with a fix as well so we should be good to go.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

fox mulder’s picture

fox mulder commented
Status: Closed (fixed) » Active

Hi populist!

I don't understand Your solution. Since we are using Panopoly rc3 this notice appears after submitting all of the node edit forms of nodes that create menu entry. Panopoly Demo is uninstalled.

I don't know what does the function panopoly_admin_menu_link_alter(&$item) make, but all of $item arrays have a 'link_path' key instead of 'path'.

Can You help me fix this?

populist’s picture

populist commented
Status: Active » Fixed

My answer in #2 was a little hasty. As it turns out, link_path is often set and the code change in #2 should solve your troubles. It just isn't in RC3, but will be in RC4!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Anonymous (not verified) commented

Fox Muldaur - In line472 of what ever your path is www/panopoly/profiles/panopoly/modules/panopoly/panopoly_admin/panopoly_admin.module)

change line 472 from - if (strpos($item['path'], 'panopoly/settings/')) {

to

if (!empty($item['path']) && strpos($item['path'], 'panopoly/settings/')) {

It worked for me.