Problem/Motivation
This is related to this issue reported for simplytest project.
When the module is installed using composer, the folder getting downloaded in modules/contrib is named 'views_fractionslider'. Even the composer command is using the same name.composer require 'drupal/views_fractionslider'. But if we try to install the module using drush drush pm:install views_fractionslider, drush will throw error because the .info.yml file of the module is named fractionslider.info.yml. This seems to be confusing. Can we update the folder name and composer command to just "fractionslider" and composer require 'drupal/fractionslider' or change the machine name to "views_fractionslider"?
Steps to reproduce
- composer require 'drupal/views_fractionslider'
- drush pm:install views_fractionslider
Proposed resolution
Update the module name accordingly
Remaining tasks
User interface changes
API changes
Data model changes
Issue fork views_fractionslider-3328794
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3328794-project-machine-name
changes, plain diff MR !2
- 3328794-machine-name-and
changes, plain diff MR !1
Comments
Comment #4
avpadernoThis is a bug: If views_fractionslider is the project machine name, there must be a views_fractionslider.info.yml file, not fractionslider.info.yml; any hook implemented by the module has a name that starts with
views_fractionslider_, notfractionslider_.Comment #5
avpadernoComment #9
sonvir249 commentedComment #10
monymirzaAlready Fixed in V2 stable branch.
Comment #11
sonvir249 commented@monymirza in V2 stable branch file names are
fractionslider.info.yml,fractionslider.libraries.ymlandfractionslider.module. It sould follow the module directory name.Comment #12
monymirza