Formatting - camelCase rather than underscore

[crhayes]

How do you feel about having the methods converted to camel case so it is compliant with the PSR-1 spec? (https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md)

If you are open to it I can definitely help with the changes!

[treffynnon]

If you can think of a way this can be done without breaking backwards
compatibility then I welcome suggestions. How would you consider
implementing this?

Naturally change for no reason is not really beneficial to the current
users of the project. Idiorm and Paris are not members of FIG either so
there is no perceived obligation to amend the API to match any of the PSRs.

Please see my recent blog post for more information:
http://simonholywell.com/post/2013/01/idiorm-and-paris-the-minimalist-orm.html
On Mar 21, 2013 6:07 PM, "Chris Hayes" [email protected] wrote:

How do you feel about having the methods converted to camel case so it is
compliant with the PSR-1 spec? (
https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-1-basic-coding-standard.md
)

If you are open to it I can definitely help with the changes!

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/108
.

[crhayes]

This is a simple approach and it's backwards compatible as well. The main advantage here is that users of the library can choose between coding styles, and (as in my case) if their whole project is in camel case the use of this library doesn't have to differ.

What do you think?

[treffynnon]

This looks great and it is what I was hoping to see. If I was ever to make this change to Idiorm myself this is how I was going to do it. Do you use Paris? These changes will also need to be reflected in Paris as well.

[crhayes]

Awesome, thanks! I will be looking to use Paris as well, so I will fork it and work on those changes over the weekend.

[tag]

Great job, straightforward solution (and what I was considering as well, after your initial suggestion). The best part is a future version could switch to camel case for the main definitions, and use underscores as a deprecated backward-compatible option, without any loss of functionality.

[treffynnon]

a future version could switch to camel case for the main definitions, and use underscores as a deprecated

And @tag , therein lies the beauty and actual eventual aim :)

[treffynnon]

@crhayes How have you been getting on with the Paris changes?

[crhayes]

@treffynnon I haven't had a chance to get to them yet, but thanks for the reminder. It's been a crazy couple of weeks with some freelance work and it slipped my mind. Still planning to do it as soon as things ease up.

[treffynnon]

Thank you for the pull request. Would it be possible to get some tests and update documentation for the functionality, please?

[treffynnon]

This is still going begging for tests and documentation. Anyone?

[treffynnon]

Merged in commit 7b1e19f
Documented and tested in commit 7f59e95