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

ruby conventions #167

Open
SampsonCrowley opened this issue Oct 5, 2018 · 4 comments
Open

ruby conventions #167

SampsonCrowley opened this issue Oct 5, 2018 · 4 comments
Labels
redesign Request for SDK redesign

Comments

@SampsonCrowley
Copy link

Your ruby code blantantly ignores most ruby conventions (e.g. underscore_method_names) and uses (what looks like) java based naming schemes everywhere

Is an expectation to migrate this sdk to use ruby conventions ever going to be reasonable for a major release? I wouldn't mind helping, if that isn't something you would be opposed to

@SampsonCrowley
Copy link
Author

also related to #143

@gnongsie
Copy link
Contributor

Hi,

Thank you for your suggestions and offer.

This message has been communicated to the team and we will try to adhere to the conventions of the language as much as possible.

We will keep you in mind, should any help be required.

@gnongsie gnongsie added the redesign Request for SDK redesign label Oct 23, 2018
@taf2
Copy link

taf2 commented Jan 13, 2020

I believe this also is breaking rails 6 with zeitwerk

Hold on, I am eager loading the application.

WARNING: The files in these directories cannot be checked because they
are not eager loaded:

  /Users/taf2/.rvm/gems/ruby-2.6.5@ctm3/gems/authorizenet-2.0.0/lib/app/helpers

You may verify them manually, or add them to config.eager_load_paths
in config/application.rb and run zeitwerk:check again.

Otherwise, all is good!

@gbs4ever
Copy link

gbs4ever commented Apr 11, 2024

I have to second this, the entire SDK was written in Ruby as if its Java also related to 143
eg you should pass in the attributes as parameters on object creation, in addition, use key-value pairs for the parameters so the order is not important

request.sorting = TransactionListSorting.new;
 request.sorting.orderBy = TransactionListOrderFieldEnum::Id;
  request.sorting.orderDescending = true;    
    vs   request.sorting =TransactionListSorting.new(orderBy:TransactionListOrderFieldEnum::Id, orderDescending:true);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
redesign Request for SDK redesign
Development

No branches or pull requests

4 participants