@@ -3,14 +3,14 @@ %tr{:id => "page-attribute-#{page_attribute_row_counter}"} %th.label - %label{:for => "page_build_page_attributes__#{page_attribute_row_counter}_class_name"} + %label{:for => "page_page_attributes__#{page_attribute_row_counter}_value"} = page_attribute_row.name %small.info = "(#{page_attribute_row.class.display_name})" - - fields_for "page[build_page_attributes][#{page_attribute_row_counter}]" do |attribute| + - fields_for "page[page_attributes][#{page_attribute_row_counter}]" do |attribute| = attribute.hidden_field :class_name, :value => page_attribute_row.class_name = attribute.hidden_field :name, :value => page_attribute_row.name %td.field - - fields_for "page[build_page_attributes][#{page_attribute_row_counter}]" do |attribute| + - fields_for "page[page_attributes][#{page_attribute_row_counter}]" do |attribute| = render :partial => "/admin/page_attributes/#{page_attribute_row.class.partial_name}", :locals => { :attribute => attribute, :page_attribute => page_attribute_row } = link_to_function image('minus', :alt => 'minus'), %{if(confirm('Delete this attribute?')) { page_attribute_removed('page-attribute-#{page_attribute_row_counter}') }}, :title => 'Remove Attribute' app/views/admin/page_attributes/_page_attribute_row.haml @@ -1,6 +1,6 @@ module PageAttributes module PageExtensions - + def self.included(base) base.class_eval do has_many :page_attributes, :dependent => :destroy do @@ -10,17 +10,36 @@ module PageAttributes end end end + after_save :save_page_attributes + alias_method_chain :page_attributes, :pending + alias_method_chain :page_attributes=, :pending end end - - def build_page_attributes=(args={}) - self.page_attributes.destroy_all - args.each_pair do |i,hash| - attr = self.page_attributes.build(hash) - attr.page = self + + def page_attributes_with_pending(reload=false) + @page_attrs = nil if reload + @page_attrs || page_attributes_without_pending(reload) + end + + def page_attributes_with_pending=(collection) + if collection.values.all? {|item| item.is_a? PageAttribute } + self.page_attributes_without_pending = collection.values + else + self.updated_at_will_change! + @page_attrs = collection.values.map { |item| PageAttribute.new(item.merge(:page => self)) } end - self.page_attributes end - + + private + + def save_page_attributes + if @page_attrs + self.page_attributes_without_pending.clear + @page_attrs.each {|p| self.page_attributes_without_pending << p } + end + @page_attrs = nil + true + end + end end \ No newline at end of file lib/page_attributes/page_extensions.rb e92ac21e454f43c06d7bd60e1e89d5e47e3e1bfd Josh French josh@vitamin-j.com http://github.com/jfrench/radiant-page-attributes/commit/6ddb1525f31ce4ebd87dd30a37addcd190f889e8 6ddb1525f31ce4ebd87dd30a37addcd190f889e8 2009-03-21T12:54:23-07:00 2009-03-21T12:54:23-07:00 Steal the nice PageParts pattern 3dce8edf578a2f551594b1b5b18a71ace460ba73 Josh French josh@vitamin-j.com