Skip to content

dx-3498 include method to handle overlapping reference paths#93

Merged
harshithad0703 merged 5 commits intodevelopmentfrom
fix/dx-3498-handle-overlapping-reference-paths-include()
Sep 10, 2025
Merged

dx-3498 include method to handle overlapping reference paths#93
harshithad0703 merged 5 commits intodevelopmentfrom
fix/dx-3498-handle-overlapping-reference-paths-include()

Conversation

@harshithad0703
Copy link
Contributor

  • Prevent duplicate entries when using paths like ["content", "content.content"]
  • Maintain 100% backward compatibility with existing implementations
  • Add analyzeReferencePaths() method to detect path overlaps
  • Add processOverlappingPaths() method to intelligently handle overlaps
  • Use most specific path from overlapping groups to avoid duplicates
  • Preserve original behavior for single fields and independent paths

Fixes: Duplication issue when including nested reference paths
Breaking: None - fully backward compatible

@harshithad0703 harshithad0703 marked this pull request as ready for review September 10, 2025 11:38
@harshithad0703 harshithad0703 requested a review from a team as a code owner September 10, 2025 11:38
@harshithad0703 harshithad0703 merged commit 82d9ca1 into development Sep 10, 2025
9 checks passed
@harshithad0703 harshithad0703 deleted the fix/dx-3498-handle-overlapping-reference-paths-include() branch September 10, 2025 11:38
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.

3 participants

Comments