rabl 0.13.0 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 9181c1a232aab327e58bd686eb3cf14403e6aac9
-  data.tar.gz: 3e2f0d536f6752cb575e5843709be0b40f8bd09a
+SHA256:
+  metadata.gz: 0fb8310726d92ea377d5be433f173207997ea6cc7e3a4a3faefe71b7de5b5df2
+  data.tar.gz: e75ceb99863c03bff3093983da87c83e79a02c8f1d74cc2ca6dd6c4755fd8404
 SHA512:
-  metadata.gz: 8ea05c8211d750c882c4b0fa6e29a20b56e4ebfcff23e12a2af18e0f50302b191be374a76eace11ed46a787ae702cb7ff5eb7ef659a6d7096db2823451c5b6d1
-  data.tar.gz: c8634ea3cfbbc11e131b484e598326888c81f15f2a304caf06ec786f8a3ab5d72cfb77f88fc47b4fd34d1dc595c3468ef5c460d44effc949c352dd6ca23d7398
+  metadata.gz: 2fc9988668325cc4dd12e75a774d14f6be984a83d2ed043712536d3753d6ae70b71d09d6784120f82b2c75809ed8d446c1965e56fcac1ddbc9c4bace259fc342
+  data.tar.gz: 58a0105a9fdf9e016ddee845ca3a4fde7fb0944a1792230f878ce380d2b9778e8d82da3722ba89d1702442e3d539c4db071ff56e201f596625d12f9ffaa90585

data/CHANGELOG.md

@@ -1,5 +1,40 @@
 # CHANGELOG
 
+## 0.15.0 (December 30, 2021)
+
+* Support for Rails 7 (@tagliala)
+* Improvements to unit testing
+* Miscellanious other bug fixes
+
+## 0.14.5 (May 29th, 2021)
+
+* Restrict and validate the `to_***` methods available within engine (Brian McFadden)
+
+## 0.14.4 (April 1st, 2021)
+
+* Test against latest Ruby versions
+* Exclude yarn lock
+* Remove non-required files from the .gem
+
+## 0.14.3 (April 22nd, 2020)
+
+* Fix regression on Rails 4.2 (@tagliala)
+
+## 0.14.2 (September 10th, 2019)
+
+* Made Rails 6 compatible (@jpawlyn)
+* Drop EOL'd property from gemspec (@olleolleolle)
+
+## 0.14.1 (April 28th, 2019)
+
+* Many adjustments and various non-breaking fixes
+* Improvements to caching (@zenspider)
+* Fix tests for Rails 5 (@robertoz)
+
+## 0.13.1 (October 21st, 2016)
+
+* Further compatibility with Rails 5
+
 ## 0.13.0 (July 15th, 2016)
 
 * Compatibility with Rails 5

data/CONTRIBUTING.md

@@ -3,7 +3,7 @@ We love pull requests. Here's a quick guide:
 1. Fork the repo.
 
 2. Run the tests. We only take pull requests with passing tests, and it's great
-to know that you have a clean slate: `bundle && rake test`
+to know that you have a clean slate: `bundle && bundle exec rake test`
 
 3. Add a test for your change. Only refactoring and documentation changes
 require no new tests. If you are adding functionality or fixing a bug, we need
@@ -34,4 +34,4 @@ Syntax:
 
 And in case we didn't emphasize it enough: we love tests!
 
-NOTE: Adapted from https://raw.github.com/thoughtbot/factory_girl_rails/master/CONTRIBUTING.md
+NOTE: Adapted from https://raw.githubusercontent.com/thoughtbot/factory_bot_rails/main/CONTRIBUTING.md

data/README.md

@@ -1,8 +1,7 @@
 # RABL #
 
-[![Continuous Integration status](https://secure.travis-ci.org/nesquena/rabl.png)](http://travis-ci.org/nesquena/rabl)
-[![Dependency Status](https://gemnasium.com/nesquena/rabl.png)](https://gemnasium.com/nesquena/rabl)
-[![Code Climate](https://codeclimate.com/github/nesquena/rabl.png)](https://codeclimate.com/github/nesquena/rabl)
+[![Continuous Integration status](https://github.com/nesquena/rabl/actions/workflows/ci.yml/badge.svg)](https://github.com/nesquena/rabl/actions)
+[![Code Climate](https://codeclimate.com/github/nesquena/rabl.svg)](https://codeclimate.com/github/nesquena/rabl)
 
 RABL (Ruby API Builder Language) is a Rails and [Padrino](http://padrinorb.com) ruby templating system for generating JSON, XML, MessagePack, PList and BSON.
 When using the ActiveRecord 'to_json' method, I find myself wanting a more expressive and powerful solution for generating APIs.
@@ -146,6 +145,7 @@ Rabl.configure do |config|
   # config.replace_empty_string_values_with_nil_values = true # Defaults to false
   # config.exclude_nil_values = true # Defaults to false
   # config.exclude_empty_values_in_collections = true # Defaults to false
+  # config.camelize_keys = :upper # Defaults to false
 end
 ```
 
@@ -184,7 +184,9 @@ If `replace_nil_values_with_empty_strings` is set to `true`, all values that are
 
 If `exclude_nil_values` is set to `true`, all values that are `nil` and would normally be displayed as `null` in the response are not included in the response.
 
-if `exclude_empty_values_in_collections` is set to `true`, all vaules in a collection that are `{}` and would normally be displayed as `{}` in the response are not included in the response.
+if `exclude_empty_values_in_collections` is set to `true`, all values in a collection that are `{}` and would normally be displayed as `{}` in the response are not included in the response.
+
+If `camelize_keys` is set to `true`, all object keys will be converted to camel case. By default the first character will be lower case. The value can be set to `:upper` to set the first character to upper case.
 
 If you wish to use [oj](https://github.com/ohler55/oj) as
 the primary JSON encoding engine simply add that to your Gemfile:
@@ -300,6 +302,13 @@ Named and aliased attributes can not be combined on the same line. This currentl
 attributes :foo, :bar => :baz # throws exception
 ```
 
+in such cases, you need to specify attributes twice.
+
+```ruby
+attributes :foo
+attributes :bar => :baz
+```
+
 ### Child Nodes ###
 
 Often a response requires including nested information from data associated with the parent model:
@@ -580,11 +589,11 @@ You can also pass in other instance variables to be used in your template as:
 Rabl::Renderer.new('posts/show', @post, :locals => { :custom_title => "Hello world!" })
 ````
 
-Then, in your template, you can use `@custom_title` as:
+Then, in your template, you can use `locals[:custom_title]` as:
 
-```
+```ruby
 attribute :content
-node(:title) { @custom_title }
+node(:title) { locals[:custom_title] }
 ```
 
 ### Content Type Headers ###
@@ -624,13 +633,9 @@ Tutorials can always be helpful when first getting started:
 
  * [Railscasts #322](http://railscasts.com/episodes/322-rabl) - Ryan Bates explains RABL
  * [BackboneRails](http://www.backbonerails.com/) - Great screencasts by Brian Mann
- * [Creating an API with RABL and Padrino](http://blog.crowdint.com/2012/10/22/rabl-with-padrino.html)
  * http://blog.joshsoftware.com/2011/12/23/designing-rails-api-using-rabl-and-devise/
- * http://engineering.gomiso.com/2011/06/27/building-a-platform-api-on-rails/
- * http://blog.lawrencenorton.com/better-json-requests-with-rabl
  * http://tech.favoritemedium.com/2011/06/using-rabl-in-rails-json-web-api.html
  * http://seesparkbox.com/foundry/better_rails_apis_with_rabl
- * http://blog.dcxn.com/2011/06/22/rails-json-templates-through-rabl
  * http://teohm.github.com/blog/2011/05/31/using-rabl-in-rails-json-web-api
 
 Let me know if there's any other useful resources not listed here.

data/lib/rabl/builder.rb

@@ -1,3 +1,5 @@
+require 'active_support/inflector' # for the sake of camelcasing keys
+
 module Rabl
   class Builder
     include Helpers
@@ -135,10 +137,13 @@ module Rabl
       # attribute :foo, :as => "bar"
       # attribute :foo, :as => "bar", :if => lambda { |m| m.foo }
       def attribute(name, options = {})
-        return unless @_object && attribute_present?(name) && resolve_condition(options)
+        return unless
+          @_object &&
+          attribute_present?(name) &&
+          resolve_condition(options)
 
         attribute = data_object_attribute(name)
-        name = (options[:as] || name).to_sym
+        name = create_key(options[:as] || name)
         @_result[name] = attribute
       end
       alias_method :attributes, :attribute
@@ -151,7 +156,7 @@ module Rabl
 
         result = block.call(@_object)
         if name.present?
-          @_result[name.to_sym] = result
+          @_result[create_key(name)] = result
         elsif result.is_a?(Hash) # merge hash into root hash
           @_result.merge!(result)
         end
@@ -174,7 +179,7 @@ module Rabl
 
         object = { object => name } if data.is_a?(Hash) && object # child :users => :people
 
-        engines << { name.to_sym => object_to_engine(object, engine_options, &block) }
+        engines << { create_key(name) => object_to_engine(object, engine_options, &block) }
       end
 
       # Glues data from a child node to the json_output
@@ -198,10 +203,11 @@ module Rabl
 
       # Evaluate conditions given a symbol/proc/lambda/variable to evaluate
       def call_condition_proc(condition, object)
-        # This will evaluate lambda, proc & symbol and call it with 1 argument
-        return condition.to_proc.call(object) if condition.is_a?(Proc) || condition.is_a?(Symbol)
-        # Else we send directly the object
-        condition
+        case condition
+        when Proc   then condition.call(object)
+        when Symbol then condition.to_proc.call(object)
+        else             condition
+        end
       end
 
       # resolve_condition(:if => true) => true
@@ -211,8 +217,10 @@ module Rabl
       # resolve_condition(:unless => lambda { |m| false }, :if => proc { true}) => true
       def resolve_condition(options)
         result = true
-        result &&= call_condition_proc(options[:if], @_object) if options.key?(:if)
-        result &&= !call_condition_proc(options[:unless], @_object) if options.key?(:unless)
+        result &&=  call_condition_proc(options[:if], @_object)     if
+          options.key?(:if)
+        result &&= !call_condition_proc(options[:unless], @_object) if
+          options.key?(:unless)
         result
       end
 
@@ -220,13 +228,9 @@ module Rabl
       # Checks if an attribute is present. If not, check if the configuration specifies that this is an error
       # attribute_present?(created_at) => true
       def attribute_present?(name)
-        if @_object.respond_to?(name)
-          true
-        elsif Rabl.configuration.raise_on_missing_attribute
-          raise "Failed to render missing attribute #{name}"
-        else
-          false
-        end
+        @_object.respond_to?(name) ||
+          (Rabl.configuration.raise_on_missing_attribute &&
+           raise("Failed to render missing attribute #{name}"))
       end
 
       # Returns a guess at the format in this context_scope
@@ -248,5 +252,13 @@ module Rabl
           yield
         end
       end
+
+      def create_key(name)
+        if Rabl.configuration.camelize_keys
+          name.to_s.camelize(Rabl.configuration.camelize_keys == :upper ? :upper : :lower).to_sym
+        else
+          name.to_sym
+        end
+      end
   end
 end

data/lib/rabl/cache_engine.rb

@@ -5,8 +5,35 @@
 #     config.cache_engine = AdvancedCacheEngine.new
 #
 
+class LRU < Hash
+  attr_accessor :max_size
+
+  def initialize
+    super
+    self.max_size = 100_000
+  end
+
+  def []= k,v
+    r = super
+    limit_size
+    r
+  end
+
+  def limit_size
+    if size > max_size then
+      delete keys.shift while size > max_size
+    end
+  end
+end
+
 module Rabl
   class CacheEngine
+    def initialize
+      unless defined?(Rails)
+        @cache = LRU.new
+      end
+    end
+
 
     # Fetch given a key and options and a fallback block attempts to find the key in the cache
     # and stores the block result in there if no key is found.
@@ -17,13 +44,15 @@ module Rabl
       if defined?(Rails)
         Rails.cache.fetch(key, cache_options, &block)
       else
-        yield
+        @cache[key] ||= yield
       end
     end
 
     def write(key, value, options = {})
       if defined?(Rails)
         Rails.cache.write(key, value, options)
+      else
+        @cache[key] = yield
       end
     end
 

data/lib/rabl/configuration.rb

@@ -7,7 +7,7 @@ end
 # We load the bson library if it is available.
 begin
   require 'bson'
-  rescue LoadError
+rescue LoadError
 end
 
 # We load the plist library if it is available.
@@ -52,6 +52,7 @@ module Rabl
     attr_accessor :replace_empty_string_values_with_nil_values
     attr_accessor :exclude_nil_values
     attr_accessor :exclude_empty_values_in_collections
+    attr_accessor :camelize_keys
 
     DEFAULT_XML_OPTIONS = { :dasherize  => true, :skip_types => false }
 
@@ -81,6 +82,7 @@ module Rabl
       @replace_empty_string_values_with_nil_values  = false
       @exclude_nil_values                           = false
       @exclude_empty_values_in_collections          = false
+      @camelize_keys                                = false
     end
 
     # @return The JSON engine used to encode Rabl templates into JSON

data/lib/rabl/digestor/rails3.rb

@@ -0,0 +1,12 @@
+require 'action_view'
+
+module Rabl
+  class Digestor < ActionView::Digestor
+    def self.digest(name, format, finder, options = {})
+      cache_key = [name, format] + Array.wrap(options[:dependencies])
+      @@cache[cache_key.join('.')] ||= begin
+        Digestor.new(name, format, finder, options).digest
+      end
+    end
+  end
+end

data/lib/rabl/digestor/rails41.rb

@@ -0,0 +1,12 @@
+require 'action_view'
+
+module Rabl
+  class Digestor < ActionView::Digestor
+    def self.digest(options = {})
+      cache_key = [options[:name]] + Array.wrap(options[:dependencies])
+      @@cache[cache_key.join('.')] ||= begin
+        Digestor.new({ :name => options[:name], :finder => options[:finder] }.merge!(options)).digest
+      end
+    end
+  end
+end

data/lib/rabl/digestor/rails5.rb

@@ -0,0 +1,6 @@
+require 'action_view'
+
+module Rabl
+  class Digestor < ActionView::Digestor
+  end
+end

data/lib/rabl/digestor.rb

@@ -1,47 +1,15 @@
+require 'action_view'
+
 module Rabl
   class Digestor < ActionView::Digestor
     # Override the original digest function to ignore partial which
     # rabl doesn't use the Rails conventional _ symbol.
     if Gem::Version.new(Rails.version) >= Gem::Version.new('5.0.0.beta1')
-      def self.digest(name:, finder:, **options)
-
-        options.assert_valid_keys(:dependencies, :partial)
-
-        cache_key = ([ name ].compact + Array.wrap(options[:dependencies])).join('.')
-
-        # this is a correctly done double-checked locking idiom
-        # (Concurrent::Map's lookups have volatile semantics)
-        finder.digest_cache[cache_key] || @@digest_monitor.synchronize do
-          finder.digest_cache.fetch(cache_key) do # re-check under lock
-            begin
-              # Prevent re-entry or else recursive templates will blow the stack.
-              # There is no need to worry about other threads seeing the +false+ value,
-              # as they will then have to wait for this thread to let go of the @@digest_monitor lock.
-
-              pre_stored = finder.digest_cache.put_if_absent(cache_key, false).nil? # put_if_absent returns nil on insertion
-
-              finder.digest_cache[cache_key] = stored_digest = Digestor.new(name, finder, options).digest
-            ensure
-              # something went wrong or ActionView::Resolver.caching? is false, make sure not to corrupt the @@cache
-              finder.digest_cache.delete_pair(cache_key, false) if pre_stored && !stored_digest
-            end
-          end
-        end
-      end
+      require 'rabl/digestor/rails5'
     elsif Gem::Version.new(Rails.version) >= Gem::Version.new('4.1')
-      def self.digest(options = {})
-        cache_key = [options[:name]] + Array.wrap(options[:dependencies])
-        @@cache[cache_key.join('.')] ||= begin
-          Digestor.new({ :name => options[:name], :finder => options[:finder] }.merge!(options)).digest
-        end
-      end
+      require 'rabl/digestor/rails41'
     else
-      def self.digest(name, format, finder, options = {})
-        cache_key = [name, format] + Array.wrap(options[:dependencies])
-        @@cache[cache_key.join('.')] ||= begin
-          Digestor.new(name, format, finder, options).digest
-        end
-      end
+      require 'rabl/digestor/rails3'
     end
 
     private

data/lib/rabl/engine.rb

@@ -6,6 +6,7 @@ module Rabl
 
     # List of supported rendering formats
     FORMATS = [:json, :xml, :plist, :bson, :msgpack]
+    SAFE_FORMATS = FORMATS + [:mpac, :dumpable, :hash]
 
     # Constructs a new ejs engine based on given vars, handler and declarations
     # Rabl::Engine.new("...source...", { :format => "xml", :root => true, :view_path => "/path/to/views" })
@@ -27,6 +28,8 @@ module Rabl
     # Renders the representation based on source, object, context_scope and locals
     # Rabl::Engine.new("...source...", { :format => "xml" }).apply(context_scope, { :foo => "bar", :object => @user })
     def apply(context_scope, locals, &block)
+      locals = locals.dup unless locals.nil?
+
       set_instance_variables!(context_scope, locals)
 
       reset_settings!
@@ -58,8 +61,10 @@ module Rabl
         if digestor_available? && respond_to?(:lookup_context) && lookup_context
           template = @_options[:template] || @virtual_path
 
-          digest = \
-            if Gem::Version.new(Rails.version) >= Gem::Version.new('4.1')
+          digest =
+            if Rails.version.to_s =~ /^[67]/
+              Digestor.digest(name: template, finder: lookup_context, format: :rabl)
+            elsif Gem::Version.new(Rails.version) >= Gem::Version.new('4.1')
               Digestor.digest(:name => template, :finder => lookup_context)
             else
               Digestor.digest(template, :rabl, lookup_context)
@@ -81,7 +86,7 @@ module Rabl
 
       options[:root_name] = determine_object_root(data, root_name, options[:root])
 
-      result = \
+      result =
         if is_object?(data) || !data # object @user
           Builder.new(data, @_settings, options).to_hash
         elsif is_collection?(data) # collection @users
@@ -165,6 +170,7 @@ module Rabl
       @_root_name_data = @_root_name_data.values.first if @_root_name_data.is_a?(Hash)
 
       # If we turn this around, `@_root_name_date ==` may trigger data to be loaded unnecessarily.
+      # TODO: is nil a different semantic? otherwise don't use `false ==`, use !
       if false == @_root_name_data
         @_object_root_name = false
         @_collection_name = false
@@ -387,7 +393,11 @@ module Rabl
       end
 
       def digestor_available?
-        defined?(Rails) && Rails.version =~ /^[45]/
+        defined?(Rails) && Rails.version =~ /^[4567]/
+      end
+
+      def valid_format?(format)
+        SAFE_FORMATS.include?(format.to_sym) && respond_to?("to_#{format}")
       end
 
       def set_instance_variables!(context_scope, locals)
@@ -398,6 +408,9 @@ module Rabl
 
         @_options[:format] ||= request_format
 
+        # Prevent calls to inherited methods `to_yaml`, `to_enum`, etc.
+        @_options[:format] = 'json' unless valid_format?(@_options[:format])
+
         set_locals(@_locals)
       end
 
@@ -407,15 +420,21 @@ module Rabl
       end
 
       def eval_source(locals, &block)
-        # Note: locals and block may be used by the eval'ed source
-
         return unless @_source.present?
 
-        if @_options[:source_location]
-          instance_eval(@_source, @_options[:source_location])
-        else
-          instance_eval(@_source)
+        msg = "cached_source_#{@_source.hash.abs}"
+
+        unless self.respond_to? msg then
+          src = "def #{msg} locals, &block\n#{@_source}\nend"
+
+          if @_options[:source_location]
+            self.class.class_eval(src, @_options[:source_location])
+          else
+            self.class.class_eval(src)
+          end
         end
+
+        send msg, locals, &block
       end
-    end
+  end
 end

data/lib/rabl/helpers.rb

@@ -14,7 +14,9 @@ module Rabl
     # data_object_attribute(data) => @_object.send(data)
     def data_object_attribute(data)
       attribute = @_object.__send__(data)
-      attribute = attribute.as_json if is_collection?(attribute, false) && attribute.respond_to?(:as_json)
+      attribute = attribute.as_json if
+        is_collection?(attribute, false) &&
+        attribute.respond_to?(:as_json)
       attribute
     end
 
@@ -81,8 +83,10 @@ module Rabl
     # is_collection?([]) => true
     def is_collection?(obj, follow_symbols = true)
       data_obj = follow_symbols ? data_object(obj) : obj
-      data_obj && data_obj.respond_to?(:map) && data_obj.respond_to?(:each) &&
-        !(data_obj.is_a?(Struct) || defined?(Hashie::Mash) && data_obj.is_a?(Hashie::Mash))
+      data_obj &&
+        data_obj.respond_to?(:map) && data_obj.respond_to?(:each) &&
+        !(data_obj.is_a?(Struct) ||
+          defined?(Hashie::Mash) && data_obj.is_a?(Hashie::Mash))
     end
 
     # Returns the context_scope wrapping this engine, used for retrieving data, invoking methods, etc

data/lib/rabl/railtie.rb

@@ -1,11 +1,28 @@
 module Rabl
   class Railtie < Rails::Railtie
     initializer "rabl.initialize" do |app|
+      # Force Rails to load view templates even in API mode
+      # Stolen shamelessly from jbuilder: https://github.com/rails/jbuilder/blob/master/lib/jbuilder/railtie.rb
+      if Rails::VERSION::MAJOR >= 5
+        module ::ActionController
+          module ApiRendering
+            include ActionView::Rendering
+          end
+        end
+
+        ActiveSupport.on_load :action_controller do
+          if self == ActionController::API
+            include ActionController::Helpers
+            include ActionController::ImplicitRender
+          end
+        end
+      end
+
       ActiveSupport.on_load(:action_view) do
         Rabl.register!
 
         # Inject dependency tracker for :rabl
-        if Rails.version =~ /^[45]/
+        if Rails.version =~ /^[4567]/
           require 'action_view/dependency_tracker'
           ActionView::DependencyTracker.register_tracker :rabl, Rabl::Tracker
         end

data/lib/rabl/template.rb

@@ -40,7 +40,7 @@ if defined?(ActionView) && defined?(Rails) && Rails.respond_to?(:version) && Rai
   ActionView::Template.register_template_handler :rabl, ActionView::TemplateHandlers::RablHandler
 end
 
-# Rails 3.X / 4.X Template
+# Rails 3.X / 4.X / 5.X Template
 if defined?(ActionView) && defined?(Rails) && Rails.respond_to?(:version) && Rails.version.to_s =~ /^[345]/
   module ActionView
     module Template::Handlers
@@ -61,3 +61,23 @@ if defined?(ActionView) && defined?(Rails) && Rails.respond_to?(:version) && Rai
 
   ActionView::Template.register_template_handler :rabl, ActionView::Template::Handlers::Rabl
 end
+
+# Rails 6.X / 7.X Template
+if defined?(ActionView) && defined?(Rails) && Rails.respond_to?(:version) && Rails.version.to_s =~ /^[67]/
+  module ActionView
+    module Template::Handlers
+      class Rabl
+        class_attribute :default_format
+        self.default_format = :json
+
+        def self.call(template, source)
+          %{ ::Rabl::Engine.new(#{source.inspect}).
+              apply(self, assigns.merge(local_assigns)).
+              render }
+        end # call
+      end # rabl class
+    end # handlers
+  end
+
+  ActionView::Template.register_template_handler :rabl, ActionView::Template::Handlers::Rabl
+end

data/lib/rabl/version.rb

@@ -1,3 +1,3 @@
 module Rabl
-  VERSION = "0.13.0"
+  VERSION = "0.15.0"
 end

data/lib/rabl.rb

@@ -16,10 +16,10 @@ require 'rabl/configuration'
 require 'rabl/renderer'
 require 'rabl/cache_engine'
 
-if defined?(Rails)
-  require 'rabl/tracker'  if Rails.version =~ /^[45]/
-  require 'rabl/digestor' if Rails.version =~ /^[45]/
-  require 'rabl/railtie'  if Rails.version =~ /^[345]/
+if defined?(Rails) && Rails.respond_to?(:version)
+  require 'rabl/tracker'  if Rails.version =~ /^[4567]/
+  require 'rabl/digestor' if Rails.version =~ /^[4567]/
+  require 'rabl/railtie'  if Rails.version =~ /^[34567]/
 end
 
 # Rabl.register!
@@ -83,6 +83,6 @@ end
 if defined?(Padrino)
   require 'padrino-core'
   Padrino.after_load { Rabl.register! }
-elsif defined?(Rails) && Rails.version =~ /^2/
+elsif defined?(Rails.version) && Rails.version =~ /^2/
   Rabl.register!
 end