From 76b1563539c258cf17d60c1b93a0c3d26be1075a Mon Sep 17 00:00:00 2001 From: Peter Gumeson Date: Thu, 4 Jun 2009 20:44:44 -0700 Subject: [PATCH] Renamed new_document? and new_model? to simply new? --- lib/couchrest/core/document.rb | 4 ++-- lib/couchrest/mixins/properties.rb | 2 +- lib/couchrest/more/casted_model.rb | 4 ++-- lib/couchrest/more/extended_document.rb | 12 ++++++------ spec/couchrest/more/casted_model_spec.rb | 20 ++++++++++---------- spec/couchrest/more/extended_doc_spec.rb | 6 +++--- spec/couchrest/more/property_spec.rb | 2 +- spec/fixtures/more/article.rb | 2 +- 8 files changed, 26 insertions(+), 26 deletions(-) diff --git a/lib/couchrest/core/document.rb b/lib/couchrest/core/document.rb index e90c5fe..93b0411 100644 --- a/lib/couchrest/core/document.rb +++ b/lib/couchrest/core/document.rb @@ -27,7 +27,7 @@ module CouchRest end # returns true if the document has never been saved - def new_document? + def new? !rev end @@ -67,7 +67,7 @@ module CouchRest # Returns the CouchDB uri for the document def uri(append_rev = false) - return nil if new_document? + return nil if new? couch_uri = "http://#{database.uri}/#{CGI.escape(id)}" if append_rev == true couch_uri << "?rev=#{rev}" diff --git a/lib/couchrest/mixins/properties.rb b/lib/couchrest/mixins/properties.rb index ae18323..953e125 100644 --- a/lib/couchrest/mixins/properties.rb +++ b/lib/couchrest/mixins/properties.rb @@ -17,7 +17,7 @@ module CouchRest end def apply_defaults - return if self.respond_to?(:new_document?) && (new_document? == false) + return if self.respond_to?(:new?) && (new? == false) return unless self.class.respond_to?(:properties) return if self.class.properties.empty? # TODO: cache the default object diff --git a/lib/couchrest/more/casted_model.rb b/lib/couchrest/more/casted_model.rb index 25e9031..029dc14 100644 --- a/lib/couchrest/more/casted_model.rb +++ b/lib/couchrest/more/casted_model.rb @@ -37,10 +37,10 @@ module CouchRest # False if the casted model has already # been saved in the containing document - def new_model? + def new? !@document_saved end - alias :new_record? :new_model? + alias :new_record? :new? # Sets the attributes from a hash def update_attributes_without_saving(hash) diff --git a/lib/couchrest/more/extended_document.rb b/lib/couchrest/more/extended_document.rb index dbc929c..44c73c3 100644 --- a/lib/couchrest/more/extended_document.rb +++ b/lib/couchrest/more/extended_document.rb @@ -64,7 +64,7 @@ module CouchRest save_callback :before do |object| object['updated_at'] = Time.now - object['created_at'] = object['updated_at'] if object.new_document? + object['created_at'] = object['updated_at'] if object.new? end EOS end @@ -145,7 +145,7 @@ module CouchRest end # for compatibility with old-school frameworks - alias :new_record? :new_document? + alias :new_record? :new? # Trigger the callbacks (before, after, around) # and create the document @@ -166,7 +166,7 @@ module CouchRest # unlike save, create returns the newly created document def create_without_callbacks(bulk =false) raise ArgumentError, "a document requires a database to be created to (The document or the #{self.class} default database were not set)" unless database - set_unique_id if new_document? && self.respond_to?(:set_unique_id) + set_unique_id if new? && self.respond_to?(:set_unique_id) result = database.save_doc(self, bulk) (result["ok"] == true) ? self : false end @@ -181,7 +181,7 @@ module CouchRest # only if the document isn't new def update(bulk = false) caught = catch(:halt) do - if self.new_document? + if self.new? save(bulk) else _run_update_callbacks do @@ -197,7 +197,7 @@ module CouchRest # and save the document def save(bulk = false) caught = catch(:halt) do - if self.new_document? + if self.new? _run_save_callbacks do save_without_callbacks(bulk) end @@ -211,7 +211,7 @@ module CouchRest # Returns a boolean value def save_without_callbacks(bulk = false) raise ArgumentError, "a document requires a database to be saved to (The document or the #{self.class} default database were not set)" unless database - set_unique_id if new_document? && self.respond_to?(:set_unique_id) + set_unique_id if new? && self.respond_to?(:set_unique_id) result = database.save_doc(self, bulk) mark_as_saved if result["ok"] == true result["ok"] == true diff --git a/spec/couchrest/more/casted_model_spec.rb b/spec/couchrest/more/casted_model_spec.rb index 678c81f..029e294 100644 --- a/spec/couchrest/more/casted_model_spec.rb +++ b/spec/couchrest/more/casted_model_spec.rb @@ -280,7 +280,7 @@ describe CouchRest::CastedModel do end end - describe "calling new_model? on a casted model" do + describe "calling new? on a casted model" do before :each do reset_test_db! @cat = Cat.new(:name => 'Sockington') @@ -289,35 +289,35 @@ describe CouchRest::CastedModel do end it "should be true on new" do - CatToy.new.new_model?.should be_true + CatToy.new.should be_new CatToy.new.new_record?.should be_true end it "should be true after assignment" do - @cat.favorite_toy.new_model?.should be_true - @cat.toys.first.new_model?.should be_true + @cat.favorite_toy.should be_new + @cat.toys.first.should be_new end it "should not be true after create or save" do @cat.create @cat.save - @cat.favorite_toy.new_model?.should be_false - @cat.toys.first.new_model?.should be_false + @cat.favorite_toy.should_not be_new + @cat.toys.first.should_not be_new end it "should not be true after get from the database" do @cat.save @cat = Cat.get(@cat.id) - @cat.favorite_toy.new_model?.should be_false - @cat.toys.first.new_model?.should be_false + @cat.favorite_toy.should_not be_new + @cat.toys.first.should_not be_new end it "should still be true after a failed create or save" do @cat.name = nil @cat.create.should be_false @cat.save.should be_false - @cat.favorite_toy.new_model?.should be_true - @cat.toys.first.new_model?.should be_true + @cat.favorite_toy.should be_new + @cat.toys.first.should be_new end end diff --git a/spec/couchrest/more/extended_doc_spec.rb b/spec/couchrest/more/extended_doc_spec.rb index dd02a91..d4100bf 100644 --- a/spec/couchrest/more/extended_doc_spec.rb +++ b/spec/couchrest/more/extended_doc_spec.rb @@ -97,12 +97,12 @@ describe "ExtendedDocument" do it "should be a new_record" do @obj = Basic.new @obj.rev.should be_nil - @obj.should be_a_new_record + @obj.should be_new end it "should be a new_document" do @obj = Basic.new @obj.rev.should be_nil - @obj.should be_a_new_document + @obj.should be_new end end @@ -405,7 +405,7 @@ describe "ExtendedDocument" do end it "should be a new document" do - @art.should be_a_new_document + @art.should be_new @art.title.should be_nil end diff --git a/spec/couchrest/more/property_spec.rb b/spec/couchrest/more/property_spec.rb index 57af71a..4f85fde 100644 --- a/spec/couchrest/more/property_spec.rb +++ b/spec/couchrest/more/property_spec.rb @@ -85,7 +85,7 @@ describe "ExtendedDocument properties" do @invoice.location = nil @invoice.should_not be_valid @invoice.save.should be_false - @invoice.should be_new_document + @invoice.should be_new end end diff --git a/spec/fixtures/more/article.rb b/spec/fixtures/more/article.rb index e0a6393..840b45b 100644 --- a/spec/fixtures/more/article.rb +++ b/spec/fixtures/more/article.rb @@ -29,6 +29,6 @@ class Article < CouchRest::ExtendedDocument save_callback :before, :generate_slug_from_title def generate_slug_from_title - self['slug'] = title.downcase.gsub(/[^a-z0-9]/,'-').squeeze('-').gsub(/^\-|\-$/,'') if new_document? + self['slug'] = title.downcase.gsub(/[^a-z0-9]/,'-').squeeze('-').gsub(/^\-|\-$/,'') if new? end end \ No newline at end of file