mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #31904
[Bug 1522659] A patch has been submitted for review
Patch for "master" branch: https://reviews.mahara.org/5816
--
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/1522659
Title:
Have require of pieform more centralised
Status in Mahara:
In Progress
Bug description:
Discovered when fixing another problem that some pages get their
Pieform class via the require_once('pieforms/pieform.php'); via the
auth_check_required_fields() function, while all the others get it via
thier own require_once('pieforms/pieform.php'); within the top of the
file.
When all we need is for is the pieform() function that calls the
Pieform class to make sure the required file is included.
So will submit a code tidying up patch
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1522659/+subscriptions
References