Rails: Support reloading of classes in development

[pfeiffer]

At the moment changing a method in a class descending from ContentfulModel::Base requires full reload of Rails environment.

Example:

class TestModel < ContentfulModel::Base
   self.content_type_id = 'test'

   def title
      "This is the title!"
   end
end

# eg. in "rails console"
irb(main):001:0> TestModel.first.title
=> "This is the title!"

Changing the #title method:

....
def title
   "New title?"
end
....

And then reloading:

irb(main):002:0> reload!
Reloading...
=> true

irb(main):003:0> TestModel.first.title
=> "This is the title!" # Would have expected "New title?" here

[dlitvakb]

Hey @pfeiffer,

I'n not entirely sure on what I could do to support this. Would you mind helping with the implementation?

Cheers

[mrGrazy]

The problem that is breaking reloading is that mappings are not readded if the class is redefined:

contentful_model/lib/contentful_model/base.rb

Line 123 in b286afd

def mapping?

If I override this method to return false, the class gets reloaded fine in development.

class TestModel < ContentfulModel::Base
   # ....
  def self.mapping?
    false
  end
end

This happens because the rails environment persists class variables like entry_mapping, and that mapping contains a reference to the old version of the model.

I don't know if that helps.

[timfjord]

Here is how this problem can be fixed

module MyApp
  class Application < Rails::Application
    config.to_prepare do
      ContentfulModel.configuration.entry_mapping.each do |content_type_id, model|
        ContentfulModel.configuration.entry_mapping[content_type_id] = Object.const_get(model.to_s)
      end
    end
  end
end

It is still an expiemintal solution I am keeping an eye on it in my current project.
And if it will work I will send a PR with a more generic fix

[arielkirkwood]

@dlitvakb Do you still need more information in order to understand how to improve this? This has been one of the most frustrating parts of using this library :(