Uploaded image for project: 'Magnolia Diff Module'
  1. Magnolia Diff Module
  2. MGNLDIFF-103

Show changes fails silenty in some cases

    XMLWordPrintable

Details

    • Bug
    • Resolution: Won't Fix
    • Neutral
    • None
    • 2.0
    • Magnolia MKG subscription package, Corp website

    Description

      As described in CLOUDHELP-86, compare Versions in the Pages app displays only a blank screen.

      eg. https://author-live-magnolia-mkg.de.magnolia-now.com/.magnolia/admincentral#app:pages:diff;/.magnolia/versionDiff?diffViewContentHandle=/www-en-intl&fromVersionNr=1.81&toVersionNr=1.79&diffLocale=en

      After investigation, we recognized that underlying FTL was changed, which caused some errors when displaying previous versioned content.

      "if you change its layout and its underlying implementation, the previous version cannot be displayed anymore, then we can't compare."

      Basically currently we just provided 'content' comparison, not totally page comparison. Also rendering errors didn't displayed properly --> this should be fixed first.

      Also below information might help when fixing this issue:

      yellow freemarker errors are being displayed only on author instances out of the box. To be sure, I’ve just tested and it works. But they are displayed only if 'mgnlPreview' is set to false, which is standard behaviour in Pages app but 'compare versions' functionality comparing a page (probably) against a page in preview mode.

      Without freemarker error: https://author-live-magnolia-mkg.de.magnolia-now.com/www-en-intl/?mgnlVersion=1.82
      With freemarker error (see the last parameter) https://author-live-magnolia-mkg.de.magnolia-now.com/www-en-intl/?mgnlVersion=1.82&mgnlPreview=false

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              Unassigned Unassigned
              viet.nguyen Viet Nguyen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD