Align project namespace and file names

Created on 30 January 2023, over 1 year ago

Problem/Motivation

After downloading the module, you automatically assume that you can install it using the same namespace, as you can everywhere else.

But you get an error:

$ composer require drupal/chatgpt_plugin
$ drush in chatgpt_plugin
In PmCommands.php line 333:
Unable to install modules chatgpt_plugin due to missing modules chatgpt_plugin

Ideally, the namespace (project URL) and file names should be aligned. The project namespace is chatgpt_plugin ( https://www.drupal.org/project/chatgpt_plugin ) but file names are chatgpt.info.yml ...

These files should probably be called chatgpt_plugin.info.yml, chatgpt_plugin.module, etc. instead.

This way people can download and install using the same namespace, without getting an error.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇩🇰Denmark ressa Copenhagen

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @ressa
  • Status changed to Needs review over 1 year ago
  • 🇮🇳India anup.sinha Bengaluru

    Hi @ressa,

    Actually earlier the folder structure was chatgpt_plugin/chatgpt. Now in the 1.0.0 version I moved out all the code from chatgpt folder to direct chatgpt_plugin to remove the sub-folder but I forgot to rename the files and namespaces. Thanks for reporting the bug.

    I have done all the changes and committed it into in the branch 1.0.x. Please test the chatgpt_plugin 1.0.x-dev version and let me know please if there is any more issue.

    Thanks & Regards,
    Anup

  • Status changed to RTBC over 1 year ago
  • 🇩🇰Denmark ressa Copenhagen

    Perfect @anup.sinha: With the latest dev-release I can use the chatgpt_plugin namespace both for downloading with Composer and installing with Drush. Thanks a lot, and have a great day.

  • Status changed to Fixed over 1 year ago
  • 🇩🇰Denmark ressa Copenhagen

    Thanks, feel free to give credit to both of us, for creating the issue, and fixing it.

Production build 0.69.0 2024