Uploaded image for project: 'User Directory'
  1. User Directory
  2. UD-179

UserDirectoryPageHandler#getUserPagesContent(Page, String) is broken if multiple roots set in different sitemaps

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Major Major
    • 2.15.0
    • 2.2.0, 2.3.0
    • None

      Put a root on 'fr' sitemap AND 'en' sitemap with same parameters (content type, classifying metadata and depth).

      It works well when navigating in the site

      BUT the method UserDirectoryPageHandler#getUserPagesContent(Page, String) is broken. It is based on a cache which is not sitemap/language aware

      As a result its callers (for instance UserPageFactory#getAmetysObjectById always return the same page for different ids

      i.e. ametysResolver.resolveById("uduser://b?rootId=root.en.id&contentId=my.user.content.1.en.id") and ametysResolver.resolveById("uduser://b?rootId=root.fr.id&contentId=my.user.content.1.fr.id") will return the same page object (either the 'fr' or 'en' one, depending on which came first to fill the cache).

      So the method is wrong.

          [UD-179] UserDirectoryPageHandler#getUserPagesContent(Page, String) is broken if multiple roots set in different sitemaps

          Raphaël Franchet made changes -
          Fix Version/s New: 2.15.0 [ 19100 ]
          Fix Version/s Original: 2.14.0 [ 19011 ]
          Cédric Damioli made changes -
          Fix Version/s New: 2.14.0 [ 19011 ]
          Fix Version/s Original: 2.13.0 [ 18923 ]
          Laurence Aumeunier made changes -
          Fix Version/s New: 2.13.0 [ 18923 ]
          Fix Version/s Original: 2.12.0 [ 18031 ]
          Laurence Aumeunier made changes -
          Fix Version/s New: 2.12.0 [ 18031 ]
          Fix Version/s Original: 2.11.0 [ 17741 ]
          Cédric Damioli made changes -
          Fix Version/s New: 2.11.0 [ 17741 ]
          Fix Version/s Original: 2.10.0 [ 17259 ]
          Laurence Aumeunier made changes -
          Fix Version/s New: 2.10.0 [ 17259 ]
          Fix Version/s Original: 2.9.0 [ 17245 ]
          Cédric Damioli made changes -
          Fix Version/s New: 2.9.0 [ 17245 ]
          Fix Version/s Original: 2.8.0 [ 17229 ]
          Laurence Aumeunier made changes -
          Fix Version/s New: 2.8.0 [ 17229 ]
          Fix Version/s Original: 2.7.0 [ 17106 ]
          Cédric Damioli made changes -
          Fix Version/s New: 2.7.0 [ 17106 ]
          Fix Version/s Original: 2.6.0 [ 16956 ]
          Laurence Aumeunier made changes -
          Fix Version/s New: 2.6.0 [ 16956 ]
          Fix Version/s Original: 2.5.0 [ 16396 ]

            Unassigned Unassigned
            sprieul Simon Prieul (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: