D7 includes/registry.inc : http://drupalcode.org/project/drupal.git/blob/e3de4550acb794e7b3589576d1... -> all modules are fetched - why not only the enabled ones? http://drupalcode.org/project/drupal.git/blob/e3de4550acb794e7b3589576d1... gives other modules a chance to change the list -> if a module is activated without admins recognizing it - this is surprising and should be avoided - so i don't see the need to fetch disabled modules in the db_query or do I miss something?

(There are other functions to get all modules )

Comments

Thomas_Zahreddin’s picture

StatusFileSize
new1.27 KB

patch shows what i mean …

Thomas_Zahreddin’s picture

Project: Drupal core » SimpleTest
Version: 7.x-dev » 7.x-2.x-dev
Component: base system » Code

http://drupalcode.org/project/drupal.git/blobdiff/b9ed1bc6e64fe82019c92c...

so it sould be a issue against simpletest.

My argument:
it is not needed for productive environment - so simpletest should be changed (and not the test object).
If simpletest needs something, then it's simpltests task to gather the stuff …

dave reid’s picture

Project: SimpleTest » Drupal core
Version: 7.x-2.x-dev » 7.x-dev
Component: Code » simpletest.module

You probably want to fix this in Core SimpleTest first then.

Thomas_Zahreddin’s picture

Status: Active » Closed (works as designed)

found with chx support:

http://drupalcode.org/project/drupal.git/blob/e9d97f1eef493de9b9cb3ae456...

means simpletest uses this to test disabled modules …