Refactoring

lib/generators/ratyrate/ratyrate_generator.rb

@@ -5,19 +5,19 @@ class RatyrateGenerator < ActiveRecord::Generators::Base
 
   source_root File.expand_path('../templates', __FILE__)
 
-  desc "copying jquery.raty files to assets directory ..."
-  def copying
-    copy_file 'jquery.raty.js', 'app/assets/javascripts/jquery.raty.js'
-    copy_file 'star-on.png', 'app/assets/images/star-on.png'
-    copy_file 'star-off.png', 'app/assets/images/star-off.png'
-    copy_file 'star-half.png', 'app/assets/images/star-half.png'
-    copy_file 'mid-star.png', 'app/assets/images/mid-star.png'
-    copy_file 'big-star.png', 'app/assets/images/big-star.png'
-    copy_file 'cancel-on.png', 'app/assets/images/cancel-on.png'
-    copy_file 'cancel-off.png', 'app/assets/images/cancel-off.png'
-    copy_file 'ratyrate.js.erb', 'app/assets/javascripts/ratyrate.js.erb'
-    copy_file 'rater_controller.rb', 'app/controllers/rater_controller.rb'
-  end
+  # desc "copying jquery.raty files to assets directory ..."
+  # def copying
+  #   copy_file 'jquery.raty.js', 'app/assets/javascripts/jquery.raty.js'
+  #   copy_file 'star-on.png', 'app/assets/images/star-on.png'
+  #   copy_file 'star-off.png', 'app/assets/images/star-off.png'
+  #   copy_file 'star-half.png', 'app/assets/images/star-half.png'
+  #   copy_file 'mid-star.png', 'app/assets/images/mid-star.png'
+  #   copy_file 'big-star.png', 'app/assets/images/big-star.png'
+  #   copy_file 'cancel-on.png', 'app/assets/images/cancel-on.png'
+  #   copy_file 'cancel-off.png', 'app/assets/images/cancel-off.png'
+  #   copy_file 'ratyrate.js.erb', 'app/assets/javascripts/ratyrate.js.erb'
+  #   copy_file 'rater_controller.rb', 'app/controllers/rater_controller.rb'
+  # end
 
   desc "model is creating..."
   def create_model
@@ -26,31 +26,17 @@ def create_model
     class_collisions 'Rate'
     template 'model.rb', File.join('app/models', "rate.rb")
     template 'cache_model.rb', File.join('app/models', "rating_cache.rb")
-    template 'average_cache_model.rb', File.join('app/models', "average_cache.rb")
-    template 'overall_average_model.rb', File.join('app/models', "overall_average.rb")
+    template 'rating_average_model.rb', File.join('app/models', "rating_average.rb")
   end
 
   def add_rate_path_to_route
     route "post '/rate' => 'rater#create', :as => 'rate'"
   end
 
-  desc "cacheable rating average migration is creating ..."
-  def create_cacheable_migration
-    migration_template "cache_migration.rb", "db/migrate/create_rating_caches.rb"
-  end
-
-  desc "migration is creating ..."
-  def create_ratyrate_migration
+  desc "copy migrations"
+  def create_migrations
     migration_template "migration.rb", "db/migrate/create_rates.rb"
-  end
-
-  desc "average caches migration is creating ..."
-  def create_average_caches_migration
-    migration_template "average_cache_migration.rb", "db/migrate/create_average_caches.rb"
-  end
-
-  desc "overall averages migration is creating ..."
-  def create_overall_averages_migration
-    migration_template "overall_average_migration.rb", "db/migrate/create_overall_averages.rb"
+    migration_template "cache_migration.rb", "db/migrate/cache_migration.rb"
+    migration_template "rating_average_migration.rb", "db/migrate/create_rating_averages.rb"
   end
 end

lib/generators/ratyrate/templates/cache_migration.rb

@@ -1,19 +1,14 @@
 class CreateRatingCaches < ActiveRecord::Migration
 
-  def self.up
-      create_table :rating_caches do |t|
-        t.belongs_to :cacheable, :polymorphic => true
-        t.float :avg, :null => false
-        t.integer :qty, :null => false
-        t.string :dimension
-        t.timestamps
-      end
-
-      add_index :rating_caches, [:cacheable_id, :cacheable_type]
-    end
-
-    def self.down
-      drop_table :rating_caches
+  def change
+    create_table :rating_caches do |t|
+      t.belongs_to :cacheable, polymorphic: true
+      t.float :avg, null: false
+      t.integer :qty, null: false
+      t.string :dimension
+      t.timestamps
     end
 
+    add_index :rating_caches, [:cacheable_id, :cacheable_type]
+  end
 end

lib/generators/ratyrate/templates/cache_model.rb

@@ -1,3 +1,3 @@
 class RatingCache < ActiveRecord::Base
-  belongs_to :cacheable, :polymorphic => true
+  belongs_to :cacheable, polymorphic: true
 end

lib/generators/ratyrate/templates/rating_average_migration.rb

@@ -0,0 +1,16 @@
+class CreateRatingAverages < ActiveRecord::Migration
+
+  def change
+    create_table :rating_averages do |t|
+      t.belongs_to :rateable, polymorphic:  true
+      t.float :avg, null:false
+      t.belongs_to :rater
+
+      t.timestamps
+    end
+
+    add_index :rating_averages, [:rateable_id, :rateable_type]
+    add_index :rating_averages, [:rateable_id, :rateable_type, :rater_id]
+  end
+end
+

lib/generators/ratyrate/templates/rating_average_model.rb

@@ -0,0 +1,5 @@
+class RatingAverage < ActiveRecord::Base
+  belongs_to :rateable, polymorphic: true
+  belongs_to :rater, class_name:  "<%= file_name.classify %>"
+end
+

lib/ratyrate.rb

@@ -1,6 +1,7 @@
 require "ratyrate/version" 
 require "ratyrate/model" 
-require "ratyrate/helpers" 
+require "ratyrate/helpers"
+require "ratyrate/engine"
 
 module Ratyrate
 

lib/ratyrate/engine.rb

@@ -0,0 +1,7 @@
+module RatyRate
+  class Engine < ::Rails::Engine
+    initializer 'ratyrate.load_static_assets' do |app|
+      app.middleware.use ::ActionDispatch::Static, "#{root}/vendor"
+    end
+  end
+end