Skip to content

Align Omeka module metadata with the HelpAssistant module namespace#10

Merged
erseco merged 1 commit into
mainfrom
copilot/fix-class-not-found-error
Mar 23, 2026
Merged

Align Omeka module metadata with the HelpAssistant module namespace#10
erseco merged 1 commit into
mainfrom
copilot/fix-class-not-found-error

Conversation

Copilot AI commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

Omeka was attempting to load Help Assistant\Module during module installation, but the module class is declared as HelpAssistant\Module. The mismatch came from config/module.ini, causing installation to fail before the module could bootstrap.

  • Module identity

    • Updated the module name in config/module.ini from Help Assistant to HelpAssistant
    • This brings Omeka’s module-class resolution into sync with the actual PHP namespace declared in Module.php
  • Regression coverage

    • Added a focused test that asserts the module name in module.ini remains aligned with the HelpAssistant namespace
    • This guards against future packaging or metadata edits reintroducing the install-time class lookup failure
  • Relevant change

    [info]
    - name         = "Help Assistant"
    + name         = "HelpAssistant"

@github-actions

Copy link
Copy Markdown

omeka-s Playground Preview

Open this PR in Omeka-S Playground
Try this PR in your browser

This preview loads the plugin directly from the PR branch ZIP in Omeka-S Playground.

@erseco erseco merged commit b5fdd50 into main Mar 23, 2026
5 checks passed
@erseco erseco deleted the copilot/fix-class-not-found-error branch March 23, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants