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

Comments

danchadwick’s picture

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 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 🇨🇦
commented
Status: Active » Needs review
StatusFileSize
new webform-access_token-2474773-3-D7.patch1.18 KB

danchadwick’s picture

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 commented
Category: Feature request » Task
Status: Fixed » Patch (to be ported)

Needs port to D8.

danchadwick’s picture

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 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 🇨🇦
commented

Thanks, Dan.

fenstrat’s picture

fenstrat
he/him
Primary language English
Location Australia
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.