Creating a new group with cleanurls active, throws a warning

Bug #1211161 reported by Aaron Wells
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Robert Lyon

Bug Description

[WAR] b9 (group/edit.php:473) Undefined index: urlid
Call stack (most recent first):

    log_message("Undefined index: urlid", 8, true, true, "/var/www/columbia-dev-mahara/releases/201308120359...", 473) at /var/www/columbia-dev-mahara/releases/20130812035927/lib/errors.php:446
    error(8, "Undefined index: urlid", "/var/www/columbia-dev-mahara/releases/201308120359...", 473, array(size 6)) at /var/www/columbia-dev-mahara/releases/20130812035927/group/edit.php:473
    editgroup_submit(object(Pieform), array(size 26)) at Unknown:0
    call_user_func_array("editgroup_submit", array(size 2)) at /var/www/columbia-dev-mahara/releases/20130812035927/lib/pieforms/pieform.php:514
    Pieform->__construct(array(size 4)) at /var/www/columbia-dev-mahara/releases/20130812035927/lib/pieforms/pieform.php:161
    Pieform::process(array(size 4)) at /var/www/columbia-dev-mahara/releases/20130812035927/lib/pieforms/pieform.php:71
    pieform(array(size 4)) at /var/www/columbia-dev-mahara/releases/20130812035927/group/edit.php:390

Revision history for this message
Aaron Wells (u-aaronw) wrote :
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

This seems to be a regression. I had not encountered this problem before.

Aaron Wells (u-aaronw)
Changed in mahara:
assignee: Aaron Wells (u-aaronw) → Robert Lyon (robertl-9)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/2399
Committed: http://gitorious.org/mahara/mahara/commit/7b3c69d192ae2762e97cd6f24ec594e3403d8408
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 7b3c69d192ae2762e97cd6f24ec594e3403d8408
Author: Aaron Wells <email address hidden>
Date: Mon Aug 12 17:33:11 2013 +1200

htdocs/group/edit.php: Squelch a cleanurls warning (bug 1211161)

- also bug 1212863

The code is designed to rely on the not set variable being automatically
converted to an empty string, but PHP throws a warning if you do that.

Change-Id: I40c2c7bd69939779bda68f4de4d47643f0212f42

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Aaron Wells (u-aaronw)
Changed in mahara:
milestone: 1.8rc1 → 1.8.0
Aaron Wells (u-aaronw)
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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