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

SiteService.allowRemove doesn't understand softly deleted sites.

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 10.0
    • Fix Version/s: 11.0
    • Component/s: Kernel
    • Labels:
    • Previous Issue Keys:
      KNL-1208

      Description

      SiteService.allowRemoveSite() is JavaDoc'd as returning true when SiteService.removeSite() will succeed, at the moment it only checks the site.del permission.

      If softly deleted sites are enabled then SiteService.removeSite() checks site.del to mark the site as softly deleted but then requires the permission site.del.softly.deleted to actually remove a softly deleted site.

      SiteService.allowRemoveSite() should have the same behaviour as SiteService.removeSite().

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  buckett Matthew Buckett
                  Reporter:
                  buckett Matthew Buckett
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration