rules_engine 0.0.1 → 0.0.3

data/rails_generators/manifests/rules_engine_complex.rb

@@ -0,0 +1,30 @@
+class RulesEngineComplexManifest
+  def self.populate_record(m, rule_name ,rule_class)
+
+    %W(
+      app/rules
+      app/views/re_rule_definitions/#{rule_name}_rule
+      lib/tasks
+      spec/lib/rules
+    ).each do |dirname|
+      m.directory dirname
+    end
+
+    %W(
+    ).each do |filename|
+      m.file filename, filename
+    end
+
+   m.template "app/rules/complex_rule.rb",  "app/rules/#{rule_name}_rule.rb"
+   m.template "app/views/re_rule_definitions/complex_rule/_edit.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_edit.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_help.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_help.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_new.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_new.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_pipeline.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_pipeline.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_script.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_script.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_title.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_title.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_word.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_word.html.erb"
+   m.template "app/views/re_rule_definitions/complex_rule/_words.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_words.html.erb"
+   m.template "spec/lib/rules/complex_rule_spec.rb",  "spec/lib/rules/#{rule_name}_rule_spec.rb"
+
+  end
+end

data/rails_generators/manifests/rules_engine_complex.yml

@@ -0,0 +1,25 @@
+directories :
+  raw:
+    - app/rules
+    - spec/lib/rules
+    - lib/tasks
+
+templates :
+  replacements :
+    - complex : rule_name
+    - Complex : rule_class
+    
+  directories : 
+    - app/views/re_rule_definitions/complex_rule
+
+  files : 
+    - app/rules/complex_rule.rb
+    - app/views/re_rule_definitions/complex_rule/_edit.html.erb
+    - app/views/re_rule_definitions/complex_rule/_help.html.erb
+    - app/views/re_rule_definitions/complex_rule/_pipeline.html.erb
+    - app/views/re_rule_definitions/complex_rule/_script.html.erb
+    - app/views/re_rule_definitions/complex_rule/_title.html.erb
+    - app/views/re_rule_definitions/complex_rule/_new.html.erb
+    - app/views/re_rule_definitions/complex_rule/_word.html.erb
+    - app/views/re_rule_definitions/complex_rule/_words.html.erb
+    - spec/lib/rules/complex_rule_spec.rb

data/rails_generators/manifests/rules_engine_simple.rb

@@ -0,0 +1,26 @@
+class RulesEngineSimpleManifest
+  def self.populate_record(m, rule_name ,rule_class)
+
+    %W(
+      app/rules
+      app/views/re_rule_definitions/#{rule_name}_rule
+      lib/tasks
+      spec/lib/rules
+    ).each do |dirname|
+      m.directory dirname
+    end
+
+    %W(
+    ).each do |filename|
+      m.file filename, filename
+    end
+
+   m.template "app/rules/simple_rule.rb",  "app/rules/#{rule_name}_rule.rb"
+   m.template "app/views/re_rule_definitions/simple_rule/_edit.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_edit.html.erb"
+   m.template "app/views/re_rule_definitions/simple_rule/_form.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_form.html.erb"
+   m.template "app/views/re_rule_definitions/simple_rule/_help.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_help.html.erb"
+   m.template "app/views/re_rule_definitions/simple_rule/_new.html.erb",  "app/views/re_rule_definitions/#{rule_name}_rule/_new.html.erb"
+   m.template "spec/lib/rules/simple_rule_spec.rb",  "spec/lib/rules/#{rule_name}_rule_spec.rb"
+
+  end
+end

data/rails_generators/manifests/rules_engine_simple.yml

@@ -0,0 +1,22 @@
+directories :
+  raw:
+    - app/rules
+    - spec/lib/rules
+    - lib/tasks
+
+templates :
+  replacements :
+    - simple : rule_name
+    - Simple : rule_class
+    
+  directories : 
+    - app/views/re_rule_definitions/simple_rule
+
+  files : 
+    - app/rules/simple_rule.rb
+    - app/views/re_rule_definitions/simple_rule/_edit.html.erb
+    - app/views/re_rule_definitions/simple_rule/_help.html.erb
+    - app/views/re_rule_definitions/simple_rule/_new.html.erb
+    - app/views/re_rule_definitions/simple_rule/_form.html.erb
+    - spec/lib/rules/simple_rule_spec.rb
+  

data/rails_generators/rules_engine_generator.rb

@@ -1,35 +1,61 @@
-require File.expand_path(File.dirname(__FILE__) + "/generator_manifest")
-require File.expand_path(File.dirname(__FILE__) + "/template_manifest")
-require File.expand_path(File.dirname(__FILE__) + "/blacklist_word_manifest")
-require File.expand_path(File.dirname(__FILE__) + "/simple_manifest")
+require File.expand_path(File.dirname(__FILE__) + '/manifests/rules_engine')
+require File.expand_path(File.dirname(__FILE__) + '/manifests/rules_engine_simple')
+require File.expand_path(File.dirname(__FILE__) + '/manifests/rules_engine_complex')
 
 class RulesEngineGenerator < Rails::Generator::Base
-  
+
+  attr_reader :rule_type
   attr_reader :rule_name, :rule_class
   
   def initialize(runtime_args, runtime_options = {})
     super
-  end
     
+    @rule_type = runtime_args[0] unless runtime_args.length < 1
+    @rule_name = runtime_args[1].downcase.gsub(/[^a-zA-Z0-9]+/i, '_') unless runtime_args.length < 2
+    @rule_class = @rule_name.camelize unless runtime_args.length < 2
+  end
+
   def after_generate
-  end  
-  
-  def manifest 
-    @rule_name = args[0] || 'generator'
-    @rule_class = @rule_name.camelize
+    puts ''
+    puts '******************************************************'
+    case @rule_type
 
-    record do |m|
-      if (@rule_name == 'generator')
-        GeneratorManifest.populate_record(m)      
-        m.readme './../USAGE'
-      elsif (@rule_name == 'blacklist_word')
-        BlacklistWordManifest.populate_record(m)        
-      elsif (@rule_name == 'simple')
-        Simple.populate_record(m)        
+    when 'simple'
+      if @rule_name.blank? || @rule_class.blank?
+        puts 'error rule name required USAGE ./script/generate simple [rule_name]'
+      else
+        puts 'run >rake spec to test the new rule'  
+      end 
+
+    when 'complex'
+      if @rule_name.blank? || @rule_class.blank?
+        puts 'error rule name required USAGE ./script/generate complex [rule_name]'
       else
-        TemplateManifest.populate_record(m, @rule_name)
-      end
-      
+        puts 'run >rake spec to test the new rule'  
+      end 
+           
+    when nil
+      puts 'open doc/README.rules_engine for more instructions'
+    else
+      puts 'error rule type not known USAGE ./script/generate [simple|complex] [rule_name]'  
+    end      
+    puts ''
+  end
+
+  def manifest
+    record do |m|
+      case @rule_type
+      when 'simple'
+        unless @rule_name.blank? || @rule_class.blank?
+          RulesEngineSimpleManifest.populate_record(m, @rule_name ,@rule_class)
+        end  
+      when 'complex'
+        unless @rule_name.blank? || @rule_class.blank?
+          RulesEngineComplexManifest.populate_record(m, @rule_name ,@rule_class)
+        end  
+      when nil  
+        RulesEngineManifest.populate_record(m)
+      end      
     end
   end
-end
+end

data/rails_generators/templates/app/controllers/re_jobs_controller.rb

@@ -1,18 +1,20 @@
 class ReJobsController < ApplicationController    
+  helper :re_pipeline 
+  layout 'rules_engine'
 
+  # before_filter :login_required
+  before_filter :rules_engine_reader_access_required
+  
   before_filter :only => [:show] do |controller|
-    controller.load_model :re_job
+    controller.re_load_model :re_job
   end    
   
-  # before_filter :login_required
-  before_filter :reader_access_required
-
   def index
-    @re_jobs = ReJob.find_jobs(:page => params[:page], :per_page => 2)
+    @re_jobs = ReJob.find_jobs(:page => params[:page], :per_page => 20)
   end
 
   def show
-    @re_job_audits = ReJobAudit.by_re_job_id(params[:id]).order_date('ASC').find(:all, :include => [:re_pipeline] )
+    @re_job_audits = ReJobAudit.by_re_job_id(@re_job.id).order_date('ASC').find(:all, :include => [:re_pipeline] )
   end
 
 end

data/rails_generators/templates/app/controllers/re_pipeline_jobs_controller.rb

@@ -1,14 +1,13 @@
 class RePipelineJobsController < ApplicationController    
+  helper :re_pipeline 
+  layout 'rules_engine'
 
+  # before_filter :login_required
+  before_filter :rules_engine_reader_access_required
+  
   before_filter do |controller|
-    controller.load_model :re_pipeline, {:param_id => :re_pipeline_id, :redirect_path => :re_pipelines_path}    
+    controller.re_load_model :re_pipeline, {:param_id => :re_pipeline_id, :redirect_path => :re_pipelines_path}    
   end
-  before_filter :only => [:show] do |controller|
-    controller.load_model :re_job
-  end    
-  
-  # before_filter :login_required
-  before_filter :reader_access_required
 
   def index
     @re_jobs = ReJob.find_jobs_by_pipeline(@re_pipeline.id, :page => params[:page], :per_page => 20)

data/rails_generators/templates/app/controllers/re_pipelines_controller.rb

@@ -1,27 +1,29 @@
 class RePipelinesController < ApplicationController    
+  helper :re_pipeline 
+  layout 'rules_engine'
+  
+  # before_filter :login_required
+  before_filter :rules_engine_editor_access_required,  :only => [:new, :create, :change, :edit, :update, :activate_all, :activate, :deactivate, :revert, :destroy]
+  before_filter :rules_engine_reader_access_required,  :only => [:lookup, :index, :show]
 
   before_filter :only => [:show, :change, :edit, :update, :activate, :deactivate, :revert, :destroy] do |controller|
-    controller.load_model :re_pipeline
+    controller.re_load_model :re_pipeline
   end    
 
-  # before_filter :login_required
-  before_filter :editor_access_required,  :only => [:new, :create, :change, :edit, :update, :activate, :deactivate, :revert, :destroy]
-  before_filter :reader_access_required,  :only => [:lookup, :index, :show]
-
   def lookup
     klass = RePipeline
     query = params[:q] || ''
     if params[:t] == 'code'
-      if query.chomp.blank?
-        render :text => klass.find(:all, :order => "title", :limit => 10 ).map(&:title).join("\n")
+      if query.blank?
+        render :text => klass.find(:all, :order => "code", :limit => 10 ).map(&:code).join("\n")
       else
-        render :text => klass.find(:all, :conditions => ["title LIKE ?", "#{params[:q]}%"], :order => "title", :limit => 10 ).map(&:title).join("\n")
+        render :text => klass.find(:all, :conditions => ["code LIKE ?", "#{query}%"], :order => "code", :limit => 10 ).map(&:code).join("\n")
       end
     else # params[:t] == 'title'
-      if query.chomp.blank?
-        render :text => klass.find(:all, :order => "code", :limit => 10 ).map(&:code).join("\n")
+      if query.blank?
+        render :text => klass.find(:all, :order => "title", :limit => 10 ).map(&:title).join("\n")
       else
-        render :text => klass.find(:all, :conditions => ["code LIKE ?", "#{params[:q]}%"], :order => "code", :limit => 10 ).map(&:code).join("\n")
+        render :text => klass.find(:all, :conditions => ["title LIKE ?", "#{query}%"], :order => "title", :limit => 10 ).map(&:title).join("\n")
       end
     end  
   end
@@ -55,8 +57,7 @@ class RePipelinesController < ApplicationController
       end
     else
       render :action => "new"
-    end
-    
+    end    
   end
 
   def change
@@ -66,7 +67,8 @@ class RePipelinesController < ApplicationController
   end
 
   def update
-    @re_pipeline.attributes = params[:re_pipeline].except(:code)
+    update_params = params[:re_pipeline] || {}
+    @re_pipeline.attributes = update_params.except(:code)
     if @re_pipeline.save
       flash[:success] = 'Pipeline Updated.'
       
@@ -80,10 +82,28 @@ class RePipelinesController < ApplicationController
       end
     else
       render :action => "edit"
-    end
-    
+    end    
   end
 
+  def activate_all
+    klass = RePipeline
+    @re_pipelines = klass.find(:all)
+    
+    @re_pipelines.each do |re_pipeline|
+      re_pipeline.activate!
+    end
+    flash[:success] = 'All Pipelines Activated.'
+
+    respond_to do |format|
+      format.html do
+        redirect_to(re_pipelines_path)
+      end  
+      format.js do
+        render :action => "index"
+      end
+    end    
+  end
+  
   def activate
     @re_pipeline.activate!
     flash[:success] = 'Pipeline Activated.'

data/rails_generators/templates/app/controllers/re_rules_controller.rb

@@ -1,34 +1,53 @@
 class ReRulesController < ApplicationController    
-
+  helper :re_pipeline 
+  layout 'rules_engine'
+  
+  # before_filter :login_required
+  before_filter :rules_engine_editor_access_required, :except => [:help, :error]
+  before_filter :rules_engine_reader_access_required, :only => [:help, :error]
+  
   before_filter do |controller|
-    controller.load_model :re_pipeline, {:param_id => :re_pipeline_id, :redirect_path => :re_pipelines_path}    
+    controller.re_load_model :re_pipeline, {:param_id => :re_pipeline_id, :redirect_path => :re_pipelines_path}    
   end
+  
   before_filter :only => [:edit, :update, :destroy, :move_up, :move_down] do |controller|
-    controller.load_model :re_rule, {:param_id => :id, :parents => [:re_pipeline], :redirect_path => :re_pipeline_path}
+    controller.re_load_model :re_rule, {:param_id => :id, :parents => [:re_pipeline], :redirect_path => :re_pipeline_path}
   end    
-  before_filter :load_rule_class_from_rule_class,  :only => [:help, :new, :create]
-  before_filter :load_rule_class_from_model,       :only => [:edit, :update, :move_up, :move_down]
   
-  # before_filter :login_required
-  before_filter :editor_access_required, :except => [:error, :help]
-
-  def error
+  def help
+    @rule_class = RulesEngine::Discovery.rule_class(params[:rule_class_name])
+    if @rule_class.nil?
+      flash[:error] = "#{params[:rule_class]} : class not found."
+      render :error
+    else
+      @rule = @rule_class.new  
+    end      
   end  
   
-  def help
+  def error
   end  
   
   def new
-    @re_rule = ReRule.new(:re_pipeline_id => @re_pipeline.id)
+    @re_rule = ReRule.new(:re_pipeline_id => @re_pipeline.id, :rule_class_name => params[:rule_class_name])
+    if @re_rule.rule.nil? 
+      render :action => "error"
+      return
+    end
+    
+    render :action => "new"      
   end
   
   def create
-    @re_rule = ReRule.new(:re_pipeline_id => @re_pipeline.id)
-    @rule.attributes = params
-    if @rule.valid? && @rule.save(@re_rule) && @re_rule.save
+    @re_rule = ReRule.new(:re_pipeline_id => @re_pipeline.id, :rule_class_name => params[:rule_class_name])
+    if @re_rule.rule.nil? 
+      render :action => "error"
+      return      
+    end
+    
+    @re_rule.rule_attributes = params
+    if @re_rule.valid? && @re_rule.save
       flash[:success] = 'Rule Created.'      
-      @rule.after_create(@re_rule)
-      
+    
       respond_to do |format|
         format.html do          
           redirect_to(change_re_pipeline_path(@re_pipeline))
@@ -43,14 +62,23 @@ class ReRulesController < ApplicationController
   end
 
   def edit
+    if @re_rule.rule.nil? 
+      render :action => "error"
+      return
+    end
+    
+    render :action => "edit"      
   end
 
   def update
-    @rule.attributes = params      
-    if @rule.valid? && @rule.save(@re_rule) && @re_rule.save
-      @re_pipeline.changed
+    if @re_rule.rule.nil? 
+      render :action => "error"
+      return
+    end
+    
+    @re_rule.rule_attributes = params
+    if @re_rule.valid? && @re_rule.save
       flash[:success] = 'Rule Updated.'      
-      @rule.after_update(@re_rule)
       
       respond_to do |format|
         format.html do          
@@ -61,18 +89,12 @@ class ReRulesController < ApplicationController
         end
       end
     else
-      render :action => "new"
+      render :action => "edit"
     end
   end
 
   def destroy
-    @rule_class = RulesEngine::Discovery.rule_class(@re_rule.rule_class)
-    if (@rule_class)
-      @rule = @rule_class.new  
-      @rule.before_destroy(@re_rule)
-    end  
     @re_rule.destroy
-    @re_pipeline.changed
     flash[:success] = 'Rule Deleted.'
 
     respond_to do |format|
@@ -88,7 +110,6 @@ class ReRulesController < ApplicationController
   
   def move_up
     @re_rule.move_higher
-    @re_pipeline.changed
     flash[:success] = 'Rule Moved Up.'      
     
     respond_to do |format|
@@ -103,7 +124,6 @@ class ReRulesController < ApplicationController
 
   def move_down
     @re_rule.move_lower
-    @re_pipeline.changed
     flash[:success] = 'Rule Moved Down.'      
     
     respond_to do |format|
@@ -115,26 +135,4 @@ class ReRulesController < ApplicationController
       end
     end
   end
-  
-  protected
-    def load_rule_class_from_rule_class
-      @rule_class = RulesEngine::Discovery.rule_class(params[:rule_class])
-      if @rule_class.nil?
-        flash[:error] = "#{params[:rule_class]} : class not found."
-        render :action => :error
-      else
-        @rule = @rule_class.new  
-      end      
-    end
-
-    def load_rule_class_from_model
-      @rule_class = RulesEngine::Discovery.rule_class(@re_rule.rule_class)
-      if @rule_class.nil?
-        flash[:error] = "#{@re_rule.rule_class} : class not found."
-        render :action => :error
-      else
-        @rule = @rule_class.new  
-        render :action => :error unless @rule.load(@re_rule)
-      end      
-    end
 end