Skip to content

Fix: Store parent of most deeply nested element during expansion#516

Merged
g105b merged 4 commits intomasterfrom
504-bind-list-bug
Apr 12, 2026
Merged

Fix: Store parent of most deeply nested element during expansion#516
g105b merged 4 commits intomasterfrom
504-bind-list-bug

Conversation

@g105b
Copy link
Copy Markdown
Member

@g105b g105b commented Apr 10, 2026

This pull request fixes an issue where the parent of the most deeply nested element was not correctly stored during expansion. It ensures proper hierarchy tracking by storing a properly-normalised version path to the component's original location in the tree.

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 10, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 11 complexity · 0 duplication

Metric Results
Complexity 11
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@g105b g105b linked an issue Apr 10, 2026 that may be closed by this pull request
@g105b
Copy link
Copy Markdown
Member Author

g105b commented Apr 10, 2026

I've been working on the DHPL project again this morning and loaded this branch using composer. It immediately fixed the issue, apart from where bindListCallback was used, which the fix didn't address properly.

It's all resolved now. I'll continue to test and then merge later if nothing else comes up.

g105b added 2 commits April 10, 2026 16:26
…list-bug

# Conflicts:
#	src/DocumentBinder.php
#	test/phpunit/DocumentBinderTest.php
@g105b g105b merged commit bb14438 into master Apr 12, 2026
53 checks passed
@g105b g105b deleted the 504-bind-list-bug branch April 12, 2026 15:13
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.

BindList: complex pages with named lists can cause extra binding

1 participant