sequenceid 0.0.3 → 0.0.4

data/README.md

@@ -20,6 +20,8 @@ SequenceId is a fairly straight forward gem and you need to ensure only ONE cond
 
 Back to our original example, if you want invoices to start from 1 for each new company, the Invoice RESOURCE is nested to the Company RESOURCE. The gem will check for the relationhip in the database (Invoice table in this case) and ensure the sequence_num (SequenceId column created in the table)  and the parent resource_id are a secondary compound key in the nested resource's table. 
 
+At no point is the sequence_num changed once its assigned since it will serve as one of the columns of the compound secondary key. If you ever do rollback the migration, be *careful* since re-running the migration might result in a different sequence_num if a nested resource is deleted.
+
 SequenceId is compatible with Active Record  **3.0**.
 
 ## Docs, Info and Support
@@ -36,6 +38,8 @@ SequenceId is compatible with Active Record  **3.0**.
     # add to Gemfile
     gem "sequenceid"
 
+    bundle install
+
     rails generate sequenceid <parent resource> <nested resource> #eg rails generate sequenceid Company Invoice 
 
     rake db:migrate
@@ -67,9 +71,11 @@ If you have a bug to report, please include the following information:
 ## Credits
 
 SequenceId was create by Syed Ali @ 7vals 
+
 *   [7vals](http://www.7vals.com)
 
 Special thanks to
+
 *   [cancan](https://github.com/ryanb/cancan)
 
 *   [FriendlyId](https://github.com/norman/friendly_id)

data/lib/generators/sequenceid/sequenceid_generator.rb

@@ -22,22 +22,22 @@ class SequenceidGenerator < ActiveRecord::Generators::Base
       @parent_resource=eval parent_resource_s.classify
       @nested_resource=eval nested_resource_s.classify
       if((!@parent_resource.ancestors.include?ORM) || (!@nested_resource.ancestors.include?ORM))
-        puts "both #{parent_resource_s} and #{nested_resource_s} need to be ActiveRecords"
+        puts "ERROR: both #{parent_resource_s} and #{nested_resource_s} need to be ActiveRecords"
         exit
       end
 
       if((!@nested_resource.new.respond_to? parent_resource_s) & (!@nested_resource.new.respond_to? parent_resource_s.pluralize))
-        puts "#{nested_resource_s} should have an association with #{parent_resource_s} otherwise its not possible to have a unique sequence number to identify #{nested_resource_s} from the url"
+        puts "ERROR: #{nested_resource_s} should have an association with #{parent_resource_s} otherwise its not possible to have a unique sequence number to identify #{nested_resource_s} from the url"
         #TODO need to check for belongs_to relation
         exit
       end
       #if not belongs to
       if(!@parent_resource.new.respond_to? @nested_resource.to_s.downcase.pluralize)
-        exit unless yes?("The resource #{nested_resource_s} should have a belongs to association with #{parent_resource_s}, do you still want to continue (risky)?")
+        exit unless yes?("ERROR: The resource #{nested_resource_s} should have a belongs to association with #{parent_resource_s}, do you still want to continue (risky)?")
       end
 
     rescue =>e
-      puts "both #{parent_resource_s} and #{nested_resource_s} need to be ActiveRecords #{e.message}"
+      puts "ERROR: both #{parent_resource_s} and #{nested_resource_s} need to be ActiveRecords.\n #{e.message}"
       exit
     end
   end
@@ -47,6 +47,6 @@ class SequenceidGenerator < ActiveRecord::Generators::Base
     migration_template "migration.rb", "db/migrate/add_sequence_num_to_#{@nested_resource.to_s.downcase.pluralize}"
     #inject into model class module includeA
     @model_path ||= File.join("app", "models", "#{@nested_resource.to_s.underscore}.rb")
-    inject_into_class(@model_path,@nested_resource,"sequenceid '#{@parent_resource.to_s.downcase}','#{@nested_resource.to_s.downcase.pluralize}'\n")
+    inject_into_class(@model_path,@nested_resource,"\tsequenceid :#{@parent_resource.to_s.downcase.to_sym} , :#{@nested_resource.to_s.downcase.pluralize.to_sym}\n")
   end
 end

data/lib/sequenceid/model_adapters/sequenceid_logic.rb

@@ -1,41 +1,43 @@
 module Sequenceid
   module Logic
 
-    def sequenceid(parent_rel,current_rel)
-      #TODO:need to ensure multiple before_saves are not added - possibly use respond_to?
+    def sequenceid(parent_rel_sym,current_rel_sym)
+      @parent_rel=parent_rel_sym.to_s
+      @current_rel=current_rel_sym.to_s
+
       class_eval do
-        before_create :set_sequence_num
-        after_rollback :reset_sequence_num
-        include InstanceMethods
+	before_create :set_sequence_num
+	after_rollback :reset_sequence_num
+	include InstanceMethods
+
       end
-      @relation_for_sequence_string=(parent_rel+"."+current_rel)
     end
 
     module InstanceMethods
       def to_param
-        "#{sequence_num}"
+	"#{sequence_num}"
       end
 
       protected
       def set_sequence_num
-        if new_record?
-	  @relation_sequence=self.send @relation_for_sequence_string
-          self.sequence_num=@relation_sequence.order("id").last.try(:sequence_num) || 0
-          self.sequence_num+=1
-        end
-        return true
+	if new_record?
+	  @relation_sequence=self.send(self.class.instance_variable_get("@parent_rel")).send(self.class.instance_variable_get("@current_rel"))
+	  self.sequence_num=@relation_sequence.order("id").last.try(:sequence_num) || 0
+	  self.sequence_num+=1
+	end
+	return true
       end
 
       def reset_sequence_num
-        @save_counter||=1
-        if new_record? && @save_counter<3
-          logger.info "SQUENCENUM:: attempt number #{@save_counter} of a max 2"
-          self.sequence_num=@relation_sequence.order("id").last.try(:sequence_num) +1
-          @save_counter+=1
-          save
-        else
-          raise
-        end
+	@save_counter||=1
+	if new_record? && @save_counter<3
+	  logger.info "SQUENCENUM:: attempt number #{@save_counter} of a max 2"
+	  self.sequence_num=@relation_sequence.order("id").last.try(:sequence_num) +1
+	  @save_counter+=1
+	  save
+	else
+	  raise
+	end
       end
     end
   end

data/lib/sequenceid/version.rb

@@ -1,3 +1,3 @@
 module Sequenceid
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: sequenceid
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: 
   segments: 
   - 0
   - 0
-  - 3
-  version: 0.0.3
+  - 4
+  version: 0.0.4
 platform: ruby
 authors: 
 - Syed Ali
@@ -15,7 +15,8 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-31 00:00:00 Z
+date: 2011-08-04 00:00:00 +05:00
+default_executable: 
 dependencies: []
 
 description: For SaaS applications, there are times when we dont want to show the resource id to the user since its being shared with multiple companies that have signed up. And for scenarios where we want the URL to show numbers and not worded URL's like what friendly_id provide, this is the gem for you! It'll create id's sequentially starting from 1 for each relation parent/nested resource relation definedi\n Feel free to get in touch for any specific questions until the help, support group and readme are not done at info@7vals.com \n http://www.7vals.com
@@ -28,21 +29,18 @@ extensions: []
 extra_rdoc_files: []
 
 files: 
-- .DS_Store
-- .README.swp
 - .gitignore
 - Gemfile
 - README.md
 - Rakefile
-- lib/.DS_Store
 - lib/generators/sequenceid/sequenceid_generator.rb
 - lib/generators/sequenceid/template/migration.rb
 - lib/sequenceid.rb
-- lib/sequenceid/.DS_Store
 - lib/sequenceid/model_adapters/active_record_adapter.rb
 - lib/sequenceid/model_adapters/sequenceid_logic.rb
 - lib/sequenceid/version.rb
 - sequenceid.gemspec
+has_rdoc: true
 homepage: ""
 licenses: []
 
@@ -72,7 +70,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: sequenceid
-rubygems_version: 1.7.2
+rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: Create resource URL's starting from 1 for each SaaS company