← Back to team overview

mahara-contributors team mailing list archive

[Bug 1099811] Re: group files error after upgrade

 

I've managed to track down exactly why this was triggering on this site,
but not on other Mahara sites (and to clear up confusion, the 'client'
who's site I was investigating was Gordon's site - I was assigned the
issue to investigate but only had the site name, not the contact name).

Usually the 'delay' field for the 'watchlist' activity is set to '1', so
that the code which was throwing an exception on this site is only
called via cron. With cron running, there is already a workaround in
place to not throw an error if the 'owner' field is 'null', on the
assumption that the owner must have been deleted before cron was called.
This means that cron was silently coping with 'group' pages, none of
which have an 'owner' value set.

On this site, the 'delay' field was set to '0', so the watchlist was
being updated immediately and throwing an exception when the 'owner'
field was null.

I would recommend fixing this by updating the ActivityTypeWatchlist
'__construct()' function to simply return if the 'owner' is not found,
rather than throwing an exception.

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/1099811

Title:
  group files error after upgrade

Status in Mahara ePortfolio:
  New

Bug description:
  Hi,
  I upgraded to Mahara 1.6.2 on two sites. When upgrading both sites I had to alter the database to remove a latin1 collation somewhere in the tables, as outlined here - https://mahara.org/interaction/forum/topic.php?id=1770 The upgrade then ran and appeared to work, but I discovered a couple of closely related issues. In the first site I'm getting an error whenever any user tries to add a file to a page within a group (see screenshot). In the second test site the Files tab in groups is missing completely.

  My best guess is that a table or index has either been corrupted or
  was missed from the upgrade - but I'm stumped about how to proceed in
  fixing this. Any suggestions would be welcome. The page not found
  issue is the priority.

  Thanks, Gordon.

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


References