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

Avoid NullPointerException for no longer defined tools in pda portal

    Details

    • Type: Bug
    • Status: CLOSED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.0
    • Fix Version/s: 2.8.0
    • Component/s: Mobile Devices, Portal
    • Labels:
      None
    • Test Plan:
      Hide

      I will use synoptic discussion tool for example here. But any other tool definition should work, too:

      1. make sure Synoptic Discussion and Discussion tool are defined and implemented in system;
      2. add Discussion tool to site, then visit the Home page, notice the Synoptic Discussion tool has been added into Home page;
      3. Go to Admin Sites tool, edit the above site, make sure to add synoptic discussion tool as a separate tool on a separate page to the site;
      4. remove the definition of Synoptic Discussion tool from the system;
      5. access through the pda portal, make sure none of the link to Home-page Synoptic Discussion tool nor the normal Synoptic Discussion page are shown.

      Show
      I will use synoptic discussion tool for example here. But any other tool definition should work, too: 1. make sure Synoptic Discussion and Discussion tool are defined and implemented in system; 2. add Discussion tool to site, then visit the Home page, notice the Synoptic Discussion tool has been added into Home page; 3. Go to Admin Sites tool, edit the above site, make sure to add synoptic discussion tool as a separate tool on a separate page to the site; 4. remove the definition of Synoptic Discussion tool from the system; 5. access through the pda portal, make sure none of the link to Home-page Synoptic Discussion tool nor the normal Synoptic Discussion page are shown.

      Description

      If a tool has been added into a site, and later gets removed from the build, the site should not show the tool in the future.

      PDA portal contains one special case: it gets all Home tools out from one Home site page into individual page navigation(s). Need to check for no-longer defined tools over there, too.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  zqian Zhen Qian
                  Reporter:
                  zqian Zhen Qian
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration