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

eZ DFS sql error storing image files when path contains apostrophe (urlalias_iri)

    Details

      Description

      When using URL transformation group "//urlalias_iri//" (site.ini, [URLTranslator]),
      and a content folder created with an apostrophe, DFS operations on files beneath that node fail with an SQL error.

      Steps to reproduce
      • Set TransformationGroup=urlalias_iri (and clear .ini caches).
      • Create a folder object: speakers'
      • inside this folder, create an image object, and publish.

      Result:
      SQL errors will be produced, such as:

      [ Sep 19 2012 20:02:09 ] [192.168.56.1] _endCacheGeneration( var/ezflow_site/storage/images/tests/speakers'/image/9741-1-ger-DE/image.png ):
      Failed to execute SQL for function:
      1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '')' at line 1
      

        Issue Links

          Activity

          Show
          Jérôme Vieilledent (Inactive) added a comment - Additional fix in master : https://github.com/ezsystems/ezpublish/commit/f75fa30884e202197ff04bdfd861a0b406867957
          Hide
          Vidar Langseid added a comment -

          The is the certification platforms ( and what we therefore needs to test on )
          4.7 : dfs/db/single : mysqli
          4.6 : dfs/db/single : mysqli
          4.5 : dfs/db/single : mysqli
          4.4 : dfs/db : mysql
          single : mysqli
          4.3 : dfs/db/single : mysql
          4.2 : dfs/db/single : mysql

          Show
          Vidar Langseid added a comment - The is the certification platforms ( and what we therefore needs to test on ) 4.7 : dfs/db/single : mysqli 4.6 : dfs/db/single : mysqli 4.5 : dfs/db/single : mysqli 4.4 : dfs/db : mysql single : mysqli 4.3 : dfs/db/single : mysql 4.2 : dfs/db/single : mysql
          Hide
          Gunnstein Lye added a comment -

          Closing to reopen

          Show
          Gunnstein Lye added a comment - Closing to reopen
          Show
          Patrick Allaert (Inactive) added a comment - Additional PR: https://github.com/ezsystems/ezpublish/pull/527
          Hide
          Patrick Allaert (Inactive) added a comment -

          New patch to be reviewed

          Show
          Patrick Allaert (Inactive) added a comment - New patch to be reviewed
          Show
          Patrick Allaert (Inactive) added a comment - Additional fix in: eZ Publish master (merge commit): https://github.com/ezsystems/ezpublish/commit/42f98baaf04e583a369a5ba47bb5191044f64b49 eZ Publish master: https://github.com/ezsystems/ezpublish/commit/af2d86b10cf2a2bb1f8e93875f8ef8bc201e87b0
          Hide
          Joao Pingo (Inactive) added a comment -

          Tested with tc-1431
          All DFS cluster tests passed
          DB Cluster tests are now handled in https://jira.ez.no/browse/EZP-20901
          QA Done

          Show
          Joao Pingo (Inactive) added a comment - Tested with tc-1431 All DFS cluster tests passed DB Cluster tests are now handled in https://jira.ez.no/browse/EZP-20901 QA Done

            People

            • Assignee:
              Unassigned
              Reporter:
              João Inácio
            • Votes:
              0 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 4 weeks, 35 minutes
                4w 35m