Problem/Motivation

Now that anonymous submissions are tracked via $_SESSION, we can setup basic submission limits for anonymous users.

Proposed resolution

Enable user submission limit for anonymous users.

Remaining tasks

  • Update \Drupal\webform\WebformSubmissionForm::checkUserLimit
  • Cleanup "Submission limit" settings.
  • Add warning to "Per user submission limit" look at D7 code.
  • Write test for anonymous submissions.

User interface changes

Updates to Webform settings.

API changes

None

Data model changes

None

CommentFileSizeAuthor
#6 enable_user_submission-2888619-5.patch8.88 KBjrockowitz
#3 enable_user_submission-2888619-3.patch7.37 KBjrockowitz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jrockowitz created an issue. See original summary.

  • jrockowitz committed 58ee56a on 2888619-anonymous-submission-limits 
    Issue #2888619: Enable user submission limit for anonymous users
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Active » Needs review
FileSize
enable_user_submission-2888619-3.patch7.37 KB

Status: Needs review » Needs work

The last submitted patch, 3: enable_user_submission-2888619-3.patch, failed testing. View results

  • jrockowitz committed ba7714a on 2888619-anonymous-submission-limits 
    Issue #2888619: Enable user submission limit for anonymous users
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs work » Needs review
FileSize
enable_user_submission-2888619-5.patch8.88 KB

  • jrockowitz committed a8246e4 on 8.x-5.x 
    Issue #2888619 by jrockowitz: Enable user submission limit for anonymous...
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.