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

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

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.6
    • 2.6
    • Assignment
    • MOODLE_26_STABLE
    • MOODLE_26_STABLE
    • MDL-42671-master
    • 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';

    Description

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

      Attachments

        Issue Links

          Activity

            People

              damyon Damyon Wiese
              damyon Damyon Wiese
              Rajesh Taneja Rajesh Taneja
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Petr Skoda Petr Skoda
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                18/Nov/13