Fixed issue #05877: all Lime replacement fields are broken.

LimeSurvey · Mar 24, 2012 · 7b20089 · 7b20089
1 parent 13d06a4
commit 7b20089
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 8 deletions.
diff --git a/application/controllers/AdminController.php b/application/controllers/AdminController.php
@@ -183,6 +183,7 @@ public function getActionClasses()
         'expressions'      => 'expressions',
         'globalsettings'   => 'globalsettings',
         'htmleditor_pop'   => 'htmleditor_pop',
+        'limereplacementfields' => 'limereplacementfields',
         'index'            => 'index',
         'kcfinder'         => 'kcfinder',
         'labels'           => 'labels',

diff --git a/application/controllers/admin/limereplacementfields.php b/application/controllers/admin/limereplacementfields.php
@@ -15,16 +15,22 @@
 class limereplacementfields extends Survey_Common_Action
 {
 
-    public function index($fieldtype, $action, $surveyid = false, $gid = false, $qid = false)
+    public function index()
     {
-        if ($surveyid != false) {
-            $surveyid = sanitize_int($surveyid);
+        if (isset($_GET['surveyid'])) {
+            $surveyid = sanitize_int($_GET['surveyid']);
         }
-        if ($gid != false) {
-            $gid = sanitize_int($gid);
+        if (isset($_GET['gid'])) {
+            $gid = sanitize_int($_GET['gid']);
         }
-        if ($qid != false) {
-            $qid = sanitize_int($qid);
+        if (isset($_GET['qid'])) {
+            $qid = sanitize_int($_GET['qid']);
+        }
+        if (isset($_GET['fieldtype'])) {
+            $fieldtype = $_GET['fieldtype'];
+        }
+        if (isset($_GET['action'])) {
+            $action = $_GET['action'];
         }
 
         $clang = Yii::app()->lang;

diff --git a/application/views/admin/limeReplacementFields_view.php b/application/views/admin/limeReplacementFields_view.php
@@ -30,7 +30,7 @@ function Ok()
 <body scroll="no" style="OVERFLOW: hidden;">
 <table height="100%" cellSpacing="0" cellPadding="0" width="100%" border="0">
     <tr>
-
+        <td>
         <?php
             if (count($replFields) > 0 || isset($cquestions) )
             {