Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-31709

Favorites doesn't work if site id is numeric

    Details

    • 19 status:
      Resolved
    • 12 status:
      Resolved
    • Test Plan:
      Hide
      • As admin in the Sites tool, manually create a site with id 12345
      • Add a page, add a tool with site info so that the site works.
      • Open the "More sites dropdown" and click the star next to this site, it should mark it as a favorite but it isn't.
      Show
      As admin in the Sites tool, manually create a site with id 12345 Add a page, add a tool with site info so that the site works. Open the "More sites dropdown" and click the star next to this site, it should mark it as a favorite but it isn't.

      Description

      We have some sites that have sites manually created that are short like 5 digits

      /portal/site/12345

      It seems like when you mark them as a favorite with the star they get saved to the database, but when they're retrieved the star doesn't appear.

      SAKAI_PREFERENCES after saving a favorite, the last favorite MTIzNDU= is this 12345.

      When portal/favorites/list?_=1471964562671 is called it returns all of the sites correctly
      37c4f47d-36d5-4c1d-bec8-dc6c0fe8dc6f;d0e2ed68-81ba-4d23-b538-95cfc258b682;12345 but it stlil doesn't star the site 12345.

      <prefs key="sakai:portal:sitenav"><properties><property enc="BASE64" list="list" name="order" value="ZDBlMmVkNjgtODFiYS00ZDIzLWI1MzgtOTVjZmMyNThiNjgy"/><property enc="BASE64" list="list" name="order" value="MzdjNGY0N2QtMzZkNS00YzFkLWJlYzgtZGM2YzBmZThkYzZm"/><property enc="BASE64" list="list" name="order" value="MTIzNDU="/></properties></prefs>
      

      So my guess is that the bug is in sakai.morpheus.more.sites.js but I'm not seeing it in a quick review.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                marktriggs Mark Triggs
                Reporter:
                jonespm Matthew Jones
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Git Source Code