tb_cms 1.3.beta1 → 1.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: de4396854d57732f8a52e8efbace10d73fdcaded
-  data.tar.gz: 93a220547d65d8c0a94fb9123ae346409a782391
+SHA256:
+  metadata.gz: d674e42a14812fcc99fc125a85514ba33f016752e5cdcc7c4d7319e841505ea3
+  data.tar.gz: 5b1669484dff7f06e78db700051107954229d95ec11819a3d05144d5929fe050
 SHA512:
-  metadata.gz: 8857e4dabfd684a60633fa40e179a1c596cc2efa9d3c5ad9485020e37959bcfdff74971aa1faf41d18ed01900b04ccfca81d8ad7cdbe78e6a6dda82c063cb737
-  data.tar.gz: 673da85578b647fe67ce656dea4908d42658d274f6157759834d97acf703978df27166926cc1d28a157800758aecc22657f4854289608a1ffa41a194efb35e2c
+  metadata.gz: b819655bcdbb65fc611da5ddbba4a159dcb9a495d05fbcbd46b7c8d04351d014b8d974b0889b2fb0b53162a089658b99cbe69a190b88916699fbfa97e9240a59
+  data.tar.gz: 6583b6a1d64056748b9252381e51d6cfda5949c7b119ed0a731e7b5e382cb9522e0e6355d1a9392ba36ed50509d2c3b9c233ed389f037a16a1d77e42f392e88a

data/README.md

@@ -72,12 +72,6 @@ Create a controller concern at `app/controllers/concerns/spud_cms_layout_actions
 
 If you ran the `spud:cms:layout` generator described earlier, the concern and empty action would have been created for you automatically.
 
-## Using Liquid Templates
-
-Pages use the [liquid](https://github.com/Shopify/liquid) template syntax engine created by Shopify. This allows you to easily inject variables into your pages in the page editor. Example:
-
-    <h2>{{page.name}}</h2>
-
 ## Using Menus
 
 A lot of cms engines allow you to render your navigation links in a ul block by using your page tree structure. In a lot of cases this is insufficient as some sites have urls that redirect to pages outside of your cms. This is where menus come in. They can be built in the admin control panel.
@@ -105,6 +99,3 @@ TB CMS uses RSpec for testing. Get the tests running with a few short commands:
         rspec spec
 
 After the tests have completed the current code coverage stats is available by opening ```/coverage/index.html``` in a browser.
-
-
-

data/app/assets/javascripts/admin/cms/menu_items.js

@@ -3,7 +3,7 @@ spud.admin.cms.menu_items = {};
 (function() {
   var menu_items = spud.admin.cms.menu_items;
   var val;
-  
+
   var setUrl = function(){
     return '/admin/menus/'+ val +'/menu_items/update_sort';
   };
@@ -35,25 +35,31 @@ spud.admin.cms.menu_items = {};
         });
         val = $("#menu-id").val();
         //Update order of menuItems
-        $.ajax(setUrl(), { 
-          method: 'PUT', 
+        $.ajax(setUrl(), {
+          method: 'PUT',
           data: {
             order: sortArr
-          }, 
+          },
           dataType: "json",
           success: function(data, status, jqXHR) {
             menu_items.updateStatus(200, '0px 0px 26px 2px rgba(4, 252, -49, 0.59)');
           },
           error: function(XMLHttpRequest, textStatus, errorThrown) {
             menu_items.updateStatus(100, '0px 0px 26px 2px rgba(255, 0, 40, 0.59)');
-            alert("Status: " + textStatus); alert("Error: " + errorThrown); 
-          }       
+            alert("Status: " + textStatus); alert("Error: " + errorThrown);
+          }
         });
       }
     });
   };
 
   menu_items.init = function() {
+    return
+
+    // Sortable relies on jQuery UI, which has been removed
+    // This code needs to migrate to sortable.js
+    // See: https://bitbucket.org/moser-inc/tb_core/commits/e6252b6aa508e103b54df7e563ecfa2d2ec26c24
+    //
     $("#sort tbody").sortable({
       helper: fixHelper,
       cursor: "move",

data/app/assets/javascripts/admin/cms/pages.js

@@ -24,6 +24,9 @@ var clickedPreviewButton = function(e){
     form.target = '';
     form.onSubmit = null;
     previewInput.val(0);
+    form.querySelectorAll('.btn').forEach(function(btn) {
+      btn.disabled = false;
+    });
   }, 200);
 };
 

data/app/controllers/cms/sitemaps_controller.rb

@@ -1,4 +1,4 @@
-class Cms::SitemapsController < Spud::ApplicationController
+class Cms::SitemapsController < TbCore::ApplicationController
   respond_to :xml
   def show
     @pages = SpudPage.published_pages.viewable.order(:spud_page_id)

data/app/controllers/pages_controller.rb

@@ -12,7 +12,7 @@ class PagesController < ApplicationController
     end
 
     url_name = !params[:id].blank? ? params[:id] : Spud::Cms.root_page_name
-    @page = SpudPage.published_pages.find_by(url_name: url_name)
+    @page = SpudPage.published_pages.find_ignoring_case(url_name)
 
     if @page.blank?
       render_404
@@ -34,11 +34,11 @@ class PagesController < ApplicationController
   private
 
   def render_404
-    raise Spud::NotFoundError, item: 'page'
-   end
+    raise TbCore::NotFoundError, item: 'page'
+  end
 
   def set_default_content_type
     request.format = :html if params[:format].blank?
-   end
+  end
 
 end

data/app/helpers/cms/application_helper.rb

@@ -85,8 +85,8 @@ module Cms::ApplicationHelper
       menu_id = menu.id
     end
 
-    cache(['sp_list_menu', menu, options]) do
-      link_options = options[:link_options] if options.key?(:link_options)
+    Rails.cache.fetch(['sp_list_menu', menu, options]) do
+      link_options = options[:link_options] if options.has_key?(:link_options)
 
       start_menu_item = options[:start_menu_item_id] if options.key?(:start_menu_item_id)
       content = if options.key?(:id)
@@ -136,8 +136,8 @@ module Cms::ApplicationHelper
 
       content += '</ul>'
 
-      concat content.html_safe
-    end
+      content
+    end.html_safe
   end
 
   def sp_menu_with_seperator(options = {})

data/app/models/spud_menu_item.rb

@@ -1,7 +1,7 @@
 class SpudMenuItem < ActiveRecord::Base
-  belongs_to :parent, polymorphic: true
-  belongs_to :spud_menu, touch: true
-  belongs_to :spud_page
+  belongs_to :parent, polymorphic: true, required: false
+  belongs_to :spud_menu, touch: true, required: false
+  belongs_to :spud_page, required: false
   has_many :spud_menu_items, as: :parent, dependent: :destroy
 
   validates :name, presence: true

data/app/models/spud_page.rb

@@ -2,13 +2,19 @@ class SpudPage < ActiveRecord::Base
   include CmsDeprecatedMultisite
   include TbRedirects::HasRedirects
 
-  spud_searchable
-  belongs_to :spud_page
+  belongs_to :spud_page,
+    required: false
   has_many :spud_page_partial_revisions
   has_many :spud_pages, dependent: :nullify
   has_many :spud_page_partials, dependent: :destroy
-  belongs_to :created_by_user, class_name: 'SpudUser', foreign_key: :created_by
-  belongs_to :updated_by_user, class_name: 'SpudUser', foreign_key: :updated_by
+  belongs_to :created_by_user,
+    class_name: 'SpudUser',
+    foreign_key: :created_by,
+    required: false
+  belongs_to :updated_by_user,
+    class_name: 'SpudUser',
+    foreign_key: :updated_by,
+    required: false
 
   before_validation :generate_url_name
   validates :name, presence: true, uniqueness: { scope: :spud_page_id }
@@ -22,11 +28,12 @@ class SpudPage < ActiveRecord::Base
   scope :viewable, -> { where(visibility: 0) }
 
   def full_content_processed
+    ActiveSupport::Deprecation.warn('#full_content_processed is deprecated; use #full_content instead.')
     spud_page_partials.collect(&:content_processed).join(' ')
   end
 
-  def to_liquid
-    return { 'name' => name, 'url_name' => url_name }
+  def full_content
+    spud_page_partials.collect(&:content).join(' ')
   end
 
   # Returns an array of pages in order of heirarchy
@@ -43,7 +50,7 @@ class SpudPage < ActiveRecord::Base
       if filter.blank? || c.id != filter.id
         list << [Array.new(level) { '- ' }.join('') + c.name, c[value]]
         list += options_tree_for_page(collection: collection, parent_id: c.id, level: level + 1, filter: filter)
-       end
+      end
     end
     return list
   end
@@ -52,6 +59,10 @@ class SpudPage < ActiveRecord::Base
     return visibility == 1
   end
 
+  def self.find_ignoring_case(url_name)
+    find_by(arel_table[:url_name].matches(url_name))
+  end
+
   private
 
   def generate_url_name
@@ -65,13 +76,13 @@ class SpudPage < ActiveRecord::Base
   end
 
   def create_redirect_if_necessary
-    if url_name_changed?
-      TbRedirect.create_smart(source: "/#{url_name_was}",
+    if saved_change_to_attribute?(:url_name)
+      TbRedirect.create_smart(source: "/#{attribute_before_last_save(:url_name)}",
                               destination: "/#{url_name}",
                               created_by: 'cms',
                               owner: self)
     end
-    return true
+    true
   end
 
 end

data/app/models/spud_page_partial.rb

@@ -1,10 +1,8 @@
 class SpudPagePartial < ActiveRecord::Base
   belongs_to :spud_page, touch: true
-  has_many :spud_liquid_tags, as: :attachment, dependent: :destroy
   validates :name, presence: true
   before_save :maintain_revisions
   before_save :update_symbol_name
-  acts_as_tb_liquid_content
 
   def update_symbol_name
     self.symbol_name = name.parameterize.underscore
@@ -14,24 +12,9 @@ class SpudPagePartial < ActiveRecord::Base
     return @symbol_name || name.parameterize.underscore
   end
 
-  def postprocess_content
-    @_template = Liquid::Template.parse(content)
-    self.content_processed = @_template.render('page' => spud_page)
-  end
-
-  def content_processed=(content)
-    self[:content_processed] = content
-  end
-
   def content_processed
-    if self[:content_processed].blank?
-      if new_record?
-        self.content_processed = postprocess_content()
-      else
-        update_column(:content_processed, postprocess_content)
-      end
-    end
-    return self[:content_processed]
+    ActiveSupport::Deprecation.warn('#content_processed is deprecated; use #content instead.')
+    content
   end
 
   def maintain_revisions
@@ -44,7 +27,7 @@ class SpudPagePartial < ActiveRecord::Base
         revision_bye = SpudPagePartialRevision.where(spud_page_id: spud_page_id, name: name).order('created_at ASC').first
         revision_bye.destroy unless revision_bye.blank?
       end
-      end
+    end
     return true
   end
 end

data/app/models/spud_snippet.rb

@@ -4,11 +4,8 @@ class SpudSnippet < ActiveRecord::Base
   validates :name, presence: true
   validates :name, uniqueness: true
 
-  acts_as_tb_liquid_content
-
   def postprocess_content
-    template = Liquid::Template.parse(content)
-    self.content_processed = template.render()
+    content
   end
 
   def content_processed=(content)

data/app/views/admin/pages/show.html.erb

@@ -1,20 +1,17 @@
-
 <%=content_for :head do%>
-	<meta name="description" content="<%=@page.meta_description%>" />
-
-	  <meta name="keywords" content="<%=@page.meta_keywords%>" />
+  <meta name="description" content="<%=@page.meta_description%>" />
+  <meta name="keywords" content="<%=@page.meta_keywords%>" />
 <% end %>
 
-<%if !@inline.blank?%>
-	<%=render :inline => @inline%>
+<% if !@inline.blank? %>
+  <%= render :inline => @inline %>
 <% end %>
-<%@page.spud_page_partials.each do |page_partial|%>
-
-	<%if(page_partial.name.downcase == 'body' && Spud::Cms.yield_body_as_content_block == false)%>
-		<%=page_partial.postprocess_content.html_safe%>
-	<%else%>
-		<%=content_for page_partial.name.parameterize.underscore.to_sym do%>
-			<%=page_partial.postprocess_content.html_safe%>
-		<% end %>
-	<% end %>
+<% @page.spud_page_partials.each do |page_partial| %>
+  <% if page_partial.name.downcase == 'body' && Spud::Cms.yield_body_as_content_block == false %>
+    <%= page_partial.content.html_safe %>
+  <% else %>
+    <%= content_for page_partial.name.parameterize.underscore.to_sym do %>
+      <%= page_partial.content.html_safe %>
+    <% end %>
+  <% end %>
 <% end %>

data/app/views/pages/show.html.erb

@@ -17,12 +17,12 @@
 <% @page.spud_page_partials.each do |page_partial| %>
 	<% if(page_partial.name.match(/^body$/i) && Spud::Cms.yield_body_as_content_block == false) %>
 		<% cache(page_partial) do %>
-			<%= page_partial.content_processed.html_safe %>
+			<%= page_partial.content.html_safe %>
 		<% end %>
 	<% else%>
 		<%= content_for page_partial.symbol_name.to_sym do %>
 			<% cache(page_partial) do %>
-				<%= page_partial.content_processed.html_safe %>
+				<%= page_partial.content.html_safe %>
 			<% end %>
 		<% end %>
 	<% end %>

data/db/migrate/20120101192412_create_spud_pages.rb

@@ -1,4 +1,4 @@
-class CreateSpudPages < ActiveRecord::Migration
+class CreateSpudPages < ActiveRecord::Migration[4.2]
   def change
     create_table :spud_pages do |t|
       t.string :name

data/db/migrate/20120101193138_create_spud_menus.rb

@@ -1,4 +1,4 @@
-class CreateSpudMenus < ActiveRecord::Migration
+class CreateSpudMenus < ActiveRecord::Migration[4.2]
   def change
     create_table :spud_menus do |t|
       t.string :name

data/db/migrate/20120101193255_create_spud_menu_items.rb

@@ -1,4 +1,4 @@
-class CreateSpudMenuItems < ActiveRecord::Migration
+class CreateSpudMenuItems < ActiveRecord::Migration[4.2]
   def change
     create_table :spud_menu_items do |t|
       t.string :parent_type

data/db/migrate/20120101194124_create_spud_templates.rb

@@ -1,4 +1,4 @@
-class CreateSpudTemplates < ActiveRecord::Migration
+class CreateSpudTemplates < ActiveRecord::Migration[4.2]
   def change
     create_table :spud_templates do |t|
       t.string :name

data/db/migrate/20120103034659_create_spud_page_partials.rb

@@ -1,4 +1,4 @@
-class CreateSpudPagePartials < ActiveRecord::Migration
+class CreateSpudPagePartials < ActiveRecord::Migration[4.2]
   def change
     create_table :spud_page_partials do |t|
       t.integer :spud_page_id

data/db/migrate/20120104194032_add_visibility_to_spud_pages.rb

@@ -1,4 +1,4 @@
-class AddVisibilityToSpudPages < ActiveRecord::Migration
+class AddVisibilityToSpudPages < ActiveRecord::Migration[4.2]
   def change
     add_column :spud_pages, :visibility, :integer,:default => 0
     add_column :spud_pages, :published, :boolean,:default => true

data/db/migrate/20120107181337_add_menu_name_to_spud_menu_items.rb

@@ -1,4 +1,4 @@
-class AddMenuNameToSpudMenuItems < ActiveRecord::Migration
+class AddMenuNameToSpudMenuItems < ActiveRecord::Migration[4.2]
   def change
     add_column :spud_menu_items, :name, :string
   end

data/db/migrate/20120111134754_add_use_custom_url_name_to_spud_pages.rb

@@ -1,4 +1,4 @@
-class AddUseCustomUrlNameToSpudPages < ActiveRecord::Migration
+class AddUseCustomUrlNameToSpudPages < ActiveRecord::Migration[4.2]
   def change
     add_column :spud_pages, :use_custom_url_name, :boolean,:default => false
   end

data/db/migrate/20120118141852_add_notes_to_spud_pages.rb

@@ -1,4 +1,4 @@
-class AddNotesToSpudPages < ActiveRecord::Migration
+class AddNotesToSpudPages < ActiveRecord::Migration[4.2]
   def change
     add_column :spud_pages, :notes, :text
   end

data/db/migrate/20120126232428_add_menu_id_to_spud_menu_items.rb

@@ -1,4 +1,4 @@
-class AddMenuIdToSpudMenuItems < ActiveRecord::Migration
+class AddMenuIdToSpudMenuItems < ActiveRecord::Migration[4.2]
 
   def change
     add_column :spud_menu_items, :spud_menu_id, :integer

data/db/migrate/20120128163601_add_classes_to_spud_menu_items.rb

@@ -1,4 +1,4 @@
-class AddClassesToSpudMenuItems < ActiveRecord::Migration
+class AddClassesToSpudMenuItems < ActiveRecord::Migration[4.2]
   def change
     add_column :spud_menu_items, :classes, :string
 

data/db/migrate/20120329132314_add_site_id_to_spud_pages.rb

@@ -1,4 +1,4 @@
-class AddSiteIdToSpudPages < ActiveRecord::Migration
+class AddSiteIdToSpudPages < ActiveRecord::Migration[4.2]
   def change
   	add_column :spud_pages, :site_id, :integer
     add_index :spud_pages,:site_id

data/db/migrate/20120329132322_add_site_id_to_spud_templates.rb

@@ -1,4 +1,4 @@
-class AddSiteIdToSpudTemplates < ActiveRecord::Migration
+class AddSiteIdToSpudTemplates < ActiveRecord::Migration[4.2]
   def change
   	add_column :spud_templates, :site_id, :integer
     add_index :spud_templates,:site_id

data/db/migrate/20120329132330_add_site_id_to_spud_menus.rb

@@ -1,4 +1,4 @@
-class AddSiteIdToSpudMenus < ActiveRecord::Migration
+class AddSiteIdToSpudMenus < ActiveRecord::Migration[4.2]
   def change
   	add_column :spud_menus, :site_id, :integer
     add_index :spud_menus,:site_id

data/db/migrate/20120510195151_create_spud_page_partial_revisions.rb

@@ -1,4 +1,4 @@
-class CreateSpudPagePartialRevisions < ActiveRecord::Migration
+class CreateSpudPagePartialRevisions < ActiveRecord::Migration[4.2]
   def change
     create_table :spud_page_partial_revisions do |t|
       t.string :name

data/db/migrate/20120911190030_add_symbol_name_to_spud_page_partials.rb

@@ -1,4 +1,4 @@
-class AddSymbolNameToSpudPagePartials < ActiveRecord::Migration
+class AddSymbolNameToSpudPagePartials < ActiveRecord::Migration[4.2]
   def change
     add_column :spud_page_partials, :symbol_name, :string
   end