Skip to content

Commit

Permalink
MDL-31117 delete obsolete TODO
Browse files Browse the repository at this point in the history
This TODO was added when we were still using moodledata folder for group images, it is not relevant any more because group images are in new file API which supports DB transaction rollbacks.
  • Loading branch information
skodak committed Jan 14, 2012
1 parent 60e9909 commit c5eeccb
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion group/externallib.php
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,6 @@ public static function delete_groups($groupids) {

$transaction = $DB->start_delegated_transaction();

// TODO: this is problematic because the DB rollback does not handle deleting of group images!
foreach ($params['groupids'] as $groupid) {
// validate params
$groupid = validate_param($groupid, PARAM_INTEGER);
Expand Down

0 comments on commit c5eeccb

Please sign in to comment.