Mahara

  1. Bug #1605127
  2. Comment #14

Comment 14 for bug 1605127

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

Reviewed: https://reviews.mahara.org/6769
Committed: https://git.mahara.org/mahara/mahara/commit/bd05e5b5a27999cb37569115d5460ba132df7d65
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.10_STABLE

commit bd05e5b5a27999cb37569115d5460ba132df7d65
Author: Aaron Wells <email address hidden>
Date: Mon Jul 25 10:56:35 2016 +1200

Bug 1605127: Method signature mismatch in Activity::update_url()

The abstract Activity class defines the function update_url()
with no parameters, but the two subclasses that override it
define it with one parameter. This will cause problems in PHP 7.

It's always called with one parameter, (even in the Activity
class itself) so the best option is to add one parameter to the
implementation in Activity.

Change-Id: I810061ed6f8c55101327e2e907bb68ebf9870380
behatnotneeded: Covered by existing tests
(cherry picked from commit b469030b12e80147d406fc00b75afe024a329402)