admin.inc files not included in pot file
Dave Reid - March 15, 2009 - 19:08
| Project: | Translation template extractor |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
I have a module directory structure like:
/sites/all/modules
/path_redirect
/genereate
path_redirect_generate.info
path_redirect_generate.module
path_redirect.admin.inc
path_redirect.info
path_redirect.install
path_redirect.module
path_redirect.testWith Extract from all in directory "sites/all/modules/path_redirect", the path_redirect.admin.inc file is included in the pot file (expected).
With Extract from path_redirect (in the fieldset), path_redirect.admin.inc is not included in the pot file (unexpected).
With Extract from path_redirect_generate in the sites/all/modules/path_redirect/generate directory (in the fieldset), path_redirect.admin.inc is not included in the pot file (expected).
| Attachment | Size |
|---|---|
| screenshot_003.png | 24.01 KB |
#1
And I'm using the very latest 6.x-2.x CVS branch.
#2
Well, potx is trying to look for path_redirect.{any of the known extensions}, and .admin.inc is not a known extension. The file lookup pattern probably needs to be changed to {modulename}{.optional whatever}.{any of the known extensions}.
#3
Try this patch.
#4
Duh, of course that was untested, so a typo was in there. Try this instead.
#5
Applied the patch and all the options worked as expected. I also tried it with a dummy path_redirect.views.inc and a path_redirect.pages.inc and it picked them up correctly.
#6
Great, committed, thanks!
#7
Since 5.x is not maintained, moving back to fixed.
#8
Automatically closed -- issue fixed for 2 weeks with no activity.