As described in #1161806: enabled_modules vs used_modules vs views_system... merge? deprecate?, there are some similarities between this module (Enabled Modules) and Views System.

Also, there are some bugs in the Drupal 7 branch at the moment (for example: #1185810: PHP error - class_exists() expects parameter 1 to be a string).

I've had a look into the Views System module, and it seems to have a good set up for describing the system module. It also gets around the problem of unwrapping the "info" field by using some custom field handlers.

Thus, i'm intending to rebuild the Enabled Modules default views on top of Views System. This will mean a dependency on Views System.

I've tested this locally and it seems to fit very well.

CommentFileSizeAuthor
#5 em--1262052-05.patch76.21 KBjuliangb
#3 EM--1262052-02.patch76.13 KBjuliangb
#1 EM--1262052-01.patch75.45 KBjuliangb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Active » Needs review
FileSize
EM--1262052-01.patch75.45 KB

Patch for review.

Status: Needs review » Needs work

The last submitted patch, EM--1262052-01.patch, failed testing.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Needs work » Needs review
FileSize
EM--1262052-02.patch76.13 KB

Amended patch.

Status: Needs review » Needs work

The last submitted patch, EM--1262052-02.patch, failed testing.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Needs work » Needs review
FileSize
em--1262052-05.patch76.21 KB

Revised patch.

Status: Needs review » Needs work

The last submitted patch, em--1262052-05.patch, failed testing.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Needs work » Fixed

I've pushed this up to 7.x-1.x-dev.

(The patch applied fine for me).

Status: Fixed » Closed (fixed)

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