[LIVECOPY-312] getReferencedLiveCopyNodeById does not match the correct candidate Created: 12/Apr/22 Updated: 13/Oct/22 Resolved: 12/Oct/22 |
|
| Status: | Closed |
| Project: | Live Copy |
| Component/s: | None |
| Affects Version/s: | 3.2.7 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Medium |
| Reporter: | Horst Loepprich | Assignee: | Andrei Ichimescu |
| Resolution: | Cannot Reproduce | Votes: | 0 |
| Labels: | Livecopy | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Date of First Response: | |||||||||
| Epic Link: | AuthorX Support | ||||||||
| Team: | |||||||||
| Description |
|
AS-IS:
A possible solution for the filter would be to remove the site from the path before comparison:
Node resultNode = candidates.stream()
.filter(n -> Exceptions.sneak().get(() ->
masterPath.replaceFirst("^/\\w*", "")
.equals(n.getPath().replaceFirst("^/\\w*", ""))
))
.findFirst()
.orElse(candidates.get(0));
But this will only work if the rest of the path is the same as the master. Another solution would be to skip the further processing of the node if the children do not match the master and display a warning to the user that a node was skipped. Currently the whole push process just fails with an error. |
| Comments |
| Comment by Jesus Alonso [ 09/Sep/22 ] |
|
Hello hloepprich , would you mind to detail the steps to reproduce this issue? |
| Comment by Horst Loepprich [ 19/Sep/22 ] |
|
Hi Jesus, this error occurred when our customer copied a component from a live copy page to a non-live copy page, but we only noticed this later. The behavior was always reproducible through this process. However, the Pages-App must be configured to allow copying between pages. |
| Comment by Laura Delnevo [ 12/Oct/22 ] |
|
Thank you hloepprich for your time today. As we couldn't replicate the issue anymore (on 6.2.8), as agreed, we are closing the ticket as "Cannot Reproduce". // cc aichimescu jalonso miruela fmangold
|