Skip to content

Commit

Permalink
Revert "Fixed issue #18808: Mandatory soft only works once per survey…
Browse files Browse the repository at this point in the history
… if over multiple pages/groups (#3119)"

This reverts commit 6439cb1.
  • Loading branch information
olleharstedt committed Jun 5, 2023
1 parent b119999 commit 1b9b844
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 1,160 deletions.
4 changes: 4 additions & 0 deletions application/helpers/SurveyRuntimeHelper.php
Expand Up @@ -546,6 +546,10 @@ public function run($surveyid, $args)
if (!empty($_SESSION[$this->LEMsessid]['token']) and $this->aSurveyInfo['anonymized'] != 'Y') {
$this->aSurveyInfo['hiddenInputs'] .= \CHtml::hiddenField('token', $_SESSION[$this->LEMsessid]['token'], array('id' => 'token'));
}
/* Set sof man to true if it's already in POST */
if (App()->request->getPost('mandSoft')) {
$this->aSurveyInfo['hiddenInputs'] .= \CHtml::hiddenField('mandSoft', App()->request->getPost('mandSoft'), array('id' => 'mandSoft'));
}
}

// For "clear all" buttons
Expand Down
7 changes: 1 addition & 6 deletions tests/LimeSurveyWebDriver.php
Expand Up @@ -4,7 +4,6 @@

use Facebook\WebDriver\Remote\RemoteWebDriver;
use Facebook\WebDriver\WebDriverBy;
use Facebook\WebDriver\WebDriverExpectedCondition;
use Facebook\WebDriver\Exception\NoSuchElementException;

/**
Expand Down Expand Up @@ -86,11 +85,7 @@ public function answerTextQuestion($sgqa, $answer)
*/
public function next()
{
$nextButton = $this->wait(1)->until(
WebDriverExpectedCondition::elementToBeClickable(
WebDriverBy::id('ls-button-submit')
)
);
$nextButton = $this->findElement(WebDriverBy::id('ls-button-submit'));
$nextButton->click();
}

Expand Down

0 comments on commit 1b9b844

Please sign in to comment.