Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-6527

Image embedded with RTE in the old pages app "broken" when opening component in new pages app

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 6.2.7
    • 6.2.5
    • None
    • Maintenance 43
    • 5

      The rich text editor has an option to insert images from the dam.

      When an image is selected and saved in past versions of Magnolia the format would look like this:

      <p>Hello World!</p><p><img alt="" src="${link:{uuid:{795449d2-6c16-402e-9e8a-bbb101fe22bb},repository:{dam},path:{/tours/shark_brian_warrick_0824.JPG},nodeData:{},extension:{2021-01-26-09-01-43.JPG}}}" style="height:866px; width:1197px" /></p>
      

      Now it's being stored like this:

      <p>Hello World!</p><p><img alt="" src="/dam/jcr:795449d2-6c16-402e-9e8a-bbb101fe22bb/shark_brian_warrick_0824.JPG" style="height:866px; width:1197px" /></p>
      

      Reproduce

      • Edit the dialog of mtk:components/textImage to use images.
      • Using the old pages app, create a textImage component which has an embedded image.
      • Using the new pages app, create a textImage component which has an embedded image.
      • Notice how the embedded image created in the old pages app is "broken" when opening the component in the new page app.
      • Also open the JCR Browser to compare the new format and the old format.

      Expected
      The format should have stayed the same between the two apps.

      Actual
      The format has changed which breaks installations coming from older versions of Magnolia. Anyone starting a new project on 6.2 will be embedding the new format.

      Workaround
      Projects using the old format will need to use the old pages when editing components of this type. Otherwise the image is not displayed in the editor. A white box is there representing the space where the image would be.

        Acceptance criteria

              fgrilli Federico Grilli
              rgange Richard Gange
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: