... because in Drupal 7 there are some modules for which it won't work; see #1539940-94: Encourage best practices in /sites/README.txt, /modules/README.txt, /profiles/README.txt and /themes/README.txt for further details.

The current text:

It is safe to organize modules into subdirectories, such as "contrib" for
contributed modules, and "custom" for custom modules. Note that if you move a
module to a subdirectory after it has been enabled, you may need to clear the
Drupal cache so that it can be found.

needs to be rewritten somehow to acknowledge that.

CommentFileSizeAuthor
#1 move-modules-2500101-1.patch835 bytesDavid_Rothstein

Comments

David_Rothstein’s picture

David_Rothstein commented
Status: Active » Needs review
StatusFileSize
new move-modules-2500101-1.patch835 bytes

Maybe something like this, based loosely on #1539940-98: Encourage best practices in /sites/README.txt, /modules/README.txt, /profiles/README.txt and /themes/README.txt. I guess if they move the module while it's enabled and it fails spectacularly, they can always move it back and then try the alternate method?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Fixed

That looks fine to me, very reasonable. As you're the branch maintainer, and probably don't want to commit your own patch :), I'll just go ahead and review/commit it.

  • jhodgdon committed dc84091 on 7.x 
    Issue #2500101 by David_Rothstein: sites/all/modules/README.txt should...
David_Rothstein’s picture

David_Rothstein commented

Thanks. You probably should have given yourself a commit credit too, though - about 80% of the wording of that was yours :)

Status: Fixed » Closed (fixed)

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