pages_core 3.4.3 → 3.7.0

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

@@ -0,0 +1,42 @@
+<%
+  self.page_title = "Pages"
+  self.page_description = link_to("All pages", admin_pages_path(@locale)) +
+                          " / Deleted pages"
+  if PagesCore.config.localizations?
+    self.page_description_links = 'In ' + PagesCore.config.locales.map{ |l, n| link_to_unless_current n, deleted_admin_pages_path(l) }.join(link_separator)
+  end
+%>
+
+<% if @pages.any? %>
+  <table class="deleted-pages list">
+    <tr>
+      <th>Name</th>
+      <th>Last updated</th>
+      <th>Author</th>
+      <th>Location</th>
+    </tr>
+    <% @pages.each do |page| %>
+      <tr>
+        <td class="name">
+          <%= link_to_if(policy(page).edit?,
+                         page_name(page),
+                         edit_admin_page_url(@locale, page),
+                         class: 'name_link') %>
+        </td>
+        <td>
+          <%= l(page.updated_at, format: :short) %>
+        </td>
+        <td>
+          <%= link_to(page.author.name, admin_user_path(page.author)) %>
+        </td>
+        <td>
+          <% if page.parent %>
+            <%= page_name(page.parent, include_parents: true) %>
+          <% else %>
+            Top level
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+  </table>
+<% end %>

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

@@ -3,7 +3,11 @@
   self.page_description = "Editing " + [@page.ancestors.reverse,@page].flatten.map{|page| link_to((page.name rescue "(Untitled)"), edit_admin_page_path(@locale, page))}.join(" » ")
 
   if PagesCore.config.localizations?
-    self.page_description_links = 'In ' + PagesCore.config.locales.map{ |l, n| link_to_unless_current n, edit_admin_page_path(l, @page) }.join(link_separator)
+    self.page_description_links = "In " + safe_join(
+      PagesCore.config.locales.map do |l, name|
+        link_to_unless((l == @locale.to_sym), name, edit_admin_page_path(l, @page.localize(l)))
+      end,
+      link_separator)
   end
 %>
 
@@ -14,7 +18,8 @@
 <% content_for :sidebar do %>
   <%= form_for(@page,
                url: admin_page_url(@locale, @page),
-               html: {class: :edit_page, id: 'page-form-sidebar'}) do |f| %>
+               html: {class: "edit-page", id: 'page-form-sidebar'},
+               builder: PagesCore::Admin::FormBuilder) do |f| %>
     <%= render partial: 'edit_options', locals: {f: f} %>
   <% end %>
 <% end %>
@@ -26,15 +31,15 @@
 <%= form_for(@page,
              url: admin_page_url(@locale, @page),
              html: {
-               class: :edit_page,
+               class: "edit-page",
                id: 'page-form',
                method: :put,
                multipart: true
              },
-             builder: PagesCore::FormBuilder
+             builder: PagesCore::Admin::FormBuilder
 ) do |f| %>
 
-  <div class="hiddenOptions">
+  <div class="hidden-options">
     <%= render partial: 'edit_options', locals: {f: f} %>
   </div>
 
@@ -54,40 +59,7 @@
   <%# #### FILES #### %>
   <% if @page.template_config.value(:files) %>
     <%= content_tab "Files" do %>
-      <% if @page.files.any? %>
-        <ul class="filelist" data-url="<%= reorder_admin_page_files_url(@locale, @page) %>">
-          <% @page.files.each do |file| %>
-            <li data-file-id="<%= file.id %>">
-              <span class="actions">
-                <%= link_to "Delete", admin_page_file_path(@locale, @page, file), class: :delete, method: :delete, data: {confirm: 'Are you sure? There is no undo.'} %>
-              </span>
-              <div class="drag_handle"></div>
-              <strong><%= file.name %></strong> &ndash; <%= link_to file.filename, page_file_path(@locale, @page, file) %>
-            </li>
-          <% end %>
-        </ul>
-      <% else %>
-        <p>No files added yet.</p>
-      <% end %>
-      <p>
-        <button type="button" class="upload-file-button">
-          Upload file
-        </button>
-      </p>
-    <% end %>
-  <% end %>
-
-
-  <%# #### COMMENTS #### %>
-  <% if @page.template_config.value(:comments) %>
-    <% if @page.comments.count > 0 %>
-      <%= content_tab "Comments" do %>
-        <%= render partial: 'edit_comments', locals: {f: f} %>
-      <% end %>
-    <% else %>
-      <%= content_tab "Comments", {disabled: true} do %>
-        <p>No comments added.</p>
-      <% end %>
+      <%= render partial: 'edit_files', locals: {f: f} %>
     <% end %>
   <% end %>
 
@@ -107,55 +79,3 @@
   </div>
 
 <% end %>
-
-<%# #### UPLOAD IMAGE #### %>
-<div id="new-image">
-  <h2>Upload images</h2>
-  <%= form_for(@page.page_images.new, url: admin_page_images_url(@locale, @page), html: {multipart: true}) do |f| %>
-
-    <table>
-      <tr>
-        <th>File</th>
-        <th>Caption</th>
-      </tr>
-      <% 10.times do |i| %>
-        <tr class="">
-          <td>
-            <%= file_field_tag("page_images[#{i}][image]") %>
-          </td>
-          <td>
-            <%= text_field_tag("page_images[#{i}][image_attributes][caption]", '', size: 48) %>
-          </td>
-        </tr>
-      <% end %>
-    </table>
-
-    <p>
-      <%= submit_tag "Upload" %> or
-      <%= link_to "Cancel", "#", class: "cancel clear-modal" %>
-    </p>
-  <% end %>
-</div>
-
-
-<%# #### UPLOAD FILE #### %>
-<% if @page.template_config.value(:files) %>
-  <div id="new-file" class="page_new_file">
-    <h2>Upload new file</h2>
-    <%= form_for((@new_file = PageFile.new), url: admin_page_files_path(@locale, @page), html: {multipart: true}) do |f| %>
-      <%= labelled_field f.text_field( :name, size: 48 ), "Name",
-        errors: @new_file.errors[:name]
-      %>
-      <%= labelled_field f.text_field( :description, size: 48 ), "Description",
-        errors: @new_file.errors[:name]
-      %>
-      <%= labelled_field f.file_field( :file ), "File",
-        errors: @new_file.errors[:file]
-      %>
-      <p>
-        <%= submit_tag "Upload" %> or
-        <%= link_to "Cancel", "#", class: "cancel clear-modal" %>
-      </p>
-    <% end %>
-  </div>
-<% end %>

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

@@ -1,6 +1,6 @@
 <%
   self.page_title = "Pages"
-  self.page_description = "Viewing pages"
+  self.page_description = "All pages / " + link_to("Deleted pages", deleted_admin_pages_path(@locale))
 
   if PagesCore.config.localizations?
     self.page_description_links = 'In ' + PagesCore.config.locales.map{ |l, n| link_to_unless_current n, admin_pages_path(l) }.join(link_separator)
@@ -8,60 +8,16 @@
 %>
 
 <% content_for :sidebar do %>
-  <% if policy(Page).new? %>
-    <h2>New page</h2>
-    <p>
-      <%= form_tag new_admin_page_path, method: :get do %>
-        In <%= select_tag 'parent', options_from_collection_for_select(@root_pages, 'id', 'name') %>
-        <%= submit_tag "Go", name: nil %>
-      <% end %>
-    </p>
-    <p>
-      You can also <%= link_to "create a new root page", new_admin_page_path(@locale) %>.
-    </p>
-    <h2>Structure</h2>
-    <p>
-      <button id="reorder_link" type="button">Reorder the pages</button>
-    </p>
-    <!-- What shall with do with a drunken sailor?
-    <h2>Categories</h2>
-    <% if @categories && @categories.length > 0 %>
-      <ul>
-        <% @categories.each do |category| %>
-          <li><%= category.name %> (<%= link_to "Delete", admin_category_path(category), method: :delete, class: :delete, data: {confirm: "Are you sure? There is no undo."} %>)</li>
-        <% end %>
-      </ul>
-    <% end %>
-    <div id="new-category">
-      <%= form_for([:admin, Category.new]) do |f| %>
-        <p>
-          <label><strong>New category</strong></label>
-          <%= f.text_field :name %>
-          <%= submit_tag "Save" %>
-        </p>
-      <% end %>
-    </div>
-    <div id="new-category-button">
-      <p>
-        <button type="button" class="toggle-category">
-          Add a new category
-        </button>
-      </p>
-    </div>
-    -->
-  <% else %>
-    &nbsp;
-  <% end %>
 <% end %>
 
 <div class="content">
-  <% if @root_pages.length < 1 -%>
-    <p>No pages found</p>
-  <% else -%>
-    <ul class="pagelist reorderable">
-      <% @root_pages.each do |page| -%>
-        <%= render partial: "pagelisting", locals: { page: page } %>
-      <% end -%>
-    </ul>
-  <% end -%>
+  <% cache Page.visible.roots.to_a + [current_user, @locale] do %>
+    <%= react_component(
+      "PageTree", {
+        csrf_token: form_authenticity_token,
+        pages: @pages.map { |p| page_json(p) },
+        locale: @locale,
+        permissions: [(:create if policy(Page).create?)] }
+    ) %>
+  <% end %>
 </div>

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

@@ -1,6 +1,6 @@
 <%
   if @page.parent
-    self.page_title = "#{@page.parent.name} &raquo; New Page".html_safe
+    self.page_title = "New page"
     self.page_description = "<em>#{@page.parent.name}</em> &raquo; New Page"
   else
     self.page_title = "Create new page"
@@ -14,22 +14,23 @@
 <% content_for :sidebar do %>
   <%= form_for(@page,
                url: admin_pages_url(@locale),
-               html: {class: :edit_page, id: 'page-form-sidebar'}) do |f| %>
+               html: {class: "edit-page", id: 'page-form-sidebar'}) do |f| %>
     <%= render partial: 'edit_options', locals: {f: f} %>
   <% end %>
 <% end %>
 
 <%= form_for(@page,
              url: admin_pages_url(@locale),
+             builder: PagesCore::Admin::FormBuilder,
              html: {
-               class: :edit_page,
+               class: "edit-page",
                id: 'page-form',
                method: :post,
                multipart: true
              }
 ) do |f| %>
 
-  <div class="hiddenOptions">
+  <div class="hidden-options">
     <%= render partial: 'edit_options', locals: {f: f} %>
   </div>
 

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

@@ -1,7 +1,7 @@
 <%
   month_names = %w{January February March April May June July August September October November December}
-  self.page_title = "News"
-  self.page_description = "<em>#{month_names[@month - 1]} #{@year}</em>"
+  page_title "News"
+  page_description "<em>News: #{month_names[@month - 1] if @month} #{@year}</em>"
 
   if PagesCore.config.localizations?
     self.page_description_links = 'In ' + PagesCore.config.locales.map{ |l, n| link_to_unless_current n, news_admin_pages_path(l) }.join(link_separator)
@@ -9,61 +9,75 @@
 %>
 
 <% content_for :sidebar do %>
-  <h2>New article</h2>
-  <p>
-    <%= form_tag new_news_admin_pages_path, method: :get do %>
-      In <%= select_tag 'parent', options_for_select(@news_pages.map{|p| [page_name(p, include_parents: true).gsub("&raquo;", "»"), p.id] }) %>
-      <%= submit_tag "Go", name: nil %>
-    <% end %>
-  </p>
-  <% @archive_finder.years.reverse.each do |year| %>
-    <h2><%= year %><%= ": #{@category.name}" if @category %></h2>
-    <ul>
-      <% @archive_finder.months_in_year_with_count(year).each do |month, page_count| %>
-        <li><% if year == @year && month == @month %>
-          <strong><%= month_names[month-1] %></strong> (<%= page_count %>)</strong>
-        <% else %>
-          <%= link_to month_names[month-1], news_admin_pages_path(@locale, month: month, year: year, category: ((@category) ? @category.slug : nil)) %> (<%= page_count %>)
-        <% end %></li>
+  <% if policy(Page).new_news? %>
+    <h2>New article</h2>
+    <p>
+      <%= form_tag new_news_admin_pages_path, method: :get do %>
+        In
+        <%= select_tag('parent',
+                       options_for_select(@news_pages.map{|p| [news_section_name(p, @news_pages).gsub("&raquo;", "»"), p.id] })) %>
+        <%= submit_tag "Go", name: nil %>
       <% end %>
-    </ul>
+    </p>
   <% end %>
+  <div class="archive-index">
+    <% @archive_finder.years_with_count.reverse.each do |year, year_count| %>
+      <h2>
+        <%= link_to_unless_current(year,
+                                   news_admin_pages_path(
+                                     @locale,
+                                     year: year,
+                                     category: @category&.slug)
+        ) %>
+        <%= ": #{@category.name}" if @category %>
+        <span class="count">
+          (<%= year_count %>)
+        </span>
+      </h2>
+      <ul>
+        <% @archive_finder.months_in_year_with_count(year).reverse.each do |month, page_count| %>
+          <li>
+            <% if year == @year && month == @month %>
+              <strong>
+                <%= month_names[month-1] %>
+              </strong>
+              (<%= page_count %>)
+            <% else %>
+              <%= link_to(month_names[month-1],
+                          news_admin_pages_path(@locale,
+                                                month: month,
+                                                year: year,
+                                                category: @category&.slug)) %>
+              <span class="count">
+                (<%= page_count %>)
+              </span>
+            <% end %>
+          </li>
+        <% end %>
+      </ul>
+    <% end %>
+  </div>
 <% end %>
 
-<% if @pages.any? %>
-  <table class="list news_item_list">
+<% if @pages.length > 0 %>
+  <table class="list news-item-list">
     <tr>
       <th>Name</th>
       <th>Published</th>
       <th>Author</th>
       <th>Section</th>
     </tr>
-    <% @pages.each do |page| %><tr class="<%= [page.status_label.downcase,((page.autopublish?) ? 'autopublish' : nil), ((page.pinned?) ? 'pinned' : nil)].join(' ') %>">
-      <td class="name">
-        <%= link_to_if(policy(page).edit?, page_name(page), edit_admin_page_url(@locale, page), class: 'name_link') %>
-        <% if page.autopublish? %>
-          <br /><small>This page will be published <strong><%= publish_time(page.published_at) %></strong></small>
-        <% end %>
-      </td>
-      <td>
-        <% if page.published? %>
-          <%= l(page.published_at, format: :short) %>
-        <% else %>
-          <% if page.status_label == 'Published' %>
-            <em>Not published</em>
-          <% else %>
-            <em><%= page.status_label %></em>
-          <% end %>
-        <% end %>
-      </td>
-      <td>
-        <%= link_to page.author.name, admin_user_path(page.author) %>
-      </td>
-      <td>
-        <%= link_to_if(policy(page.parent).edit?, page_name(page.parent, include_parents: true), news_admin_pages_path(section: page.parent.id)) %>
-      </td>
-    </tr><% end %>
+    <% @pages.each do |page| %>
+      <%= render(partial: "list_item",
+                 locals: { page: page, author: true, section: true }) %>
+    <% end %>
   </table>
+
+  <%= will_paginate @pages, renderer: PagesCore::LinkRenderer %>
 <% else %>
-  <div class="content"><p>No news items have been posted yet.</p></div>
+  <div class="content">
+    <p>
+      No news items have been posted yet.
+    </p>
+  </div>
 <% end %>

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

@@ -5,15 +5,12 @@
 -%>
 
 <div class="login-form">
-  <%= form_for(@user, url: admin_password_reset_path(@password_reset_token, token: @password_reset_token.token), class: 'form') do |f| %>
-    <p>
-      <%= f.label :password %>
-      <%= f.password_field :password %>
-    </p>
-    <p>
-      <%= f.label :confirm_password %>
-      <%= f.password_field :confirm_password %>
-    </p>
+  <%= form_for(@user,
+               url: admin_password_reset_path(@password_reset_token, token: @password_reset_token.token),
+               builder: PagesCore::Admin::FormBuilder,
+               class: 'form') do |f| %>
+    <%= f.labelled_password_field :password %>
+    <%= f.labelled_password_field :confirm_password %>
     <p>
       <button type="submit">
         Change Password