Files copied during user csv upload are put the wrong folders
Bug #820180 reported by
Richard Mansfield
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mahara |
Fix Released
|
Medium
|
Richard Mansfield | ||
Bug Description
When you create multiple users at once by uploading a csv file, and the institution specifies a view to be copied for those users on creation, all the files copied (as part of copied views) are placed into a folder owned by the first user to be copied. Files owned by all the other new users are left inaccessible by their rightful owners.
| Changed in mahara: | |
| assignee: | nobody → Richard Mansfield (richard-mansfield) |
| importance: | Undecided → Medium |
| milestone: | none → 1.5.0 |
| status: | New → Confirmed |
Revision history for this message
| Mahara Bot (dev-mahara) wrote A change has been merged | #1 |
| Changed in mahara: | |
| status: | Confirmed → Fix Committed |
| Changed in mahara: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Reviewed: https:/
Committed: http://
Submitter: Hugh Davenport (<email address hidden>)
Branch: master
commit 3f4ee9b1c957fe4
Author: Richard Mansfield <email address hidden>
Date: Wed Aug 3 17:01:37 2011 +1200
Generate new parent folder for each new view copied (bug #820180)
Files copied during view copying were not dealt with correctly
when more than one view was being copied in a single request.
The copying process automatically creates a folder in which to
put any files it needs to copy, but the id of this folder was
declared as static, so it didn't work once copying of multiple
views is allowed.
There is a similar problem for blogs created during view copies
(to hold copied blogposts). I'm not sure if there's a way to
copy multiple views containing blogposts that need copying (it
can't happen in institution views). But the default blog creation
function is updated in this patch anyway.
Change-Id: Id8ea2504772f49
Signed-off-by: Richard Mansfield <email address hidden>