Closed (won't fix)
Project:
Webform
Version:
6.1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
27 Mar 2023 at 17:25 UTC
Updated:
26 May 2023 at 23:30 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
alangallery commentedIn case its of use to others, for now I've added a pipe option to the functions formatHtmlItems and formatTextItems in WebformElementBase with this patch.
Comment #3
jrockowitz commentedPlease fix the spacing in the patch and provide test coverage.
Comment #4
jrockowitz commentedComment #5
jrockowitz commentedComment #6
cilefen commentedComment #7
alangallery commentedNot sure why this patch is failing. For me the patch applies okay on 6.2-beta5 and 6.2-dev
Comment #8
cilefen commentedIs that patch made with something besides Git?
Also: you probably don’t need to test every platform combination at this stage.
Comment #9
alangallery commentedThanks cilefen. I just used diff.
The patch test failure message is error: Plugin/WebformElementBase.php: No such file or directory
Comparing a patch generated with Git its very similar to this one but with a/ b/ prefixes indicating the modified files. eg.
I could amend the patch accordingly and upload again?
Comment #10
jrockowitz commentedComment #11
jrockowitz commentedComment #12
jrockowitz commentedI just realized you could use custom Twig for pipe delimited values.
See attached the webform and below is the example Twig.
{{ items|join(''|'') }}