make the editable elements more robust when working with localization + write tests to prove it

This commit is contained in:
Didier Lafforgue 2012-03-21 03:21:29 +01:00
parent e2f60f2e15
commit ab5a4755b8
10 changed files with 146 additions and 12 deletions

View File

@ -9,9 +9,9 @@ module Locomotive
field :hint field :hint
field :priority, :type => Integer, :default => 0 field :priority, :type => Integer, :default => 0
field :fixed, :type => Boolean, :default => false field :fixed, :type => Boolean, :default => false
field :disabled, :type => Boolean, :default => false field :disabled, :type => Boolean, :default => false, :localize => true
field :from_parent, :type => Boolean, :default => false field :from_parent, :type => Boolean, :default => false
# field :locales, :type => Array TODO field :locales, :type => Array, :default => []
## associations ## ## associations ##
embedded_in :page, :class_name => 'Locomotive::Page', :inverse_of => :editable_elements embedded_in :page, :class_name => 'Locomotive::Page', :inverse_of => :editable_elements
@ -27,6 +27,16 @@ module Locomotive
## methods ## ## methods ##
def disabled?
!!self.disabled # the original method does not work quite well with the localization
end
# Determines if the current element can be edited in the back-office
#
def editable?
!self.disabled? && self.locales.include?(::Mongoid::Fields::I18n.locale.to_s) && (!self.fixed? || !self.from_parent?)
end
def _run_rearrange_callbacks def _run_rearrange_callbacks
# callback from page/tree. not needed in the editable elements # callback from page/tree. not needed in the editable elements
end end
@ -52,10 +62,19 @@ module Locomotive
# @param [ EditableElement] el The source element # @param [ EditableElement] el The source element
# #
def copy_attributes_from(el) def copy_attributes_from(el)
self.attributes = el.attributes.reject { |attr| !%w(slug block hint priority fixed disabled from_parent).include?(attr) } self.attributes = el.attributes.reject { |attr| !%w(slug block hint priority fixed disabled locales from_parent).include?(attr) }
self.from_parent = true self.from_parent = true
end end
# Make sure the current locale is added to the list
# of locales for the current element so that we know
# in which languages the element was translated.
#
def add_current_locale
locale = ::Mongoid::Fields::I18n.locale.to_s
self.locales << locale unless self.locales.include?(locale)
end
protected protected
def _selector def _selector

View File

@ -8,6 +8,7 @@ module Locomotive
## methods ## ## methods ##
def content=(value) def content=(value)
self.add_current_locale
self.default_content = false unless self.new_record? self.default_content = false unless self.new_record?
super super
end end

View File

@ -26,7 +26,7 @@ module Locomotive
end end
def enabled_editable_elements def enabled_editable_elements
self.editable_elements.by_priority.reject { |el| el.disabled? || (el.fixed? && el.from_parent?) } self.editable_elements.by_priority.find_all(&:editable?) # { |el| !el.editable? }
end end
def editable_elements_grouped_by_blocks def editable_elements_grouped_by_blocks
@ -43,15 +43,17 @@ module Locomotive
end end
def add_or_update_editable_element(attributes, type) def add_or_update_editable_element(attributes, type)
# locale = attributes.delete(:locale) # TODO
element = self.find_editable_element(attributes[:block], attributes[:slug]) element = self.find_editable_element(attributes[:block], attributes[:slug])
if element if element
element.copy_attributes(attributes) element.copy_attributes(attributes)
else else
self.editable_elements.build(attributes, type) element = self.editable_elements.build(attributes, type)
end end
element.add_current_locale
element
end end
def enable_editable_elements(block) def enable_editable_elements(block)
@ -68,7 +70,7 @@ module Locomotive
new_el = self.editable_elements.build({}, el.class) new_el = self.editable_elements.build({}, el.class)
new_el.copy_attributes_from(el) new_el.copy_attributes_from(el)
else else
existing_el.attributes = { :disabled => false } existing_el.disabled = false # = { :disabled => false }
end end
end end
end end
@ -77,7 +79,7 @@ module Locomotive
return unless self.editable_elements.any? { |el| el.disabled? } return unless self.editable_elements.any? { |el| el.disabled? }
# super fast way to remove useless elements all in once # super fast way to remove useless elements all in once
self.collection.update(self.atomic_selector, '$pull' => { 'editable_elements' => { 'disabled' => true } }) self.collection.update(self.atomic_selector, '$pull' => { 'editable_elements' => { "disabled.#{::Mongoid::Fields::I18n.locale}" => true } })
end end
end end

View File

@ -38,6 +38,8 @@ module Locomotive
@parsing_errors << I18n.t(:liquid_syntax, :fullpath => self.fullpath, :error => error.to_s, :scope => [:errors, :messages, :page]) @parsing_errors << I18n.t(:liquid_syntax, :fullpath => self.fullpath, :error => error.to_s, :scope => [:errors, :messages, :page])
rescue ::Locomotive::Liquid::PageNotFound => error rescue ::Locomotive::Liquid::PageNotFound => error
@parsing_errors << I18n.t(:liquid_extend, :fullpath => self.fullpath, :scope => [:errors, :messages, :page]) @parsing_errors << I18n.t(:liquid_extend, :fullpath => self.fullpath, :scope => [:errors, :messages, :page])
rescue ::Locomotive::Liquid::PageNotTranslated => error
@parsing_errors << I18n.t(:liquid_translation, :fullpath => self.fullpath, :scope => [:errors, :messages, :page])
end end
end end
end end

View File

@ -11,11 +11,11 @@ en:
protected_page: "You can not remove index or 404 pages" protected_page: "You can not remove index or 404 pages"
extname_changed: "New file does not have the original extension" extname_changed: "New file does not have the original extension"
array_too_short: "is too small (minimum element number is %{count})" array_too_short: "is too small (minimum element number is %{count})"
liquid_syntax: "Liquid Syntax error ('%{error}')"
invalid_theme_file: "can't be blank or isn't a zip file" invalid_theme_file: "can't be blank or isn't a zip file"
page: page:
liquid_syntax: "Liquid Syntax error ('%{error}' on '%{fullpath}')" liquid_syntax: "Liquid Syntax error ('%{error}' on '%{fullpath}')"
liquid_extend: "The page '%{fullpath}' extends a template which does not exist" liquid_extend: "The page '%{fullpath}' extends a template which does not exist"
liquid_translation: "The page '%{fullpath}' extends a template which is not translated"
too_few_custom_fields: "At least, one custom field is required" too_few_custom_fields: "At least, one custom field is required"
security: "presents a security problem" security: "presents a security problem"

View File

@ -32,11 +32,11 @@ fr:
protected_page: "Vous ne pouvez pas supprimer les pages index et 404" protected_page: "Vous ne pouvez pas supprimer les pages index et 404"
extname_changed: "Nouveau fichier n'a pas l'extension original" extname_changed: "Nouveau fichier n'a pas l'extension original"
array_too_short: "est trop petit (le nombre minimum d'éléments est %{count})" array_too_short: "est trop petit (le nombre minimum d'éléments est %{count})"
liquid_syntax: "Erreur de syntaxe ('%{error}')"
security: "présente un problème de sécurité" security: "présente un problème de sécurité"
page: page:
liquid_syntax: "Erreur de syntaxe dans les sections de page, veuillez vérifier la syntaxe ('%{error}'/'%{fullpath}')" liquid_syntax: "Erreur de syntaxe dans les sections de page, veuillez vérifier la syntaxe ('%{error}'/'%{fullpath}')"
liquid_extend: "La page '%{fullpath}' étend le contenu d'une page qui n'existe pas" liquid_extend: "La page '%{fullpath}' étend le contenu d'une page qui n'existe pas"
liquid_translation: "La page '%{fullpath}' étend le contenu d'une page qui n'a pas été traduite"
attributes: attributes:
defaults: defaults:

View File

@ -1,5 +1,7 @@
module Locomotive module Locomotive
module Liquid module Liquid
class PageNotFound < ::Liquid::Error; end class PageNotFound < ::Liquid::Error; end
class PageNotTranslated < ::Liquid::Error; end
end end
end end

View File

@ -34,7 +34,9 @@ module Locomotive
raise PageNotFound.new("Page with fullpath '#{@template_name}' was not found") if @context[:parent_page].nil? raise PageNotFound.new("Page with fullpath '#{@template_name}' was not found") if @context[:parent_page].nil?
# be sure to work with a copy of the parent template otherwise there will be conflicts # be sure to work with a copy of the parent template otherwise there will be conflicts
parent_template = @context[:parent_page].template.clone parent_template = @context[:parent_page].template.try(:clone)
raise PageNotTranslated.new("Page with fullpath '#{@template_name}' was not translated") if parent_template.nil?
@context[:parent_page].instance_variable_set(:@template, parent_template) @context[:parent_page].instance_variable_set(:@template, parent_template)

View File

@ -0,0 +1,46 @@
require 'spec_helper'
describe Locomotive::Liquid::Tags::Extends do
before(:each) do
@home = FactoryGirl.build(:page, :raw_template => 'Hello world')
@home.send :serialize_template
@home.instance_variable_set(:@template, nil)
@site = FactoryGirl.build(:site)
@site.stubs(:pages).returns([@home])
end
it 'works' do
lambda {
page = FactoryGirl.build(:page, :slug => 'sub_page_1', :parent => @home)
parse('parent', page)
}.should_not raise_error
end
context '#errors' do
it 'raises an error if the source page does not exist' do
lambda {
@site.pages.expects(:where).with(:fullpath => 'foo').returns([])
parse('foo')
}.should raise_error(Locomotive::Liquid::PageNotFound, "Page with fullpath 'foo' was not found")
end
it 'raises an error if the source page is not translated' do
lambda {
::Mongoid::Fields::I18n.with_locale 'fr' do
page = FactoryGirl.build(:page, :slug => 'sub_page_1', :parent => @home)
parse('parent', page)
end
}.should raise_error(Locomotive::Liquid::PageNotTranslated, "Page with fullpath 'parent' was not translated")
end
end
def parse(source = 'index', page = nil)
page ||= @home
Liquid::Template.parse("{% extends #{source} %}", { :site => @site, :page => page })
end
end

View File

@ -18,6 +18,66 @@ describe Locomotive::EditableShortText do
@sub_page_1_1 = FactoryGirl.create(:page, :slug => 'sub_page_1_1', :parent => @sub_page_1, :raw_template => "{% extends 'parent' %}") @sub_page_1_1 = FactoryGirl.create(:page, :slug => 'sub_page_1_1', :parent => @sub_page_1, :raw_template => "{% extends 'parent' %}")
end end
context '#locales' do
it 'assigns the default locale' do
@sub_page_1_el.locales.should == ['en']
end
it 'adds new locale' do
::Mongoid::Fields::I18n.with_locale 'fr' do
page = Locomotive::Page.find(@home._id)
page.editable_elements.first.content = 'Lorem ipsum (FR)'
page.save
page.editable_elements.first.locales.should == %w(en fr)
end
end
it 'adds new locale within sub page elements' do
::Mongoid::Fields::I18n.with_locale 'fr' do
@home.update_attributes :title => 'Accueil', :raw_template => "{% block body %}{% editable_short_text 'body' %}Lorem ipsum{% endeditable_short_text %}{% endblock %}"
page = Locomotive::Page.find(@sub_page_1._id)
page.editable_elements.first.content = 'Lorem ipsum (FR)'
page.save
page.editable_elements.first.locales.should == %w(en fr)
end
end
end
context '#editable?' do
it 'is editable' do
@sub_page_1_el.editable?.should be_true
end
it 'is not editable if the element is disabled' do
@sub_page_1_el.disabled = true
@sub_page_1_el.editable?.should be_false
end
it 'is not editable if it is a fixed one' do
@sub_page_1_el.fixed = true
@sub_page_1_el.editable?.should be_false
end
it 'is not editable if it does exist for the current locale' do
::Mongoid::Fields::I18n.with_locale 'fr' do
@sub_page_1_el.editable?.should be_false
end
end
it 'is among the editable elements of the page' do
@sub_page_1.enabled_editable_elements.map(&:slug).should == %w(body)
end
it 'is not among the editable elements of the page if disabled' do
@sub_page_1_el.disabled = true
@sub_page_1.enabled_editable_elements.map(&:slug).should == []
end
end
context 'in sub pages level #1' do context 'in sub pages level #1' do
before(:each) do before(:each) do