Problem/Motivation

Several links and one menu item in MyTube do not currently reflect the elements of menu structure that changed in Drupal 7. As a consequence, these links are broken. This is especially concerning for new users of the module, who are presumably most dependent upon the help text.

Proposed resolution

  • Update the links to reflect the current locations (admin/people/permissions instead of admin/user/permissions, for instance)
  • Update the location of the settings page from admin/config/mytube to admin/config/content/mytube.

Remaining tasks

Review patch.

User interface changes

  • Fixes broken links in help text.
  • Moves settings page to admin/config/content/mytube.

API changes

None.

Original report by grndlvl

d7 changed the structure of configuration menu and now this settings page should exist under "Administration >> Configuration >> Content authoring"

CommentFileSizeAuthor
#4 mytube-update_urls_to_be_consistent_with_drupal_7-2423509-4.patch4.24 KBmilodesc
#3 mytube-update_urls_to_be_consistent_with_drupal_7-2423509-3.patch3.83 KBfureigh
#1 mytube-move_settings_page_under_content_authoring-2423509-1.patch531 bytesgrndlvl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grndlvl’s picture

grndlvl CreditAttribution: grndlvl commented
Status: Active » Needs review
FileSize
mytube-move_settings_page_under_content_authoring-2423509-1.patch531 bytes
fureigh’s picture

fureigh CreditAttribution: fureigh commented
Title: Move settings page to under admin/config/content to match d7 » Update URLs and move settings page to match D7 menu structure
Issue summary: View changes
fureigh’s picture

fureigh CreditAttribution: fureigh commented
FileSize
mytube-update_urls_to_be_consistent_with_drupal_7-2423509-3.patch3.83 KB

The attached patch updates URLs in a few links that were broken and moves the settings page. Grndlvl's patch (hi, grndlvl!) did the latter too, so I've updated the commit info accordingly.

milodesc’s picture

milodesc
he/him
CreditAttribution: milodesc commented
FileSize
mytube-update_urls_to_be_consistent_with_drupal_7-2423509-4.patch4.24 KB

In previous patches for this issue, the configuration page for mytube was moved from 'admin/config/mytube' to 'admin/config/content/mytube'. However the path for the 'Configure' link under the operations column on the modules page was not updated to reflect this change. This patch fixes that issue so that users can get to mytube's configuration page from the site's modules page.

  • mfb committed 5c37240 on 7.x-1.x authored by grndlvl 
    Issue #2423509 by grndlvl, fureigh, milodesc: Update URLs and move...
mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
Status: Needs review » Fixed

Thanks all!

milodesc’s picture

milodesc
he/him
CreditAttribution: milodesc as a volunteer and at American Civil Liberties Union commented
grndlvl’s picture

grndlvl CreditAttribution: grndlvl at GollyGood Software for Advomatic commented

Status: Fixed » Closed (fixed)

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