refinerycms-pages 0.9.9.8 → 0.9.9.9

data/app/controllers/admin/pages_controller.rb

@@ -2,13 +2,15 @@ module Admin
   class PagesController < Admin::BaseController
 
     crudify :page,
-            :conditions => {:parent_id => nil},
+            :conditions => nil,
             :order => "lft ASC",
-            :include => [:parts, :slugs, :children, :parent, :translations],
+            :include => [:slugs, :translations],
             :paging => false
 
     rescue_from FriendlyId::ReservedError, :with => :show_errors_for_reserved_slug
 
+    after_filter lambda{::Page.expire_page_caching}, :only => [:update_positions]
+
     def new
       @page = Page.new
       Page.default_parts.each_with_index do |page_part, index|

data/app/models/page.rb

@@ -27,19 +27,22 @@ class Page < ActiveRecord::Base
                               :custom_title, :browser_title, :all_page_part_content]
 
   before_destroy :deletable?
-  after_save :reposition_parts!
-  after_save :invalidate_child_cached_url
+  after_save :reposition_parts!, :invalidate_child_cached_url, :expire_page_caching
+  after_destroy :expire_page_caching
 
   scope :live, where(:draft => false)
+  scope :by_title, lambda {|t|
+    where(:id => Page::Translation.where(:locale => Globalize.locale, :title => t).map(&:page_id))
+  }
 
-  # shows all pages with :show_in_menu set to true, but it also
+  # Shows all pages with :show_in_menu set to true, but it also
   # rejects any page that has not been translated to the current locale.
+  # This works using a query against the translated content first and then
+  # using all of the page_ids we further filter against this model's table.
   scope :in_menu, lambda {
-    pages = Arel::Table.new(Page.table_name)
-    translations = Arel::Table.new(Page.translations_table_name)
-
-    includes(:translations).where(:show_in_menu => true).where(
-      translations[:locale].eq(Globalize.locale)).where(pages[:id].eq(translations[:page_id]))
+    where(:show_in_menu => true).joins(:translations).includes(:translations).where(
+      :id => Page::Translation.where(:locale => Globalize.locale).map(&:page_id)
+    )
   }
 
   # when a dialog pops up to link to a page, how many pages per page should there be
@@ -126,6 +129,8 @@ class Page < ActiveRecord::Base
   end
 
   def link_url_localised?
+    return link_url unless defined?(::Refinery::I18n)
+
     current_url = link_url
 
     if current_url =~ %r{^/} && ::Refinery::I18n.current_frontend_locale != ::Refinery::I18n.default_frontend_locale
@@ -231,6 +236,12 @@ class Page < ActiveRecord::Base
     def use_marketable_urls?
       RefinerySetting.find_or_set(:use_marketable_urls, true, :scoping => 'pages')
     end
+
+    def expire_page_caching
+      if File.writable?(Rails.cache.cache_path)
+        Pathname.glob(File.join(Rails.cache.cache_path, '**', '*pages*')).each(&:delete)
+      end
+    end
   end
 
   # Accessor method to get a page part from a page.
@@ -282,7 +293,7 @@ class Page < ActiveRecord::Base
     sluggified
   end
 
-  private
+private
 
   def invalidate_child_cached_url
     return true unless self.class.use_marketable_urls?
@@ -292,4 +303,8 @@ class Page < ActiveRecord::Base
       Rails.cache.delete(child.path_cache_key)
     end
   end
+
+  def expire_page_caching
+    self.class.expire_page_caching
+  end
 end

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

@@ -28,8 +28,10 @@
     </span>
   </div>
   <ul class='nested'>
-    <% if (children = page.children).any? -%>
-      <%= render :partial => 'page', :collection => children %>
-    <% end -%>
+    <%= render :partial => 'page',
+               :collection => collection.select{|p| p.parent_id == page.id},
+               :locals => {
+                 :collection => collection
+               } if page.has_descendants? %>
   </ul>
 </li>

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

@@ -1,5 +1,9 @@
 <ul id='sortable_list'>
-  <%= render :partial => "page", :collection => @pages %>
+  <%= render :partial => "page",
+             :collection => @pages.select{|p| p.parent_id.nil?},
+             :locals => {
+               :collection => @pages
+             },  %>
 </ul>
 <%= render :partial => "/shared/admin/sortable_list",
            :locals => {:continue_reordering => !!local_assigns[:continue_reordering]} %>

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

@@ -1,5 +1,8 @@
 <section id='records' class='tree'>
-  <%= render :partial => 'records' %>
+  <% caching = RefinerySetting.find_or_set(:cache_pages_backend, false) && File.writable?(Rails.cache.cache_path) %>
+  <% cache_if(caching, [Refinery.base_cache_key, "pages_backend", Globalize.locale].join('_')) do %>
+    <%= render :partial => 'records' %>
+  <% end %>
 </section>
 <section id='actions'>
   <%= render :partial => 'actions' %>

data/app/views/admin/pages_dialogs/_page_link.html.erb

@@ -10,4 +10,4 @@
            :locals => {
              :child => (child ||= 0) + 1,
              :link_to_arguments => link_args
-           } if page_link.children.any? -%>
+           } if page_link.has_descendants? -%>

data/config/locales/jp.yml

@@ -0,0 +1,83 @@
+jp:
+  plugins:
+    refinery_pages:
+      title: ページ
+      description: ページの管理
+  admin:
+    pages_dialogs:
+      page_link:
+        link_to_this_page: このページへのリンク
+      link_to:
+        insert: 挿入
+        your_page:
+          tab_name: Your page
+        web_address:
+          tab_name: サイト
+          location: 場所
+          new_window: 新規ウィンドウ
+          step1: リンクするページを探す
+          step2: ブラウザのアドレスバーに表示されているアドレスをコピーして上欄にペーストする
+        email_address:
+          tab_name: メールアドレス
+          subject_line_optional: 件名（無くても可）
+          body_optional: 本文（無くても可）
+          not_sure: 何を記入すべきか解りませんか？
+          step1_html: "メールアドレスを上の'<strong>メールアドレス</strong>'欄に直接書き込むかアドレス帳などからコピーしたものをペーストする"
+          step2_html: "<strong>予め決められた件名</strong>を使ってメールを作成するには上の'<strong>件名</strong>'欄を使います"
+          step3_html: "<strong>予め決められた本文</strong>を使ってメールを作成するには上の'<strong>本文</strong>'欄を使います"
+        your_resource:
+          tab_name: ファイル
+          link_to_this_resource: このファイルへのリンク
+    pages:
+      delete: ページを削除
+      edit: ページを編集
+      reserved_system_word: "このタイトルはシステム用ですので使用出来ません。"
+      page:
+        view_live_html: "このページを表示する<br/><em>（新しいウィンドウが開きます）</em>"
+        hidden: 隠す
+        draft: 準備稿
+      form_new_page_parts:
+        title: タイトル
+      form_page_parts:
+        create_content_section: コンテンツを追加
+        delete_content_section: コンテンツを削除
+      form_advanced_options:
+        toggle_advanced_options: メタタグの設定とメニューオプションの設定
+        page_options: ページの設定
+        parent_page: 親ページ
+        advanced_options: その他の設定
+        custom_title: カスタマイズしたタイトル
+        title_types:
+          none: 無し
+          text: テキスト
+        show_in_menu_title: メニューで表示する
+        show_in_menu_description: このページをサイトメニューで表示する
+        show_in_menu_help: サイトメニューからこのページを削除する時はこのチェックボックスのチェックを外して下さい。ユーザに直接アクセスしてもらいたいページで、サイトメニューには表示したくないページがある場合などに便利です。
+        save_as_draft: 準備稿として保存する
+        skip_to_first_child: トップレベルのページを飛ばす
+        skip_to_first_child_label: この直ぐ下の階層のページにユーザを飛ばす
+        skip_to_first_child_help: このオプションを使うと、リンクは直ぐ下の階層のページのうちの先頭のものにリンクします。幾つかのページを一まとめにする場合などに便利です。
+        link_url: 他のウェブサイトやページに転送する
+        link_url_help: メニューでこのページへのリンクをクリックした時、他のウェブサイトやページに転送するようにするにはURL（例 http://google.com）を書き入れます。そうでない場合は空欄のままにします。
+        parent_page_help: リストから選んで、ページを他のページの下に置く事が出来ます。空にするとこのページは一番上の階層のページとして扱われます。
+        custom_title_help: このページにメニューに表示されているもの以外のタイトルを付けたい時にはここに書いて下さい。
+      form_advanced_options_seo:
+        seo: 検索エンジン最適化
+        seo_override_title: ブラウザのタイトル
+        seo_override_title_help: このページの内容を要約する短いタイトルを入れて下さい。
+        meta_keywords_title: メタキーワード
+        meta_keywords_help: このページに関係のあるキーワードを5つから10個ほど入れて下さい。区切りの記号はコンマです。
+        meta_description_title: メタの説明
+        meta_description_help: このページの内容を説明する簡潔な文章を二三入れて下さい。
+      actions:
+        create_new_page: 新規ページを追加
+        reorder_pages: ページを並べ替える
+        reorder_pages_done: 並べ替えを終了する
+      records:
+        no_pages_yet: まだページはありません。"新規ページを追加"をクリックして下さい。
+  activerecord:
+    models:
+      page: ページ
+    attributes:
+      page:
+        title: タイトル

data/db/migrate/20110314213540_remove_translated_fields_from_pages.rb

@@ -0,0 +1,13 @@
+class RemoveTranslatedFieldsFromPages < ActiveRecord::Migration
+  def self.up
+    ::Page.translated_attribute_names.map(&:to_sym).each do |column_name|
+      remove_column ::Page.table_name, column_name if ::Page.column_names.map(&:to_sym).include?(column_name)
+    end
+  end
+
+  def self.down
+    ::Page.translated_attribute_names.map(&:to_sym).each do |column_name|
+      add_column ::Page.table_name, column_name, Page::Translation.columns.detect{|c| c.name.to_sym == column_name}.type
+    end
+  end
+end

data/features/step_definitions/page_steps.rb

@@ -1,12 +1,11 @@
 Given /^I (only )?have a page titled "?([^\"]*)"? with a custom url "?([^\"]*)"?$/ do |only, title, link_url|
   Page.delete_all if only
 
-  Page.create(:title => title,
-              :link_url => link_url)
+  Page.create(:title => title, :link_url => link_url)
 end
 
 Given /^the page titled "?([^\"]*)"? has a menu match "?([^\"]*)"?$/ do |title, menu_match|
-  Page.where(:title => title).first.update_attribute(:menu_match, menu_match)
+  Page.by_title(title).first.update_attribute(:menu_match, menu_match)
 end
 
 Given /^I (only )?have pages titled "?([^\"]*)"?$/ do |only, titles|
@@ -29,15 +28,15 @@ Given /^I (only )?have a page titled "?([^\"]*)"?$/ do |only, title|
 end
 
 Given /^the page titled "?([^\"]*)"? is a child of "?([^\"]*)"?$/ do |title, parent_title|
-  Page.where(:title => title).first.update_attribute(:parent, Page.where(:title => parent_title).first)
+  Page.by_title(title).first.update_attribute(:parent, Page.by_title(parent_title).first)
 end
 
 Given /^the page titled "?([^\"]*)"? is not shown in the menu$/ do |title|
-  Page.where(:title => title).first.update_attribute(:show_in_menu, false)
+  Page.by_title(title).first.update_attribute(:show_in_menu, false)
 end
 
 Given /^the page titled "?([^\"]*)"? is draft$/ do |title|
-  Page.where(:title => title).first.update_attribute(:draft, true)
+  Page.by_title(title).first.update_attribute(:draft, true)
 end
 
 Then /^I should have ([0-9]+) pages?$/ do |count|

data/features/support/paths.rb

@@ -11,7 +11,7 @@ module NavigationHelpers
           new_admin_page_path
         else
           begin
-            if page_name =~ /the page titled "?([^\"]*)"?/ and (page = Page.where(:title => $1).first).present?
+            if page_name =~ /the page titled "?([^\"]*)"?/ and (page = Page.by_title($1).first).present?
               self.url_for(page.url)
             else
               nil

data/refinerycms-pages.gemspec

@@ -2,10 +2,10 @@
 
 Gem::Specification.new do |s|
   s.name              = %q{refinerycms-pages}
-  s.version           = %q{0.9.9.8}
+  s.version           = %q{0.9.9.9}
   s.summary           = %q{Pages engine for Refinery CMS}
   s.description       = %q{The default content engine of Refinery CMS. This engine handles the administration and display of user-editable pages.}
-  s.date              = %q{2011-03-11}
+  s.date              = %q{2011-03-15}
   s.email             = %q{info@refinerycms.com}
   s.homepage          = %q{http://refinerycms.com}
   s.rubyforge_project = %q{refinerycms}
@@ -63,6 +63,7 @@ Gem::Specification.new do |s|
     'config/locales/es.yml',
     'config/locales/fr.yml',
     'config/locales/it.yml',
+    'config/locales/jp.yml',
     'config/locales/lolcat.yml',
     'config/locales/lt.yml',
     'config/locales/lv.yml',
@@ -85,6 +86,7 @@ Gem::Specification.new do |s|
     'db/migrate/20101214040815_translate_page_plugin.rb',
     'db/migrate/20101216194133_remove_cached_slug_from_pages.rb',
     'db/migrate/20110307025652_translate_custom_title_on_pages.rb',
+    'db/migrate/20110314213540_remove_translated_fields_from_pages.rb',
     'db/seeds',
     'db/seeds/pages.rb',
     'features',
@@ -109,5 +111,5 @@ Gem::Specification.new do |s|
     'spec/models/page_spec.rb'
   ]
 
-  s.add_dependency 'refinerycms-core', '~> 0.9.9.8'
+  s.add_dependency 'refinerycms-core', '~> 0.9.9.9'
 end

metadata

@@ -2,7 +2,7 @@
 name: refinerycms-pages
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.9.9.8
+  version: 0.9.9.9
 platform: ruby
 authors: 
 - Resolve Digital
@@ -13,7 +13,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-03-11 00:00:00 +13:00
+date: 2011-03-15 00:00:00 +13:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -24,7 +24,7 @@ dependencies:
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        version: 0.9.9.8
+        version: 0.9.9.9
   type: :runtime
   version_requirements: *id001
 description: The default content engine of Refinery CMS. This engine handles the administration and display of user-editable pages.
@@ -71,6 +71,7 @@ files:
 - config/locales/es.yml
 - config/locales/fr.yml
 - config/locales/it.yml
+- config/locales/jp.yml
 - config/locales/lolcat.yml
 - config/locales/lt.yml
 - config/locales/lv.yml
@@ -91,6 +92,7 @@ files:
 - db/migrate/20101214040815_translate_page_plugin.rb
 - db/migrate/20101216194133_remove_cached_slug_from_pages.rb
 - db/migrate/20110307025652_translate_custom_title_on_pages.rb
+- db/migrate/20110314213540_remove_translated_fields_from_pages.rb
 - db/seeds/pages.rb
 - features/manage_pages.feature
 - features/step_definitions/page_steps.rb