backdrop-contrib / taxonomy_menu

Adds links to taxonomy terms into a menu.
GNU General Public License v2.0
0 stars 3 forks source link

Add default arg to taxonomy_menu_handler to avoid too few args fatal … #12

Open eric2pin opened 3 months ago

eric2pin commented 3 months ago

Proposed fix for issue #11 that's working on my local. The $node arg seems unnecessary in these cases; several calls are made to taxonomy_menu_handler throughout the code without providing the $node arg, resulting in an error.

nattywebdev commented 2 weeks ago

I applied this update and it fixed issue #11 for me too, except my two-level taxonomy vocab is flattened into one level in the menu.

argiepiano commented 2 weeks ago

I've submitted a more comprehensive PR that fixes this problem and also the term hierarchy issue #13. You may want to close this PR.