Undefined index: sendnow when adding groups by CSV

Bug #1429646 reported by Son Nguyen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Cecilia Vela Gurovic

Bug Description

Version: 1.9

I got the following warning when adding groups via CSV

[WAR] 48 (lib/group.php:438) Undefined index: sendnow
Call stack (most recent first):
log_message("Undefined index: sendnow", 8, true, true, "/home/sonn/code/mahara/1.9/htdocs/lib/group.php", 438) at /home/sonn/code/mahara/1.9/htdocs/lib/errors.php:439
error(8, "Undefined index: sendnow", "/home/sonn/code/mahara/1.9/htdocs/lib/group.php", 438, array(size 3)) at /home/sonn/code/mahara/1.9/htdocs/lib/group.php:438
group_create(array(size 5)) at /home/sonn/code/mahara/1.9/htdocs/admin/groups/uploadcsv.php:288
uploadcsv_submit(object(Pieform), array(size 5)) at Unknown:0
call_user_func_array("uploadcsv_submit", array(size 2)) at /home/sonn/code/mahara/1.9/htdocs/lib/pieforms/pieform.php:527
Pieform->__construct(array(size 2)) at /home/sonn/code/mahara/1.9/htdocs/lib/pieforms/pieform.php:162
Pieform::process(array(size 2)) at /home/sonn/code/mahara/1.9/htdocs/lib/pieforms/pieform.php:71
pieform(array(size 2)) at /home/sonn/code/mahara/1.9/htdocs/admin/groups/uploadcsv.php:352

This won't stop adding groups

Solution:

Change the code at the line 438 in htdocs/lib/group.php
 'sendnow' => $data['sendnow'],
to
  'sendnow' => isset($data['sendnow']) ? $data['sendnow'] : 0,

Tags: bite-sized
Changed in mahara:
assignee: nobody → Cecilia Vela Gurovic (ceciliavg)
Revision history for this message
Cecilia Vela Gurovic (ceciliavg) wrote :

Cannot reproduce bug:

This was fixed in lib/group.php file

by lines 411

  if (!isset($data['sendnow'])) {
        $data['sendnow'] = null;
    }

and 445
   'sendnow' => isset($data['sendnow']) ? $data['sendnow'] : null,

Revision history for this message
Cecilia Vela Gurovic (ceciliavg) wrote :

It was fix back in 2014 by commit 1750dd4f80edb38dab0841cc8131a6b0162a6ab4

Changed in mahara:
status: Confirmed → 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.