skylinecms 3.0.8 → 3.1.0

data/app/controllers/skyline/sections_controller.rb

@@ -8,17 +8,24 @@ class Skyline::SectionsController < Skyline::ApplicationController
       section.sectionable = params[:sectionable_type].constantize.new
       section_guid = Guid.new
       
+      if params[:after_section]
+        pos, id = :after, params[:after_section]          
+      else
+        pos, id = :bottom, "contentlist"
+      end
+            
       fields_for params[:object_name] do |variant_form|
-        page.insert_html(:bottom, "contentlist", :partial => "form", :locals => {:variant_form => variant_form, :section => section, :guid => section_guid})
+        page.insert_html(pos, id, :partial => "form", :locals => {:variant_form => variant_form, :section => section, :guid => section_guid})
       end
+      
       page << "$('contentlist').retrieve('application.sections').addSection('section_#{section_guid}');"
-      page << "var myFx = new Fx.Scroll(\"contentEditPanel\").toBottom();"
+      page << "$('contentEditPanel').scrollTo(0, $('contentEditPanel').getScroll().y + $('section_#{section_guid}').getPosition(\"contentEditPanel\").y)"
     end
   end
   
   protected
   def find_renderable_scope
-    @renderable_scope = Skyline::RenderableScopeInterface.unserialize(params[:renderable_scope]) if params[:renderable_scope]
+    @renderable_scope = Skyline::Rendering::Scopes::Interface.unserialize(params[:renderable_scope]) if params[:renderable_scope]
     raise "Can't load renderable_scope from params[:renderable_scope]: '#{params[:renderable_scope]}'" unless @renderable_scope
   end
 end

data/app/controllers/skyline/settings_controller.rb

@@ -2,12 +2,14 @@ class Skyline::SettingsController < Skyline::Skyline2Controller
   
   layout "skyline/layouts/settings"
   
-  self.default_menu_item = :admin
+  self.default_menu = :admin
   
   authorize :index, :edit, :update, :by => :settings_update
   
   def index
-    redirect_to edit_skyline_setting_path(@implementation.settings.page_names.first)
+    if @implementation.has_settings?
+      redirect_to edit_skyline_setting_path(@implementation.settings.page_names.first)
+    end
   end
   
   def edit

data/app/controllers/skyline/user_preferences_controller.rb

@@ -0,0 +1,18 @@
+class Skyline::UserPreferencesController < Skyline::ApplicationController
+  def create
+    unless params[:skyline_up].blank?
+      user_preference = ActiveSupport::JSON.decode(params[:skyline_up])
+      
+      if user_preference.values.first == "_delete"
+        current_user.user_preferences.remove(user_preference.keys.first)
+        up = {}
+      else
+        current_user.user_preferences.set(user_preference.keys.first, user_preference.values.first)
+        up = {user_preference.keys.first => user_preference.values.first}
+      end
+    end
+    respond_to do |format|
+      format.json  { render :json => up }
+    end
+  end
+end

data/app/controllers/skyline/users_controller.rb

@@ -1,7 +1,7 @@
 class Skyline::UsersController < Skyline::ApplicationController
   layout "skyline/layouts/settings"
   
-  self.default_menu_item = :admin
+  self.default_menu = :admin
   
   authorize :index, :by => "user_show"
   authorize :new,:create, :by => "user_create"

data/app/controllers/skyline/variant_current_editor_controller.rb

@@ -1,6 +1,8 @@
 class Skyline::VariantCurrentEditorController < Skyline::ApplicationController
   layout false
   
+  skip_before_filter :handle_user_preferences
+  
   def poll
     return render(:nothing => true) if session[:user_id].blank?
     

data/app/helpers/skyline/application_helper.rb

@@ -1,11 +1,8 @@
 module Skyline::ApplicationHelper  
   # Place a tick or a cross depending on the value of bool
   #
-  # ==== Parameters
-  # bool<Boolean>:: The value of the tick/cross
-  # options<Hash>:: Options will be passed to the image_tag method
-  # 
-  # --
+  # @param [Boolean] bool The value of the tick/cross
+  # @param [Hash] options ({}) Options will be passed to the image_tag method
   def tick_image(bool,options={})
     name = bool ? "true" : "false"
     src = "/skyline/images/icons/#{name}.gif"
@@ -15,56 +12,63 @@ module Skyline::ApplicationHelper
     image_tag(src,options)
   end
   
+  # Set the Skyline JS layout to use. Used in templates so the layout can initialize the correct layout.
+  #
+  # @param [Symbol] name The name of the js layout to use @see #render_js_layout
+  def use_js_layout(name)
+    @_js_layout = name
+  end
+  
+  # Does this template have  a Skyline JS Layout set?
+  #
+  # @return [Boolean]
+  def has_js_layout?
+    @_js_layout.present?
+  end  
+  
+  # Actually render the JS code to initialize the previously set Skyline JS Layout.
+  # 
+  # @todo Currently only implemented in EntopicEditor plugin
+  def render_js_layout
+    return unless @_js_layout
+    raise "TO BE IMPLEMENTED"
+  end  
+  
   # You can use this method to place a message directly in your view. This also
   # works directly from a render(:update) or an update_page block. 
   # 
-  # ==== Parameters
-  # type<Symbol>:: The type of the message (:error,:notification,:success)
-  # message<String>:: The message to show
-  # optiosn<Hash>:: Options to be passed to the MessageGenerator (javascript)
-  #
-  # --
+  # @param [Symbol] type The type of the message (:error,:notification,:success)
+  # @param [String] message The message to show
+  # @param [Hash] options ({}) Options to be passed to the MessageGenerator (javascript)
   def message(type,message,options={})
-    MessageGenerator.new(type,message,options)
+    Skyline::MessageGenerator.new(type,message,options)
   end
 
   # You can use this method to place a notification directly in your view. This also
   # works directly from a render(:update) or an update_page block. 
   # 
-  # ==== Parameters
-  # type<Symbol>:: The type of the message (:error,:notification,:success)
-  # message<String>:: The message to show
-  # optiosn<Hash>:: Options to be passed to the MessageGenerator (javascript)
-  #
-  # --  
+  # @param [Symbol] type The type of the message (:error,:notification,:success)
+  # @param [String] message The message to show
+  # @param [Hash] options ({}) Options to be passed to the MessageGenerator (javascript)
   def notification(type,message,options={})
-    MessageGenerator.new(type,message,options)    
+    Skyline::NotificationGenerator.new(type,message,options)    
   end
   
+  # Actually render the messages on screen.
+  # 
+  # @option options [Class] :generator (Skyline::MessageGenerator) The generator to use to render the messages.
   def render_messages(options={})
     _render_volatiles(self.messages,options)
   end
   
+  # Actually render the notifications on screen.
+  #
+  # @option options [Class] :generator (Skyline::NotificationGenerator) The generator to use to render the messages.
   def render_notifications(options={})
+    options.reverse_merge! :generator => Skyline::NotificationGenerator
     _render_volatiles(self.notifications,options)
   end
-  
-  def plugin_hook(name)
-    template = caller.first[/app\/views\/([^:]*):/,1]
-    raise "Cannot determine template from caller: #{caller}" unless template
-    plugin_template = template.sub(".html.erb", "_#{name}.html.erb")
     
-    logger.debug "Looking for template #{plugin_template} in plugins..."
-    Dir[Rails.root + "vendor/skyline_plugins/*/app/views/#{plugin_template}"].each do |file|      
-      if RAILS_ENV == "development"
-        concat render(:inline => File.read(file), :layout => nil)
-      else
-        # render :file caches the file somehow, so only use it in production mode
-        concat render(:file => file, :layout => nil)
-      end
-    end    
-  end
-  
   protected
   
   def _render_volatiles(messages_hash, options={})

data/app/helpers/skyline/dialog_helper.rb

@@ -8,23 +8,24 @@ module Skyline::DialogHelper
       content = args.last 
       options = options_for_render
     else
-      options = options_for_render.slice!(:partial, :locals)
-      options.each do |k,v|
-        options[k] = case v
-          when String,Symbol : "'" + escape_javascript(v.to_s) + "'"
-          when Hash : options_for_javascript(v)
-          else v
-        end
-      end      
+      options = options_for_render.slice!(:partial, :locals)   
       content = render(options_for_render)
     end
     
-    p =  "var s = function(){"
+    options.each do |k,v|
+      options[k] = case v
+        when String,Symbol : "'" + escape_javascript(v.to_s) + "'"
+        when Hash : options_for_javascript(v)
+        else v
+      end
+    end    
+    
+    p =  "(function(){"
     p << "var sd = new Skyline.Dialog(#{options_for_javascript(options)});"
     p << "sd.setTitle('#{escape_javascript(title)}');"
     p << "sd.setContent('#{escape_javascript(content)}');"
     p << "sd.setup(); sd.show();"
-    p << "}()"
+    p << "})()"
   end
   
 end

data/app/helpers/skyline/editors/boolean.rb

@@ -1,6 +1,6 @@
 class Skyline::Editors::Boolean < Skyline::Editors::Editor
   def output_without_errors
-    check_box_tag(input_name(self.attribute_names), "1" , field.value(record)) +
-    hidden_field_tag(input_name(self.attribute_names), "0") + " " + field.singular_label   
+    hidden_field_tag(input_name(self.attribute_names), "0") + 
+    check_box_tag(input_name(self.attribute_names), "1" , field.value(record)) + " " + field.singular_label  
   end
 end

data/app/helpers/skyline/editors/page_browser.rb

@@ -1,5 +1,7 @@
 class Skyline::Editors::PageBrowser < Skyline::Editors::Editor
   def output_without_errors
+    return "" unless Skyline::Configuration.enable_pages
+    
     attr_names = self.attribute_names.dup
     attr_names[-1] = attr_names[-1].to_s.gsub(/_id$/, "_attributes")
     input_name_prefix = input_name(attr_names)                    # ie: element[image_attributes]

data/app/helpers/skyline/editors/wysiwyg.rb

@@ -2,7 +2,7 @@ class Skyline::Editors::Wysiwyg < Skyline::Editors::Editor
   def output_without_errors
     out = content_tag("div",text_area_tag(
       input_name(self.attribute_names), 
-      record.send(field.name,true), 
+      record.send(field.name), 
       :class => "wysiwyg", 
       :rows => 15,
       :cols => 90,

data/app/helpers/skyline/js_layout_helper.rb

@@ -0,0 +1,19 @@
+module Skyline::JsLayoutHelper
+  def use_js_layout(name)
+    @_js_layout = name
+  end
+  
+  def has_js_layout?
+    @_js_layout.present?
+  end
+  
+  def render_js_layout
+    return unless @_js_layout
+    
+    t = ["Layout"]
+    t << {:content => "Content", :media => "Media"}[@_js_layout.to_sym]
+    
+    layout_js = "new Application.#{t.compact.join(".")}()"
+    javascript_tag(layout_js)
+  end
+end

data/app/helpers/skyline/media/dirs_helper.rb

@@ -0,0 +1,2 @@
+module Skyline::Media::DirsHelper
+end

data/app/helpers/skyline/media/files_helper.rb

@@ -0,0 +1,6 @@
+module Skyline::Media::FilesHelper
+  def skyline_media_dir_files_path_with_session_information(media_dir)
+    session_key = ActionController::Base.session_options[:session_key] || ActionController::Base.session_options[:key]
+    skyline_media_dir_files_path(media_dir, session_key => cookies[session_key], request_forgery_protection_token => form_authenticity_token)
+  end
+end

data/app/helpers/skyline/media_files_helper.rb

@@ -1,7 +1,7 @@
 # @private
 module Skyline::MediaFilesHelper
-  def skyline_media_file_path_with_session_information(media_dir)
+  def skyline_media_dir_files_path_with_session_information(media_dir)
     session_key = ActionController::Base.session_options[:session_key] || ActionController::Base.session_options[:key]
-    skyline_media_dir_media_files_path(media_dir, session_key => cookies[session_key], request_forgery_protection_token => form_authenticity_token)
+    skyline_media_dir_files_path(media_dir, session_key => cookies[session_key], request_forgery_protection_token => form_authenticity_token)
   end
 end

data/app/helpers/skyline/menu_helper.rb

@@ -0,0 +1,33 @@
+module Skyline::MenuHelper
+  
+  # @param scope [Array]
+  # @param url [String,Hash]
+  # @param options [Hash]
+  def menu_item(*scope)
+    scope = Array(scope)
+    url_options = scope.extract_options!
+    options = url_options.slice!(:method)
+        
+    url = scope.pop
+    scope = current_menu_scope(scope)
+    if controller.current_menu[0, scope.size] == scope
+      (options[:class] ||= "") << " active"
+    end
+    
+    
+    content_tag("li", link_to(t(scope.last, :scope => [:navigation, scope[-2]]), url, url_options), options)
+  end
+
+  def menu_for(*scope, &block)
+    return unless controller.current_menu[0, scope.size] == scope
+    @_menu_scope = scope
+    menu = capture(&block)
+    @_menu_scope = []
+    concat(menu)
+  end
+  
+  def current_menu_scope(scope)
+    (@_menu_scope || []) + scope
+  end
+    
+end

data/app/helpers/skyline/message_generator.rb

@@ -15,11 +15,15 @@ class Skyline::MessageGenerator
         else v
       end
     end    
-    "new Application.Message('#{self.escape_javascript(@message)}',#{self.options_for_javascript(options)})"
+    "new #{self.js_object}('#{self.escape_javascript(@message)}',#{self.options_for_javascript(options)})"
   end
   
   def to_str
     to_s
   end
   
+  def js_object
+    "Application.Message"
+  end
+  
 end

data/app/helpers/skyline/notification_generator.rb

@@ -0,0 +1,6 @@
+# @private
+class Skyline::NotificationGenerator < Skyline::MessageGenerator
+  def js_object
+    "Application.Notification"
+  end  
+end

data/app/helpers/skyline/plugin_helper.rb

@@ -0,0 +1,21 @@
+module Skyline::PluginHelper
+  
+  # Add a hook to the template where a plugin can place it's own template.
+  # If templates from multiple plugins match the templates will be concatenated.
+  def plugin_hook(name)
+    template = caller.first[/app\/views\/([^:]*):/,1]
+    raise "Cannot determine template from caller: #{caller}" unless template
+    plugin_template = template.sub(".html.erb", "_#{name}.html.erb")
+    
+    logger.debug "Looking for template #{plugin_template} in plugins..."
+    Dir[Skyline::PluginsManager.plugin_path + "*/app/views/#{plugin_template}"].each do |file|      
+      if Rails.env == "development"
+        concat render(:inline => File.read(file), :layout => nil)
+      else
+        # render :file caches the file somehow, so only use it in production mode
+        concat render(:file => file, :layout => nil)
+      end
+    end    
+  end
+  
+end

data/app/helpers/skyline/presenters/presenter.rb

@@ -4,11 +4,7 @@
 class Skyline::Presenters::Presenter
   class << self
     def create(presenter,records,fieldset,template,options={})
-      klass = case presenter
-        when :article : Skyline::Presenters::ArticleList
-        else Skyline::Presenters::Table
-      end      
-      klass.new(records,fieldset,template,options)
+      Skyline::Presenters::Table.new(records,fieldset,template,options)
     end
   end
   
@@ -31,14 +27,9 @@ class Skyline::Presenters::Presenter
   def content_tag(*params); @template.content_tag(*params); end # :nodoc:
   
   def output
-    [header,body,footer].join("\n")
+    @template.render :partial => "skyline/content/presenters/table", :locals => {:presenter => self}
   end
-  
-  def header; ""; end
-  def body; ""; end
-  def footer; ""; end
-  
-  protected
+    
   
   def edit_button(record)
 		link_to button_text(:edit),{:action => "edit", :types => stack.url_types(:down => [record.id]), :return_to  => url_for({:filter => params[:filter]})}, :class => "button small"
@@ -118,7 +109,7 @@ class Skyline::Presenters::Presenter
           truncate(simple_format(strip_tags(content.gsub("<br />", "<br />\n").gsub("</p>", "</p>\n"))),150)
         end
       when String : truncate(content,150)
-      when TrueClass,FalseClass : [(content ? image_tag("icons/tick.png") : image_tag("icons/cross.png")),{:class => "center"}]
+      when TrueClass,FalseClass : [(content ? image_tag("/skyline/images/icons/true.gif") : image_tag("/skyline/images/icons/false.gif")),{:class => "center"}]
       when Date,Time : l(content, :format => :long)
       else content.to_s
     end 

data/app/helpers/skyline/presenters/table.rb

@@ -1,30 +1,3 @@
 class Skyline::Presenters::Table < Skyline::Presenters::Presenter
-  
-  def output
-    content_tag("table",super, :class => "listing")
-  end
-        
-  def header
-    "<thead><tr>#{self.heading_collection.map{|f| content_tag("th",f.singular_label.capitalize)}.join("\n")}<th></th><th></th></tr></thead>"
-  end
-  
-  def body
-    if self.collection.any?
-      self.collection.map{|el| next if el.new_record?; content_tag("tr",self.row(el).join("\n"), :class => cycle("odd","even")) }
-    else
-      "<tr class=\"blank-slate\"><td colspan=\"#{self.heading_collection.size + 2}\">#{t(:blank_slate, :scope => [:content,:list], :class => self.fieldset.plural_name.downcase)}.</td></tr>"
-    end
-  end
-  
-  def row(record)
-    return if record.new_record? # there can't be any new records in a list!
-    content_cells = self.heading_collection.map do |field| 
-      content,options = self.value(record,field); 
-      content_tag("td",content,options)
-    end
-    
-    [content_cells, content_tag("td",edit_button(record), :class => "edit"), content_tag("td",delete_button(record),:class => "delete")]
-  end
-  
         
 end

data/app/helpers/skyline/ref_object_helper.rb

@@ -76,9 +76,9 @@ module Skyline::RefObjectHelper
         l << browse_button
       end
       
-      c = content_tag("div", c.join, :class => "relatesTo #{"linked" if linked_form.object.andand.referable && !linked_form.object.andand.marked_for_destruction?}")
+      c = content_tag("div", c.join, :class => "relatesTo #{"linked" if linked_form.object.present? && !linked_form.object.andand.marked_for_destruction?}")
     end
 
-    content_tag "div", c, :id => form_builder.dom_id(field), :class => css_class
+    content_tag("div", c, :id => form_builder.dom_id(field), :class => css_class) + form_builder.fieldset_errors(field).to_s
   end
 end

data/app/helpers/skyline/tree_helper.rb

@@ -20,7 +20,13 @@ module Skyline::TreeHelper
       edit_skyline_article_path(page["id"])
     end
     
-    options.reverse_merge! :id_prefix => "article", :node_content => node_content, :node_url => node_url, :node_title => node_title
+    options.reverse_merge!(
+      :id_prefix => "article", 
+      :node_content => node_content, 
+      :node_url => node_url, 
+      :node_title => node_title,
+      :open_nodes => preferred_open_nodes("pages.tree.nodes.open", options[:selected])
+    )
     node_tree(pages,roots,options)
   end
   
@@ -36,27 +42,90 @@ module Skyline::TreeHelper
     node_url = Proc.new do |node|
       skyline_media_dir_media_files_path(node)
     end
-    options.reverse_merge! :id_prefix => "media", :node_content => node_content, :node_url => node_url, :node_title => node_title
+    options.reverse_merge!(
+      :id_prefix => "media", 
+      :node_content => node_content, 
+      :node_url => node_url, 
+      :node_title => node_title,
+      :open_nodes => preferred_open_nodes("media.dirs.tree.nodes.open", options[:selected])
+      )
     node_tree(dirs,roots,options)    
   end
   
-  
+  # Build a nested UL/LI construct used for trees.
+  #
+  # @param node_collection [Hash] A flat Hash which is keyed on parent_id and have as value all the child elements
+  # @param nodes [Array] The nodes for the current level to place (mostly the root nodes).
+  # 
+  # @option options id_prefix [String] The prefix to place before the node id (this is used as the DOM id of the LI tag)
+  # @option options node_content [lambda{|node| }] A proc returning a string that will put as the content of the node
+  # @option options node_url [lambda{|node| }] A proc returning the URL for the node
+  # @option options node_title [lambda{|node| }] The title to use for node A tag.
+  # @option options selected [~id] The selected node, will be compared against #id
+  # @option options class [String] The CSS class to give the node
   def node_tree(node_collection,nodes,options={})
     node_url = Proc.new{|node| "" }
     node_content = Proc.new{|node| node }
     node_title = Proc.new{|node| "" }
-    options.reverse_merge! :id_prefix => "node", :node_content => node_content, :node_url => node_url, :node_title => node_title, :selected => nil, :open => []
+    options.reverse_merge!(
+      :id_prefix => "node", 
+      :node_content => node_content, 
+      :node_url => node_url, 
+      :node_title => node_title, 
+      :selected => nil,
+      :open_nodes => []
+    )
+    
     tags = []
     nodes ||= []
+    open_nodes = options[:open_nodes] || []
+    
+    logger.warn("ON #{open_nodes.inspect}");
     
     nodes.each do |node|
       selected = options[:selected].present? ? options[:selected].id == node.id : false
       li = link_to(options[:node_content].call(node), options[:node_url].call(node), :class => (selected ? "selected" : nil), :title => options[:node_title].call(node))
       li << node_tree(node_collection,node_collection[node.id],options) if node_collection.has_key?(node.id)
-      tags << content_tag("li",li , :id => "#{options[:id_prefix]}_#{node.id}", :class => (options[:open].include?(node)))
+      
+      node_class = []
+      # node_class << node.open ? "open" : "closed" if node_collection.has_key?(node.id) && node.respond_to?(:open)
+      node_class << (open_nodes.include?("#{node.id}") ? "open" : "closed") if node_collection.has_key?(node.id) 
+      node_class << options[:class] if options[:class]
+      
+      tags << content_tag("li",li , :id => "#{options[:id_prefix]}_#{node.id}", :class => (node_class.any? ? node_class.join(" ") : nil))
     end
     
     content_tag("ul",tags.join("\n"));
   end
   
+  # Get's the current user preferred open nodes
+  #
+  # @param String up_key The User preferences key to use ("a.b.c")
+  # @param ~id selected The currently selected node
+  #
+  def preferred_open_nodes(up_key, selected = nil)
+    open_nodes = []
+    if user_preferences = current_user.user_preferences.get(up_key)
+      open_nodes = user_preferences.inject([]) do |result, (k,v)|
+        if v
+          id = k
+          result << id
+        end
+      end
+    end
+    
+    # Make sure parents are open.
+    if selected.present?
+      if selected.respond_to?("nesting")
+        selected.nesting.each do |p|
+          open_nodes << p.id.to_s
+        end
+      else
+        open_nodes << selected.parent_id.to_s
+      end
+    end
+    
+    open_nodes    
+  end
+  
 end

data/app/models/skyline/article.rb

@@ -14,9 +14,20 @@ class Skyline::Article < ActiveRecord::Base
       super      
       subclass.set_table_name subclass.name.underscore.gsub("/","_")
       
-      parent = subclass.parent
-      parent.send(:belongs_to, :default_variant_data, :class_name => subclass.name)
-      parent.send(:belongs_to, :published_publication_data, :class_name => subclass.name)
+      parentclass = subclass.parent
+      parentclass.class_eval do
+        belongs_to :default_variant_data, :class_name => subclass.name
+        belongs_to :published_publication_data, :class_name => subclass.name
+      end
+      
+      subclass.class_eval do
+        has_one :article, :foreign_key => "published_publication_data_id", :class_name => parentclass.name
+        
+        named_scope :published, {
+          :include => [:article],
+          :conditions => "skyline_articles.published_publication_data_id = #{self.table_name}.id"
+        }
+      end
       
       if !Rails.configuration.cache_classes && !(ActiveSupport::Dependencies.load_once_path?(__FILE__) && subclass.parents[-2] == ::Skyline)        
         ActiveSupport::Dependencies.autoloaded_constants << subclass.to_s
@@ -183,7 +194,8 @@ class Skyline::Article < ActiveRecord::Base
   
   def set_default_variant!(variant)
     return if variant.id == self.default_variant_id && variant.data_id == self.default_variant_data_id
-    self.update_attributes(:default_variant_id => variant.id, :default_variant_data_id => variant.data_id)
+    self.attributes = {:default_variant_id => variant.id, :default_variant_data_id => variant.data_id}
+    self.save(false)
   end
   
   def set_default_variant(variant)
@@ -228,7 +240,7 @@ class Skyline::Article < ActiveRecord::Base
   end  
   
   def renderable_scope
-    Skyline::WildcardRenderableScope.new
+    Skyline::Rendering::Scopes::Wildcard.new
   end  
   
   protected
@@ -249,7 +261,7 @@ class Skyline::Article < ActiveRecord::Base
     raise StandardError, "can't be destroyed because this page is persistent" if self.persistent?
   end  
 
-  # reset ref objects that refer to removed media file
+  # Reset ref objects that refer to this now removed Article.
   # by setting referable_id = nil
   def reset_ref_object
     Skyline::RefObject.update_all({:referable_id => nil}, {:referable_id => self.id, :referable_type => self.class.name})    

data/app/models/skyline/article_version.rb

@@ -25,7 +25,7 @@ class Skyline::ArticleVersion < ActiveRecord::Base
   end
   
   def clone
-    returning super do |clone|
+    super.tap do |clone|
       clone.created_at = nil
       clone.updated_at = nil
       clone.sections = self.sections.collect{|section| section.clone}