I've installed
drush @TEST status | grep version
Drupal version : 7.19
Drush version : 6.0-dev
and latest zen,
drush @TEST pm-info zen | egrep "Version|Status"
Version : 7.x-5.1+93-dev
Status : enabled
If I subtheme zen, using an underscore in my subtheme name,
drush @TEST zen "test_zen"
the subtheme is create, but *incorrectly* with a hyphen replacing the underscore in the dir name,
Starter kit for "test_zen" created in: /dev/web/www.test.loc/sites/all/themes/test-zen
^^^^^^^^
Comments
Comment #1
makangus CreditAttribution: makangus commentedTrying running
drush @TEST zen "Test Zen" test_zen
, the first argument is the theme name and the second optional argument is machine name.Comment #2
JohnAlbinSo the relavent line in zen.drush.inc is:
My thinking was that the path to assets like CSS, JS, images shouldn't use underscores. e.g. sites/all/themes/test-zen/images, sites/all/themes/test-zen/css, etc.
Note that the sub-theme works just fine. Even though the directory is named "test-zen", the .info file is test_zen.info and all the functions in template.php and theme-settings.php use "test_zen" in their names.
drupsup, is there a specific problem/bug this is causing? I'm open to discussing this.
Comment #3
drupsup CreditAttribution: drupsup commentedhi john, sorry for the delay ... i got caught up in abandoning all other themes on our sites, and migrating over to Zen.
to this issue -- no, no specific *bug*.
simply maintaining consistency & managing confusion.
as you've pointed out, currently,
for my own prefs,
bottom line -- give me an OPTION, and sure, I may be glad for it. FORCE me to "do what you think is best", and, eventually, one bumps into an issue.
my request? just let the drush installer do what I tell it to do.
similarly, open to discuss ...
thanks!
Comment #4
drupsup CreditAttribution: drupsup commentedComment #5
jwilson3Is there a technical reason why paths to assets should be referenced with dashes in the URL?
Drupal core doesn't contain any folders with dashes in them.
Modules and themes that have spaces in their names stored on Drupal.org and downloaded with drush all get generated inside folders with underscores, to match the underscore in the module and theme name, and their respective assets are referenced using underscores in the URLs.
I guess the dash is technically not an invalid, but it just strikes me as odd, and have to agree with the arguments in #3.
Comment #6
drupsup CreditAttribution: drupsup commented> Is there a technical reason why paths to assets should be referenced with dashes in the URL?'
If there is, then it's news to me ...
None of the other theme systems I've used have had any issues with underscores, or dashes for that matter.
The only argument I've found re: "dashes vs underscores" is SEO ... e.g.:
@ http://www.ecreativeim.com/blog/2011/03/seo-basics-hyphen-or-underscore-...
In general, if true, that *amplifies* the "leave it alone" argument made here ... it's up to ME to decide if I want Google et. al. ranking an asset at a given URL as "test_zen" or "test-zen" (which, 'in SEO' == "test" + "zen", apparently).
In Drupal, of course, my subtheme's dir isn't being exposed to search engines ...
*This* just sits in the 'annoyance' category.
Comment #7
JohnAlbinI'm sorry this annoys you, but there's no bug here. Most people don't know that a theme can exist in a directory with dashes in it. It's only the .info file name that defines the machine name of the theme. And that file is created with the proper underscore.
Also, the Zen Classic theme lives at http://drupal.org/project/zen-classic without issue; it can be downloaded with drush using: drush dl zen-classic
Comment #8
JohnAlbinSlept on this. The dash in the theme directory is inconsistent with other themes in the Drupal world. I shouldn't be trying to create a new standard with my drush commands. :-)
Comment #9
JohnAlbinFixed! http://drupalcode.org/project/zen.git/commitdiff/4c7f1c7