Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-61431

plagiarism_get_links doesn't uniquely identify content for forum posts.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Reopened
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.4.1
    • Fix Version/s: None
    • Component/s: Forum, Plagiarism
    • Labels:
    • Testing Instructions:
      Hide

      This is hard to test as no plagiarism plugins exist in core, and plagiarism plugins that support forum submission all require subscriptions.

      Configure a plagiarism plugin to submit forum post content to the service (not all plagiarism plugins support this), submit some text and make sure the link for the external system still displays.

      this patch has already been tested using the Turnitin and Urkund plugins - Dan may be able to supply test credentials for the tester during integration, but I suspect in this case a sanity check of the code would be sufficient.

      Show
      This is hard to test as no plagiarism plugins exist in core, and plagiarism plugins that support forum submission all require subscriptions. Configure a plagiarism plugin to submit forum post content to the service (not all plagiarism plugins support this), submit some text and make sure the link for the external system still displays. this patch has already been tested using the Turnitin and Urkund plugins - Dan may be able to supply test credentials for the tester during integration, but I suspect in this case a sanity check of the code would be sufficient.
    • Affected Branches:
      MOODLE_34_STABLE
    • Pull Master Branch:
      MDL-61431-master

      Description

      The forum doesn't pass enough information to plagiarism_get_links() to be able to identify the actual forum post. It should pass the forum post id as well.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                johnmcgettrick John McGettrick
                Peer reviewer:
                Dan Marsden
                Integrator:
                Eloy Lafuente (stronk7)
                Participants:
                Component watchers:
                Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze, Dan Marsden, Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                2 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: