Problem/Motivation

responsive_image module still defines the obsolete hook_menu() which was removed in favour of the new routing system

Proposed resolution

Remove the hook_menu() definition from responsive_image module

User interface changes

none

API changes

none

Data model changes

none

CommentFileSizeAuthor
#1 remove_obsolte-2531568-1.patch1.23 KBwillzyx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

willzyx’s picture

willzyx CreditAttribution: willzyx commented
FileSize
remove_obsolte-2531568-1.patch1.23 KB
cilefen’s picture

cilefen CreditAttribution: cilefen commented
Title: Remove obsolte hook_menu() from responsive_image module » Remove obsolete hook_menu() from responsive_image module
Status: Needs review » Reviewed & tested by the community
Issue tags: +@deprecated

I checked over the patch and found that all the hook_menu() functionality was migrated to YAML, so this can be removed.

cilefen’s picture

cilefen CreditAttribution: cilefen commented

#2531622: The description for the responsive image module admin link should end with a period

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed 5e15b9b and pushed to 8.0.x. Thanks!

  • alexpott committed 5e15b9b on 8.0.x 
    Issue #2531568 by willzyx: Remove obsolete hook_menu() from...

Status: Fixed » Closed (fixed)

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