Code cleanup and stabilisation #870
Open
+1,478,926
−42,224
Conversation
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
Hotfix for used/uses endpoint
- Run PHPCBF multiple passes to auto-fix 183+ coding standard violations - Add missing @return tags in controllers - Fix parameter documentation mismatches - Fix line length issues in docblocks - Fix array/function call formatting - Fix concatenation spacing - Fix indentation and alignment issues Remaining: 7,062 errors (down from 7,245) - Large files (ObjectService, FileService, SaveObjects) need manual fixes - Many errors in complex files that PHPCBF cannot auto-fix
ARCHITECTURE IMPROVEMENT: - Move massValidateObjects business logic (443 lines) from controller to service - Controller now properly thin: validates HTTP params, delegates to service - Service contains all business logic: batch processing, metrics, error handling PHPCS IMPROVEMENTS: - SettingsController: 123 → 83 errors (40 fixed, 33% reduction) - Fix file comment format (/** instead of /*) - Add missing @param documentation - Fix parameter documentation formatting - Remove business logic helper methods from controller NEW SERVICE METHODS: - SettingsService::massValidateObjects() - main business logic - SettingsService::createBatchJobs() - batch job creation - SettingsService::processJobsSerial() - serial batch processing - SettingsService::processJobsParallel() - parallel batch processing - SettingsService::processBatchDirectly() - single batch processing - SettingsService::formatBytes() - utility method BENEFITS: - Better separation of concerns (MVC pattern) - Easier to test business logic independently - Reusable service methods for other consumers - Improved code maintainability
…m ObjectService - Created new SearchQueryHandler with 8 search-related helper methods: * buildSearchQuery() - Convert request parameters to structured queries * applyViewsToQuery() - Apply view filters to search queries * isSolrAvailable() - Check SOLR availability * cleanQuery() - Normalize query parameters * addPaginationUrls() - Generate pagination links * logSearchTrail() - Log search analytics * isSearchTrailsEnabled() - Check trail settings - Registered SearchQueryHandler in Application.php dependency injection - Updated ObjectService to inject and use SearchQueryHandler - Reduced code duplication by delegating 4 methods to handler - PHPCS compliance: 570+ lines with only 2 minor errors Part of ObjectService refactoring to separate concerns and improve maintainability.
Replace force-push dev-to-beta with PR-creation workflow, re-enable unstable release for development branch, fix feature/php-linting refs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Enforce that PRs to main can only come from beta or hotfix branches, and PRs to beta can only come from development or hotfix branches. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Handle JSON strings from database in setRequired/getRequired and setConfiguration/getConfiguration methods. When reading JSON fields from database, they come as strings but setters expected arrays. Updated docblocks to match getFacets()/setFacets() style. Fixes configuration import errors.
Fix JSON field type handling in Schema and Register entities
Multitenancy fix & webhook POST/GET call fixes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
No description provided.