In the method getRemoteIdByUUID() when the request ends up with an exception the caller will not notice it. The logic in the doSync() method will then evaluate this as the remote entity does not exist and will attempt for create action.

To fix this the exception from getRemoteIdByUUID() should bubble up. See the attached patch.

CommentFileSizeAuthor
sc_throw_exception_on_get_remote_id_call.patch1.38 KBblueminds
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

blueminds created an issue. See original summary.

irek02’s picture

irek02 CreditAttribution: irek02 at Acquia commented
Status: Active » Reviewed & tested by the community

The patch looks sane!

  • ad72435 committed on 7.x-2.x 
    Issue #2761417 by blueminds: Allow the exception to bubble up when...
blueminds’s picture

blueminds CreditAttribution: blueminds commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.