From f543eea9d6b598d304cdc6d61929d24cffc91e12 Mon Sep 17 00:00:00 2001 From: Petr Skoda Date: Sun, 15 Jan 2012 00:43:28 +0100 Subject: [PATCH] MDL-31117 delete obsolete TODO 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. --- group/externallib.php | 1 - 1 file changed, 1 deletion(-) diff --git a/group/externallib.php b/group/externallib.php index 160dcbf1b19..7ff57d53d91 100644 --- a/group/externallib.php +++ b/group/externallib.php @@ -267,7 +267,6 @@ class core_group_external extends external_api { $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);