Fix: Refactor folder structure for consistency with dynamix plugins#76
Open
Fix: Refactor folder structure for consistency with dynamix plugins#76
Conversation
…ase files Align compose.manager directory layout with dynamix.docker.manager conventions: - php/ -> include/ (matches dynamix include/ pattern) - styles/ -> sheets/ (matches dynamix sheets/ pattern) - All PHP files renamed to PascalCase - All CSS files renamed to PascalCase Path references not yet updated - next commit.
Update all require_once, include, URL, and asset references to use the new directory names (include/ instead of php/, sheets/ instead of styles/) and PascalCase filenames. Files updated: - 14 PHP source files (require_once paths) - 4 .page files (asset hrefs and include paths) - 1 JS file (AJAX endpoint URLs) - 1 shell script (inline PHP paths) - 1 CSS file (comment reference) - 3 build/test configs (phpstan, phpunit, bootstrap) - 19 test files (include paths and docblock comments) - 1 framework example test (asset path)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull request summary
Update all require_once, include, URL, and asset references to use the new directory names (include/ instead of php/, sheets/ instead of styles/) and PascalCase filenames.
Files updated:
What changed:
source/compose.manager/php/→source/compose.manager/include/source/compose.manager/styles/→source/compose.manager/sheets/compose_manager_main.php→ComposeManager.phpcompose_list.php→ComposeList.phpcompose_util.php→ComposeUtil.phpcompose_util_functions.php→Helpers.phpexec.php→Exec.phpexec_functions.php→ExecHelpers.phpinclude/similarly.pagetemplate files to new assets and include URL pathssource/compose.manager/javascript/common.jssource/compose.manager/scripts/backup_cron.shphpstan.neonphpunit.xmltests/bootstrap.php(PluginBootstrap::init path/subPath)test.ps1execution passes with PHPUnit, PHPStan, Bats (skipped tests are known and unrelated to this refactor)Related issues
Closes # (if applicable)
Checklist
Testing notes
Run locally from repo root:
pwsh -NoProfile -ExecutionPolicy Bypass -File test.ps1php vendor/bin/phpunit --configuration phpunit.xmlphp vendor/bin/phpstan analyse --memory-limit=512Mbash tests/framework/bin/run-bats.sh tests/unitcompose.manager/phporcompose.manager/stylesreferences:grep -R "compose.manager/php" source/compose.managergrep -R "compose.manager/styles" source/compose.managerNotes
plugin_rootpath remains/usr/local/emhttp/plugins/compose.manager/php/andstyles/plugin URLs are intentionally replaced,