Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-24279

liip_imagine variation settings are not set in filter configuration

    Details

      Description

      When defining settings for image variations, under liip_imagine block,
      these are not applied to existing image variations.

      For example, for the following ezpublish.yml config:

      liip_imagine:
          driver: imagick
          filter_sets:
              campaign:
                  jpeg_quality: 50
      

      The existing "campaign" image variation will not have the option(s) applied.

        Issue Links

          Activity

          Joao Inacio (Inactive) created issue -
          Joao Inacio (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to EZP-24167 [ EZP-24167 ]
          Joao Inacio (Inactive) made changes -
          Description When defining settings for image variations, under {{liip_imagine}} block,
          these are not applied to the (eZ) image variations.

          For example, for the following ezpublish.yml config:
          {code}
          liip_imagine:
              driver: imagick
              filter_sets:
                  campaign:
                      jpeg_quality: 50
          {code}

          The existing "campaign" image variation will not have the option(s) applied.
          When defining settings for image variations, under {{liip_imagine}} block,
          these are not applied to existing image variations.

          For example, for the following ezpublish.yml config:
          {code}
          liip_imagine:
              driver: imagick
              filter_sets:
                  campaign:
                      jpeg_quality: 50
          {code}

          The existing "campaign" image variation will not have the option(s) applied.
          Joao Inacio (Inactive) made changes -
          Status Open [ 1 ] Confirmed [ 10037 ]
          Show
          Joao Inacio (Inactive) added a comment - Suggested PR: https://github.com/ezsystems/ezpublish-kernel/pull/1232
          Yannick Roger (Inactive) made changes -
          Fix Version/s Customer request [ 11018 ]
          Yannick Roger (Inactive) made changes -
          Status Confirmed [ 10037 ] InputQ [ 10001 ]
          Yannick Roger (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ]
          Yannick Roger (Inactive) made changes -
          Status Development [ 3 ] Development Review [ 10006 ]
          Yannick Roger (Inactive) made changes -
          Assignee Yannick Roger [ yannick.roger@ez.no ] Joao Inacio [ joao.inacio@ez.no ]
          Yannick Roger (Inactive) made changes -
          Fix Version/s Engineering tracked issues [ 11179 ]
          Fix Version/s Customer request [ 11018 ]
          Joao Inacio (Inactive) made changes -
          Fix Version/s Customer request [ 11018 ]
          Show
          Joao Inacio (Inactive) added a comment - Merged in master: https://github.com/ezsystems/ezpublish-kernel/commit/3d6e48db6320526f572c8ef320b41197aa89d795
          Joao Inacio (Inactive) made changes -
          Status Development Review [ 10006 ] Development Review done [ 10028 ]
          Affects Version/s 5.4.2 [ 13979 ]
          Joao Inacio (Inactive) made changes -
          Status Development Review done [ 10028 ] Development Acceptance Done [ 10030 ]
          Fix Version/s 2015.03 [ 14179 ]
          Fix Version/s 5.4.3 [ 14184 ]
          Joao Inacio (Inactive) made changes -
          Link This issue blocks EZP-24167 [ EZP-24167 ]
          Joao Inacio (Inactive) made changes -
          Link This issue relates to EZP-24167 [ EZP-24167 ]
          Yannick Roger (Inactive) made changes -
          Assignee Joao Inacio [ joao.inacio@ez.no ] Yannick Roger [ yannick.roger@ez.no ]
          Yannick Roger (Inactive) made changes -
          Status Development Acceptance Done [ 10030 ] Documentation Review done [ 10011 ]
          Affects Version/s 2015.01 [ 13680 ]
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ] Rui Silva [ rui.silva@ez.no ]
          Joao Inacio (Inactive) made changes -
          Component/s Documentation [ 10793 ]
          Rui Silva (Inactive) made changes -
          Status QA [ 10008 ] Development Acceptance Done [ 10030 ]
          Rui Silva (Inactive) made changes -
          Assignee Rui Silva [ rui.silva@ez.no ]
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Status Development Acceptance Done [ 10030 ] Documentation [ 10010 ]
          Assignee Sarah Haïm-Lubczanski [ sarah.haim-lubczanski@ez.no ]
          Hide
          Sarah Haïm-Lubczanski (Inactive) added a comment -

          Sorry for not answering, I see this issue just today, because you mentionned it during the Daily Meeting.

          A good example would be config block on the description.

          I don't understand this sentence. Do you prefer some example with configuration for all LiipBundle filters ?

          Show
          Sarah Haïm-Lubczanski (Inactive) added a comment - Sorry for not answering, I see this issue just today, because you mentionned it during the Daily Meeting. A good example would be config block on the description. I don't understand this sentence. Do you prefer some example with configuration for all LiipBundle filters ?
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Status Documentation [ 10010 ] Documentation Review done [ 10011 ]
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Assignee Sarah Haïm-Lubczanski [ sarah.haim-lubczanski@ez.no ] Rui Silva [ rui.silva@ez.no ]
          Rui Silva (Inactive) made changes -
          Status QA [ 10008 ] Development Acceptance Done [ 10030 ]
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Status Development Acceptance Done [ 10030 ] Documentation [ 10010 ]
          Assignee Rui Silva [ rui.silva@ez.no ] Sarah Haïm-Lubczanski [ sarah.haim-lubczanski@ez.no ]
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Status Documentation [ 10010 ] Documentation Review done [ 10011 ]
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Assignee Sarah Haïm-Lubczanski [ sarah.haim-lubczanski@ez.no ] Rui Silva [ rui.silva@ez.no ]
          Hide
          Rui Silva (Inactive) added a comment -

          Approved by QA.
          [~sarah.haim-lubczanski@ez.no], I myself altered the name of an image variation in an example you provided, for consistency, i.e., to show that the same variation can have settings both from ezpublish.yml and liip configuration, at the same time (which was the focus of João's comment, last paragraph).

          Show
          Rui Silva (Inactive) added a comment - Approved by QA. [~sarah.haim-lubczanski@ez.no] , I myself altered the name of an image variation in an example you provided, for consistency, i.e., to show that the same variation can have settings both from ezpublish.yml and liip configuration, at the same time (which was the focus of João's comment, last paragraph).
          Rui Silva (Inactive) made changes -
          Assignee Rui Silva [ rui.silva@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          André Rømcke made changes -
          Workflow eZ Engineering Scrumban Workflow [ 67207 ] EZ* Development Workflow [ 85822 ]
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 85822 ] EZEE Development Workflow [ 124745 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          1m 7s 1 joao.inacio@ez.no 16/Apr/15 5:47 PM
          Confirmed Confirmed InputQ InputQ
          22h 33m 1 yannick.roger@ez.no 17/Apr/15 4:21 PM
          InputQ InputQ Development Development
          5s 1 yannick.roger@ez.no 17/Apr/15 4:21 PM
          Development Development Development Review Development Review
          5s 1 yannick.roger@ez.no 17/Apr/15 4:21 PM
          Development Review Development Review Development Review done Development Review done
          9d 19h 20m 1 joao.inacio@ez.no 27/Apr/15 11:41 AM
          Development Review done Development Review done Removed Status Removed Status
          5m 49s 1 joao.inacio@ez.no 27/Apr/15 11:47 AM
          Removed Status Removed Status Documentation Review done Documentation Review done
          1d 22h 8m 1 yannick.roger@ez.no 29/Apr/15 9:55 AM
          QA QA Removed Status Removed Status
          2h 6m 2 rui.silva@ez.no 12/May/15 3:59 PM
          Removed Status Removed Status Documentation Documentation
          18d 1h 15m 2 sarah.haim-lubczanski@ez.no 18/May/15 5:05 PM
          Documentation Documentation Documentation Review done Documentation Review done
          1d 3h 29m 2 sarah.haim-lubczanski@ez.no 18/May/15 5:05 PM
          Documentation Review done Documentation Review done QA QA
          19h 28m 3 rui.silva@ez.no 19/May/15 12:15 PM
          QA QA Closed Closed
          16m 30s 1 rui.silva@ez.no 19/May/15 12:31 PM

            People

            • Assignee:
              Unassigned
              Reporter:
              Joao Inacio (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: