Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-32225 add plagiarism api hooks to other areas of Moodle code
  3. MDL-34382

Add new option to module_supports() to flag modules that support the use of plagiarism plugins

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Plagiarism
    • Labels:
    • Testing Instructions:
      Hide

      This can only be reviewed/tested by a developer - no core code uses this - a developer could write their own php file to test that the function returns valid options?
      (Also - no contrib code uses this yet either so you can't use any plagiarism plugins to test this - it won't be added until support is available in core)

      Show
      This can only be reviewed/tested by a developer - no core code uses this - a developer could write their own php file to test that the function returns valid options? (Also - no contrib code uses this yet either so you can't use any plagiarism plugins to test this - it won't be added until support is available in core)
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      master_MDL-34382_add_new_option_to_flag_modules_that_support_plagiarism_plugins

      Description

      now that multiple plugins support the use of the plagiarism API it would be nice to add a new option to the module_supports() functions so we can check this.

      for example - mod/assign/lib.php function assign_supports() should have an item like this:
      case FEATURE_PLAGIARISMAPI: return true;
      (FEATURE_PLAGIARISMAPI should be defined in moodlelib.php like the other FEATURE defines)

      For every module we have added support we should add this to the relevant module_supports function

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              danmarsden Dan Marsden
              Reporter:
              danmarsden Dan Marsden
              Peer reviewer:
              Dan Poltawski
              Integrator:
              Aparup Banerjee
              Tester:
              Frédéric Massart
              Participants:
              Component watchers:
              Dan Marsden, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                3/Dec/12