Problem/Motivation

The machine name for this module is pb_drupal_packagist, but the info.yml and the install files are named project_browser_drupal_packagist.

You would normally enable a module using the machine name that is the folder name/used in the composer.json, but the way this is set up, you need to run drush en -y project_browser_drupal_packagist instead.

Steps to reproduce

composer require drupal/pb_drupal_packagist
drush en -y pb_drupal_packagist
You will get this error: Unable to install modules pb_drupal_packagist due to missing modules pb_drupal_packagist.

Proposed resolution

Rename project_browser_drupal_packagist.info.yml and project_browser_drupal_packagist.install

Remaining tasks

TBD

User interface changes

TBD

API changes

TBD

Data model changes

TBD

Issue fork pb_drupal_packagist-3464136

Command icon 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:

About issue forks

Comments

thejimbirch created an issue. See original summary.

thejimbirch opened merge request !1

thejimbirch’s picture

thejimbirch commented
Status: Active » Needs review
fjgarlin’s picture

fjgarlin commented
Status: Needs review » Needs work

We also need to rename function names in the install/module files and namespaces to reflect the change.

fjgarlin’s picture

fjgarlin commented
Status: Needs work » Fixed

Changes merged and also some little fixes so it's compatible with the latest 2.0.x

There are still ongoing issues listed here #3453884: Fix many small things but the current code gets a POC up and running with a live endpoint.

Status: Fixed » Closed (fixed)

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