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

    • Sub-task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.4
    • 2.4
    • Plagiarism
    • MOODLE_24_STABLE
    • MOODLE_24_STABLE
    • master_MDL-34382_add_new_option_to_flag_modules_that_support_plagiarism_plugins
    • 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)

    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

              danmarsden Dan Marsden
              danmarsden Dan Marsden
              Dan Poltawski Dan Poltawski
              Aparup Banerjee Aparup Banerjee
              Frédéric Massart Frédéric Massart
              Dan Marsden, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                3/Dec/12