Skip to content

Commit

Permalink
Fixed issue #11391: Clicking a question code in EM expression opens s…
Browse files Browse the repository at this point in the history
…urvey home page instead of particular question
  • Loading branch information
c-schmitz committed Jun 17, 2016
1 parent 089c9de commit 155bb0c
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
6 changes: 3 additions & 3 deletions application/helpers/common_helper.php
Expand Up @@ -342,7 +342,7 @@ function getQuestions($surveyid,$gid,$selectedqid)
{
$qrow = $qrow->attributes;
$qrow['title'] = strip_tags($qrow['title']);
$link = Yii::app()->getController()->createUrl("/admin/survey/sa/view/surveyid/".$surveyid."/gid/".$gid."/qid/".$qrow['qid']);
$link = Yii::app()->getController()->createUrl("/admin/questions/sa/view/surveyid/".$surveyid."/gid/".$gid."/qid/".$qrow['qid']);
$sQuestionselecter .= "<option value='{$link}'";
if ($selectedqid == $qrow['qid'])
{
Expand All @@ -369,7 +369,7 @@ function getQuestions($surveyid,$gid,$selectedqid)
}
else
{
$link = Yii::app()->getController()->createUrl("/admin/survey/sa/view/surveyid/".$surveyid."/gid/".$gid);
$link = Yii::app()->getController()->createUrl("/admin/questiongroups/sa/view/surveyid/".$surveyid."/gid/".$gid);
$sQuestionselecter = "<option value='{$link}'>".gT("None")."</option>\n".$sQuestionselecter;
}
return $sQuestionselecter;
Expand Down Expand Up @@ -1002,7 +1002,7 @@ function getGroupListLang($gid, $language, $surveyid)
$gv = $gv->attributes;
$groupselecter .= "<option";
if ($gv['gid'] == $gid) {$groupselecter .= " selected='selected'"; $gvexist = 1;}
$link = Yii::app()->getController()->createUrl("/admin/survey/sa/view/surveyid/".$surveyid."/gid/".$gv['gid']);
$link = Yii::app()->getController()->createUrl("/admin/questiongroups/sa/view/surveyid/".$surveyid."/gid/".$gv['gid']);
$groupselecter .= " value='{$link}'>";
if (strip_tags($gv['group_name']))
{
Expand Down
2 changes: 1 addition & 1 deletion application/helpers/expressions/em_core_helper.php
Expand Up @@ -1573,7 +1573,7 @@ public function GetPrettyPrintString()

if ($this->hyperlinkSyntaxHighlighting && isset($gid) && isset($qid) && $qid>0)
{
$editlink = Yii::app()->getController()->createUrl('admin/survey/sa/view/surveyid/' . $this->sid . '/gid/' . $gid . '/qid/' . $qid);
$editlink = Yii::app()->getController()->createUrl('admin/questions/sa/view/surveyid/' . $this->sid . '/gid/' . $gid . '/qid/' . $qid);
$stringParts[] = "<a title='{$message}' class='em-var {$class}' href='{$editlink}' >";
}
else
Expand Down
6 changes: 3 additions & 3 deletions application/helpers/expressions/em_manager_helper.php
Expand Up @@ -9216,7 +9216,7 @@ static public function ShowSurveyLogicFile($sid, $gid=NULL, $qid=NULL,$LEMdebugL
$LEM->ProcessString($sGroupText, $qid,NULL,false,1,1,false,false);
$bGroupHaveError=$bGroupHaveError || $LEM->em->HasErrors();
$sGroupText= viewHelper::purified(viewHelper::filterScript($LEM->GetLastPrettyPrintExpression()));
$editlink = Yii::app()->getController()->createUrl('admin/survey/sa/view/surveyid/' . $LEM->sid . '/gid/' . $gid);
$editlink = Yii::app()->getController()->createUrl('admin/questiongroups/sa/view/surveyid/' . $LEM->sid . '/gid/' . $gid);
if($bGroupHaveError)
{
$errClass='text-danger';
Expand Down Expand Up @@ -9606,12 +9606,12 @@ static public function ShowSurveyLogicFile($sid, $gid=NULL, $qid=NULL,$LEMdebugL

if ($varNameErrorMsg == '')
{
$editlink = Yii::app()->getController()->createUrl('admin/survey/sa/view/surveyid/' . $sid . '/gid/' . $gid . '/qid/' . $qid);
$editlink = Yii::app()->getController()->createUrl('admin/questions/sa/view/surveyid/' . $sid . '/gid/' . $gid . '/qid/' . $qid);
$questionRow .= $rootVarName;
}
else
{
$editlink = Yii::app()->getController()->createUrl('admin/survey/sa/view/surveyid/' . $LEM->sid . '/gid/' . $varNameError['gid'] . '/qid/' . $varNameError['qid']);
$editlink = Yii::app()->getController()->createUrl('admin/questions/sa/view/surveyid/' . $LEM->sid . '/gid/' . $varNameError['gid'] . '/qid/' . $varNameError['qid']);
$questionRow .= "<span class='highlighterror' title='" . $varNameError['message'] . "' "
. "onclick='window.open(\"$editlink\",\"_blank\")'>"

This comment has been minimized.

Copy link
@Shnoulle

Shnoulle Jun 17, 2016

Collaborator

It's not this part to remove ?

Sorry : bad bug number :)

. $rootVarName . "</span>";
Expand Down
Expand Up @@ -117,7 +117,7 @@
<?php else: ?>

<!-- there is at least one question having a condition on its content -->
<a href='<?php echo $this->createUrl("admin/survey/sa/view/surveyid/$surveyid/gid/$gid"); ?>' class="btn btn-default" onclick="alert('<?php eT("Impossible to delete this group because there is at least one question having a condition on its content","js"); ?>'); return false;">
<a href='<?php echo $this->createUrl("admin/questiongroups/sa/view/surveyid/$surveyid/gid/$gid"); ?>' class="btn btn-default" onclick="alert('<?php eT("Impossible to delete this group because there is at least one question having a condition on its content","js"); ?>'); return false;">
<span class="glyphicon glyphicon-trash"></span>
<?php eT("Delete current question group"); ?>
</a>
Expand Down

0 comments on commit 155bb0c

Please sign in to comment.