8 Nov
2024
8 Nov
'24
2:56 p.m.
Branch: refs/heads/feature-deploy-refactor-links
Home: https://github.com/xwiki/xwiki-platform
Commit: ce2be6872b96da123060688d80af85d9374a7f05
https://github.com/xwiki/xwiki-platform/commit/ce2be6872b96da123060688d80af…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker/src/test/it/org/xwiki/flamingo/test/docker/RenamePageIT.java
M xwiki-platform-core/xwiki-platform-oldcore/src/main/java/com/xpn/xwiki/XWiki.java
M
xwiki-platform-core/xwiki-platform-oldcore/src/main/java/com/xpn/xwiki/internal/render/DefaultOldRendering.java
M
xwiki-platform-core/xwiki-platform-oldcore/src/test/java/com/xpn/xwiki/XWikiTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/checkstyle/checkstyle-suppressions.xml
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/ReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/ReferenceUpdater.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/AbstractCopyOrMoveJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/AbstractEntityJobWithChecks.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/CopyJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/MoveJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/listener/BackLinkUpdaterListener.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultMacroRefactoring.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultReferenceUpdater.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultMacroRefactoringTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultReferenceUpdaterTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/ResourceReferenceRenamerTest.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-xwiki/src/main/java/org/xwiki/rendering/macro/MacroRefactoring.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
WIP
The idea of this work is to:
1. Provide a way to access all documents that are moved as part of a
move job
2. Use that information when performing a call to ReferenceRenamer to
define if a relative untyped link should be handled or not
On top of it, the idea is also to check if the doc exists in case of
refactoring of a link to avoid refactoring unexisting relative links.
One problem is remaining about relative link pointing to sibling pages
(e.g. the link to Alice in Bob page in the ticket): we rely apparently
to an old mechanism for backward compatibility reason for this to work
in the UI, we might need same thing in the check, or to decide to
ignore that UC.
I started to add an integration tests but for some reason it's not
passing, though it seemed to be working locally for the scenario
described in the ticket (except for the link in Bob page).
Commit: 85844e63a987ff62680277a46bc5dd598d5684b6
https://github.com/xwiki/xwiki-platform/commit/85844e63a987ff62680277a46bc5…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker/src/test/it/org/xwiki/flamingo/test/docker/RenamePageIT.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacroRefactoring.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/include/IncludeMacroRefactoringTest.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Fix integration test setup
* Fix some signatures
* Work on the conditions for performing link update: WIP
Commit: 38d9797b4e11751bec3a045a978611f9ed70dee4
https://github.com/xwiki/xwiki-platform/commit/38d9797b4e11751bec3a045a9786…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Fix conditions to make all RenamePageIT passing
* WIP: need to double check that some conditions are not redundant and
double check side effects
Commit: 6b5089b0230717bc0bb64a90299bc0563eeaba6e
https://github.com/xwiki/xwiki-platform/commit/6b5089b0230717bc0bb64a90299b…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/test/java/org/xwiki/refactoring/internal/listener/BackLinkUpdaterListenerTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultReferenceUpdaterTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/ResourceReferenceRenamerTest.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Simplify a bit the conditions in ResourceReferenceRenamer and ensure
all unit tests are passing in refactoring module
Commit: eb72a90a7150723d8ed78a281230fa958154879d
https://github.com/xwiki/xwiki-platform/commit/eb72a90a7150723d8ed78a281230…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-oldcore/src/test/java/com/xpn/xwiki/XWikiTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/ReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/ReferenceUpdater.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/AbstractEntityJobWithChecks.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/listener/BackLinkUpdaterListener.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultMacroRefactoring.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-xwiki/src/main/java/org/xwiki/rendering/macro/MacroRefactoring.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Fix checkstyle
* WIP: try to find proper oracle for renaming absolute references,
without success so far.
Commit: 46f3452fd035ccd113bd2d8137f214b8655a0d38
https://github.com/xwiki/xwiki-platform/commit/46f3452fd035ccd113bd2d8137f2…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultReferenceUpdaterTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/ResourceReferenceRenamerTest.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacroRefactoring.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Find proper conditions to perform or not link renames
* Fix unit tests to add missing conditions
* WIP: need to fix coverage and check on subwikis / with more
conditions (e.g. with holes in hierarchy)
Commit: 6a1aa5e085565fb44436d45d7cec38ce80446692
https://github.com/xwiki/xwiki-platform/commit/6a1aa5e085565fb44436d45d7cec…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker/src/test/it/org/xwiki/flamingo/test/docker/RenamePageIT.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultModelBridgeTest.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Fix a regression and provide a test to cover it
Commit: 5f2f34bcf40ba276869675fca4b43d5c208c8fe5
https://github.com/xwiki/xwiki-platform/commit/5f2f34bcf40ba276869675fca4b4…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker/src/test/it/org/xwiki/flamingo/test/docker/RenamePageIT.java
M
xwiki-platform-core/xwiki-platform-wiki/xwiki-platform-wiki-test/xwiki-platform-wiki-test-docker/src/test/it/org/xwiki/wiki/test/ui/SubWikiIT.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Provide subwiki integration tests
* Minor improvment in RenamePageIT
Commit: 4a6cb235583389021bc4badf9f0efcdeff35f5fc
https://github.com/xwiki/xwiki-platform/commit/4a6cb235583389021bc4badf9f0e…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-wiki/xwiki-platform-wiki-test/xwiki-platform-wiki-test-docker/src/test/it/org/xwiki/wiki/test/ui/SubWikiIT.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Improve SubWikiIT to add more checks
Commit: 022c1e244565b0f51115dfc5f65ecc7fccbac0a2
https://github.com/xwiki/xwiki-platform/commit/022c1e244565b0f51115dfc5f65e…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-test/xwiki-platform-flamingo-skin-test-docker/src/test/it/org/xwiki/flamingo/test/docker/RenamePageIT.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/AbstractCopyOrMoveJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/CopyJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/MoveJob.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Few improvments following review
Commit: b67c19779c5b8b09df1f30b7c013e5663c5585e5
https://github.com/xwiki/xwiki-platform/commit/b67c19779c5b8b09df1f30b7c013…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M xwiki-platform-core/xwiki-platform-oldcore/src/main/java/com/xpn/xwiki/XWiki.java
M
xwiki-platform-core/xwiki-platform-oldcore/src/main/java/com/xpn/xwiki/internal/render/DefaultOldRendering.java
M
xwiki-platform-core/xwiki-platform-oldcore/src/test/java/com/xpn/xwiki/XWikiTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/ReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/ReferenceUpdater.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/AbstractCopyOrMoveJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/AbstractEntityJobWithChecks.java
A
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/InternalCopyOrMoveJobException.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/job/RenameJob.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/internal/listener/BackLinkUpdaterListener.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/main/java/org/xwiki/refactoring/job/question/EntitySelection.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/test/java/org/xwiki/refactoring/internal/job/MoveJobTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/test/java/org/xwiki/refactoring/internal/job/RenameJobTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-api/src/test/java/org/xwiki/refactoring/internal/listener/BackLinkUpdaterListenerTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultMacroRefactoring.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/DefaultReferenceUpdater.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/main/java/org/xwiki/refactoring/internal/ResourceReferenceRenamer.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultMacroRefactoringTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultReferenceUpdaterTest.java
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/ResourceReferenceRenamerTest.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacroRefactoring.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/include/IncludeMacroRefactoringTest.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-xwiki/src/main/java/org/xwiki/rendering/macro/MacroRefactoring.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Change APIs to use a Map<EntityReference, EntityReference>
corresponding to the source and target of refactorings in renamers
* Change some logic of AbstractCopyOrMoveJob to compute the actual
couple source/destination before performing any operation and store
the info in EntitySelection
* Add a log in RenameJob if it's not executed because of the number of
entities (not needed for this issue, but felt better to understand
what's happening)
Commit: 7a9cdfe228b511c286f2c8ac5a357a791a299e7c
https://github.com/xwiki/xwiki-platform/commit/7a9cdfe228b511c286f2c8ac5a35…
Author: Simon Urli <simon.urli(a)xwiki.com>
Date: 2024-11-08 (Fri, 08 Nov 2024)
Changed paths:
M
xwiki-platform-core/xwiki-platform-refactoring/xwiki-platform-refactoring-default/src/test/java/org/xwiki/refactoring/internal/DefaultModelBridgeTest.java
M
xwiki-platform-core/xwiki-platform-rendering/xwiki-platform-rendering-macros/xwiki-platform-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/include/IncludeMacroRefactoringTest.java
Log Message:
-----------
XWIKI-12987: Relative links are made absolute or even broken after moving a page
* Fix remaining coverage problems
Compare: https://github.com/xwiki/xwiki-platform/compare/1d8b51bfcace...7a9cdfe228b5
To unsubscribe from these emails, change your notification settings at
https://github.com/xwiki/xwiki-platform/settings/notifications