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

Restricted Zuora Bank Transfer Account Number is included in PaymentMethod queries #25

Closed
cdelrosario opened this issue Dec 14, 2014 · 3 comments

Comments

@cdelrosario
Copy link
Contributor

When doing a select on PaymentMethod, the Zuora write-only field BankTransferAccountNumber is included in queries which is causing the query to fail.

Suggestion, these types of restricted fields should be configurable via config so we do not have to wait for GEM updates to fix them. At the moment we had to enumerate all the fields in our queries, not ideal but it works.

cdelrosario added a commit to cdelrosario/active_zuora that referenced this issue Dec 15, 2014
cdelrosario added a commit to cdelrosario/active_zuora that referenced this issue Dec 15, 2014
@anfleene
Copy link
Contributor

I think thats a decent idea. We've run into this issue a few times. I think the main issue is that the wsdl doesn't include any metadata about whether a field is queryable or not. It will be hard for us to always be 100% up to date on which fields need to be excluded. I think your proposed solution is a good stop gap.

@cdelrosario
Copy link
Contributor Author

Thank you and also for doing the merge so quickly. Will update today :-)

When does it get published to RubyGems?

On Wednesday, 17 December 2014, Andy Fleener notifications@github.com
wrote:

I think thats a decent idea idea. We've run into this issue a few times. I
think the main issue is that the wsdl doesn't include any metadata about
whether a field is queryable or not. It will be hard for use to alway be
100% up to date on which fields need to be excluded. I think your proposed
solution is a good stop gap.


Reply to this email directly or view it on GitHub
#25 (comment)
.

Cenon Del Rosario
Fairfax Media Australia
1 Darling Island Road, Pyrmont

The information contained in this e-mail message and any accompanying files
is or may be confidential. If you are not the intended recipient, any use,
dissemination, reliance, forwarding, printing or copying of this e-mail or
any attached files is unauthorised. This e-mail is subject to copyright. No
part of it should be reproduced, adapted or communicated without the
written consent of the copyright owner. If you have received this e-mail in
error please advise the sender immediately by return e-mail or telephone
and delete all copies. Fairfax Media does not guarantee the accuracy or
completeness of any information contained in this e-mail or attached files.
Internet communications are not secure, therefore Fairfax Media does not
accept legal responsibility for the contents of this message or attached
files.

@anfleene
Copy link
Contributor

These changes went out. Sorry for the screwup on releasing it initially.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants