Please maintainers, rename this module properly following Drupal standards. I saw a few posts dealing with this and apparently it has not been dealt with: Either rename the enclosing folder to "clone" or rename the full module "node_clone". Not doing so breaks workflow with Drush and adds an unneccesary glitch to it.

Then have a margarita.

Great module btw.

Comments

tezalsec’s picture

Subscribing..

I am building a custom module that retrieves info from all other contrib modules, and this node_clone module which I am happily using, seems to be the only one not respecting namespaces behaviour. It seems node_clone and clone are both used, and it messes up my dynamic functions building arrays collecting info from all installed contrib modules.
If it is at all possible, please consider respecting namespaces, maybe renaming the project to clone, or the files and functions to node_clone_etc...

thanks!

r2integrated’s picture

+1

pwolanin’s picture

Status: Active » Closed (works as designed)

Everything in the module is "clone". I don't feel that relocating the project would be in the best interest of existing users.

Agileware’s picture

Status: Closed (works as designed) » Needs review

Hi,

I ask that you re-consider bringing this module in-line with the Drupal module standards, having the module called "node_clone", yet the module itself called "clone" is confusing and problematic.

If you are using drush then you can get stuck in this infinite loop:

$ drush dl node_clone
Project node_clone (7.x-1.0-rc1) downloaded to [success]
/public_html/sites/all/modules/contrib/node_clone.
Project node_clone contains a module named clone.

$ drush -y en node_clone
node_clone was not found. [warning]
The following projects provide some or all of the extensions not found: [ok]
node_clone
Would you like to download them? (y/n): y
Install location /public_html/sites/all/modules/contrib/node_clone already exists. Do you want to overwrite it? (y/n): n
Skip installation of node_clone to /public_html/sites/all/modules/contrib/node_clone.[warning]
node_clone was not found.
...
(infinite loop starts)

So please consider renaming the module and all functions completely to "node_clone"

Thanks!

pwolanin’s picture

Status: Needs review » Postponed

not for any stable release - maybe for 8.x

pwolanin’s picture

Also, you appear not to be reading the drush output which clearly says:

"Project node_clone contains a module named clone"

rdworianyn’s picture

Version: 7.x-1.x-dev » 6.x-1.3
Status: Postponed » Active

The project should be moved to "clone". Every module needs to follow the proper naming conventions, this one included. Either that, or all of the code needs to be updated to use node_clone instead so that the project doesn't have to be moved.

pwolanin’s picture

Version: 6.x-1.3 » 7.x-1.x-dev
Issue summary: View changes
Status: Active » Closed (won't fix)

There is no possibility that I am going to rename it all for 7 or 6.