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

Editpdf requires zlib (for tcpdf). Make it unavailable if not installed.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Edit your php config to disable zlib
      2. Go to the Annotate PDF admin settings page
      3. Verify there is a warning about zlib being required
      4. Create an assignment and upload a pdf as a student
      5. Mark the assignment and verify no option to "Launch PDF editor" appears.
      6. Edit your php config and enable zlib again
      7. Visit the same admin page and verify there is no warning
      8. Grade the assignment and verify there is now a "Launch PDF editor" button in the grading page.

      OPTIONAL, if you are a developer and have access to the DB, verify that the assignfeedback_editpdf/zlibenabled isn't ever written to DB, aka:

       select * from mdl_config_plugins where plugin = 'assignfeedback_editpdf';
      

      Show
      Edit your php config to disable zlib Go to the Annotate PDF admin settings page Verify there is a warning about zlib being required Create an assignment and upload a pdf as a student Mark the assignment and verify no option to "Launch PDF editor" appears. Edit your php config and enable zlib again Visit the same admin page and verify there is no warning Grade the assignment and verify there is now a "Launch PDF editor" button in the grading page. OPTIONAL, if you are a developer and have access to the DB, verify that the assignfeedback_editpdf/zlibenabled isn't ever written to DB, aka: select * from mdl_config_plugins where plugin = 'assignfeedback_editpdf';
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-42671-master

      Description

      This should show a warning page on the test editpdf admin page if zlib is not installed.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              damyon Damyon Wiese
              Reporter:
              damyon Damyon Wiese
              Peer reviewer:
              Rajesh Taneja
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Petr Skoda
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13