The token [submission:access-token] should exist with value returned by webform_get_submission_access_token(). This would make it easier to generate links to the submission results page. If the maintainer agrees with this request and the token name, I will find time to work on this.

CommentFileSizeAuthor
#3 webform-access_token-2474773-3-D7.patch1.18 KBLiam Morland
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented

Seems like a good idea. It's unfortunate that we are using "token" to mean two totally different things. I don't have a suggestion for what to do about that though.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented

For the record, I'm okay with the proposed token name. A token that gives you a token. Sigh.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Active » Needs review
FileSize
webform-access_token-2474773-3-D7.patch1.18 KB

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Status: Needs review » Fixed

Tested and confirmed that token works as advertised. Nice.

For reference, here's a link to the submission that could be included in an e-mail or confirmation page:
Go to your submission: <a href="/node/[node:nid]/submission/[submission:sid]?token=[submission:access-token]">Submission [submission:serial]</a>

Committed to 7.x. Thanks, Liam.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Category: Feature request » Task
Status: Fixed » Patch (to be ported)

Needs port to D8.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Version: 7.x-4.x-dev » 8.x-4.x-dev

D'oh.

Status: Patch (to be ported) » Needs work

The last submitted patch, 3: webform-access_token-2474773-3-D7.patch, failed testing.

DanChadwick’s picture

DanChadwick CreditAttribution: DanChadwick commented
Status: Needs work » Patch (to be ported)

Um, thanks, testbot. You can't apply the patch because it's already committed and you've already passed the branch with the commit.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented

Thanks, Dan.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
CreditAttribution: fenstrat commented
Version: 8.x-4.x-dev » 7.x-4.x-dev
Category: Task » Feature request
Status: Patch (to be ported) » Fixed

Committed and pushed to 8.x-4.x. Thanks!

  • fenstrat committed a7b47e1 on 8.x-4.x authored by Liam Morland 
    Issue #2474773 by Liam Morland: Token [submission:access-token] should...

Status: Fixed » Closed (fixed)

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