Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Camel_SalesForce: Give possibility to update OwnerId #1869

Closed
wants to merge 1 commit into from

Conversation

Sw1m1k
Copy link
Contributor

@Sw1m1k Sw1m1k commented Aug 4, 2017

Commented: OwnerId = null

Sometimes need to update Owner.

Commented OwnerId = null
@Sw1m1k Sw1m1k changed the title Update AbstractSObjectBase.java Camel_SalesForce: Give possibility to update OwnerId Aug 4, 2017
@davsclaus
Copy link
Contributor

@Sw1m1k can you maybe tell a bit more about your use-case for this need?

@zregvart do you mind looking at this PR ?

@zregvart
Copy link
Member

zregvart commented Aug 7, 2017

@Sw1m1k thank you for the pull request, clearBaseFields method is used when trying to update the record in Salesforce, we need to set certain fields to null as they cannot be modified via REST API. Or at least that stands at the moment that code was written, it could be that the newer editions of Salesforce removed this restriction.
I think using workflow rules would be a better way of managing ownership.

@zregvart
Copy link
Member

@Sw1m1k Hi, thanks for the PR, can you elaborate on the use case?

@Sw1m1k
Copy link
Contributor Author

Sw1m1k commented Aug 28, 2017

@zregvart Hi, I didn't found any restriction for updating field "OwnerId" by SalesForce API. You can check it by invoking SalesForce REST API.
Very often we need to create or update, for example, opportunity and associate it with some person. Now this is restricted only by camel-salesforce liblary.

@zregvart
Copy link
Member

@Sw1m1k thank you! I'll just make sure that all tests pass before merging this.

@zregvart
Copy link
Member

zregvart commented Sep 3, 2017

Hi @Sw1m1k, thank you for this I've logged it as CAMEL-11744 and it has been fixed in the current master branch scheduled to be released in version 2.20. Can you close this pull request? 👍

@Sw1m1k
Copy link
Contributor Author

Sw1m1k commented Sep 6, 2017

Ok, Thanks

@Sw1m1k Sw1m1k closed this Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants