New 'handle event' events break upgrade

Bug #1718562 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon

Bug Description

We need to wrap some bits in if(!defined('INSTALLER')) { ... }

Robert Lyon (robertl-9)
Changed in mahara:
assignee: nobody → Robert Lyon (robertl-9)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

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

Revision history for this message
Robert Lyon (robertl-9) wrote :

To Test: Try upgrading from 15.04 site to 17.10

Without patch - will fail as some db structure is missing

With patch - should upgrade fine

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8044
Committed: https://git.mahara.org/mahara/mahara/commit/8c590b1399aae00d66b34c1da675d65f119f51e8
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 8c590b1399aae00d66b34c1da675d65f119f51e8
Author: Robert Lyon <email address hidden>
Date: Thu Sep 21 14:08:00 2017 +1200

Bug 1718562: Stop the install/upgrade system from doing 'handle_event()'

As we don't need to record user interation at this point and it breaks
things on upgrade from old systems as some of the
tables/columns/sequences do not exist

behatnotneeded

Change-Id: I7d6f8969f9b94ba93761c0c25303e2e4916fa778
Signed-off-by: Robert Lyon <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.