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

Comments

Yago Elias created an issue. See original summary.

yago elias’s picture

yago elias commented
StatusFileSize
new 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, 🇳🇱/🇪🇺
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 commented
Status: Needs work » Needs review
StatusFileSize
new 2886842-4.patch1.14 KB

Reroll for #2 with some changes. Please review.

andrey.troeglazov’s picture

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

asya_asina 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
commented
Status: Needs work » Needs review
StatusFileSize
new 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
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 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.