Closed (fixed)
Project:
Node Hierarchy
Version:
7.x-2.x-dev
Component:
Drupal/PHP Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Apr 2011 at 21:21 UTC
Updated:
5 Jan 2012 at 18:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
CUclimber commentedI'm getting exactly the same error for exactly the same reason. I installed Node Hierarchy today and I can no longer save changes to my menus without getting this error.
Comment #2
tonycpsu commentedMy temporary workaround has been to comment out line 350 of nodehierarchy.module:
# $form['#validate'][] = menu_overview_form_validate';
Then I can edit my menu. No bad side effects so far.
Comment #3
jorge.suarez commentedI'm affected, too. I did your workaround, but now I can't edit my menus properly. I even uninstalled the module via the uninstall option, and no dice.
Since then, the links list seems to be messed up. By example, the user menu. The «User account» link appears as disabled on the links list, but it's enabled and if I edit it, I can setup it properly from the edit form. So the only thing that seems to be broken is the links list. Saving the menu item, reset it to default options don't seem to help.
This could be NOT Node Hierarchy fault, however. Any ideas on how can I debug this? Can you tonycpsu and CUclimber test this behaviour? I attached a screenshot for better understanding.
Thank you.
Comment #4
CUclimber commentedThe fix suggested by tonycpsu in Post #2 is working for me, although I'm always just a little uncomfortable with patches like this in production. It does allow me to save changes to my menus with no errors and no apparent side effects.
Comment #5
jbylsma commentedJust a word of caution: NodeHierarchy isn't ready for production on 7.x. ronan closed the port issue (#828244-68: Drupal 7 - Port of Node Hierarchy) asking for people to test in non-production environments. I know that there was a very long standing, pretty much stable 6.x dev branch that I was personally using in production envs, but we are striving to leave that mindset behind.
Not trying to be negative, just trying to help avoid any unnecessary headaches!
Comment #6
Cybnext commentedHere is a patch that should fix the re-ordering problem. It also fixes the ability to drag and drop to reorder. Core comes with a table drag JS already in place.
Comment #7
kirby14 commentedThis patch also works in my basic testing. D7 core doesn't have menu_overview_form_validate anymore. Does anyone know why it was being called and is there something that needs to replace it in D7?
Comment #8
murzCybnext, with your patch this bug is disappear! Thanks!
Comment #9
jsj9832 commentedCybernxt and Murz. Can you please state where should I place the patch. If inside the nodehierachy.module or elsewhere. Thanks.
Comment #10
jsj9832 commentedCybernxt can you please state where should I place the patch. If inside the nodehierachy.module or elsewhere. Thanks.
Comment #11
Cybnext commentedThe changes go in the nodehierarchy.module file.
Comment #12
acbramley commentedPatch worked for me too.
Comment #13
murzComment #14
carl.brown commentedI was having a very similar problem here - trying to reorder menu link items in the draggable table and couldn't save the changes. Applied patch from #6 and am now able to save the changes. Thanks @Cybnext - that's the second time one of your patches has fixed my problems in the past 10 minutes!
Comment #15
jvandooren commentedI can also confirm the patch provided in #6 fixes the issue.
Comment #16
eric.toupin commentedFixed reordering menu links fatal error w/ patch listed in #6. Thanks!
Comment #17
pbfleetwood commentedAny chance that the patch will be committed to the distribution? In the four months since it was provided, several people have confirmed that it works and none have complained of ill effects.
I'm changing the status of this thread to 'patch (to be ported)'; I apologize if that is not correct--I'm kind of a newb.
Comment #18
pbfleetwood commentedAfter reading through the documentation on applying a patch, I realized that I should not have changed the status as I did in #17, so I'm changing it back. Sorry about that. :/
Comment #19
organicHack commentedLooking at this patch too - does what I need it to (meaning fixes the crash). Also voting to get it committed. Thanks!
Comment #20
organicHack commentedYup. This is a good patch. Fixes a number of bugs. Definitely need to get it in there asap!
Comment #21
jnettikAny menu item that node hierarchy created was disabled and I was unable to update it. With the help of patch #6 everything is good again.
+1 to committing this patch.
Comment #22
ronan commentedI've applied the part of the patch that fixes the fatal error, but I didn't use the bit that removes the tabledrag override. The purpose of the js override is to have the drag and drop respect the rules of parentage from node hierarchy. If there is a problem with this code (my very brief testing indicates it works ok, but I might have missed something) please open a new ticket and we can figure out what's wrong with it.
Thanks once again for your help everyone.
Happy Holidays!
-R
Comment #23
ronan commented