Hi all, following this #2403371: Add option to display module path on admin/modules table we could implement the small thing for the same reasons. I found that very useful, I will be working in a patch for that. Let's add this feature.

Examples of how should appear on the UI:

example

CommentFileSizeAuthor
#7 module_filter-Add_option_to_display_module_path-2886842-d8-7.patch2.64 KBpaulocs
#4 2886842-4.patch1.14 KBAkashKumar07
#2 add_option_to_display-2886842-2.patch1.49 KBYago Elias
aaaa.PNG24.25 KBYago Elias
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Yago Elias created an issue. See original summary.

Yago Elias’s picture

Yago Elias CreditAttribution: Yago Elias as a volunteer and at CI&T commented
FileSize
add_option_to_display-2886842-2.patch1.49 KB

Added a new option in the config options.

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
CreditAttribution: clemens.tolboom commented
Issue summary: View changes
Status: Active » Needs work

Why is this patch not added into same issue #2403371: Add option to display module path on admin/modules table? I think it was better to bump other issue to 8.x-3.x so we later can backport it to 7.x-2.x. IMHO that would be better for the maintainers.

+++ b/module_filter.routing.yml
@@ -4,4 +4,4 @@ module_filter.settings:
\ No newline at end of file

No newline at end of file

AkashKumar07’s picture

AkashKumar07 CreditAttribution: AkashKumar07 at OpenSense Labs commented
Status: Needs work » Needs review
FileSize
2886842-4.patch1.14 KB

Reroll for #2 with some changes. Please review.

andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented
Assigned: Unassigned » andrey.troeglazov
asya_asina’s picture

asya_asina CreditAttribution: asya_asina as a volunteer and at DrupalJedi commented
Status: Needs review » Needs work

Unfortunately, adding a new option in the config options is not enough. You also need to implement the logic of the checkbox in the code. This is not in the patch.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented
Status: Needs work » Needs review
FileSize
module_filter-Add_option_to_display_module_path-2886842-d8-7.patch2.64 KB

Follow the patch to add the module's path in the table if the checkbox is checked.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented
Status: Needs review » Reviewed & tested by the community

  • andrey.troeglazov authored 2211623 on 8.x-3.x 
    Issue #2886842 by paulocs, andrey.troeglazov, asya_asina: Add option to...
andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -ciandt-contrib +Module Filter Release 8.x-3.2

Status: Fixed » Closed (fixed)

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