View Issue Details

Jump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
05746Bug reportsSurvey takingpublic2012-02-01 22:042012-02-07 21:20
Reporteruser11071Assigned Toc_schmitz  
PriorityhighSeverityblock 
Status closedResolutionfixed 
Product Version2.00a1 
Fixed in Version1.92RC4 
Summary05746: array merging error in em_manager_helper.php
Description

Retaking a survey raises an error that tells my that the second parameter isn't an array.

could be fixed with

if (is_array($result['updatedValues']))
$updatedValues = array_merge($updatedValues,$result['updatedValues']);

Additional Information

see attachment

TagsNo tags attached.
Attached Files
em_manager_helper.php.diff (825 bytes)    
Index: helpers/expressions/em_manager_helper.php
===================================================================
--- helpers/expressions/em_manager_helper.php	(revision 12258)
+++ helpers/expressions/em_manager_helper.php	(working copy)
@@ -2951,7 +2951,8 @@
                 {
                     $result = $LEM->_ValidateGroup($LEM->currentGroupSeq);
                     $message .= $result['message'];
-                    $updatedValues = array_merge($updatedValues,$result['updatedValues']);
+					if (is_array($result['updatedValues']))
+                    	$updatedValues = array_merge($updatedValues,$result['updatedValues']);
                     if (!is_null($result) && ($result['mandViolation'] || !$result['valid']))
                     {
                         // redisplay the current group
em_manager_helper.php.diff (825 bytes)   
Bug heat4
Complete LimeSurvey version number (& build)2
I will donate to the project if issue is resolvedNo
Browserfirefox
Database type & versionMysql
Server OS (if known)mac os x
Webserver software & version (if known)apache
PHP Version5.3

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2012-02-03 17:18

administrator   ~17211

Can you please re-test this with the latest developemnt version. If this issues persists, please describe in detail how you can reproduce it and if possible attach a small sample survey.
c_schmitz

c_schmitz

2012-02-06 22:15

administrator   ~17266

?

user11071

2012-02-07 20:58

  ~17291

It works from my point of view. Thanks

Issue History

Date Modified Username Field Change
2012-02-01 22:04 user11071 New Issue
2012-02-01 22:04 user11071 File Added: em_manager_helper.php.diff
2012-02-03 13:04 c_schmitz Assigned To => TMSWhite
2012-02-03 13:04 c_schmitz Status new => assigned
2012-02-03 17:17 c_schmitz Assigned To TMSWhite => c_schmitz
2012-02-03 17:18 c_schmitz Note Added: 17211
2012-02-03 17:18 c_schmitz Status assigned => feedback
2012-02-06 22:15 c_schmitz Note Added: 17266
2012-02-07 20:58 user11071 Note Added: 17291
2012-02-07 20:58 user11071 Status feedback => assigned
2012-02-07 21:20 c_schmitz Status assigned => closed
2012-02-07 21:20 c_schmitz Resolution open => fixed
2012-02-07 21:20 c_schmitz Fixed in Version => 1.92RC4