Fixed #6801: Reporting Graph Issue

dev: prefix can be T, D or nothing checking for not numeric now and stripping if present
LimeSurvey · Nov 5, 2012 · e4d6315 · e4d6315
1 parent bd5657e
commit e4d6315
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/application/controllers/admin/statistics.php b/application/controllers/admin/statistics.php
@@ -528,7 +528,11 @@ function graph()
         $sStatisticsLanguage=sanitize_languagecode($_POST['sStatisticsLanguage']);
         $oStatisticsLanguage = new Limesurvey_lang($sStatisticsLanguage);        
 	    if (isset($_POST['cmd']) && isset($_POST['id'])) {
-	        list($qsid, $qgid, $qqid) = explode("X", substr($_POST['id'], 1), 3);
+	        list($qsid, $qgid, $qqid) = explode("X", substr($_POST['id'], 0), 3);
+            if(!is_numeric(substr($qsid,0,1))) {
+                // Strip first char when not numeric (probably T or D)
+                $qsid=substr($qsid,1);
+            }
 	        $qtype = substr($_POST['id'], 0, 1);
             $aattr = getQuestionAttributeValues($qqid);
             $field = substr($_POST['id'], 1);