momomoto 0.1.3 → 0.1.4

data/lib/momomoto/base.rb

@@ -170,7 +170,7 @@ module Momomoto
               store = instance_variable_get(:@data)
               value = data_type.filter_set( value )
               if store[index] != value
-                instance_variable_set(:@dirty, true)
+                mark_dirty( field_name )
                 store[index] = value
               end
             end

data/lib/momomoto/row.rb

@@ -22,18 +22,27 @@ module Momomoto
       @data == other.instance_variable_get( :@data )
     end
 
-    def dirty?
+    def dirty
       @dirty
     end
 
-    def dirty=( value )
-      @dirty = !!value
+    def dirty?
+      @dirty.length > 0
+    end
+
+    def mark_dirty( field )
+      field = field.to_sym
+      @dirty.push( field ) if not @dirty.member?( field )
+    end
+
+    def clean_dirty
+      @dirty = []
     end
 
     def initialize( data = [] )
       @data = data
       @new_record = false
-      @dirty = false
+      clean_dirty
     end
 
     def new_record?
@@ -72,7 +81,7 @@ module Momomoto
       value = table.columns[column.to_sym].filter_set( value )
       index = table.column_order.index( column.to_sym )
       if @data[index] != value
-        @dirty = true
+        mark_dirty( column )
         @data[index] = value
       end
     end

data/lib/momomoto/table.rb

@@ -21,7 +21,7 @@ module Momomoto
       # set the columns of the table this class operates on
       def columns=( columns )
         # we store the order separate because it's quite important 
-        # that it's constant otherwise get_colum and set_column 
+        # that it's constant otherwise get_column and set_column 
         # on the row class might stop working 
         @column_order = columns.keys
         @columns = columns
@@ -93,7 +93,7 @@ module Momomoto
       def primary_keys( keys = nil )
         return self.primary_keys=( keys ) if keys
         if not instance_variable_defined?( :@primary_keys )
-          self.primary_keys=( database.fetch_primary_keys( table_name(), schema_name()) )
+          initialize_table
         end
         @primary_keys
       end
@@ -243,7 +243,7 @@ module Momomoto
           return false unless row.dirty?
           update( row )
         end
-        row.dirty = false
+        row.clean_dirty
         true
       end
 
@@ -262,7 +262,7 @@ module Momomoto
           end
           next if row.send( field_name ).nil?
           fields << field_name
-          values << datatype.escape( row.send( field_name ))
+          values << datatype.escape( row.get_column( field_name ))
         end
         raise Error, "insert with all fields nil" if fields.empty?
         sql = "INSERT INTO " + full_name + '(' + fields.join(',') + ') VALUES (' + values.join(',') + ');'
@@ -275,7 +275,8 @@ module Momomoto
         raise CriticalError, 'Updating is only allowed for tables with primary keys' if primary_keys.empty?
         setter, conditions = [], {}
         columns.each do | field_name, data_type |
-          setter << field_name.to_s + ' = ' + data_type.escape(row.send(field_name))
+          next if not row.dirty.member?( field_name )
+          setter << field_name.to_s + ' = ' + data_type.escape(row.get_column(field_name))
         end
         primary_keys.each do | field_name |
           raise Error, "Primary key fields must not be empty!" if not row.send( field_name )

data/test/test_table.rb

@@ -216,6 +216,26 @@ class TestTable < Test::Unit::TestCase
     p2.delete
   end
 
+  def test_update
+    Person.select(:first_name=>'test_update').each do | p | p.delete end
+
+    assert_raise( Momomoto::Nothing_found ) do
+      Person.select_single(:first_name=>'test_update')
+    end
+
+    p1 = Person.new(:first_name=>'test_select_single')
+    p1.write
+    p1.first_name = 'Chunky'
+    p1.last_name = 'Bacon'
+    p1.write
+    p2 = Person.select_single(:person_id=>p1.person_id)
+    assert_equal( p1.person_id, p2.person_id)
+    assert_equal( p1.first_name, p2.first_name )
+    assert_equal( p1.last_name, p2.last_name)
+
+    p1.delete
+  end
+
   def test_defaults
     conf = Conference.select_or_new(:conference_id=>1)
     conf.acronym = 'Pentabarf'

metadata

@@ -1,10 +1,10 @@
 --- !ruby/object:Gem::Specification 
-rubygems_version: 0.8.11
+rubygems_version: 0.9.4
 specification_version: 1
 name: momomoto
 version: !ruby/object:Gem::Version 
-  version: 0.1.3
-date: 2007-08-17 00:00:00 +02:00
+  version: 0.1.4
+date: 2007-09-01 00:00:00 +02:00
 summary: Momomoto is an object relational mapper for PostgreSQL.
 require_paths: 
 - lib
@@ -25,6 +25,7 @@ required_ruby_version: !ruby/object:Gem::Version::Requirement
 platform: ruby
 signing_key: 
 cert_chain: 
+post_install_message: 
 authors: 
 - Sven Klemm
 files: