Let MultiJson select default JSON adapter (engine), support for latest MultiJson syntax

ccocchi / rabl-rails

Rails 4.2+ templating system with JSON, XML and Plist support.

MIT License

209 stars 51 forks source link

Let MultiJson select default JSON adapter (engine), support for latest MultiJson syntax #6

Closed shmeltex closed 11 years ago

shmeltex commented 11 years ago

This change should give one more reason to use default configuration. json_engine configuration option will be needed only if fastest installed JSON engine is not a desired option.

ccocchi commented 11 years ago

It is a good idea to let MultiJson choose the faster JSON installed.

Concerning old syntax, I'm ok to use it. As long as it aliased, I prefer to use it instead of addind the respond_to? overhead.

If you can just changed back to the old syntax, I will be happy to merge.

ccocchi commented 11 years ago

If you can also add an entry to the changelog that would be perfect :D

shmeltex commented 11 years ago

Please let me know if it's good to go.

ccocchi commented 11 years ago

Seems good