Show overview of modules in the admin plugin

[thijskh]

Previously, the admin module did not provide any information about which modules you have installed or wheter they are enabled. This seems a logical thing to show there.

This is now combined with the "SAML 2.0 IdP" enabled/disabled flag
which was a bit lonely before, as it's the only flag of its type.

[codecov]

Codecov Report

Merging #1561 (0547855) into master (51e3e55) will decrease coverage by 0.68%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1561      +/-   ##
============================================
- Coverage     42.39%   41.70%   -0.69%     
- Complexity     3427     3604     +177     
============================================
  Files           144      144              
  Lines         10330    10813     +483     
============================================
+ Hits           4379     4510     +131     
- Misses         5951     6303     +352     

[tvdijen]

I'm not sure... You've kinda reinvented the modinfo-module here...

[thijskh]

Yes. But:

• I think this information really belongs in the admin module, is actually a core part of its functionality. The config page shows various parts of your config, it really seems logical to me that which modules you've enabled is quite an important part, maybe the most important part, of your config.
• It does not make much sense to require users to fetch a separate module, enable it (!), and only then see simply what modules are installed and are enabled.
• Given it's only 10 lines of code and a few lines of template, having that hosted in its own separate module with all the overhead seems overkill.

[tvdijen]

I'm fine with it, but can we archive the modinfo-module then, to reduce maintenance?

[thijskh]

Definitely!