Problem/Motivation

Currently we alter the modules admin form to remove modules. This works but is now superseded by the hidden flag that can be set on extensions.

Proposed resolution

Instead use hook_system_info_alter() to set the hidden flag on the relevant modules (including paranoia).

Remaining tasks

User interface changes

If all modules in a given package (grouping) are hidden, the name of the empty package will no longer appear.

API changes

Data model changes

CommentFileSizeAuthor
#4 interdiff.txt552 bytesnedjo
#4 paranoia-system-info-2894994-2.patch1.55 KBnedjo
#3 paranoia-system-info-2894994-2.patch1.8 KBnedjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo created an issue. See original summary.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Version: 7.x-1.x-dev » 8.x-1.x-dev
nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Active » Needs review
FileSize
paranoia-system-info-2894994-2.patch1.8 KB
nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Issue summary: View changes
FileSize
paranoia-system-info-2894994-2.patch1.55 KB
interdiff.txt552 bytes

Oh, right, we still need to dynamically hide paranoia rather than hard-coding its hidden flag in the .info.yml file, so it can be manually enabled.

  • nedjo committed 666564f on 8.x-1.x 
    Issue #2894994 by nedjo: Hide modules via hook_system_info_alter()
nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Needs review » Fixed

In this fix I also modified the behaviour of hook_paranoia_hide_modules() to also return modules designated as disabled, since there isn't a use case where a module that is forcefully disabled should not also be hidden.

Status: Fixed » Closed (fixed)

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