Make engine generators work correctly.

`rails generate/destroy` now correctly place files in the engine rather than test/dummy.

* Move dependencies loading to gem file (lib/browsercms) which avoids loading all of them during rails tasks.

Author: peakpg

lib/browsercms.rb

@@ -1,3 +1,30 @@
+# Load all dependencies needed at boot time.
+require 'rails'
+require 'cms/configuration'
+require 'cms/version'
+require 'browsercms'
+
+require 'bootstrap-sass'
+require 'compass-rails'
+
+# Gem name is different than file name
+# Must be required FIRST, so that our assets paths appear before its do.
+# This allows app/assets/ckeditor/config.js to set CMS specific defaults.
+require 'ckeditor-rails'
+
+# Explicitly require this, so that CMS projects do not need to add it to their Gemfile
+# especially while upgrading
+require 'jquery-rails'
+require 'jquery-ui-rails'
+
+require 'underscore-rails'
+require 'will_paginate'
+require 'will_paginate/active_record'
+require 'actionpack/page_caching'
+require 'panoramic'
+require 'simple_form'
+require 'devise'
+
 require 'cms/engine'
 require 'cms/extensions'
 require 'cms/route_extensions'
@@ -38,4 +65,4 @@
 ActiveRecord::Base.send(:include, Cms::Acts::CmsUser)
 require 'cms/responders/content_responder'
 
-require "panoramic"
+require "panoramic"

lib/cms/engine.rb

@@ -1,32 +1,7 @@
-require 'rails'
+# Load just enough dependencies for this file to be loadable.
 require 'cms/module'
-require 'cms/configuration'
-require 'cms/version'
-require 'browsercms'
-
-require 'bootstrap-sass'
-require 'compass-rails'
-
-# Gem name is different than file name
-# Must be required FIRST, so that our assets paths appear before its do.
-# This allows app/assets/ckeditor/config.js to set CMS specific defaults.
-require 'ckeditor-rails'
-
-# Explicitly require this, so that CMS projects do not need to add it to their Gemfile
-# especially while upgrading
-require 'jquery-rails'
-require 'jquery-ui-rails'
-
-require 'underscore-rails'
-require 'will_paginate'
-require 'will_paginate/active_record'
-require 'actionpack/page_caching'
-require 'panoramic'
-require 'simple_form'
-require 'devise'
 
 module Cms
-
   class Engine < Rails::Engine
     include Cms::Module
     isolate_namespace Cms

script/rails

@@ -1,21 +1,27 @@
 #!/usr/bin/env ruby
-# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
 
-ENGINE_PATH = File.expand_path('../..',  __FILE__)
-load File.expand_path('../../test/dummy/script/rails',  __FILE__)
+#This is standard Rails 4 Engine commands behavior.
+ENGINE_ROOT = File.expand_path('../..', __FILE__)
+ENGINE_PATH = File.expand_path('../../lib/cms/engine', __FILE__)
 
+# Modified the core `rails` script so that commands other than generate/destroy work in context of the test/dummy app.
 
-# Rails 3.2.2 generates the following instead, but you only get access to:
-# - rails generate
-# - rails destroy
-# I kept the above (for now anyway) so I can get access to rails server, though I will need to test that it works for other commands too.
+ARGV << '--help' if ARGV.empty?
 
-# Bugs:  Adding the following would:
-# 1. Enable generators to work correctly (right now they put code into test/dummy)
-# 2. Prevent loading, since engine requires other files directly.
+aliases = {
+    "g" => "generate",
+    "d" => "destroy"
+}
 
-#ENGINE_ROOT = File.expand_path('../..', __FILE__)
-#ENGINE_PATH = File.expand_path('../../lib/cms/engine', __FILE__)
-##
-#require 'rails/all'
-#require 'rails/engine/commands'
+command = ARGV.first
+command = aliases[command] || command
+
+case command
+  when 'generate', 'destroy'
+    require 'rails/all'
+    require 'rails/engine/commands'
+  else
+    # Commands like 'server' and 'console' will use test/dummy's rails script.
+    load File.expand_path('../../test/dummy/script/rails', __FILE__)
+
+end