[MAGNOLIA-6977] CK editor image not updated after editing it in Assets app Created: 15/Feb/16  Updated: 17/Mar/17  Resolved: 17/Mar/17

Status: Closed
Project: Magnolia
Component/s: None
Affects Version/s: 5.4.4
Fix Version/s: 5.5.3

Type: Improvement Priority: Neutral
Reporter: Ondrej Chytil Assignee: Evzen Fochr
Resolution: Fixed Votes: 0
Labels: support
Remaining Estimate: 7h
Time Spent: 3d 1h
Original Estimate: 4d

Issue Links:
Relates
relates to MGNLCACHE-164 Browser cache policy "never" is too b... Closed
causality
caused by MGNLDAM-568 ImagingBasedAssetRenderer should not ... Closed
dependency
is depended upon by MGNLDAM-698 Remove usage of private IMAGE_PATTERN Closed
relation
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)
Date of First Response:
Sprint: Kromeriz 87
Story Points: 8

 Description   

Steps to reproduce:

  • allow images in CK editor at Text & Image component
  • add component to a page and select any image from dam workspace
  • crop the image in Assets app and save
  • reload the page and see the image, it's unchanged


 Comments   
Comment by Ngoc Nguyenthanh [ 02/Mar/16 ]

First look
Reproduce steps

  • Open 2 browser tabs.
  • One open PageEditor of a page which have textImage component.
  • Another one open an Asset app.
  • Select an image in textImage component
  • Open the image and edit in Asset app
  • Back to the page editor tab. Try to refresh, the image is uncharged.

What's happen?

  • Cache module use SetExpirationHeaders in order to set expiration date.
  • The page editor use IFrame in order to display the preview page
  • The IFrame didn't reload completely. It still use the old image with the expiration date set.

Observed behaviors

  • Refresh by using hard refresh doesn't show a new image.
  • Refresh the preview iframe by using browser menu context "Reload frame" in order to get new image.
  • Open the image in the new tab and back to page editor and press hard refresh can get new image also.
Comment by Mikaël Geljić [ 14/Mar/16 ]

To address this automatically, this requires further research, proposal, and most likely API changes.

Meanwhile, I can see two potential ways to work around that:

  • tuning cache configuration on the author (ochytil is this an option for this support case, by any chance?)
  • "reloading frame" is a workaround for some browsers which enable it.
Comment by Roman Kovařík [ 03/Feb/17 ]

A fingerprint should be added to assets links as the resource servlet does. This issue is probably caused by MGNLDAM-568.

Generated at Mon Feb 12 04:19:35 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.