Fixed issue : 500 error just after survey creation

Dev: QuestionGroup and its behaviors do not have a method or closure named "getByPk". Dev: and pk for QuestionGroup (and Question) in 3.X is an array
LimeSurvey · Jan 21, 2019 · 2577e46 · 2577e46
1 parent bfd9dd4
commit 2577e46
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/application/models/SurveymenuEntryData.php b/application/models/SurveymenuEntryData.php
@@ -138,13 +138,13 @@ private function _getValueForLinkData($getDataPair)
                 $oTypeObject = Template::model()->findByPk($oSurvey->template);
                 break;
             case 'questiongroup':
-                if (isset($_REQUEST['gid'])) {
-                    $oTypeObject = QuestionGroup::model()->getByPk(((int) $_REQUEST['gid']));
+                if (App()->getRequest()->getParam('gid')) {
+                    $oTypeObject = QuestionGroup::model()->findByPk(array('gid'=>App()->getRequest()->getParam('gid'),'language'=>App()->getLanguage()));
                 }
                 break;
             case 'question':
-                if (isset($_REQUEST['qid'])) {
-                    $oTypeObject = QuestionGroup::model()->getByPk(((int) $_REQUEST['qid']));
+                if (App()->getRequest()->getParam('qid')) {
+                    $oTypeObject = QuestionGroup::model()->findByPk(array('gid'=>App()->getRequest()->getParam('qid'),'language'=>App()->getLanguage()));
                 }
                 break;
             break;