mahara-contributors team mailing list archive

[Mahara Bot <1534383@xxxxxxxxxxxxxxxxxx>]

Patch for "master" branch: https://reviews.mahara.org/5911

-- 
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:
  In Progress

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