Skip to content

Moved the constants to the dedicated class.#40

Merged
akohan91 merged 1 commit intodevelopfrom
refactor/move-constants-to-separeted-class
Jan 19, 2026
Merged

Moved the constants to the dedicated class.#40
akohan91 merged 1 commit intodevelopfrom
refactor/move-constants-to-separeted-class

Conversation

@akohan91
Copy link
Copy Markdown
Owner

Summary

Moved all static constants from libak_RestFramework to a new dedicated libak_RestConstants class to improve code organization and separation of concerns.

Changes

  • New Class: Created libak_RestConstants class containing all HTTP status codes, header names, content types, and error messages
  • Updated Classes: Refactored libak_RestFramework and all dependent classes to reference libak_RestConstants instead
  • Updated Tests: Modified all test classes to use the new constant references
  • Documentation: Updated API_Reference.md to reflect the new class structure

Impact

  • ✅ Better separation of concerns - constants are now in a dedicated class
  • ✅ Improved maintainability - constants have a single source of truth
  • ✅ No breaking changes to functionality - all tests pass
  • ⚠️ Breaking change: External code referencing libak_RestFramework.HTTP_CODE_* or similar constants must update to libak_RestConstants.*

Files Changed

  • New: libak_RestConstants.cls and metadata
  • Modified: 7 framework classes, 6 test classes
  • Documentation: API_Reference.md

This is a refactoring improvement that enhances code organization while maintaining all existing functionality.

@akohan91 akohan91 changed the base branch from main to develop January 19, 2026 12:32
@akohan91 akohan91 closed this Jan 19, 2026
@akohan91 akohan91 reopened this Jan 19, 2026
@github-actions
Copy link
Copy Markdown

💚 PMD Analysis: No issues found

@akohan91 akohan91 closed this Jan 19, 2026
@akohan91 akohan91 reopened this Jan 19, 2026
@github-actions
Copy link
Copy Markdown

💚 PMD Analysis: No issues found

@github-actions
Copy link
Copy Markdown

💚 Deployment Validation Results:

  • Status: Success

@akohan91 akohan91 merged commit 464c7f6 into develop Jan 19, 2026
3 of 4 checks passed
@akohan91 akohan91 deleted the refactor/move-constants-to-separeted-class branch January 19, 2026 12:40
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.

1 participant