I'm slowly getting back to using this module, directly from the web (no drush use yet) and I am puzzled by the "module_builder_write_directory" variable. Doesn't seem like it's used anywhere, am I wrong?

I could provide a patch to use that variable and actually write the .module and .info files to said directory. Assign me the issue if that's the case, or close it if I am missing something.

Update

After looking at the code more closely, I found a "module_builder_module_write_directory" variable in module_builder.module and what looks like incomplete code. My offer still stands to fix this gap.

Comments

Robin Millette’s picture

Issue summary: View changes

Updated issue summary.

Robin Millette’s picture

Issue summary: View changes

Updated issue summary.

joachim’s picture

It's used by Drush, probably, with the --parent option.

Also, there used to be a system to write files directly, and I was told off by various people with more sense than me for it being a whopping security risk (search the issue queue maybe... or maybe it was on IRC).

There is certainly a case to be made for it anyway, and saying that this is a module for developers only which you should only enable on your development environment and NEVER a product site.

There was also a while ago either a patch or a feature to download the files as a zip. Possibly on the 6.x version. IIRC it required a plugin, and I never quite got it to work, so I didn't bother porting it to D7.

You're welcome to work on any of the above if you like! (I use only the drush part these days... I'll probably just commit any patches you post that look reasonably sane ;)

joachim’s picture

Category: bug » feature
joachim’s picture

Issue summary: View changes

Updated issue summary.

sobi3ch’s picture

Issue summary: View changes

+1