← Back to team overview

mahara-contributors team mailing list archive

[Bug 1401210] A change has been merged

 

Reviewed:  https://reviews.mahara.org/4223
Committed: http://gitorious.org/mahara/mahara/commit/efb321c370382257e510ca832fc669915472505e
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit efb321c370382257e510ca832fc669915472505e
Author: Aaron Wells <aaronw@xxxxxxxxxxxxxxx>
Date:   Sun Jan 25 22:23:16 2015 +1300

Adding ArtefactType::postcommit_hook function

Bug 1401210. In order for watchlists to properly handle blog
blocks, we need to give the Blog artefact a chance to update
the view_artefacts table before the "saveartefact" event gets
fired at the bottom of ArtefactType::commit().

So I'm adding a postcommit_hook() method which gets called in
the middle of commit(), and shifting ArtefactType's commit()
method into there.

Change-Id: I23986597fd411fa8f93da7cdbfa8d0b577de66fc

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

Title:
  Watching a page with a journal block

Status in Mahara ePortfolio:
  Fix Committed

Bug description:
  A page of user A is on the watchlist of user B.

  The page contains a journal block. When user A updates is journal by
  adding a new article, the new article is displayed in the page but
  there is no notification to user B that user A has updated is journal.

  This situation is the same with a journal block, a recent journal
  entry block and tagged journal entry block.

  It would be nice either to have the option to add the block to the
  watchlist, specifically or to make sure that when a page is on a
  watchlist all modifications including a new journal entry in a block
  trigger a notification.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1401210/+subscriptions


References