This is a cool big improvement.

The original idea coming from the jurgenhaas Feedback.

Comments

jurgenhaas’s picture

This should be very easy to implement: just remove the .info and .module files and include everything that is now in the .module into the .drush.inc - that should be it.

Then, drush dl entity_scaffold should download and install this module into ~/.drush/entity_scaffold and it can be used globally.

EDIT:

In addition a small change in the first define instruction is required:

define('ENTITY_SCAFFOLD_MODEL_MODULE_PATH', dirname(__FILE__) . '/model/');

I.e. use dirname() instead of drupal_get_path().

luizsgpetri’s picture

Assigned: Unassigned » luizsgpetri

Fixed

  • luizsgpetri committed 13b1977 on 7.x-2.x
    Issue #2396257 by luizsgpetri: Update module to be a global Drush...

  • luizsgpetri committed c69af66 on 7.x-2.x
    Issue #2396257 by luizsgpetri: Define drush bootstrap level "es" command...
jurgenhaas’s picture

Looks like this drush command is using functions that are defined by Drupal and therefore requires a higher bootstrap level.

jribeiro’s picture

Assigned: luizsgpetri » Unassigned

hey @jurgenhaas,

We created the branch 7.x-2.x with the changes to convert the module to be a 'global drush' command.

But, when I use drush dl entity_scaffold --select and select the 2.x version, the drush dl don't saves the module in the drush folder, instead saves in the relative path.

any idea?

jurgenhaas’s picture

Should be working just fine. Is that only happening when you'r using the --select option? If so, that might be a drush issue then.

  • jribeiro committed 25b15f7 on 7.x-2.x
    Issue #2396257 by jurgenhaas, jribeiro, luizsgpetri: Changed the...
jribeiro’s picture

Status: Active » Fixed

Done!

Implemented and released on 7.x-1.0-beta2. Check the commit.

Thanks guys, you were mentioned in the commit.

Status: Fixed » Closed (fixed)

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

luizsgpetri’s picture

luizsgpetri’s picture

luizsgpetri’s picture