Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
The salesforce_webform submodule is an excellent add-on, and arrived just in time for my project. It would be a great addition to handle fields that are entity references (fields that support the WebformEntityTrait) akin to how the related elements field from the main part of the suite so that the SFID of mapped fields gets sent instead of the Drupal ID.
Comment | File | Size | Author |
---|---|---|---|
#10 | interdiff.txt | 1.41 KB | AaronBauman |
#10 | salesforce-webform_entity_references-3047748-10.patch | 4.29 KB | AaronBauman |
#7 | Webform-Entity-Field-3047748-7.patch | 4.28 KB | acrosman |
Comments
Comment #2
acrosmanI'm happy to work on getting this started, but I'm looking for feedback on whether this should be handled as a separate plugin entirely or if it should be included as part of the existing plugin and just detect the trait and search of a map. Left to my own devices I'm inclined toward to separate plugin.
Comment #3
acrosmanFixed totally incorrect title.
Comment #4
acrosmanInitial patch attached. Provides a second webform field mapping plugin to handle webform module's entity plugins. Right now it detects them via naming convention, which seems less than ideal but works on the webform core element just fine.
Comment #5
acrosmanComment #6
AaronBaumanLooks like this should implement getDependencies and isAllowed to add conditionality on webform_entity module
See \Drupal\salesforce_mapping\Plugin\SalesforceMappingField\PropertiesExtended for example
Comment #7
acrosmanUpdated patch to include dependency checking. Also fixes an issue with entity autocomplete fields getting skipped for mappings.
Comment #8
acrosmanComment #9
AaronBaumanHere's an updated version with changes from #3065470: getConfigurationOptions should return all the element for the webform
Also, now that I'm looking more closely, why are we doing this in ::value()?
I'm guessing this is cruft from 7.x, but let me know if you know a reason we should keep it.
Comment #10
AaronBaumanand the patch
Comment #11
AaronBaumanComment #13
AaronBaumanThis is in