Uploaded image for project: 'PUBLIC - Liferay Portal Community Edition'
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-155811

Incorrect assets shown for collections when combined segments enabled and user belongs to segments without personalized variations

Details

    Description

      Summary

      When a user:

      1. belongs to more than one segment, but
      2. not all the corresponding personalized variations for a collection (manual or dynamic) have been created, and
      3. segments combination is enabled,

      the user should see:

      • The assets corresponding to the existing personalized variations they belong to if at least one exists.
      • The assets corresponding to the Anyone personalized variation if no personalized variations that they belong to, exist.

      Differences between master and 7.3.x/7.2.x

      Due to an issue in master (LPS-147235) the reported problem can only be seen for manual collections in master, but for any type of collection (manual or dynamic) in 7.3.x and 7.2.x.

      Reproducing steps

      1. Create three users: user01user02, and user03.
      2. Create a new site and add a new page.
      3. Create a new segment, segment01 with condition: User equals user01 OR User equals user03.
      4. Create another segment, segment02 with condition: User equals user02 OR User equals user03.
      5. Create a basic web content, bwcForSegment01.
      6. Create a blog entry: blogEntryForAnyone.
      7. Create two content folders: wcFolderForAnyone and wcFolderForSegment01.
      8. Create a documents folder, documentFolder01.
      9. Create 
      10. Create a dynamic collection (in 7.2.x a collection is called a content set) with:
        1. PV for Anyone with Item Type Blogs Entry.
        2. PV for segment01 with Item Type Web Content Article and Item Subtype Basic Web Content.
        3. (Important!) Do not create a PV for segment02.
        • Make sure to have segment01 above Anyone in master so that it gets higher priority.
      11. Add an Asset Publisher and configure it to show the dynamic collection created above.
      12. Create a manual collection with:
        1. PV for Anyone with wcFolderForAnyone.
        2. PV for segment01 with wcFolderForAnyone.
        3. (Important!) Do not create a PV for segment02.
        • Make sure to have segment01 above Anyone in master so that it gets higher priority.
      13. Add an Asset Publisher to the page and configure it to show the manual collection created above.
      14. Results for different users, which are the expected ones:
      All branches Combin. OFF
        Dynamic ... ... Collection Manual ... .. Collection
      User Observed Expected Observed Expected
      Test  blogEntryForAnyone  blogEntryForAnyone  wcFolderForAnyone  wcFolderForAnyone
      user01  bwcForSegment01  bwcForSegment01  wcFolderForSegment01  wcFolderForSegment01
      user02 blogEntryForAnyone blogEntryForAnyone wcFolderForAnyone wcFolderForAnyone
      user03  bwcForSegment01  bwcForSegment01  wcFolderForSegment01  wcFolderForSegment01

      Now we activate segment combination:

      1. Go to Control Panel > System Settings > Assets > Collections and enable Combine Assets from All Segments in Asset Publisher for Dynamic Collections and Combine Assets from All Segments in Asset Publisher for Manual Collections.

      For master we only focus on manual collections, where we can see the issue at play without being masked by a different one. (Once that other issue is fixed, the expected results for dynamic collections should be the same as for 7.3.x/7.2.x; see below.)

      master Combin. ON
        Manual ... .. Collection
      User Observed Expected
      Test  wcFolderForAnyone  wcForderForAnyone
      user01  wcFolderForSegment01  wcFolderForSegment01
      user02 Empty wcFolderForAnyone
      user03 wcFolderForSegment01 wcFolderForSegment01

       
      For 7.3.x/7.2.x we describe for dynamic and manual collections.

      7.(3|2).x Combin. ON
        Dynamic ... ... Collection Manual ... .. Collection
      User Observed Expected Observed Expected
      Test  blogEntryForAnyone  blogEntryForAnyone  wcFolderForAnyone  wcFolderForAnyone
      user01  bwcForSegment01  bwcForSegment01  wcFolderForSegment01  wcFolderForSegment01
      user02  all assets blogEntryForAnyone Empty  wcFolderForAnyone
      user03  all assets bwcForSegment01  wcFolderForSegment01  wcFolderForSegment01

      Attachments

        Activity

          People

            team-tango Product Team Tango
            ricardo.couso Ricardo Couso
            Kiyoshi Lee Kiyoshi Lee
            Ricardo Couso Ricardo Couso
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              10 weeks, 3 days ago

              Packages

                Version Package
                7.4.3.31 CE GA31
                7.4.13 DXP U31
                7.4.3.34 CE GA34
                7.4.3.35 CE GA35
                Master