Currently the mobile_jquery drush command install's the sub-theme into the sites folder that the Mobile jQuery theme is installed in.

It would be great if the command would be able to specify a "site" to install the theme to. This would be of great help to developers working in multi-site environments.

CommentFileSizeAuthor
#2 mobile_jquery-drush_now_checks_specified_sites_folder_exists-1054616.patch1.33 KBcharlietoleary
#1 mobile_jquery-allow_drush_sites-1542860.patch2.54 KBcharlietoleary

Comments

charlietoleary’s picture

charlietoleary commented
Assigned: Unassigned » charlietoleary
Status: Active » Needs review
StatusFileSize
new mobile_jquery-allow_drush_sites-1542860.patch2.54 KB

I had a go at implementing the functionality - please review.

You should now be able to specify a site with the "--site" option in the drush command.

drush mobile_jquery "My Mobile Theme" mobile_theme --site="mysite"

charlietoleary’s picture

charlietoleary commented
StatusFileSize
new mobile_jquery-drush_now_checks_specified_sites_folder_exists-1054616.patch1.33 KB

I realised that the code should probably check that the sites folder specified actually exists - patch attached.

Depends on patch #1.

jasonsavino’s picture

jasonsavino commented
Status: Needs review » Fixed

patch added to dev

Status: Fixed » Closed (fixed)

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