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

Legacy installer should allow utf8mb4 charset

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Medium Medium
    • Resolution: Done
    • Affects Version/s: 5.3.12, 5.4.12
    • Fix Version/s: 2017.12.3, 5.4.13
    • Component/s: Database related
    • Labels:
      None

      Description

      When creating a database in utf8mb4, the legacy installer complains that only utf-8 is supported. Since utf8mb4 is recommended now we should support both.

        Issue Links

          Activity

          Show
          Gunnstein Lye added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1386
          Hide
          Gunnstein Lye added a comment - - edited

          QA reports broken content type icons, gateway timeout when publishing a folder, and transaction error when creating an article. See screenshots.

          This may be related, needs to be tested. https://github.com/ezsystems/ezpublish-kernel/commit/b2a1ffd65a7792b541f170bb684cdc82f78ab4be#diff-079518e72cd553a190335b8aeaaf92de

          See also https://github.com/ezsystems/ezpublish-legacy-ee/commit/f475f5c9032eaf395adf9429927880d7357a3d9d

          Show
          Gunnstein Lye added a comment - - edited QA reports broken content type icons, gateway timeout when publishing a folder, and transaction error when creating an article. See screenshots. This may be related, needs to be tested. https://github.com/ezsystems/ezpublish-kernel/commit/b2a1ffd65a7792b541f170bb684cdc82f78ab4be#diff-079518e72cd553a190335b8aeaaf92de See also https://github.com/ezsystems/ezpublish-legacy-ee/commit/f475f5c9032eaf395adf9429927880d7357a3d9d
          Hide
          Michał Szołtysek added a comment -

          Tried again with v5.4.12 final which already includes https://github.com/ezsystems/ezpublish-legacy-ee/commit/f475f5c9032eaf395adf9429927880d7357a3d9d.

          Added the change to Charset in site.ini under DatabaseSettings from https://github.com/ezsystems/ezpublish-kernel/commit/b2a1ffd65a7792b541f170bb684cdc82f78ab4be#diff-079518e72cd553a190335b8aeaaf92de.

          Followed the steps like in the PR.
          (also tried changing charset in ezplatform.yml and i18n.ini.)

          Still having issues with timeouts when publishing (although drafts are created in the process) and icons on Sub items list.

          Maybe I'm missing some layer which needs to be reloaded?
          Tried in UI Setup / Cache management / Clear all caches and on server side from CLI (assets:install, ezpublish:legacy:assets_install, assetic:dump, cache:clear).

          Show
          Michał Szołtysek added a comment - Tried again with v5.4.12 final which already includes https://github.com/ezsystems/ezpublish-legacy-ee/commit/f475f5c9032eaf395adf9429927880d7357a3d9d . Added the change to Charset in site.ini under DatabaseSettings from https://github.com/ezsystems/ezpublish-kernel/commit/b2a1ffd65a7792b541f170bb684cdc82f78ab4be#diff-079518e72cd553a190335b8aeaaf92de . Followed the steps like in the PR. (also tried changing charset in ezplatform.yml and i18n.ini.) Still having issues with timeouts when publishing (although drafts are created in the process) and icons on Sub items list. Maybe I'm missing some layer which needs to be reloaded? Tried in UI Setup / Cache management / Clear all caches and on server side from CLI (assets:install, ezpublish:legacy:assets_install, assetic:dump, cache:clear).
          Hide
          Michał Szołtysek added a comment -

          QA Approved.

          Show
          Michał Szołtysek added a comment - QA Approved.
          Show
          Gunnstein Lye added a comment - Merged in master, 2018.09, 2017.12: https://github.com/ezsystems/ezpublish-legacy/commit/b9435eb1521f763900ee8faab5b296f4b3e496ee

            People

            • Assignee:
              Unassigned
              Reporter:
              Gunnstein Lye
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 hours, 30 minutes
                3h 30m