Changeset 715

Timestamp:

02/01/08 14:00:02 (7 months ago)

Author:

mislav

Message:

minor optimization for Page; remove the unneccessary 'virtual' accessor

Files:

• branches/rails2/app/models/page.rb (modified) (2 diffs)

branches/rails2/app/models/page.rb

@@ -86 +86 @@
-  def status=(value)
-    self.status_id = value.id
-  end
-  
-  def virtual
-    !(read_attribute('virtual').to_s =~ /^(false|f|0|)$/)
-  end
-  
@@ -222 +218 @@
-  
-    def update_published_at
-'published_at', Time.now) if (status_id.to_i == Status[:published].id) and published_at.nil?
+:published_at, Time.now) if published? and !published_at
-      true
-    end
-    
-    def update_virtual
-if self.class !
-write_attribute('virtual', Page.descendant_class(class_name).new.virtual?)
+unless self.class =
+self.virtual = Page.descendant_class(class_name).new.virtual?
-      else
-write_attribute('virtual', virtual?)
+self.virtual = virtual?
-      end
-      true