Skip to content

Conversation

@wiewiurdp
Copy link
Contributor

@wiewiurdp wiewiurdp commented Jan 16, 2026

🎫 Issue IBX-10186

Related PRs:

#1605
ibexa/core#696

Description:

This one is a community contribution from @ryanolee 🎉 .

This pull request introduces performance optimizations for repository operations by implementing limits on child count queries and subtree operations.

More info here:
#1605

For QA:

See orginal PR

Documentation:

New configuration of limits added

@wiewiurdp wiewiurdp force-pushed the IBX-10186-added-limits-to-repository-count branch 5 times, most recently from 0926a1a to 6596463 Compare January 16, 2026 15:20
@wiewiurdp wiewiurdp force-pushed the IBX-10186-added-limits-to-repository-count branch 14 times, most recently from 3910b82 to 28628a2 Compare January 19, 2026 12:43
@wiewiurdp wiewiurdp force-pushed the IBX-10186-added-limits-to-repository-count branch from 28628a2 to 4e34452 Compare January 19, 2026 13:43
@sonarqubecloud
Copy link

@wiewiurdp wiewiurdp marked this pull request as ready for review January 19, 2026 14:16
@wiewiurdp wiewiurdp requested a review from a team January 19, 2026 14:16
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 bot requested review from Steveb-p, ViniTou, alongosz, barw4, ciastektk, konradoboza, mikadamczyk and tbialcz and removed request for a team January 19, 2026 14:16
@wiewiurdp wiewiurdp changed the title IBX-10186: Added limits to repository child count queries and subtree operations [DON'T MERG - TEMP COMMIT] IBX-10186: Added limits to repository child count queries and subtree operations Jan 19, 2026
$currentScope,
$scopeSettings['subtree_operations']['copy_subtree']['limit']
);
if (isset($scopeSettings['subtree_operations']['copy_subtree']['limit'])) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those changes should be reflected in Ibexa\Tests\Bundle\AdminUi\DependencyInjection\Configuration\Parser\SubtreeOperationsTest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants