Marked the interval fields as required.
The attached patch adds timestamp value push support by converting timestamp into supported by Salesforce format.
The code originally from the last patch in old discussion branch
https://www.drupal.org/project/salesforce/issues/3122412#comment-14116705
π
Mapped date fields are not properly syncing data.
Closed: outdated
,
additionally fixed bugs
- 'date' salesforce type converting
- possibly missing requested drupal field definition (due to 'drupal_field_value' may have not a drupal field name value)
Tests coverage still needed.
Confirm that solution from #5resolves the issue.
I agree that the example should be added to SalesforceExampleSubscriber
added patch according to above comment
@audiomason, nice job for finding such an issue. I have some adjustments for your patch.
It's better to use a setter method setEntityValue(), which indicates that change is intentional and controlled, rather than getting the reference of the value and change it directly.
public function __construct(&$value, SalesforceMappingFieldPluginInterface $fieldPlugin, MappedObjectInterface $mappedObject) {
$this->entityValue = &$value;
// rest of code...
}
/**
* Setter for entity value.
*
* @param mixed $value
* The value to be assigned.
*/
public function setEntityValue($value) {
$this->entityValue = $value;
}
knyshuk.vova β created an issue.