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

niceurls create wrong path on multiple siteaccess configuration

    Details

      Description

      • two languages, english and spanish for example
      • two siteaccess, site-eng and site-esp.
        SA site-eng has english and spanish in site.ini declaration
        SA site-esp has spanish and english in site.ini declaration
        (respect the language priority order)
      • 1 folder under root - people/personas (titles for each translation)
      • 2 articles under the folder: 1 in english titled "Jack", 1 in spanish titled "Julio"

      in eng siteaccess browse tro root, then to the "people" folder. verify these urls:
      site.com/people/jack OK
      site.com/personas/julio BAD

      in spanish siteaccess navigation gives these:
      site.com/people/jack BAD
      site.com/personas/julio OK

      so, the case is simple: if i'm on site-eng SA, and i have jack, john, james and julio the urls are:
      people/jack
      people/john
      people/james
      personas/julio

      the last url should resolve to: people/julio

      additional notes:

      • people/julio is accesible typing the direct url
      • an object that only exists in one language appears to "force" the SA to use the object's parent with the same translation

        Issue Links

          Activity

          Hide
          Patrick Allaert (Inactive) added a comment -

          Issue has been introduced in the fix for EZP-17632.

          Show
          Patrick Allaert (Inactive) added a comment - Issue has been introduced in the fix for EZP-17632 .
          Hide
          Patrick Allaert (Inactive) added a comment -
          Show
          Patrick Allaert (Inactive) added a comment - Pull request available at: https://github.com/ezsystems/ezpublish-legacy/pull/572
          Hide
          Paulo Bras (Inactive) added a comment -

          tested the patch and it fixes the issue

          Show
          Paulo Bras (Inactive) added a comment - tested the patch and it fixes the issue
          Show
          Patrick Allaert (Inactive) added a comment - - edited Fixed in: eZ Publish master (merge commit): https://github.com/ezsystems/ezpublish-legacy/commit/405c47193ac87ddcb66677dea7bd144c6d5e2d5b eZ Publish master: https://github.com/ezsystems/ezpublish-legacy/commit/5bb176bc3b86208fe0477cfb238f6f396de6512e + Fixed (syntax error) in: eZ Publish master: https://github.com/ezsystems/ezpublish-legacy/commit/fc9ca9c65f3455e36e1c49f34335b81b2a2ff41a
          Hide
          Patrick Allaert (Inactive) added a comment -

          Merging and backporting.

          Show
          Patrick Allaert (Inactive) added a comment - Merging and backporting.
          Hide
          Joao Pingo (Inactive) added a comment -

          Tested on master and 4.5 using tc-1539
          Test passed ... QA Approved

          Show
          Joao Pingo (Inactive) added a comment - Tested on master and 4.5 using tc-1539 Test passed ... QA Approved
          Hide
          Gunnstein Lye added a comment -

          Note: This is reverted in EZP-23753

          Show
          Gunnstein Lye added a comment - Note: This is reverted in EZP-23753

            People

            • Assignee:
              Unassigned
              Reporter:
              Paulo Bras (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              4 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 - 1 day, 4 hours, 20 minutes
                1d 4h 20m