[MAGNOLIA-4383] Images uploaded in fckEdit show 404 - Not Found Created: 10/Jan/12  Updated: 22/Nov/16  Resolved: 02/May/12

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

Type: Bug Priority: Neutral
Reporter: Marco Glur Assignee: Milan Divilek
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified
Environment:

demoauthor.magnolia-cms.com


Attachments: XML File config.modules.data.dialogs.Contact.personalTab.html.xml     XML File data.contacts.Test.xml    
Issue Links:
causality
is causing MAGNOLIA-6882 Remove legacy code that allows to acc... 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 adding an fckEdit Control with enabled 'images' configured, the Uploaded File in the FCK Image-Dialog is lost after saving the dialog.

Reproduce

  • Extend the default Contact data-type dialog e.g. importing the attached XML File (personalTab)
  • Create a new Contact
  • Close to be sure a node now exists now
  • Open the Contact again
  • Insert an image using the toolbars
  • Navigate to the 'Upload' Tab in the image dialog
  • Choose a file from the local file system
  • Send the file to server
    • The image is inserted in the editor (still in tmp-folder)
  • Close the Dialog (save)
  • Re-Open the Dialog
    • The image is not found (404)
    • Referenc should resolve to UUID or handle


 Comments   
Comment by Ondrej Chytil [ 23/Jan/12 ]

Thanks for reporting this issue Marco. It should be fixed for the next Data module version.

Comment by Jan Haderka [ 26/Apr/12 ]

This issue is still not completely fixed. URI2RepoMapping should be used to resolve repo 2 uri mapping rather then passing repo name as part of uri directly.

Comment by Milan Divilek [ 02/May/12 ]

Test for info.magnolia.module.admininterface.SaveHandlerImplTest.java isn't backported, because info.magnolia.test.mock.MockUtil works differently in 4.5 and 4.4

Comment by Marco Glur [ 17/Jul/12 ]

For me, still the image (binary) is lost, while the Export shows there is a file0 node with a jcr:data does exist according the JCR-Export XML but it is never found afterwards when re-opening the dialog.
I could still reproduce this on the demoauthor.

Comment by Marco Glur [ 17/Jul/12 ]

For me it is not fixed on demo-author, yet.

Comment by Jan Haderka [ 17/Jul/12 ]

Hi Marco,

demoauthor is still running on 4.5.2. This issue have been fixed in version 4.5.3.

Regards,
Jan

Comment by Marco Glur [ 17/Jul/12 ]

Oops, OK thanks for the hint. In 4.4.8 it looks just fine now after re-testing!

Best regards,
Marco

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