[MAGNOLIA-3972] Image broken links when using both DMS referencing and user upload Created: 20/Feb/12  Updated: 21/Aug/13  Resolved: 03/May/12

Status: Closed
Project: Magnolia
Component/s: fckeditor
Affects Version/s: 4.4.6, 4.5.2
Fix Version/s: 4.4.7, 4.5.3

Type: Bug Priority: Neutral
Reporter: Matteo Pelucco Assignee: Ondrej Chytil
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

standard magnolia installation on windows/linux


Attachments: JPEG File ImageImportInRichTextArea.jpg.rb.jpg    
Issue Links:
Cloners
is cloned by MGNLETK-78 CLONE -Image broken links when using ... Closed
causality
relation
is related to MAGNOLIA-4433 Templating samples don't render images Closed
is related to MAGNOLIA-5250 fckEditor: make repository browser si... Closed
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:

 Description   

When using FCK editor image button, there are broken links for images.
Reproduce with:

1) Upload an image to DMS
2) Open a page, new paragraph "textImage"
3) In FCK, upload a new image from your PC. The link is correct (e.g.: /magnoliaAuthor/tmp/fckeditor/22ba03f1-5b9b-11e1-834c-df984c86872c/banner_FlashSubstitute.jpg)
4) Now save and re-edit
5) In FCK, reference a DMS image (see 1.). The previous correct link now is broken (e.g.: /test/test-section/text/mainArea/013/text_files/file/banner_FlashSubstitute.jpg) but the second (dms) is working.

probably related to: http://jira.magnolia-cms.com/browse/MGNLDATA-143

Update from SUPPORT-1566:

IMPORTANT: use magnolia with site recognition based on domain mapping. Otherwise the bug is not visible.


 Comments   
Comment by Jan Haderka [ 02/May/12 ]

there's more elegant way to set boolean props then using if/else:

 uuidLink.setEditor(uuidLink.getNodeData()!=null);

instead of

      if(uuidLink.getNodeData()!=null){
          uuidLink.setEditor(true);
      }

But more importantly, you can't modify object that was passed in. It might be referenced elsewhere.

Generated at Mon Feb 12 03:51:27 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.