PageTree controller class

[distantnative]

Description

Summary of changes

• Moved page tree request logic from area config to PageTree Panel controller clas
• Added unit tests

Reasoning

Make more code testable.

Ready?

• In-code documentation (wherever needed)
• Unit tests for fixed bug/feature
• Tests and CI checks all pass

[afbora]

@distantnative I couldn't test moving the page for this branch or v5 (disabled all parents). Works great for v4. Could you able to move any page with this branch?

[distantnative]

@afbora does it work now for you?

[afbora]

@distantnative Still broken for me. Now I can move any page to any target. This is not correct behavior.

[distantnative]

@afbora what's your test case where the result of this PR is different to develop-minor?

Mine is using the starter kit and creating a test note page on the top-level. That note page I can move in the same pages on this branch as on develop-minor with certain pages disabled:

develop-minor:

This PR:

[afbora]

I was tested for starterkit with v5/refactor/page-tree-controller-class branch.

Something wrong. I've tested on 4.4.1 for Exploring the universe page. It should be like that as default.

[distantnative]

Oh interesting, this seems to be rather a difference main vs. develop-minor branch. Not this PR. I wonder why.

[distantnative]

@afbora I think it's because of #6717

[bastianallgeier]

@distantnative haha, I posted the exact same comment basically 4 seconds after yours :)

[bastianallgeier]

Is the failing unit test a false positive?

[distantnative]

They were still set up for the no pages section needed state.

Fixed them and added more specific ones too.