Problem/Motivation

Drupal 7 allows you to add a configure line to a module's .info file to let users on the admin/modules page quickly jump to the module's settings.

The Bracket Tabs submodule uses this, but it points to admin/config/bracket-tabs, but that path is not defined in bracket_tabs_menu(). However, admin/config/content/bracket/bracket-tabs is.

Proposed resolution

Update bracket_tabs.info, and change the configure line to configure = admin/config/content/bracket/bracket-tabs.

Remaining tasks

  1. Write a patch
  2. Review and RTBC
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 bracket_tabs_module_s-2624444-2.patch735 bytesmparker17
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mparker17 created an issue. See original summary.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-4
CreditAttribution: mparker17 at Northern Commerce commented
Assigned: mparker17 » Unassigned
Issue summary: View changes
Status: Active » Needs review
FileSize
bracket_tabs_module_s-2624444-2.patch735 bytes

Here is a patch that makes the change proposed in the issue summary. Feedback welcome.

jacob.embree’s picture

jacob.embree CreditAttribution: jacob.embree commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community