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

Add siteaccess to debug tool bar

        Details

        • Type: Story Story
        • Status: Closed
        • Priority: Medium Medium
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 1.7.0
        • Component/s: Symfony
        • Labels:
          None

          Description

          Make it easy to see what siteaccess and matching type where used in the debug tool bar

            Issue Links

            relates to

            Task - A task that needs to be done. EZP-22496 Document the Web Debug Toolbar

            • High - Major loss of functionality.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Bertrand Dunogier added a comment -

              QA, could you please test this ? I'd try different siteaccess matching methods, and think about places where a siteaccess switch might occur (preview ?).

              One change was requested, but it's only phpdoc, so please consider it approved

              Show
              Bertrand Dunogier added a comment - QA, could you please test this ? I'd try different siteaccess matching methods, and think about places where a siteaccess switch might occur (preview ?). One change was requested, but it's only phpdoc, so please consider it approved
              Hide
              Permalink
              Rui Silva (Inactive) added a comment -

              PR tested and approved by QA for master.
              Sending back to dev needed so merge can be done.
              Please close issue afterwards.

              Show
              Rui Silva (Inactive) added a comment - PR tested and approved by QA for master. Sending back to dev needed so merge can be done. Please close issue afterwards.
              Hide
              Permalink
              Bertrand Dunogier added a comment -

              I'd like this to be tested a bit more thoroughly, with at least an uri and a host mapping.

              Show
              Bertrand Dunogier added a comment - I'd like this to be tested a bit more thoroughly, with at least an uri and a host mapping.
              Hide
              Permalink
              Rui Silva (Inactive) added a comment -

              Bertrand Dunogier, QA's test case updated accordingly to your request.

              Show
              Rui Silva (Inactive) added a comment - Bertrand Dunogier , QA's test case updated accordingly to your request.
              Hide
              Permalink
              Bertrand Dunogier added a comment -

              Do we really expect "default" to be the matching type at step 6 ? I'd expect the value to be "uri:map". At step 11, I'd expect it to be "host:map" and not "uri:map".

              Show
              Bertrand Dunogier added a comment - Do we really expect "default" to be the matching type at step 6 ? I'd expect the value to be "uri:map". At step 11, I'd expect it to be "host:map" and not "uri:map".
              Hide
              Permalink
              Rui Silva (Inactive) added a comment -

              You are correct. It was wrongly set that way due to faulty copy-paste on the test-case (yes, testlink is messy when it comes to copy-pasting procedures..).
              Thanks.

              Show
              Rui Silva (Inactive) added a comment - You are correct. It was wrongly set that way due to faulty copy-paste on the test-case (yes, testlink is messy when it comes to copy-pasting procedures..). Thanks.
              Hide
              Permalink
              Bertrand Dunogier added a comment -

              Merged to ezsystems/ezpublish-kernel master@e1864c6 (v1.7.0).

              Show
              Bertrand Dunogier added a comment - Merged to ezsystems/ezpublish-kernel master@e1864c6 (v1.7.0).
              Hide
              Permalink
              Bertrand Dunogier added a comment -

              @doc: if we have screenshots of the Sf toolbar, we might wanna update them with this entry added.

              Show
              Bertrand Dunogier added a comment - @doc: if we have screenshots of the Sf toolbar, we might wanna update them with this entry added.
              Hide
              Permalink
              Paulo Nunes (Inactive) added a comment -

              Verified on master after merge

              QA Approved

              Show
              Paulo Nunes (Inactive) added a comment - Verified on master after merge QA Approved

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Douglas Hammond
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: