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

Policy(?) issue with index_treemenu breaks backoffice content tree when using legacy layer.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: High High
    • Resolution: Fixed
    • Affects Version/s: 5.0.0-dev
    • Component/s: None
    • Labels:
    • Environment:

      Database Server: mysql Ver 14.14 Distrib 5.1.61, for redhat-linux-gnu (x86_64) using readline 5.1
      OS: RHEL 6.2
      PHP: PHP 5.3.3

      Description

      The content tree to the left of the admin dashboard does not work when using the legacy layer.

      Looking at the network traffic, the request to content/treemenu/... returns a 500 Internal Server Error; through some poking around we've traced the origin of the error to line ~158 of index_treemenu.php:

      if ( !$hasAccessToSite )
      {
          exitWithInternalError();
          return;
      }

      The content tree works correctly if accessing the legacy installation directly (bypassing the symfony stack).

        Issue Links

          Activity

          Hide
          Jérôme Vieilledent (Inactive) added a comment -

          Workaround : remove or comment out the following rewrite rule in your apache vhost :

          RewriteRule ^/([^/]+/)?content/treemenu.* /index_treemenu\.php [L]

          Show
          Jérôme Vieilledent (Inactive) added a comment - Workaround : remove or comment out the following rewrite rule in your apache vhost : RewriteRule ^/([^/]+/)?content/treemenu.* /index_treemenu\.php [L]
          Hide
          Filipe Dobreira (Inactive) added a comment -

          I can confirm that removing the rewrite rule re-enables the tree menu. Re-enabling the rewrite rule will also not affect the tree menu until the cache is cleared.

          Show
          Filipe Dobreira (Inactive) added a comment - I can confirm that removing the rewrite rule re-enables the tree menu. Re-enabling the rewrite rule will also not affect the tree menu until the cache is cleared.
          Hide
          Jérôme Vieilledent (Inactive) added a comment -

          Steps to fix it for good :

          • Create a specific legacy kernel handler class in eZ Publish legacy reproducing code from index_treemenu.php
          • index_treemenu.php would then use ezpKernel like index.php.
          • Create a new controller in the Symfony stack for /content/treemenu to use the new treeMenuKernelHandler.

          With this solution the rewrite rule specified above won't be needed any more.

          Show
          Jérôme Vieilledent (Inactive) added a comment - Steps to fix it for good : Create a specific legacy kernel handler class in eZ Publish legacy reproducing code from index_treemenu.php index_treemenu.php would then use ezpKernel like index.php. Create a new controller in the Symfony stack for /content/treemenu to use the new treeMenuKernelHandler. With this solution the rewrite rule specified above won't be needed any more.
          Show
          Jérôme Vieilledent (Inactive) added a comment - Fixed in : eZ Publish legacy https://github.com/ezsystems/ezpublish/commit/da4113fdae02dc9c24a82b929e50c992a8d976b8 http://git.io/ooEV3Q https://github.com/ezsystems/ezpublish/commit/e969bd87fa02de1d0903cac2d74c305c66c256ec eZ Publish 5 https://github.com/ezsystems/ezp-next/compare/b91106df...89c99202 https://github.com/ezsystems/ezpublish5/commit/cb6ba2f2fd9fa550f89ce69593dfb1cef8d90d52
          Hide
          Jérôme Vieilledent (Inactive) added a comment -

          Please not that you don't need to use the rewrite rule any more for eZ Publish 5.

          Show
          Jérôme Vieilledent (Inactive) added a comment - Please not that you don't need to use the rewrite rule any more for eZ Publish 5.
          Hide
          Filipe Dobreira (Inactive) added a comment -

          QA approved.

          Show
          Filipe Dobreira (Inactive) added a comment - QA approved.
          Hide
          Filipe Dobreira (Inactive) added a comment -

          Issue is again present.

          Show
          Filipe Dobreira (Inactive) added a comment - Issue is again present.
          Hide
          Jérôme Vieilledent (Inactive) added a comment -

          I fixed it yesterday. Can you please test against master ?

          Show
          Jérôme Vieilledent (Inactive) added a comment - I fixed it yesterday. Can you please test against master ?

            People

            • Assignee:
              Unassigned
              Reporter:
              Filipe Dobreira
            • 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 - 2 days, 1 hour, 30 minutes
                2d 1h 30m