Fatal error trying to manually update a user's quota

Bug #1606432 reported by Aaron Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Aaron Wells
15.04
Fix Released
Medium
Aaron Wells
15.10
Fix Released
Medium
Aaron Wells
16.04
Fix Released
Medium
Aaron Wells
16.10
Fix Released
Medium
Aaron Wells

Bug Description

If you try to manually update a user's file quota setting, and you change it to a low enough value that it will send that user an "over the threshold" notification, Mahara crashes with this message:

 Fatal error: Class 'PluginArtefact' not found in /home/aaronw/www/mahara/htdocs/artefact/file/lib.php on line 14
 Stack trace:, referer: http://vegas.wgtn.cat-it.co.nz/mahara/htdocs/admin/users/edit.php?id=2
   1. {main}() /home/aaronw/www/mahara/htdocs/admin/users/edit.php:0
   2. pieform() /home/aaronw/www/mahara/htdocs/admin/users/edit.php:265
   3. Pieform::process() /home/aaronw/www/mahara/htdocs/lib/mahara.php:4730
   4. Pieform->__construct() /home/aaronw/www/mahara/htdocs/lib/pieforms/pieform.php:161
   5. call_user_func_array() /home/aaronw/www/mahara/htdocs/lib/pieforms/pieform.php:540
   6. edituser_site_submit() /home/aaronw/www/mahara/htdocs/lib/pieforms/pieform.php:540
   7. require_once() /home/aaronw/www/mahara/htdocs/admin/users/edit.php:402

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/6752

Revision history for this message
Aaron Wells (u-aaronw) wrote :

To replicate:

1. Create a User A
2. Log in as User A and upload a 1 MB file into your Contents -> File area
3. Log out as User A
4. Log in as an admin
5. Go to the account settings for User A
6. Set User A's file quota to 500 KB.

Expected result: The user's quota is update
Actual result: Fatal error

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6752
Committed: https://git.mahara.org/mahara/mahara/commit/d77fcb12d1327eb1d20c53460ef9433fd1da71b8
Submitter: Son Nguyen (<email address hidden>)
Branch: master

commit d77fcb12d1327eb1d20c53460ef9433fd1da71b8
Author: Aaron Wells <email address hidden>
Date: Tue Jul 26 17:06:58 2016 +1200

Bug 1606432: Use safe_require to make sure PluginArtefact is loaded

Change-Id: I2e047be4ecad4594b9d27bb6d0058fa5971966ce
behatnotneeded: Test to come later

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "16.04_STABLE" branch: https://reviews.mahara.org/6775

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "15.10_STABLE" branch: https://reviews.mahara.org/6776

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "15.04_STABLE" branch: https://reviews.mahara.org/6777

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6776
Committed: https://git.mahara.org/mahara/mahara/commit/99f9da4dae8e5d61ac55f5d261c7856f7e31c6cd
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.10_STABLE

commit 99f9da4dae8e5d61ac55f5d261c7856f7e31c6cd
Author: Aaron Wells <email address hidden>
Date: Tue Jul 26 17:06:58 2016 +1200

Bug 1606432: Use safe_require to make sure PluginArtefact is loaded

Change-Id: I2e047be4ecad4594b9d27bb6d0058fa5971966ce
behatnotneeded: Test to come later
(cherry picked from commit d77fcb12d1327eb1d20c53460ef9433fd1da71b8)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/6775
Committed: https://git.mahara.org/mahara/mahara/commit/2e25b3eb9fbc05fee9589ee83b7dce90a65e5e42
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.04_STABLE

commit 2e25b3eb9fbc05fee9589ee83b7dce90a65e5e42
Author: Aaron Wells <email address hidden>
Date: Tue Jul 26 17:06:58 2016 +1200

Bug 1606432: Use safe_require to make sure PluginArtefact is loaded

Change-Id: I2e047be4ecad4594b9d27bb6d0058fa5971966ce
behatnotneeded: Test to come later
(cherry picked from commit d77fcb12d1327eb1d20c53460ef9433fd1da71b8)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/6777
Committed: https://git.mahara.org/mahara/mahara/commit/78e825edfc8f523a269c0ef40176f61aa452b8e3
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.04_STABLE

commit 78e825edfc8f523a269c0ef40176f61aa452b8e3
Author: Aaron Wells <email address hidden>
Date: Tue Jul 26 17:06:58 2016 +1200

Bug 1606432: Use safe_require to make sure PluginArtefact is loaded

Change-Id: I2e047be4ecad4594b9d27bb6d0058fa5971966ce
behatnotneeded: Test to come later
(cherry picked from commit d77fcb12d1327eb1d20c53460ef9433fd1da71b8)

Robert Lyon (robertl-9)
Changed in mahara:
milestone: 16.10.0 → none
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.