-
Bug
-
Resolution: Fixed
-
Major
-
1.2.2
This feature was implemented with MGNLDIFF-70. But there is problem with multisite environment.
DiffLink.getLocales() always return locales from default site because storageNode path start with "/activation/..." and not with actual site prefix.
Instead getOriginalContent() should be used to aquire the proper node.
To Reproduce:
- In demo-features site, enable i18n
- Add an additional locale, fr. (This is important because the problem only shows up when the locale is something that is not present in Default site definition.)
- Edit Page /demo-features/content-templates/article, Edit the title of the "Page Header" component.
- Make the english version 'Article EN 1', make the french version 'Article FR 1'.
- Publish the change & Go to the Inbox and Approve the workitem.
- Edit the page again
- Make the english version 'Article EN 2'. make the french version 'Article FR 2'.
- Publish the change.
- Go to the inbox, double click on the workitem, you'll see links at the bottom to Compare for each language.
- You'll see that English works correctly, but that French shows the English changes too!
Acceptance criteria
- depends upon
-
MGNLETK-129 ETKSiteManager#getAssignedSite always returns the Default site for locale prefixed uri's
- Closed
- is cloned by
-
MGNLDIFF-82 i18n aware comparison doesn't work in multisite environment
- Closed
- is related to
-
MGNLDIFF-70 Diff module is not i18n aware on single site with multiple languages
- Closed