drg_cms 0.6.1.0 → 0.6.1.6

data/app/helpers/dc_application_helper.rb

@@ -1,4 +1,3 @@
-#coding: utf-8
 #--
 # Copyright (c) 2012+ Damjan Rems
 #
@@ -95,17 +94,17 @@ end
 ############################################################################
 def dc_render(renderer, opts={})
   return dc_render_design_part(renderer[:part]) if renderer.class == Hash
-# 
+
   opts[:edit_mode]  = session[:edit_mode] 
   opts[:editparams] = {}
   
   opts = @options.merge(opts) # merge options with parameters passed on site, page, design ...
   opts.symbolize_keys!        # this makes lots of things easier
-# Create renderer object
+  # Create renderer object
   klass = renderer.to_s.downcase
   klass += '_renderer' unless klass.match('_renderer') #
   obj = Kernel.const_get(klass.classify, Class.new).new(self, opts) rescue nil
-# 
+
   if obj
     html = obj.render_html
     @css  << obj.render_css.to_s
@@ -133,7 +132,7 @@ def dc_replace_in_design(opts={})
   design = opts[:piece] ? DcPiece.find(name: opts[:piece]).script : dc_get_site.design
   layout = opts[:layout] || (dc_get_site.site_layout.size > 2 ? dc_get_site.site_layout : nil)
   if opts[:replace]
-# replace more than one part of code
+    # replace more than one part of code
     if opts[:replace].class == Array
       0.upto(opts[:replace].size - 1) {|i| design.sub!(opts[:replace][i], opts[:with][i])}
     else
@@ -179,13 +178,12 @@ end
 # 
 # This helper is replacement dc_render_from_site method which will soon be deprecated. 
 ########################################################################
-def dc_render_design_part(part) 
-  
+def dc_render_design_part(part)
   case
   when part.nil? then logger.error('ERROR dc_render_design_part! part is NIL !'); ''
-# Send as array. Part may be defined with options on page. First element has
-# name of element which defines what to do. If not defined default behaviour is 
-# called. That is what is defined in second part of array.
+  # Send as array. Part may be defined with options on page. First element has
+  # name of element which defines what to do. If not defined default behaviour is
+  # called. That is what is defined in second part of array.
   when part.class == Array then
     if @options.dig(:settings, part.first)
       #TODO to be defined 
@@ -196,9 +194,9 @@ def dc_render_design_part(part)
   when part.class == Proc then
     result = part.call
     result.class == Array ? result.first : result
-# Send as string. Evaluate content of string    
+  # Send as string. Evaluate content of string
   when part.class == String then eval part
-# For future maybe. Just call objects to_s method.
+  # For future maybe. Just call objects to_s method.
   else
     part.to_s
   end.html_safe
@@ -231,9 +229,9 @@ end
 ########################################################################
 # Helper for rendering top CMS menu when in editing mode
 ########################################################################
-def dc_page_top()
-  if @design and !@design.rails_view.blank?
-# Evaluate parameters in design body    
+def dc_page_top
+  if @design && @design.rails_view.present?
+    # Evaluate parameters in design body
     eval(@design.body)
   end
   session[:edit_mode] > 0 ? render(partial: 'cmsedit/edit_stuff') : ''
@@ -282,18 +280,18 @@ end
 def dc_edit_title
   session[:form_processing] = "form:title:"
   title = @form['form']['title']
-# defined as form:title:edit
+  # defined as form:title:edit
   if title and title['edit'] and !@form['readonly']
     t( title['edit'], title['edit'] )
   elsif title and title['show'] and @form['readonly']
     t( title['show'], title['show'] )
   else
-# concatenate title 
+    # concatenate title
     c = (@form['readonly'] ? t('drgcms.show') : t('drgcms.edit')) + " : "
-    c << (@form['title'] ? t( @form['title'], @form['title'] ) : t_tablename(@form['table'])) + ' : '
+    c << (@form['title'].class == String ? t( @form['title'], @form['title'] ) : t_tablename(@form['table'])) + ' : '
     title = (title and title['field']) ? title['field'] : @form['form']['edit_title']
     dc_deprecate('form:edit_title will be deprecated. Use form:title:field instead.') if @form['form']['edit_title']
-#
+
     c << "#{@record[ title ]} : " if title and @record.respond_to?(title)
     c << @record.id if @record
   end
@@ -309,15 +307,11 @@ end
 def dc_new_title()
   session[:form_processing] = "form:title:"
   title = @form['form']['title']
-# defined as form:title:new
+  # defined as form:title:new
   if title and title['new']
     t( title['new'], title['new'] )
   else
-    if @form['table'] == 'dc_dummy'
-      dc_deprecate('dc_dummy will be deprecated. Use dc_memory instead.')
-      @form['table'] = 'dc_memory'
-    end
-# in memory variables    
+    # in memory structures
     if @form['table'] == 'dc_memory'
       t( @form['title'], @form['title'] )
     else
@@ -366,7 +360,8 @@ end
 # Returns icon code if icon is specified
 ############################################################################
 def dc_icon_for_link(icon)
-  return nil unless icon
+  return nil if icon.nil?
+
   if icon.match(/\./)
     _origin.image_tag(icon, class: 'dc-link-img dc-animate')
   elsif icon.match('<i')
@@ -439,22 +434,13 @@ end
 # Therefore it is very unwise to use non ascii chars for table (collection) names.
 # 
 # Parameters: 
-# [string] String. String to be converted into decamelized string.
+# [Object] model_string. String or model to be converted into decamelized string.
 # 
 # Returns:
 # String. Decamelized string.
 ########################################################################
-def decamelize_type(string)
-  return nil if string.nil?
-  r = ''
-  string.to_s.each_char do |c|
-    r << case 
-      when r.size == 0     then c.downcase
-      when c.downcase != c then '_' + c.downcase
-      else c      
-    end
-  end
-  r
+def decamelize_type(model_string)
+  model_string ? model_string.to_s.underscore : nil
 end
 
 ####################################################################
@@ -468,20 +454,49 @@ end
 # String. HTML code formatted for display.
 ####################################################################
 def dc_error_messages_for(doc)
-  return '' unless doc and doc.errors.any?
-  msgs = ''
-  doc.errors.each do |attribute, errors_array|
-    label = t("helpers.label.#{decamelize_type(doc.class)}.#{attribute}", attribute)
-    msgs << "<li>#{label} : #{errors_array}</li>"
+  return '' unless doc && doc.errors.any?
+
+  msgs = doc.errors.inject('') do |r, error|
+    label = t("helpers.label.#{decamelize_type(doc.class)}.#{error.attribute}", error.attribute)
+    r << "<li>#{label} : #{error.message}</li>"
   end
-  
-c = <<eot
+
+  html = <<eot
 <div class="dc-form-error"> 
   <h2>#{t('drgcms.errors_no')} #{doc.errors.size}</h2>  
   <ul>#{msgs}</ul>  
 </div>
 eot
-  c.html_safe
+  html.html_safe
+end
+
+####################################################################
+# Returns warning messages if any set in a model.
+#
+# When warnings array is added to model its content can be written on top of the form.
+#
+# Parameters:
+# [doc] Document. Document record which will be checked for errors.
+#
+# Returns:
+# String. HTML code formatted for display.
+####################################################################
+def dc_warning_messages_for(doc)
+  return ''
+  return '' unless doc && doc.respond_to?(:warnings)
+
+  msgs = doc.warnings.inject('') do |r, error|
+    label = t("helpers.label.#{decamelize_type(doc.class)}.#{error.attribute}", error.attribute)
+    msgs << "<li>#{label} : #{error.message}</li>"
+  end
+
+  html = <<eot
+<div class="dc-form-warning"> 
+  <h2>#{t('drgcms.warnings_no')} #{doc.warnings.size}</h2>  
+  <ul>#{msgs}</ul>  
+</div>
+eot
+  html.html_safe
 end
 
 ####################################################################
@@ -555,14 +570,14 @@ end
 # Create edit link with edit picture. Subroutine of dc_page_edit_menu.
 ####################################################################
 def dc_link_menu_tag(title) #:nodoc:
-  html = %Q[
+  html = %(
 <dl>
   <dt><div class='drgcms_popmenu' href="#">
     #{_origin.fa_icon('file-text-o lg', class: 'dc-inline-link', title: title)}
   </div></dt>
   <dd>
     <ul class=' div-hidden drgcms_popmenu_class'>
-]
+)
 
   yield html
   html << "</ul></dd></dl>"
@@ -596,10 +611,11 @@ end
 # Returns: 
 # String. HTML code required for manipulation of currently processed document.
 ########################################################################
-def dc_page_edit_menu(opts=@opts)
+def dc_page_edit_menu(opts = @opts)
   opts[:edit_mode] ||= _origin.session[:edit_mode]
   return '' if opts[:edit_mode] < 2
-# save some data to cookie. This can not go to session.
+
+  # save some data to cookie. This can not go to session.
   page  = opts[:page] || @page
   table = _origin.site.page_class.underscore
   kukis = { "#{table}.dc_design_id" => page.dc_design_id,
@@ -616,11 +632,9 @@ def dc_page_edit_menu(opts=@opts)
     opts[:editparams].merge!( :id => page.id, :table => _origin.site.page_class.underscore, form_name: opts[:form_name], edit_only: 'body' )
     html << dc_link_for_edit1( opts[:editparams], t('drgcms.edit_content') )
     
-#    opts[:editparams][:edit_only] = nil
     opts[:editparams].merge!( edit_only: nil, 'icon' => 'pencil' )
     html << dc_link_for_edit1( opts[:editparams], t('drgcms.edit_advanced') )
     
-#    opts[:editparams][:action] = 'new'
     opts[:editparams].merge!( action: 'new', 'icon' => 'plus' )
     html << dc_link_for_edit1( opts[:editparams], t('drgcms.edit_new_page') )
 
@@ -642,8 +656,7 @@ end
 #      type: text_with_select
 #      eval: 'dc_page_class.all_pages_for_site(@parent.dc_get_site)'
 ########################################################################
-def dc_page_class()
-#  dc_get_site.page_class.classify.constantize  
+def dc_page_class
   dc_get_site.page_klass
 end
 
@@ -658,7 +671,7 @@ end
 #      type: tree_view
 #      eval: 'dc_menu_class.all_menus_for_site(@parent.dc_get_site)'
 ########################################################################
-def dc_menu_class()
+def dc_menu_class
   dc_get_site.menu_class.classify.constantize
 end
 
@@ -672,23 +685,20 @@ end
 # Returns:
 # DCSite. Site document.
 ####################################################################
-def dc_get_site()
+def dc_get_site
   return @site if @site # already cached
-#
+
   req = _origin.request.url # different when called from renderer
   uri  = URI.parse(req)
   @site = DcSite.find_by(name: uri.host)
-# Site can be aliased
-  if @site and !@site.alias_for.blank?
-    @site = DcSite.find_by(name: @site.alias_for)
-  end
-# Development environment. Check if site with name test exists and use 
-# alias_for field as pointer to real site name.
-  if @site.nil? and ENV["RAILS_ENV"] != 'production'
+  # Site can be aliased
+  @site = DcSite.find_by(name: @site.alias_for) if @site&.alias_for.present?
+  # Development. If site with name test exists use alias_for field as pointer to real site data
+  if @site.nil? && ENV["RAILS_ENV"] != 'production'
     @site = DcSite.find_by(name: 'test')
     @site = DcSite.find_by(name: @site.alias_for) if @site
   end 
-  @site = nil if @site and !@site.active # site is disabled
+  @site = nil if @site && !@site.active # site is disabled
   @site
 end
 
@@ -704,9 +714,9 @@ end
 #    html:
 #      include_blank: true      
 ############################################################################
-def dc_choices4_site_policies()
+def dc_choices4_site_policies
   site = dc_get_site()
-  site.dc_policies.all.inject([]) { |r, policy| r << [ policy.name, policy.id] if policy.active }
+  site.dc_policies.where(active: true).order_by(name: 1).map { |policy| [ policy.name, policy.id] }
 end
 
 ############################################################################
@@ -727,7 +737,7 @@ def dc_choices4_all_collections
   DrgCms.paths(:forms).reverse.each do |path|
     filename = "#{path}/cms_menu.yml"
     next unless File.exist?(filename)
-#
+
     menu = YAML.load_file(filename) rescue nil      # load menu
     next if menu.nil? or !menu['menu']              # not menu or error
     menu['menu'].each do |section|
@@ -754,14 +764,14 @@ def forms_merge(hash1, hash2) #:nodoc:
     end
     target[key] = hash2[key] == '/' ? nil :  hash2[key]
   end
-# delete keys with nil value  
-  target.delete_if{ |k,v| v.nil? }
+  # delete keys with nil value
+  target.delete_if{ |k, v| v.nil? }
 end
 
 ##########################################################################
 # Returns choices for creating collection edit select field on CMS top menu.
 ##########################################################################
-def dc_choices4_cmsmenu()
+def dc_choices4_cmsmenu
   menus = {}
   DrgCms.paths(:forms).reverse.each do |path|
     filename = "#{path}/cms_menu.yml"
@@ -770,7 +780,7 @@ def dc_choices4_cmsmenu()
     next if menu.nil? or !menu['menu']              # not menu or error
     menus = forms_merge(menu['menu'], menus)        # ignore top level part
  end
-#
+
   html = '<ul>'
   menus.to_a.sort.each do |index, menu|    # sort menus, result is array of sorted hashes
     next unless menu['caption']
@@ -835,9 +845,11 @@ def dc_choices4(model, name, id='_id', options = {})
     sites << nil if param == :with_nil 
     qry   = qry.in(dc_site_id: sites) if sites
   end
-  qry   = qry.and(active: true) if model.method_defined?(:active)
-  choices = qry.inject([]) {|result,e| result << [ e[name], e[id] ]}
-  choices.sort_alphabetical_by(&:first) # use UTF-8 sort
+  qry = qry.and(active: true) if model.method_defined?(:active)
+  qry = qry.order_by(name => 1).collation(locale: I18n.locale.to_s)
+  #choices = qry.inject([]) {|result,e| result << [ e[name], e[id] ]}
+  #choices.sort_alphabetical_by(&:first) # use UTF-8 sort
+  qry.map { |e| [e[name], e[id]] }
 end
 
 ############################################################################
@@ -898,35 +910,35 @@ end
 ############################################################################
 def dc_user_can_view(ctrl, policy_id)
   policy_id = policy_id.policy_id if policy_id and policy_id.respond_to?(:policy_id)
-# Eventualy object without policy_id will be checked. This is to prevent error
+  # Eventualy object without policy_id will be checked. This is to prevent error
   policy_id = nil unless policy_id.class == BSON::ObjectId
-#
+
   site = ctrl.site
   policies = if site.inherit_policy.blank? 
     site.dc_policies
   else
     Mongoid::QueryCache.cache { DcSite.find(site.inherit_policy) }.dc_policies
   end
-# permission defined by default policy
+  # permission defined by default policy
   default_policy = Mongoid::QueryCache.cache { policies.find_by(is_default: true) }
   return false, 'Default access policy not found for the site!' unless default_policy
-#  
+
   permissions = {}
   default_policy.dc_policy_rules.to_a.each { |v| permissions[v.dc_policy_role_id] = v.permission }
-# update permissions with defined policy 
+  # update permissions with defined policy
   part_policy = nil
   if policy_id
     part_policy = Mongoid::QueryCache.cache { policies.find(policy_id) }
     return false, 'Access policy not found for part!' unless part_policy
     part_policy.dc_policy_rules.to_a.each { |v| permissions[v.dc_policy_role_id] = v.permission }
   end
-# apply guest role if no roles defined
+  # apply guest role if no roles defined
   if ctrl.session[:user_roles].nil?
     role = Mongoid::QueryCache.cache { DcPolicyRole.find_by(system_name: 'guest', active: true) }
     return false, 'System guest role not defined!' unless role
     ctrl.session[:user_roles] = [role.id]
   end
-# Check if user has any role that allows him to view part
+  # Check if user has any role that allows him to view part
   can_view, msg = false,''
   ctrl.session[:user_roles].each do |role|
     next unless permissions[role]          # role not yet defined. Will die in next line.
@@ -962,10 +974,10 @@ def dc_user_has_role( role, user=nil, roles=nil )
   roles = _origin.session[:user_roles] if roles.nil?
   user  = _origin.session[:user_id] if user.nil?
   return false if user.nil? or roles.nil?
-#  
+
   role = DcPolicyRole.get_role(role)
   return false if role.nil?
-# role is included in roles array
+  # role is included in roles array
   roles.include?(role._id)
 end
 
@@ -1004,6 +1016,7 @@ end
 ############################################################################
 def dc_limit_string(string, size)
   return string if string.size < size
+
   string = string[0,size]
   string.chop! until (string[-1,1] == ' ' or string == '')
   string << '...'
@@ -1031,10 +1044,11 @@ def dc_big_table(key)
   bt = DcBigTable.find_by(key: key, site: dc_get_site._id, active: true)
   bt = DcBigTable.find_by(key: key, site: nil, active: true) if bt.nil?
   return ret if bt.nil? 
-# 
+
   locale = I18n.locale.to_s
   bt.dc_big_table_values.each do |v|   # iterate each value
     next unless v.active
+
     desc = ''
     v.dc_big_table_locales.each do |l| # iterate each locale
       if l.locale == locale

data/app/models/concerns/dc_page_concern.rb

@@ -36,11 +36,12 @@ field :subject_link, type: String,  default: ''
 field :alt_link,     type: String,  default: ''
 field :sub_subject,  type: String,  default: ''
 field :picture,      type: String   
-field :gallery,      type: Boolean, default: false   
+field :gallery,      type: Mongoid::Boolean, default: false
 field :body,         type: String,  default: ''
 field :css,          type: String,  default: ''
 field :script,       type: String,  default: ''
 field :params,       type: String
+field :div_class,    type: String
 field :menu_id,      type: String
 field :author_id,    type: BSON::ObjectId
 field :dc_poll_id,   type: BSON::ObjectId
@@ -50,7 +51,7 @@ field :user_name,    type: String
 field :valid_from,   type: DateTime
 field :valid_to,     type: DateTime
 field :comments,     type: Integer, default: 1  # 0 => not allowed, 1 => allowed
-field :active,       type: Boolean, default: true 
+field :active,       type: Mongoid::Boolean, default: true
 field :created_by,   type: BSON::ObjectId
 field :updated_by,   type: BSON::ObjectId
 field :kats,         type: Array         # Categories

data/app/models/concerns/dc_piece_concern.rb

@@ -45,7 +45,7 @@ module DcPieceConcern
   field :div_id,        type: String
   field :site_id,       type: BSON::ObjectId
   field :order,         type: Integer, default: 0
-  field :active,        type: Boolean, default: true  
+  field :active,        type: Mongoid::Boolean, default: true
   field :valid_from,    type: DateTime
   field :valid_to,      type: DateTime
   

data/app/models/concerns/dc_site_concern.rb

@@ -40,14 +40,13 @@ field :css,                 type: String, default: ''
 field :route_name,          type: String, default: ''
 field :page_title,          type: String
 field :document_extension,  type: String
-field :page_table,          type: String
 field :page_class,          type: String, default: 'DcPage'
 field :site_layout,         type: String, default: 'content'
 field :menu_class,          type: String, default: 'DcSimpleMenu'
 field :request_processor,   type: String
 field :files_directory,     type: String
 field :logo,                type: String
-field :active,              type: Boolean, default: true
+field :active,              type: Mongoid::Boolean, default: true
 field :created_by,          type: BSON::ObjectId
 field :updated_by,          type: BSON::ObjectId
 field :menu_name,           type: String
@@ -87,10 +86,17 @@ def params(what=nil)
 end
 
 ########################################################################
-# Returns class object of collection name
+# Returns class object of page collection
 ########################################################################
 def page_klass
-  (self.page_table.blank? ? self.page_class : self.page_table).classify.constantize
+  page_class.classify.constantize
+end
+
+########################################################################
+# Returns collection name for page collection
+########################################################################
+def page_table
+  page_class.underscore
 end
 
 ########################################################################

data/app/models/concerns/dc_user_concern.rb

@@ -50,13 +50,13 @@ field :picture,     type: String
 field :birthdate,   type: Date
 field :about,       type: String
 field :last_visit,  type: Time
-field :active,      type: Boolean, default: true
+field :active,      type: Mongoid::Boolean, default: true
 field :valid_from,  type: Date
 field :valid_to,    type: Date
 field :created_by,  type: BSON::ObjectId
 field :updated_by,  type: BSON::ObjectId
 
-field :group,       type: Boolean, default: false # false => User, true => Group
+field :group,       type: Mongoid::Boolean, default: false # false => User, true => Group
 field :member,      type: Array
 
 embeds_many :dc_user_roles
@@ -151,7 +151,7 @@ private
 # when entry is left empty.
 ##########################################################################
 def do_before_save
-  self.name  = "#{title} #{first_name} #{middle_name + ' ' unless middle_name.blank?}#{last_name}".strip
+  self.name  = "#{title} #{first_name} #{middle_name + ' ' unless middle_name.blank?}#{last_name}".squish
   # to ensure unique e-mail
   self.email = "unknown@#{id}" if email.blank?
 end

data/app/models/dc_big_table.rb

@@ -78,10 +78,10 @@ class DcBigTable
 ########################################################################
 # Will return possible choices for specified key prepared for usega in select input field.
 ########################################################################
-def self.choices4(key, site, locale=nil)
+def self.choices4(key, site = nil, locale = nil)
   result = []
   choices = find_by(key: key, site: site)
-  choices = find_by(key: key, site: nil) if choices.nil?
+  choices = find_by(key: key, site: nil) if site && choices.nil?
   if choices
     choices.dc_big_table_values.each do |choice| 
       description = choice.description

data/app/models/dc_filter.rb

@@ -106,8 +106,9 @@ end
 ############################################################################
 # Return filter input field for entering variable filter values on index form 
 ############################################################################
-def self.get_filter_field(parent) 
+def self.get_filter_field(parent)
   return '' if parent.session[ parent.form['table'] ].nil?
+
   filter = parent.session[ parent.form['table'] ][:filter]
   return '' if filter.nil?
 
@@ -115,8 +116,11 @@ def self.get_filter_field(parent)
   return '' if filter.nil?
 
   field = get_field_form_definition(filter['field'], parent)
-  return '' if field.nil? and filter['input'].nil?
-  field = {} if field.nil?
+  return '' if field.nil? && filter['input'].nil?
+
+  saved_readonly = parent.form['readonly']
+  parent.form['readonly'] = false # must be
+  field ||= {}
   # If field has choices available in labels, use them. This is most likely select input field.
   if field['name']
     choices = parent.t('helpers.label.' + parent.form['table'] + '.choices4_' + field['name'] )
@@ -129,35 +133,37 @@ def self.get_filter_field(parent)
   field['type']     = filter['input'] if filter['input'].to_s.size > 5
   field['type']   ||= 'text_field'
   field['readonly'] = false   # must be
-  field['html']   ||= {} 
-  field['html']['size']  = 20
+  field['html']   ||= {}
+  field['html']['size'] = 20
   # Start with last entered value
   field['html']['value']    = filter['value'] unless filter['value'] == '#NIL'
   field['html']['selected'] = field['html']['value'] # for select field
   # url for filter ON action
   field['html']['data-url'] = parent.url_for(
-    controller: 'cmsedit',action: :index, filter: 'on',
+    controller: 'cmsedit', action: :index, filter: 'on',
     table: parent.form['table'], form_name: parent.params['form_name'])
   url = field['html']['data-url']
   # remove if present
   field['with_new'] = nil if field['with_new']
-# create input field object
-  klas_string = field['type'].camelize
-  klas = DrgcmsFormFields::const_get(klas_string) rescue nil
-  return '' if klas.nil?
-# return data from object and create html code to display field
-  object = klas.new(parent, nil, field).render rescue nil
-  # Error. Forget filter and return 
-  if object.nil?
-    parent.session[ parent.form['table'] ][:filter] = nil
-    return ''
+  # create input field object
+  html = ''
+  klass_string = field['type'].camelize
+  klass = DrgcmsFormFields::const_get(klass_string) rescue nil
+  # return data from object and create html code to display field
+  if klass
+    if drg_field = klass.new(parent, nil, field).render rescue nil
+      js = drg_field.js.blank? ? '' : parent.javascript_tag(drg_field.js)
+      html = %(<li class="no-background">
+<span class="filter_field" data-url="#{url}">#{drg_field.html}
+  #{parent.fa_icon('search lg', class: 'record_filter_field_icon')}
+  #{js}</span></li>)
+    else
+      # Error. Forget filter
+      parent.session[ parent.form['table'] ][:filter] = nil
+    end
   end
-  js = object.js.blank? ? '' : parent.javascript_tag(object.js)
-%Q[<li class="no-background">
-<span class="filter_field" data-url="#{url}">#{object.html}
-#{parent.fa_icon('search lg', class: 'record_filter_field_icon')}
-#{js}</span>
-</li>]
+  parent.form['readonly'] = saved_readonly
+  html
 end
 
 ######################################################################

data/app/models/dc_site.rb

@@ -38,7 +38,6 @@
 #  route_name           String               Default route name for creating page link. ex. page. Leave blank if not used.
 #  page_title           String               Default page title displayed in browser's top menu when title can not be extracted from document
 #  document_extension   String               Default document extension eg. html
-#  page_table           String               Name of table holding data for pages
 #  page_class           String               Rails model class name which defines table holding pages data usually DcPage
 #  site_layout          String               Rails layout used to draw response. This is by default content layout.
 #  menu_class           String               Rails model class name which defines table holding menu data usually DcMenu

data/app/models/dc_temp.rb

@@ -137,4 +137,11 @@ def self.prepare(key:, clear: nil)
   self.where(key: key) 
 end
 
+########################################################################
+# Order data by new key. Will update order field with values from new field
+########################################################################
+def self.reorder_by(key, new_order)
+  self.where(key: key).each { |doc| doc.order = data[key]; doc.save }
+end
+
 end