COU-349 - RTE images do not get added as dependencies

Created by Claus Jensen 14 Jun 2016, 15:50:09 Updated by Morten Christensen 24 Jun 2016, 11:03:36

Depends on: U4-6551

Dependencies are not resolved for images inserted into the RTE so they need to deployed manually.

Comments

Claus Jensen 14 Jun 2016, 17:04:12

PR: https://github.com/umbraco/UmbracoDeploy/pull/42

Note that if this is tested in a recent version of Umbraco, U4-6551 needs to be merged first. Core is not able to resolve images from a path, if the umbracoFile data is stored in the newer format.


Morten Christensen 20 Jun 2016, 07:59:18

Should we then have a version flag if it doesn't work prior to 7.2.2 ? Or maybe it doesn't matter?


Claus Jensen 20 Jun 2016, 08:02:15

If the image is stored "the old way" (old media picker) it will resolve. It will however not resolve with the new way of storing images, unless you have the fix above merged in.

No errors when it doesn't find anything - it just won't add the dependency.

Edit: and it's not only 7.2.2 - its also anything newer, the bug just wasn't flagged for latest versions :)


Morten Christensen 20 Jun 2016, 12:45:57

This doesn't appear to work for me on https://www.s1.umbraco.io/project/cg16-workshop-v2

Steps:

  1. In http://dev-cg16-workshop-v2.s1.umbraco.io/ upload an image in the Media section, add that image to the Content page through the rich text editor (Media picker).
  2. Queue for transfer - only content item
  3. Check http://cg16-workshop-v2.s1.umbraco.io/ Result image was not transferred. Only the Content item and its property


Morten Christensen 20 Jun 2016, 12:49:21

Error in log file:

2016-06-20 12:41:00,055 [47] WARN  Umbraco.Courier.DataResolvers.PropertyDataResolvers.RTEImages - [Thread 109] Could not find media from url: '/media/1003/installpackageumbracocms.png' (property 'bodyText' on '9fe55214-3237-42e6-a10c-4f8048228394')
2016-06-20 12:41:00,087 [47] WARN  Umbraco.Courier.DataResolvers.PropertyDataResolvers.RTEImages - [Thread 109] Could not find media from url: '/media/1003/installpackageumbracocms.png' (property 'bodyText' on '9fe55214-3237-42e6-a10c-4f8048228394')


Morten Christensen 24 Jun 2016, 11:01:31

Pulling this in now that 2.52.9 has been released


Priority: Normal

Type: Bug

State: Fixed

Assignee:

Difficulty:

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions: 2.52.7

Due in version: 2.52.10

Sprint: Sprint 36

Story Points:

Cycle: