maglevcms 1.1.6 → 1.2.0

data/app/frontend/editor/spec/__mocks__/page.js

@@ -9,7 +9,7 @@ export const pageSections = [
         settings: [
           {
             id: 'link',
-            value: { href: '#', text: 'Home', link_type: 'url' },
+            value: { href: '#', text: 'Home', linkType: 'url' },
           },
         ],
       },
@@ -19,7 +19,7 @@ export const pageSections = [
         settings: [
           {
             id: 'link',
-            value: { href: '#', text: 'About us', link_type: 'url' },
+            value: { href: '#', text: 'About us', linkType: 'url' },
           },
         ],
       },
@@ -33,11 +33,11 @@ export const pageSections = [
               href: '//contact',
               text: 'Contact',
               email: null,
-              link_id: 9,
-              link_type: 'page',
-              link_label: 'Contact us',
-              section_id: null,
-              open_new_window: false,
+              linkId: 9,
+              linkType: 'page',
+              linkLabel: 'Contact us',
+              sectionId: null,
+              openNewWindow: false,
             },
           },
         ],
@@ -52,11 +52,11 @@ export const pageSections = [
               href: '/products',
               text: 'Products #1',
               email: null,
-              link_id: 'd870133f9a075477a96a58e7639d40c5',
-              link_type: 'page',
-              link_label: 'Products',
-              section_id: null,
-              open_new_window: true,
+              linkId: 'd870133f9a075477a96a58e7639d40c5',
+              linkType: 'page',
+              linkLabel: 'Products',
+              sectionId: null,
+              openNewWindow: true,
             },
           },
         ],
@@ -71,7 +71,7 @@ export const pageSections = [
           width: 572,
           height: 290,
           filename: 'Screen Shot 2021-06-30 at 3.44.04 PM.png',
-          byte_size: 35070,
+          byteSize: 35070,
         },
       },
     ],
@@ -96,7 +96,7 @@ export const pageSections = [
           width: 516,
           height: 320,
           filename: 'img-91-3.jpg',
-          byte_size: 41683,
+          byteSize: 41683,
         },
       },
       { id: 'button', value: { url: '#', text: 'Click here' } },
@@ -119,7 +119,7 @@ export const pageSections = [
               width: 1920,
               height: 1200,
               filename: 'img-97.jpg',
-              byte_size: 458107,
+              byteSize: 458107,
             },
           },
         ],
@@ -137,7 +137,7 @@ export const pageSections = [
               width: 516,
               height: 400,
               filename: 'inner-74-2.jpg',
-              byte_size: 63881,
+              byteSize: 63881,
             },
           },
         ],
@@ -155,7 +155,7 @@ export const pageSections = [
               width: 516,
               height: 320,
               filename: 'img-91-4.jpg',
-              byte_size: 21808,
+              byteSize: 21808,
             },
           },
         ],
@@ -173,7 +173,7 @@ export const pageSections = [
               width: 1920,
               height: 1200,
               filename: 'img-91.jpg',
-              byte_size: 180178,
+              byteSize: 180178,
             },
           },
         ],
@@ -220,7 +220,7 @@ export const normalizedPage = {
         id: 'RiEo8C3f',
         type: 'navbar_item',
         settings: [
-          { id: 'link', value: { href: '#', text: 'Home', link_type: 'url' } },
+          { id: 'link', value: { href: '#', text: 'Home', linkType: 'url' } },
         ],
       },
       P1fGieWs: {
@@ -229,7 +229,7 @@ export const normalizedPage = {
         settings: [
           {
             id: 'link',
-            value: { href: '#', text: 'About us', link_type: 'url' },
+            value: { href: '#', text: 'About us', linkType: 'url' },
           },
         ],
       },
@@ -243,11 +243,11 @@ export const normalizedPage = {
               href: '//contact',
               text: 'Contact',
               email: null,
-              link_id: 9,
-              link_type: 'page',
-              link_label: 'Contact us',
-              section_id: null,
-              open_new_window: false,
+              linkId: 9,
+              linkType: 'page',
+              linkLabel: 'Contact us',
+              sectionId: null,
+              openNewWindow: false,
             },
           },
         ],
@@ -262,11 +262,11 @@ export const normalizedPage = {
               href: '/products',
               text: 'Products #1',
               email: null,
-              link_id: 'd870133f9a075477a96a58e7639d40c5',
-              link_type: 'page',
-              link_label: 'Products',
-              section_id: null,
-              open_new_window: true,
+              linkId: 'd870133f9a075477a96a58e7639d40c5',
+              linkType: 'page',
+              linkLabel: 'Products',
+              sectionId: null,
+              openNewWindow: true,
             },
           },
         ],
@@ -284,7 +284,7 @@ export const normalizedPage = {
               width: 1920,
               height: 1200,
               filename: 'img-97.jpg',
-              byte_size: 458107,
+              byteSize: 458107,
             },
           },
         ],
@@ -302,7 +302,7 @@ export const normalizedPage = {
               width: 516,
               height: 400,
               filename: 'inner-74-2.jpg',
-              byte_size: 63881,
+              byteSize: 63881,
             },
           },
         ],
@@ -320,7 +320,7 @@ export const normalizedPage = {
               width: 516,
               height: 320,
               filename: 'img-91-4.jpg',
-              byte_size: 21808,
+              byteSize: 21808,
             },
           },
         ],
@@ -338,7 +338,7 @@ export const normalizedPage = {
               width: 1920,
               height: 1200,
               filename: 'img-91.jpg',
-              byte_size: 180178,
+              byteSize: 180178,
             },
           },
         ],
@@ -358,7 +358,7 @@ export const normalizedPage = {
               width: 572,
               height: 290,
               filename: 'Screen Shot 2021-06-30 at 3.44.04 PM.png',
-              byte_size: 35070,
+              byteSize: 35070,
             },
           },
         ],
@@ -383,7 +383,7 @@ export const normalizedPage = {
               width: 516,
               height: 320,
               filename: 'img-91-3.jpg',
-              byte_size: 41683,
+              byteSize: 41683,
             },
           },
           { id: 'button', value: { url: '#', text: 'Click here' } },

data/app/frontend/editor/spec/__mocks__/section.js

@@ -18,7 +18,7 @@ export const simpleContentSection = {
         width: 516,
         height: 320,
         filename: 'img-91-3.jpg',
-        byte_size: 41683,
+        byteSize: 41683,
       },
     },
     { id: 'button', value: { url: '#', text: 'Click here' } },
@@ -48,7 +48,7 @@ export const normalizedSimpleContentSection = {
               width: 516,
               height: 320,
               filename: 'img-91-3.jpg',
-              byte_size: 41683,
+              byteSize: 41683,
             },
           },
           { id: 'button', value: { url: '#', text: 'Click here' } },
@@ -69,7 +69,7 @@ export const navContentSection = {
       settings: [
         {
           id: 'link',
-          value: { href: '#', text: 'Home', link_type: 'url' },
+          value: { href: '#', text: 'Home', linkType: 'url' },
         },
       ],
     },
@@ -79,7 +79,7 @@ export const navContentSection = {
       settings: [
         {
           id: 'link',
-          value: { href: '#', text: 'About us', link_type: 'url' },
+          value: { href: '#', text: 'About us', linkType: 'url' },
         },
       ],
     },
@@ -93,11 +93,11 @@ export const navContentSection = {
             href: '//contact',
             text: 'Contact',
             email: null,
-            link_id: 9,
-            link_type: 'page',
-            link_label: 'Contact us',
-            section_id: null,
-            open_new_window: false,
+            linkId: 9,
+            linkType: 'page',
+            linkLabel: 'Contact us',
+            sectionId: null,
+            openNewWindow: false,
           },
         },
       ],
@@ -112,11 +112,11 @@ export const navContentSection = {
             href: '/products',
             text: 'Products #1',
             email: null,
-            link_id: 'd870133f9a075477a96a58e7639d40c5',
-            link_type: 'page',
-            link_label: 'Products',
-            section_id: null,
-            open_new_window: true,
+            linkId: 'd870133f9a075477a96a58e7639d40c5',
+            linkType: 'page',
+            linkLabel: 'Products',
+            sectionId: null,
+            openNewWindow: true,
           },
         },
       ],
@@ -131,7 +131,7 @@ export const navContentSection = {
         width: 572,
         height: 290,
         filename: 'Screen Shot 2021-06-30 at 3.44.04 PM.png',
-        byte_size: 35070,
+        byteSize: 35070,
       },
     },
   ],
@@ -144,7 +144,7 @@ export const normalizedNavContentSection = {
         id: 'RiEo8C3f',
         type: 'navbar_item',
         settings: [
-          { id: 'link', value: { href: '#', text: 'Home', link_type: 'url' } },
+          { id: 'link', value: { href: '#', text: 'Home', linkType: 'url' } },
         ],
       },
       P1fGieWs: {
@@ -153,7 +153,7 @@ export const normalizedNavContentSection = {
         settings: [
           {
             id: 'link',
-            value: { href: '#', text: 'About us', link_type: 'url' },
+            value: { href: '#', text: 'About us', linkType: 'url' },
           },
         ],
       },
@@ -167,11 +167,11 @@ export const normalizedNavContentSection = {
               href: '//contact',
               text: 'Contact',
               email: null,
-              link_id: 9,
-              link_type: 'page',
-              link_label: 'Contact us',
-              section_id: null,
-              open_new_window: false,
+              linkId: 9,
+              linkType: 'page',
+              linkLabel: 'Contact us',
+              sectionId: null,
+              openNewWindow: false,
             },
           },
         ],
@@ -186,11 +186,11 @@ export const normalizedNavContentSection = {
               href: '/products',
               text: 'Products #1',
               email: null,
-              link_id: 'd870133f9a075477a96a58e7639d40c5',
-              link_type: 'page',
-              link_label: 'Products',
-              section_id: null,
-              open_new_window: true,
+              linkId: 'd870133f9a075477a96a58e7639d40c5',
+              linkType: 'page',
+              linkLabel: 'Products',
+              sectionId: null,
+              openNewWindow: true,
             },
           },
         ],
@@ -210,7 +210,7 @@ export const normalizedNavContentSection = {
               width: 572,
               height: 290,
               filename: 'Screen Shot 2021-06-30 at 3.44.04 PM.png',
-              byte_size: 35070,
+              byteSize: 35070,
             },
           },
         ],
@@ -226,7 +226,7 @@ export const navContentSectionBlock = {
   settings: [
     {
       id: 'link',
-      value: { href: '#', text: 'Foo', link_type: 'url' },
+      value: { href: '#', text: 'Foo', linkType: 'url' },
     },
   ],
 }

data/app/helpers/maglev/editor_helper.rb

@@ -16,7 +16,7 @@ module Maglev
     end
 
     def api_base_path
-      "#{root_path}api"
+      api_root_path
     end
 
     def editor_window_title

data/app/helpers/maglev/page_preview_helper.rb

@@ -16,11 +16,15 @@ module Maglev
     # rubocop:enable Rails/OutputSafety
 
     def render_maglev_section(type, site: nil, theme: nil, page: nil, page_sections: nil)
+      sections = (page_sections || maglev_page_sections).find_all do |section|
+        (section['type'] || section[:type]).start_with?(type.to_s)
+      end
+
       render_maglev_sections(
         site: site,
         theme: theme,
         page: page,
-        page_sections: (page_sections || maglev_page_sections).find_all { |section| section['type'] == type.to_s }
+        page_sections: sections
       )
     end
 

data/app/helpers/maglev/sitemap_helper.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module Maglev
+  module SitemapHelper
+    def sitemap_url(host, page, locale = nil)
+      path = maglev_services.get_page_fullpath.call(page: page, locale: locale)
+
+      return path if path =~ %r{^https?://}
+
+      [host, path].join
+    end
+  end
+end

data/app/models/concerns/maglev/sections_concern.rb

@@ -11,6 +11,16 @@ module Maglev::SectionsConcern
     end
   end
 
+  def prepare_sections_translations
+    return if sections_translations.blank?
+
+    sections_translations.each_key do |locale|
+      Maglev::I18n.with_locale(locale) do
+        prepare_sections
+      end
+    end
+  end
+
   private
 
   def prepare_section(section)

data/app/models/maglev/page/path_concern.rb

@@ -27,7 +27,14 @@ module Maglev::Page::PathConcern
   end
 
   def path=(value)
-    current_path.value = value
+    unless value.respond_to?(:each_pair)
+      current_path.value = value
+      return
+    end
+
+    value.each_pair do |locale, new_path|
+      Maglev::I18n.with_locale(locale) { self.path = new_path }
+    end
   end
 
   def current_path

data/app/models/maglev/section/setting.rb

@@ -21,6 +21,7 @@ class Maglev::Section::Setting
     when :image then build_default_image_content(default)
     when :link then build_default_link_content(default)
     when :checkbox then build_default_checkbox_content(default)
+    when :collection_item then build_default_collection_item_content(default)
     else
       default || label
     end
@@ -42,6 +43,10 @@ class Maglev::Section::Setting
     default.presence || false
   end
 
+  def build_default_collection_item_content(default)
+    { id: default }
+  end
+
   ## class methods ##
   def self.build(hash)
     attributes = hash.slice('id', 'label', 'type', 'default')

data/app/models/maglev/site/locales_concern.rb

@@ -24,6 +24,14 @@ module Maglev::Site::LocalesConcern
     locales.map { |locale| locale.prefix.to_sym }
   end
 
+  def each_locale
+    locale_prefixes.each do |locale|
+      Maglev::I18n.with_locale(locale) do
+        yield(locale)
+      end
+    end
+  end
+
   class LocalesSerializer
     def self.dump(array)
       (array || []).map(&:as_json)

data/app/services/concerns/maglev/get_page_sections/transform_collection_item_concern.rb

@@ -3,7 +3,12 @@
 # rubocop:disable Style/ClassAndModuleChildren
 module Maglev::GetPageSections::TransformCollectionItemConcern
   def transform_collection_item_content_setting(content, setting)
-    item_id = content.dig('value', 'id')
+    # the content might come from the default value of the collection_item setting
+    # rubocop:disable Style/StringHashKeys
+    content['value'] = { 'id' => 'any' } if content&.dig('value') == 'any'
+    # rubocop:enable Style/StringHashKeys
+
+    item_id = content&.dig('value', 'id')
     return if item_id.blank?
 
     item = fetch_collection_items.call(
@@ -11,6 +16,8 @@ module Maglev::GetPageSections::TransformCollectionItemConcern
       id: item_id
     )
 
+    return unless item
+
     content['value']['label'] = item.label
     content['value']['item'] = item.source
   end

data/app/services/maglev/app_container.rb

@@ -31,6 +31,8 @@ module Maglev
     dependency :persist_section_screenshot, class: Maglev::PersistSectionScreenshot,
                                             depends_on: %i[fetch_theme fetch_section_screenshot_path]
 
+    dependency :change_site_locales,        class: Maglev::ChangeSiteLocales
+
     dependency :fetch_page,                 class: Maglev::FetchPage, depends_on: :fetch_site
     dependency :get_page_fullpath,          class: Maglev::GetPageFullpath, depends_on: %i[fetch_site get_base_url]
     dependency :get_page_sections,          class: Maglev::GetPageSections,

data/app/services/maglev/fetch_collection_items.rb

@@ -33,7 +33,7 @@ module Maglev
 
     def fetch_item
       build_item(
-        fetch_original_items.find_by(id: id)
+        id == 'any' ? fetch_original_items.first : fetch_original_items.find_by(id: id)
       )
     end
 
@@ -78,7 +78,11 @@ module Maglev
     end
 
     def collection
-      config.collections[collection_id.to_sym]
+      config.collections[collection_id.to_sym].tap do |collection|
+        next if collection
+
+        raise "[Maglev] unregistered '#{collection_id}' collection in the Maglev configuration."
+      end
     end
 
     def build_item(original_item)

data/app/services/maglev/persist_page.rb

@@ -25,15 +25,31 @@ module Maglev
 
     def persist_page!
       page.attributes = page_attributes
-      page.prepare_sections
+
+      # the sections_translations attribute is put by the SetupPages service
+      # when we generate a brand new site
+      if page_attributes.key?(:sections_translations)
+        page.prepare_sections_translations
+      else
+        page.prepare_sections
+      end
+
       page.save!
     end
 
     def persist_site!
       return unless can_persist_site?
 
-      site.attributes = site_attributes_with_consistent_sections
-      site.prepare_sections
+      # the sections_translations attribute is put by the SetupPages service
+      # when we generate a brand new site
+      if site_attributes.key?(:sections_translations)
+        site.attributes = site_attributes
+        site.prepare_sections_translations
+      else
+        site.attributes = site_attributes_with_consistent_sections
+        site.prepare_sections
+      end
+
       site.save!
     end
 
@@ -68,7 +84,10 @@ module Maglev
     end
 
     def can_persist_site?
-      site_attributes.present? && site_attributes[:sections].present?
+      site_attributes.present? && (
+        site_attributes[:sections].present? ||
+        site_attributes[:sections_translations].present?
+      )
     end
   end
 end

data/app/services/maglev/setup_pages.rb

@@ -13,7 +13,9 @@ module Maglev
 
     def call
       pages&.map do |page_attributes|
-        create_page(page_attributes)
+        create_page(
+          attributes_in_all_locales(page_attributes.with_indifferent_access)
+        )
       end
     end
 
@@ -34,12 +36,46 @@ module Maglev
     end
 
     def site_attributes_from(page_attributes)
+      if page_attributes.include?(:sections_translations)
+        { sections_translations: site_sections_translations(page_attributes) }
+      else
+        { sections: select_site_scoped_sections(page_attributes[:sections]) }
+      end.compact || {}
+    end
+
+    def site_sections_translations(page_attributes)
+      sections_translations = page_attributes[:sections_translations].transform_values do |sections|
+        select_site_scoped_sections(sections)
+      end
+      sections_translations.any? { |_, sections| sections.blank? } ? nil : sections_translations
+    end
+
+    def attributes_in_all_locales(attributes)
       {
-        sections: (page_attributes[:sections] || []).find_all do |section|
-          definition = theme.sections.find(section['type'])
-          definition.site_scoped?
-        end
+        title_translations: value_in_all_locales(attributes[:title]),
+        sections_translations: value_in_all_locales(attributes[:sections]),
+        path: value_in_all_locales(attributes[:path])
       }
     end
+
+    def value_in_all_locales(value)
+      if value.respond_to?(:each_pair)
+        default_value = value[site.default_locale_prefix]
+        fill_translations(default_value).merge(value)
+      else
+        fill_translations(value)
+      end
+    end
+
+    def fill_translations(value)
+      site.locale_prefixes.index_with { |_locale| value }
+    end
+
+    def select_site_scoped_sections(sections)
+      (sections || []).find_all do |section|
+        definition = theme.sections.find(section['type'])
+        definition.site_scoped?
+      end
+    end
   end
 end

data/app/views/maglev/api/pages/_show.json.jbuilder

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-json.sections services.get_page_sections.call(page: page)
-
 json.key_format! camelize: :lower
 json.deep_format_keys!
 
@@ -20,5 +18,6 @@ json.og_image_url page.og_image_url
 json.preview_url services.get_page_fullpath.call(page: page, preview_mode: true, locale: content_locale)
 json.live_url services.get_page_fullpath.call(page: page, preview_mode: false, locale: content_locale)
 json.section_names services.get_page_section_names.call(page: page)
+json.sections services.get_page_sections.call(page: page)
 json.lock_version page.lock_version
 json.translated page.path.present?

data/app/views/maglev/sitemap/index.xml.builder

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+xml.instruct!
+
+xml.urlset xmlns: 'http://www.google.com/schemas/sitemap/0.9', "xmlns:xhtml": 'http://www.w3.org/1999/xhtml' do
+  @pages.each do |page|
+    xml.url do
+      xml.loc sitemap_url(@host, page, maglev_site.default_locale_prefix)
+      xml.lastmod page.updated_at.strftime('%Y-%m-%d')
+
+      if maglev_site.locales.size > 1
+        maglev_site.locales.each do |locale|
+          xml.xhtml :link,
+                    rel: 'alternate',
+                    hreflang: locale.prefix,
+                    href: sitemap_url(@host, page, locale.prefix)
+        end
+      end
+    end
+  end
+end

data/config/routes.rb

@@ -5,6 +5,7 @@ Maglev::Engine.routes.draw do
   # API
   constraints format: :json do
     namespace :api do
+      root to: proc { [200, {}, ['{}']] }
       resource :site, only: :show
       resources :pages do
         resources :clones, controller: :page_clones, only: :create

data/db/migrate/20210830085101_create_maglev_page_paths.rb

@@ -1,4 +1,4 @@
-class CreateMaglevPagePaths < ActiveRecord::Migration[6.1]
+class CreateMaglevPagePaths < ActiveRecord::Migration[6.0]
   def change
     create_table :maglev_page_paths do |t|
       t.references :maglev_page