Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Only files that contain classes belong in the .info
. I consider it a bug. Obviously not important, but a bug.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1927176.patch | 955 bytes | RobLoach |
Comments
Comment #1
RobLoachGood catch.
Comment #2
rudiedirkx CreditAttribution: rudiedirkx commentedDid you mean to add
?, because I don't think that's related... It might be necessary, but I think that's another issue? The files in files[] aren't always or auto-included...
Comment #3
RobLoachYeah, was just concerned that admin.inc might not be included on locale pages... Like: stringoverrides/fr for French. Seems like we should stick it in this issue. Why not :-) .
Think it's good? I'd really love to add some tests for this module haha.
Comment #4
rudiedirkx CreditAttribution: rudiedirkx commentedIt's kinda weird that wasn't necessary... Wasn't it?? Yes, definitely a good thing.
Haven't actually patched and tested though... Is that necessary? (I've actually never used the module =) I was just browsing.)
Comment #6
joelpittetLooks like this got in, closing the issue, thanks @RobLoach