** Changed in: mahara Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Mahara Contributors, which is subscribed to Mahara. Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it! https://bugs.launchpad.net/bugs/1534383
Title: Refactor BlockInstance::bulk_delete_artefacts($records) Status in Mahara: Fix Released Bug description: See https://mahara.org/interaction/forum/topic.php?id=7465&offset=0&limit=10#post30151. This method is old and has some problems that make it confusing. It'd be nice to clean it up a little. In particular, it has a misleading name (it doesn't delete artefacts; it only de-selects them from block instances). And it has a very particular parameter format that is tightly coupled to a particular SQL query, which could easily be moved inside the function in order to make its API more understandable. To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1534383/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : mahara-contributors@lists.launchpad.net Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp