Problem/Motivation
We'd like to use the webform signature element with the private file system.
Proposed resolution
Patch the signature element so that there is an option to store private:// or public://
User interface changes
Extra option on the element widget.
API changes
N/A internal to the widget.
Data model changes
N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #28 | 3241469-28.patch | 111.25 KB | tannguyenhn |
| #27 | 3241469-27.patch | 112.79 KB | tannguyenhn |
| #22 | 3241469-22.patch | 113.92 KB | tannguyenhn |
| #15 | 3241469-15.patch | 25.46 KB | tannguyenhn |
| #14 | 3241469-13.patch | 25.46 KB | tannguyenhn |
Issue fork webform-3241469
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
simeInitial patch to follow.
Comment #3
tannguyenhn commentedComment #4
jrockowitz commentedI will try to find time to review the patch.
Ideally, we should backport this patch to 8.x-5.x.
The failing test is very easy to fix.
Comment #5
tannguyenhn commentedThanks Jrockowitz
I have updated the patch to fix the failing test.
Comment #6
tannguyenhn commentedComment #7
pandaski commentedWe are testing it in GovCMS distribution
Comment #8
jrockowitz commentedFirst off, the solution and approach are very good.
The attached patch has few tweaks
Created WebformElementFileDownloadAccessInterface
Improve readabilty of WebformSignature::accessFileDownload
Adds very basic test coverage
A few remaining questions/issues.
PDF generation with private signatures is not working. I think we can fix this.
@see \webform_entity_print_webform_submission_access
Comment #9
jrockowitz commentedThis patch add PDF support to the private signature element.
Comment #10
jrockowitz commentedI think the last step is to add test coverage for including a "private" signature in PDFs via WebformEntityPrintFunctionalTest.php and webform.webform.test_entity_print.yml
If someone has time to manually review the patch, it would be greatly appreciated.
Comment #11
tannguyenhn commentedAdded test coverage for including a "private" signature in PDFs via WebformEntityPrintFunctionalTest.php and webform.webform.test_entity_print.yml
Comment #13
tannguyenhn commentedComment #14
tannguyenhn commentedAdded test coverage for including a "private" signature in PDFs via WebformEntityPrintFunctionalTest.php and webform.webform.test_entity_print.yml
Comment #15
tannguyenhn commentedChange patch name to match with comment number.
Comment #16
tannguyenhn commentedComment #17
tannguyenhn commentedUpdate the patch
Comment #18
tannguyenhn commentedThis patch works with version 6.0.5.
Comment #19
tannguyenhn commentedThis patch works with version 6.x-dev.
Comment #21
tannguyenhn commentedComment #22
tannguyenhn commentedComment #23
tannguyenhn commentedDefault uri schema is private.
Comment #24
tannguyenhn commentedAnother patch for 6.0.5 webform version.
Comment #25
tannguyenhn commentedComment #26
tannguyenhn commentedComment #27
tannguyenhn commentedComment #28
tannguyenhn commentedThis patch is for 6.0.5 version
Comment #29
tannguyenhn commentedComment #30
jrockowitz commentedI am hoping to do a manual review this week.
Comment #31
jrockowitz commentedComment #32
jrockowitz commentedI am moving the patch to branch and PR (i.e. GitLab).
We must continue to support the public file system for signature elements to support backward compatibility.
Steps to manually review
Comment #34
jrockowitz commented