[#MDL-12101] bulk user operations cleanup/fixes

Issue Details (XML | Word | Printable)

Key:	MDL-12101
Type:	Bug
Status:	Resolved
Resolution:	Fixed
Priority:	Critical
Assignee:	Petr Skoda
Reporter:	Petr Skoda
Votes:	0
Watchers:	0

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

bulk user operations cleanup/fixes

Created: 12/Nov/07 04:56 AM Updated: 27/Dec/07 06:48 PM

Return to search

Component/s:

Administration

Affects Version/s:

1.9

Fix Version/s:

1.9

File Attachments:

big_bulk1.patch (136 kB)

Issue Links:

Dependency

This issue will help resolve:

MDL-11424 mform1 form has no properties in moodle19/admin/user/user_bulk.php

~~MDL-11755~~ hard coded lang strings in user_bulk.php

Relates

This issue has been marked as being related by:

~~MDL-10464~~ Allow administrators dowloading all users to a file

~~MDL-3982~~ Searching by username

Participants:	Petr Skoda
Security Level:	None
Resolved date:	13/Nov/07
Affected Branches:	MOODLE_19_STABLE
Fixed Branches:	MOODLE_19_STABLE

Description

« Hide

There are several problems:
1/ unitialized arrays - XSS
2/ bad sql syntax - wrong quotes used in filters
3/ incorrect use of formslib - not nice at all, very fragile and hackish
4/ user filters ORed, should be ANDed - we can simulate OR by select all + clearing the filters, we can not do AND filters now
5/ localization unfinished and problematic
6/ hardcoded guest user id
7/ some more...

Solution:
* split the forms
* move the data processing logic from forms
* general cleanup

All

Comments

Change History

Version Control

Sort Order:

Petr Skoda made changes - 12/Nov/07 04:58 AM

Field	Original Value	New Value
Link		This issue will help resolve ~~MDL-11755~~ [ ~~MDL-11755~~ ]

Petr Skoda made changes - 12/Nov/07 04:59 AM

Link

This issue will help resolve MDL-11424 [ MDL-11424 ]

Petr Skoda made changes - 12/Nov/07 05:00 AM

Summary

bulk user operation cleanup/fixes

bulk user operations cleanup/fixes

Petr Skoda made changes - 13/Nov/07 05:45 AM

Attachment

big_bulk1.patch [ 12350 ]

Petr Skoda made changes - 13/Nov/07 05:09 PM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Petr Skoda made changes - 20/Dec/07 07:55 PM

Link

This issue has been marked as being related by ~~MDL-10464~~ [ ~~MDL-10464~~ ]

Petr Skoda made changes - 27/Dec/07 06:48 PM

Link

This issue has been marked as being related by ~~MDL-3982~~ [ ~~MDL-3982~~ ]

All content on this web site is made available under the GNU General Public License, unless otherwise stated.