Is the path in the module correct or should it be 'admin/structure/form-builder/add'?

CommentFileSizeAuthor
#9 2725789-form_builder_menu_update-1.patch6.85 KBsocialnicheguru
#5 2725789-form_builder_menu_update.patch1.99 KBsocialnicheguru

Comments

SocialNicheGuru created an issue. See original summary.

torotil’s picture

torotil commented

Hi,

I think I need a bit more context here. What are you trying to do?

socialnicheguru’s picture

socialnicheguru commented

The configuration pages?
There is no 'admin/build' in drupal 7. I think it is 'admin/structure'.

torotil’s picture

torotil commented

Perhaps you are right. Do you want to create a patch for that?

socialnicheguru’s picture

socialnicheguru commented
StatusFileSize
new 2725789-form_builder_menu_update.patch1.99 KB
socialnicheguru’s picture

socialnicheguru commented
Status: Active » Needs review
socialnicheguru’s picture

socialnicheguru commented

They are of type Menu_callback which id defined as A hidden, internal callback, typically used for API calls.

torotil’s picture

torotil commented
Status: Needs review » Needs work

There is plenty of other places (besides the hook_menu() implementation) in the module where this paths are used. They need to be changed accordingly.

Thanks for creating the patch ;)

socialnicheguru’s picture

socialnicheguru commented
Status: Needs work » Needs review
StatusFileSize
new 2725789-form_builder_menu_update-1.patch6.85 KB

torotil’s picture

torotil commented

Thanks again. It's committed to 7.x-1.x and will be part of the next release.

torotil’s picture

torotil commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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