Skip to content

Commit

Permalink
Dev fixed issue #5951: force GetBaseLanguageFromSurveyID for activate…
Browse files Browse the repository at this point in the history
… function, some error with $_SESSION
  • Loading branch information
Shnoulle committed Mar 26, 2012
1 parent fa7b34c commit e62b75f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions admin/activate_functions.php
Expand Up @@ -232,7 +232,7 @@ function checkQuestions($postsid, $surveyid, $qtypes)
}
}
//CHECK THAT ALL THE CREATED FIELDS WILL BE UNIQUE
$fieldmap=createFieldMap($surveyid, 'full', false, false, GetBaseLanguageFromSurveyID($surveyid));
$fieldmap=createFieldMap($surveyid, $style='full', $force_refresh=true,$questionid=false, $sQuestionLanguage=GetBaseLanguageFromSurveyID($surveyid));
$clang = new limesurvey_lang($_SESSION['adminlang']);
if (isset($fieldmap))
{
Expand Down Expand Up @@ -289,7 +289,7 @@ function activateSurvey($postsid,$surveyid, $scriptname='admin.php')
}

//Get list of questions for the base language
$fieldmap=createFieldMap($surveyid, $style='full', $force_refresh=true);
$fieldmap=createFieldMap($surveyid, $style='full', $force_refresh=true,$questionid=false, $sQuestionLanguage=GetBaseLanguageFromSurveyID($surveyid));
foreach ($fieldmap as $arow) //With each question, create the appropriate field(s)
{
if ($createsurvey!='') {$createsurvey .= ",\n";}
Expand Down

2 comments on commit e62b75f

@c-schmitz
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shnoulle, this commit looks pretty strange because you give '$force_refresh=true' etc as param. Please shorten it to correct calls, just 'true' is fine ;).

@Shnoulle
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done,

Please sign in to comment.