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 |
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 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 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