• Type: Improvement
• Resolution: Fixed
• Priority: Major
• Fix Version/s: 3.7
• Affects Version/s: None
• Component/s: None
• Labels:

  None

1. WEB_AmetysXSLTHelper.skinImage.patch

   19/Feb/15 09:57

   3 kB

   Quentin Glinel-Mortreuil

2. WEB_ResourceNotExistsAction.patch

   25/Feb/15 16:51

   6 kB

   Quentin Glinel-Mortreuil

[CMS-6048] AmetysXSLTHelper.skinImage

Collapse comment: Laurence Aumeunier added a comment - 03/Mar/15 10:12

Laurence Aumeunier added a comment - 03/Mar/15 10:12

Do not work. Bad source resolver use in ResourceNotExistsAction

Expand comment: Laurence Aumeunier added a comment - 03/Mar/15 10:12

Laurence Aumeunier added a comment - 03/Mar/15 10:12 Do not work. Bad source resolver use in ResourceNotExistsAction

Collapse comment: Raphaël Franchet added a comment - 25/Feb/15 18:44

Raphaël Franchet added a comment - 25/Feb/15 18:44

errors in patch. see comit for corrections

Expand comment: Raphaël Franchet added a comment - 25/Feb/15 18:44

Raphaël Franchet added a comment - 25/Feb/15 18:44 errors in patch. see comit for corrections

Collapse comment: Laurence Aumeunier added a comment - 25/Feb/15 15:31, Edited by Laurence Aumeunier - 25/Feb/15 15:45

Laurence Aumeunier added a comment - 25/Feb/15 15:31 - edited

The new added pipelines should be protected:

• skins/*/resources/**_max*x*.*

• skins/*/resources/**_*x*.*

Indeed if we have an image named "ui-bg_glass_75_e6e6e6_1x400.png" or "input_text_right.gif " for exemple, it is not served anymore

=> Add a "resource-not-exists" action.

Expand comment: Laurence Aumeunier added a comment - 25/Feb/15 15:31, Edited by Laurence Aumeunier - 25/Feb/15 15:45

Laurence Aumeunier added a comment - 25/Feb/15 15:31 - edited The new added pipelines should be protected: skins/*/resources/**_max*x*.* skins/*/resources/**_*x*.* Indeed if we have an image named "ui-bg_glass_75_e6e6e6_1x400.png" or "input_text_right.gif " for exemple, it is not served anymore => Add a "resource-not-exists" action.

Collapse comment: Quentin Glinel-Mortreuil (Inactive) added a comment - 19/Feb/15 09:57

Quentin Glinel-Mortreuil (Inactive) added a comment - 19/Feb/15 09:57

Sure. The new patch gathers both skinImage and skinBoundedImage

Expand comment: Quentin Glinel-Mortreuil (Inactive) added a comment - 19/Feb/15 09:57

Quentin Glinel-Mortreuil (Inactive) added a comment - 19/Feb/15 09:57 Sure. The new patch gathers both skinImage and skinBoundedImage

Collapse comment: Raphaël Franchet added a comment - 19/Feb/15 09:48

Raphaël Franchet added a comment - 19/Feb/15 09:48

Could you also add the helper skinBoundedImage ?

Expand comment: Raphaël Franchet added a comment - 19/Feb/15 09:48

Raphaël Franchet added a comment - 19/Feb/15 09:48 Could you also add the helper skinBoundedImage ?