Uploaded image for project: 'Contrib: TurnItIn'
  1. Contrib: TurnItIn
  2. TII-175

Add implementation for isSiteAcceptable using site type and site property

    Details

    • Type: Task
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 10.x
    • Fix Version/s: 10.x
    • Component/s: Service
    • Labels:
      None

      Description

      Currently Turnitin can be enabled or disabled in a site through the isSiteAcceptable() call. The default implementation returns true for all sites. Alternate implementations can be selected in components.xml.

      This change adds a default implementation that will consult site type if configured and site property.

      Behaviour:

      • Site property is checked first. If the site contains the site property "turnitin", then the property value will be used ("true" or any other value = false).
      • Site type is checked if the sakai.properties setting turnitin.sitetypes is set, e.g.:

      turnitin.sitetypes=course,project

      The site is allowed to use turnitin if the site type is listed in the comma-separated list of allowed site types. For example:

      turnitin.sitetypes=course

      restricts to course sites only. However, turnitin can still be disabled in an individual course site by setting the site property "turnitin" to "false" as properties are consulted before site type.

      If no site property is set and no list of sitetypes is set, always return true (i.e. turnitin permitted in all sites).

      Alternate implementations can still be configured in components.xml in which case the logic above does not apply and the call will be delegated to the provided class.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  smarquard Stephen Marquard
                  Reporter:
                  smarquard Stephen Marquard
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration