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:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Matteo Scaramuccia, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              3 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

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