Commit 83e75855 authored by Ryan McVeigh's avatar Ryan McVeigh
Browse files

Issue #3270874 by rymcveigh: Validation bypass via multiple submit button clicks

parent e1f33559

src/Plugin/WebformHandler/WebformNavigationHandler.php

+1 −1
Original line number Diff line number Diff line
@@ -178,7 +178,7 @@ class WebformNavigationHandler extends WebformHandlerBase { 
    // Get navigation webform settings.

    $forward_navigation = $webform->getThirdPartySetting('webformnavigation', 'forward_navigation');

    // Actions to perform if forward navigation is enabled and there are pages.

    if ($forward_navigation) {

    if ($forward_navigation && $webform->hasWizardPages()) {

      $this->webformNavigationHelper->logPageErrors($webform_submission, $form_state);

    }

  }

src/WebformNavigationHelper.php

+1 −1
Original line number Diff line number Diff line
@@ -378,7 +378,7 @@ class WebformNavigationHelper { 
   *

   * @throws \Drupal\Core\Entity\EntityStorageException

   */

  public function validateSinglePage(WebformSubmissionInterface $webform_submission, string $page) {

  private function validateSinglePage(WebformSubmissionInterface $webform_submission, string $page) {

    // Stash the current page.

    $current_page = $webform_submission->getCurrentPage();

    // Let's ensure we are on the page that needs to be validated.