mahara-contributors team mailing list archive

[Robert Lyon <robertl@xxxxxxxxxxxxxxx>]

Public bug reported:

With the new pdfexport option there are a few things that need to be
true before it is used.

1) The chrome-php plugin needs to exist
2) The $cfg->usepdfexport setting needs to be true
3) The active setting in export_installed table needs to be true

However, it will still do the export if point 1 and 3 are true -
ignoring the usepdfexport flag - if installing a new site (not
upgrading)

To test (before patch):

1) checkout master
2) run make pdfexport - and also make sure any other pdfexport dependencies are met eg pdfunite
3) set config.php with: $cfg->usepdfexport = false;
3) install site
4) do an export and see pdf being built in it

So we neeed to adjust the code so that all three things are respected

** Affects: mahara
     Importance: High
     Assignee: Robert Lyon (robertl-9)
         Status: In Progress

** Changed in: mahara
       Status: New => In Progress

** Changed in: mahara
   Importance: Undecided => High

** Changed in: mahara
     Assignee: (unassigned) => Robert Lyon (robertl-9)

** Changed in: mahara
    Milestone: None => 20.04.0

** Changed in: mahara
    Milestone: 20.04.0 => 20.04rc1

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1870993

Title:
  Site not respecting the usepdfexport flag

Status in Mahara:
  In Progress

Bug description:
  With the new pdfexport option there are a few things that need to be
  true before it is used.

  1) The chrome-php plugin needs to exist
  2) The $cfg->usepdfexport setting needs to be true
  3) The active setting in export_installed table needs to be true

  However, it will still do the export if point 1 and 3 are true -
  ignoring the usepdfexport flag - if installing a new site (not
  upgrading)

  To test (before patch):

  1) checkout master
  2) run make pdfexport - and also make sure any other pdfexport dependencies are met eg pdfunite
  3) set config.php with: $cfg->usepdfexport = false;
  3) install site
  4) do an export and see pdf being built in it

  So we neeed to adjust the code so that all three things are respected

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1870993/+subscriptions