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

Ensure the test file uses unoconv each time someone uses the unoconv test page.

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 3.1.2
    • Component/s: Assignment, Files API
    • Labels:
    • Testing Instructions:
      Hide

      Setup unoconv (properly - follow the docs)
      Use the test page (/mod/assign/feedback/editpdf/testunoconv.php).
      BREAK UNOCONV (rename the script)
      View the test page again.
      VERIFY THE FILE IS BORKED!
      UNBREAK UNOCONV (rename it back)
      View the test page again.
      VERIFY THE FILE IS UNBORKED!

      Show
      Setup unoconv (properly - follow the docs) Use the test page (/mod/assign/feedback/editpdf/testunoconv.php). BREAK UNOCONV (rename the script) View the test page again. VERIFY THE FILE IS BORKED! UNBREAK UNOCONV (rename it back) View the test page again. VERIFY THE FILE IS UNBORKED!
    • Affected Branches:
      MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-54647-master

      Description

      The API currently caches the result of the document conversions - so the test page will never fail once it has converted the document successfully. It would be better if the conversion result was deleted each time before running the test.

      (This page: /mod/assign/feedback/editpdf/testunoconv.php)

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                damyon Damyon Wiese
                Reporter:
                damyon Damyon Wiese
                Peer reviewer:
                Simey Lameze
                Integrator:
                Andrew Nicols
                Tester:
                Mark Nelson
                Participants:
                Component watchers:
                Damyon Wiese, Adrian Greeve, Mihail Geshoski, Peter Dias, Matteo Scaramuccia, Jake Dallimore, Jun Pataleta
              • Votes:
                3 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Sep/16