mahara-contributors team mailing list archive

[Aaron Wells <1534383@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

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.

** Affects: mahara
     Importance: Low
     Assignee: Aaron Wells (u-aaronw)
         Status: In Progress


** Tags: api refactoring

-- 
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