Reloading not working correctly (Rails engine)

[lucaspiller]

I have grape set up within a Rails engine for better separation of concerns, however the reloading suggestion in the documentation doesn't work for me. I also have to set ActiveSupport::Dependencies.explicitly_unloadable_constants.

In the engine config I have this:

# auto reload api in development mode
initializer 'reload_api' do
  if Rails.env.development?
    ActiveSupport::Dependencies.explicitly_unloadable_constants << "API::Resources"

    ["app/resources/**/*.rb"].each do |glob|
      files = Dir["#{API::Engine.root}/#{glob}"]
      reloader = ActiveSupport::FileUpdateChecker.new(files) do
        Rails.logger.info ">> Reloading API"
        Rails.application.reload_routes!
      end
      ActionDispatch::Callbacks.to_prepare do
        reloader.execute_if_updated
      end
    end
  end
end

I have no idea why this works, but heh. Might be worth adding this to the readme or wiki :)

[dblock]

I would love this added to the README, but I do also want an explanation of what explicitly_unloadable_constants really does :)

[lucaspiller]

Sorry I forgot about this.

From the Rails docs:

"An array of constant names that need to be unloaded on every request. Used to allow arbitrary constants to be marked for unloading."

https://github.com/rails/rails/blob/3-2-stable/activesupport/lib/active_support/dependencies.rb#L51-54

Again, why it works I do not know. It doesn't reload them on every request, only when the resources have changed - but without that line nothing is reloaded.

[dblock]

Ok, this makes at least some sense. Could you make a pull request with the README update, please?

[changx]

finally ActiveSupport::Dependencies.explicitly_unloadable_constants << "API" rescued me

[dblock]

@ccpp0 Can you PR this into the README, please?

[dblock]

@lucaspiller Bump?

[pusewicz]

@lucaspiller Bump?

[tomconroy]

Using require_dependency instead of require when mounting other API classes seems to be important

[dblock]

This seems relevant: #658. Can someone take a look and finish this?

[dblock]

I made a change based on all of this in #849, if something is not right there, please open a new issue / PR.