Skip to content

[Bugfix]: use PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY in LocateFileTrait#18709

Merged
kingjia90 merged 1 commit intopimcore:11.5from
jdreesen:patch-4
Nov 7, 2025
Merged

[Bugfix]: use PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY in LocateFileTrait#18709
kingjia90 merged 1 commit intopimcore:11.5from
jdreesen:patch-4

Conversation

@jdreesen
Copy link
Copy Markdown
Contributor

In case PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY is overwritten, using PIMCORE_CUSTOM_CONFIGURATION_DIRECTORY/classes/ would be wrong.

@github-actions
Copy link
Copy Markdown

Review Checklist

  • Target branch (11.5 for bug fixes, others 11.x)
  • Tests (if it's testable code, there should be a test for it - get help)
  • Docs (every functionality needs to be documented, see here)
  • Migration incl. install.sql (e.g. if the database schema changes, ...)
  • Upgrade notes (deprecations, important information, migration hints, ...)
  • Label
  • Milestone

@sonarqubecloud
Copy link
Copy Markdown

@jdreesen jdreesen changed the title Use PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY in LocateFileTrait [Bugfix]: use PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY in LocateFileTrait Oct 29, 2025
@jdreesen
Copy link
Copy Markdown
Contributor Author

jdreesen commented Nov 7, 2025

@kingjia90 since you just fixed #18809... maybe you could have a look at this one, too?

@kingjia90
Copy link
Copy Markdown
Contributor

kingjia90 commented Nov 7, 2025

Thank you for your PR!

LGTM by looking at

$resolveConstant('PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY', PIMCORE_CUSTOM_CONFIGURATION_DIRECTORY . '/classes');

@kingjia90 kingjia90 added this to the 11.5.13 milestone Nov 7, 2025
@kingjia90 kingjia90 self-assigned this Nov 7, 2025
@kingjia90 kingjia90 merged commit 52e8c82 into pimcore:11.5 Nov 7, 2025
20 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 7, 2025
@jdreesen jdreesen deleted the patch-4 branch November 7, 2025 11:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants