Attached is a patch for adding the hs widget to the Parent item select box under the menu settings field set in the node/add en node/x/edit pages.
The code is very minimal as all hs specific functionality was already in place for the menu administration section.
This one is very important for me as I regularly work on sites with big menu trees. Making hs work there would be great.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 285246.patch | 1.88 KB | wim leers |
| #7 | hs_menu.module.patch | 1.12 KB | davyvdb |
| #5 | hs_menu.module.patch | 899 bytes | davyvdb |
| #3 | hs_menu.module.patch | 1.63 KB | davyvdb |
| #1 | hs_menu.module.patch | 2.09 KB | davyvdb |
Comments
Comment #1
davyvdb commentedSorry, the one above is not the patch but the original file.
Find attach the correct patch.
Comment #2
wim leersHah, great, seems I missed a spot :)
However, what is that db query for? I *really* doubt you need it, and if you do need it, it needs some comments. Also, the doxygen for your helper function is missing.
Comment #3
davyvdb commentedTrue, I created the first patch in a hurry and didn't think about it thorougly. This one should be better now.
Comment #4
wim leersPlease create proper patches (using cvs diff) next time.
Now you're not using your helper function. And the documentation for it is still missing :)
Comment #5
davyvdb commentedOk, sorry. Left that function is. Is removed now, so no need for docs.
Should be fine now.
Comment #6
wim leers*Still* no CVS patch. I see you're a Drupal developer, so you sure must know that only CVS patches are used in the Drupal issue queues…
And the helper function was *good*, it prevented duplicate code. :)
Comment #7
davyvdb commentedSorry, I'm not fully into CVS ;)
This one should work?
The helper is now out since there is some difference between the two ('menu' index + no 'params').
Comment #8
wim leersActually… that doesn't change the fact that you should still have a helper function, you can simply pass a parameter to cope with the difference.
Patch attached for illustration. This patch also has been committed.
Comment #9
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #10
jbrown commentedI had to adjust the module weights before this would work for me.
Set Hierarchical Select Menu to 1 and Hierarchical Select to 2.
Otherwise the menu module hasn't put its fieldset in the form yet.
Comment #11
wim leersHS Menu doesn't work if it has weight 0? Could you triple-check that for me? I've always used it with weight 0 without problems!
HS should have a higher weight than HS Menu, that's true.
Comment #12
jbrown commentedThe reason I needed Hierarchical Select Menu to be heavier than Menu was because I had installed Hierarchical Select in 'modules', as opposed to 'sites/all/modules'.
You can see in this function that hooks are triggered in filename order when the weight is the same:
http://api.drupal.org/api/function/module_list/5
Hierarchical Select Menu should ensure that it is always called after Menu.
Comment #13
wim leersI see. Good to know :)
However, I disagree. Contributed modules such as HS Menu should never be installed in the modules directory, since it interferes with upgradability. If it's because you're creating an install profile, I can understand.
Because of that latter (valid) use case, I will add .install files for all included HS modules.
Also, I will add auto-correction of module weight soon, see this issue: http://drupal.org/node/296488.
Comment #14
wim leers