nocms-pages 0.1.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cd58af22ca24e3b0617e52dc00f4a12e186a7cf7
-  data.tar.gz: 336c772c261531d84e1d84b58df5a83f240cddff
+  metadata.gz: cb02fabad9eccd5316c27ba743e461bab475d77a
+  data.tar.gz: 055dd8a7dde5d2e0c937a3775e589470238f2ce2
 SHA512:
-  metadata.gz: f8981fcf6322dd3d5d91d93908f6ae184c5938b7ba03a91222930cc550826b3d4b34cc99b1e49fc680fc8c79a94452c42a627a5c4a0fbce355037f6c2175f098
-  data.tar.gz: 2dd795d0b8cbb767baff11b16c36f094f3df9714cdc067a9a09d0388eb2d251aa71d86dc07d248ec20ac02e9db58656bbc0130a4f042c26124d2753e7a522901
+  metadata.gz: 7fc360044fff30e339c8421d0b483fc24aa5e370074fd72534a431a74ceff84b27158918ca72cf80d3494eceeaa400839053eb99f2b75ee392e3d017b55904af
+  data.tar.gz: e909a3cfd7ec6faa68f0ec9dd92e62b9176b19b5b85427846aba08be7fc3b968ac792b717efd666965e49f63469473b5dccc128ca3318de1b5459d35f27afbe9

data/README.md

@@ -23,6 +23,7 @@ bundle install
 And then import all the migrations:
 
 ```
+rake no_cms_blocks:install:migrations
 rake no_cms_pages:install:migrations
 ```
 
@@ -68,133 +69,23 @@ This `template` is set from the template attribute, being `show` the default val
 
 ### Blocks
 
-Blocks are the unit of contents the pages are made of. They are thought to be independent and customizable modules that can be created, edited or removed on their own, without dependency of any other module.
+In a previous gem version blocks were a model within the NoCms::Pages namespace, but now everything was moved to the [nocms-blocks](https://github.com/simplelogica/nocms-blocks) and in this repo there's only left a rake task for the migration.
 
-#### Block layouts
-
-In NoCMS Pages, block layouts define two main things:
-
-1. What kind of information a block contains and other settings (i.e. cache settings).
-2. How this information is displayed within the page.
-
-Block settings are configured in the file `config/initializers/nocms/pages.rb`. Here we declare all the available layouts for a block.
-
-The following code
-
-```ruby
-NoCms::Pages.configure do |config|
-
-  config.block_layouts = {
-    'default' => {
-      template: 'default',
-      fields: {
-        title: :string,
-        body: :text
-      }
-    },
-    'title-3_columns' => {
-      template: 'title_3_columns',
-      fields: {
-        title: :string,
-        column_1: :text,
-        column_2: :text,
-        column_3: :text
-      },
-    },
-    'logo-caption' => {
-      template: 'logo_caption',
-      fields: {
-        caption: :string,
-        logo: TestImage
-      }
-    }
-  }
-
-end
-```
-
-declares 3 layouts ('default', 'title-3_columns' and 'logo-caption'). Each layout has a template and some declared fields. These fields will be available in the ruby object for that block. As an example, if `@block` is an instance of the NoCms::Pages::Block model which layout attribute is set to 'default' you will be able to do `@block.title`
-
-```ruby
-block = NoCms::Pages::Block.new
-block.layout = 'default'
-
-block.title = 'a title'
-block.title # => 'a title'
-
-block.column_1 = 'a column' # => NoMethodError
-block.column_1 # => NoMethodError
-
-
-block.layout = 'title-3_columns'
-
-block.title # => 'a title'
-block.column_1 = 'a column'
-block.column_1 # => 'a column'
-block.body # => NoMethodError
-
-block.layout = 'logo_caption'
-block.title # => NoMethodError
-block.logo = { name: 'testing logo' } # Currently this is the way to assign objects
-block.logo.name # => 'testing logo'
-block.logo.class # => TestImage
-block.logo = TestImage.new name: 'testing logo' # Error! Currently assigning the object is not allowed :(
-```
-
-#### Block templates
-
-Blocks are rendered using the `render_block` helper which controls all the logic related with renderinf a block, including fragment cache control.
-
-In the end a partial is rendered using the block as a local variable to obtain the information. This partial must be found at `no_cms/pages/blocks` views folder and have the name configured in the `template` setting of the block. This way, rendering a 'title-3_columns' would render the partial `/no_cms/pages/blocks/title_3_columns`.
-
-This partial is a regular Rails partial (nothing special here). AS an example, this could be the content of our  `/no_cms/pages/blocks/title_3_columns.html.erb` partial:
-
-```html
-<div class='columns_block'>
-  <h2 class="title"><%= block.title %></h2>
-  <p class="column_1"><%= block.column_1 %></p>
-  <p class="column_2"><%= block.column_2 %></p>
-  <p class="column_3"><%= block.column_3 %></p>
-</div>
-```
-
-As you can see, the partial has a block variable containing the block object you are rendering.
-
-Since this is plain old rails you can do everything you can do with a partial (i.e. having a `/no_cms/pages/blocks/title_3_columns.en.html.erb` for the english version and a `/no_cms/pages/blocks/title_3_columns.es.html.erb` for the spanish one).
+You can read the steps for a sucessful block migration in its own [migration documentation](doc/migration-blocks-steps.md)
 
 ### Block Cache
 
-Since blocks are independent units of content within a page, the standard Rails fragment cache seemed to fit well with them. That's why the `render_block` helper decides wether Rails cache should be used for rendering an individual block.
-
-Cache for the blocks are configured at 3 levels:
+Blocks are rendered through the `render_page_block` helper instead of the `render_block` helper from `nocms-blocks`. This helper uses the `render_block` helper but add some extra cache levels:
 
 1. The page may have its `cache_enabled` attribute set to false. If this is the case then the cache will be disabled without any further check. This way, a page can be marked as "not cacheable" (e.g. in an admin interface) and no other setting can overwrite it.
 
-2. The `render_block` helper may be called with a `cache_enabled` option set to true or false. This option will enable/disable the cache. This allow us to render a block without using the cache (maybe on a preview action).
+2. The `render_page_block` helper may be called with a `cache_enabled` option set to true or false. This option will enable/disable the cache. This allow us to render a block without using the cache (maybe on a preview action).
 
   ```ruby
     render_block block, cache: false
   ```
 
-3. In the blocks configuration we can enable/disable the cache for all the blocks of a kind. We just have to add the cache_enabled setting.
-
-  ```ruby
-  NoCms::Pages.configure do |config|
-
-    config.block_layouts = {
-      'default' => {
-        template: 'default',
-        fields: {
-          title: :string,
-          body: :text
-        },
-        cache_enabled: false
-      }
-    }
-  end
-  ```
-
-4. In the blocks configuration file we can enable/disable cache for all the blocks that doesn't have a cache_enabled setting. This configuration will be stored at `NoCms::Pages.cache_enabled`
+3. In the page configuration file we can enable/disable cache for all the blocks that doesn't have a cache_enabled setting. This configuration will be stored at `NoCms::Pages.cache_enabled`
 
   ```ruby
   NoCms::Pages.configure do |config|
@@ -208,23 +99,25 @@ As a summary:
 
 ```ruby
 
-  b = NoCms::Pages::Block.new layout: 'default', title: 'Foo', description: 'Bar', page: page
-  b.page.cache_enabled # => true
+  b = NoCms::Blocks::Block.new layout: 'default', title: 'Foo', description: 'Bar'
+  page.blocks << b
+  page.cache_enabled # => true
   NoCms::Pages.cache_enabled # => true
   b.cache_enabled # => false, since the block configuration sets it to false
-  render_block b # => This won't use fragment cache since this block layout have cache disabled
+  render_page_block b # => This won't use fragment cache since this block layout have cache disabled
 
-  b = NoCms::Pages::Block.new layout: 'title-3_columns', title: 'Foo', description: 'Bar', page: page
-  b.page.cache_enabled # => true
+  b = NoCms::Blocks::Block.new layout: 'title-3_columns', title: 'Foo', description: 'Bar'
+  page.blocks << b
+  page.cache_enabled # => true
   NoCms::Pages.cache_enabled # => true
   b.cache_enabled # => true, since this block configuration doesn't override NoCms::Pages.cache_enabled
-  render_block b # => This will use fragment cache since, by default, it's enabled for all blocks
+  render_page_block b # => This will use fragment cache since, by default, it's enabled for all blocks
 
-  render_block b, cache_enabled: false # => This won't use fragment cache as the option in the helper overrides the block configuration
+  render_page_block b, cache_enabled: false # => This won't use fragment cache as the option in the helper overrides the block configuration
 
   page.cache_enabled = false
-  render_block b # => This won't use fragment cache sincs it's been disabled for the page and blocks configuration has been override
-  render_block b, cache_enabled: true # => This won't use fragment cache even when saying the helper to do it. Power for the users!
+  render_page_block b # => This won't use fragment cache sincs it's been disabled for the page and blocks configuration has been override
+  render_page_block b, cache_enabled: true # => This won't use fragment cache even when saying the helper to do it. Power for the users!
 
 ```
 

data/Rakefile

@@ -14,8 +14,11 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
+Bundler::GemHelper.install_tasks
 
-
+APP_RAKEFILE = File.expand_path("../spec/dummy/Rakefile", __FILE__)
+load 'rails/tasks/engine.rake'
 
 Bundler::GemHelper.install_tasks
 
+task :default => [:"app:spec"]

data/app/controllers/no_cms/pages/application_controller.rb

@@ -1,6 +1,7 @@
 module NoCms
   module Pages
     class ApplicationController < ::ApplicationController
+      helper NoCms::Blocks::BlocksHelper
     end
   end
 end

data/app/decorators/no_cms/blocks/block_decorator.rb

@@ -0,0 +1,12 @@
+NoCms::Blocks::Block.class_eval do
+
+  after_create :set_default_position
+
+  has_and_belongs_to_many :pages, class_name: 'NoCms::Pages::Page'
+
+  def set_default_position
+    self.update_attribute :position, (((pages.map{|p| p.blocks.pluck(:position)}).flatten.compact.max || 0) + 1) if self[:position].blank? && self.pages.exists?
+  end
+
+end
+

data/app/helpers/no_cms/pages/pages_helper.rb

@@ -1,23 +1,13 @@
 module NoCms
   module Pages
     module PagesHelper
-      def render_block block, options = {}
-        # If cache is disabled for this block then we disable no matter what the block or the options passed have to say about it. This way, the user in the back has the last word about disabling cache
-        options[:cache_enabled] = false unless block.page.cache_enabled
-        # If we don't have any option about cache enabled then we ask the block
-        options[:cache_enabled] = block.cache_enabled unless options.has_key? :cache_enabled
-
-        block_template = "no_cms/pages/blocks/#{block.template}"
-
-        # And now decide if we use cache or not
-        if options[:cache_enabled]
-          Rails.cache.fetch "#{block_template}/#{block.id}/#{block.updated_at.to_i}" do
-            render block_template, block: block
-          end
-        else
-          render block_template, block: block
-        end
-
+      def render_page_block page, block, options = {}
+        # If cache is disabled for all pages or is disabled for this block
+        # then we disable no matter what the block or the options passed have to say about it.
+        # This way, the user in the back has the last word about disabling cache and the NoCms::Pages engine
+        # control the cache over the NoCms::Block engine
+        options[:cache_enabled] = false if !NoCms::Pages.cache_enabled || !page.cache_enabled
+        render_block block, options
       end
     end
   end

data/app/models/no_cms/pages/deprecated/block.rb

@@ -0,0 +1,45 @@
+class NoCms::Pages::Deprecated::Block < ActiveRecord::Base
+
+  self.table_name = "no_cms_pages_blocks"
+
+
+  belongs_to :page
+  belongs_to :parent, class_name: "NoCms::Pages::Deprecated::Block"
+  has_many :children, class_name: "NoCms::Pages::Deprecated::Block", foreign_key: 'parent_id', inverse_of: :parent, dependent: :destroy
+
+  scope :roots, ->() { where parent_id: nil }
+
+  translates :layout, :fields_info, :draft
+
+  class Translation
+    self.table_name = "no_cms_pages_block_translations"
+    serialize :fields_info, Hash
+  end
+
+  def readonly?
+    true
+  end
+
+  def position
+    self[:position] || 0
+  end
+
+  def self.dump
+    roots.map(&:dump)
+  end
+
+  def dump
+    {
+      page: page,
+      position: position,
+      children: children.map(&:dump),
+      translations: Hash[translations.map{|t| [t.locale, {
+        layout: t.layout,
+        draft: t.draft,
+        fields_info: t.fields_info.to_hash
+        }]}]
+
+    }
+  end
+
+end

data/app/models/no_cms/pages/page.rb

@@ -12,10 +12,11 @@ module NoCms::Pages
 
     acts_as_nested_set
 
-    has_many :blocks, inverse_of: :page, class_name: 'NoCms::Pages::Block'
+    has_and_belongs_to_many :blocks, class_name: "NoCms::Blocks::Block"
     accepts_nested_attributes_for :blocks, allow_destroy: true
 
     translates :title, :body, :slug, :path, :draft, :css_class, :css_id, :cache_enabled
+    accepts_nested_attributes_for :translations
 
     validates :title, presence: true
     validates :body, presence: true if NoCms::Pages.use_body?
@@ -28,14 +29,18 @@ module NoCms::Pages
 
     def set_slug_and_path
       self.slug = title.parameterize if slug.nil? && !title.nil? # If there's no slug then we create it
-      self.slug = title.parameterize if slug.blank? && !parent.nil? # If slug is blank and this page has a parent then we recreate it
-      self.slug = title.parameterize if slug.blank? && Page.home && (Page.home != self) # If slug is blank and there's already a home (and it's another page) then we recreate it
+      self.slug = title.parameterize if slug.blank? && !title.nil? && !parent.nil? # If slug is blank and this page has a parent then we recreate it
+      self.slug = title.parameterize if slug.blank? &&  !title.nil? && Page.home && (Page.home != self) # If slug is blank and there's already a home (and it's another page) then we recreate it
       self.rebuild_path if path.nil? || attribute_changed?('slug')
     end
 
     def rebuild_path
-      self.update_attribute :path, "#{parent.path unless parent.nil?}/#{slug}"
-      descendants.each(&:rebuild_path)
+      if self.persisted?
+        self.update_attribute :path, "#{parent.path unless parent.nil?}/#{slug}"
+        descendants.each(&:rebuild_path)
+      else
+        self.path = "#{parent.path unless parent.nil?}/#{slug}"
+      end
     end
 
     def self.templates

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

@@ -2,5 +2,5 @@
 <p><%= @page.body %></p>
 
 <% @blocks.sort_by(&:position).each do |block| %>
-  <%= render_block block %>
+  <%= render_page_block @page, block %>
 <% end %>

data/config/locales/en.yml

@@ -11,6 +11,9 @@ en:
         css_class: CSS classes
         css_id: CSS id
         layout: Layout
+        parent: Parent
+        template: Template
+        draft: Draft
       no_cms/pages/block:
         layout: Layout
         title: Title

data/config/locales/es.yml

@@ -11,6 +11,9 @@ es:
         css_class: Clases CSS
         css_id: Id CSS
         layout: Layout
+        parent: Padre
+        template: Plantilla
+        draft: Borrador
       no_cms/pages/block:
         layout: Layout
         title: Título

data/db/migrate/20141211224243_create_no_cms_blocks_no_cms_pages_relation.rb

@@ -0,0 +1,11 @@
+class CreateNoCmsBlocksNoCmsPagesRelation < ActiveRecord::Migration
+  def change
+    create_table :no_cms_blocks_blocks_pages_pages, id: false do |t|
+
+      t.belongs_to :page
+      t.belongs_to :block
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20141211224348_destroy_no_cms_pages_blocks.rb

@@ -0,0 +1,15 @@
+class DestroyNoCmsPagesBlocks < ActiveRecord::Migration
+  def change
+
+    if NoCms::Blocks::Block.table_exists?
+      begin
+        Rake::Task["no_cms:pages:migrate_blocks"].invoke
+      rescue
+        Rake::Task["app:no_cms:pages:migrate_blocks"].invoke
+      end
+    end
+
+    drop_table :no_cms_pages_blocks
+
+  end
+end

data/lib/generators/nocms/pages_generator.rb

@@ -7,6 +7,10 @@ module NoCms
       template "config/initializers/nocms/pages.rb", File.join(destination_root, "config", "initializers", "nocms", "pages.rb")
     end
 
+    def generate_nocms_blocks_initializer
+      generate "nocms:blocks"
+    end
+
     def self.namespace
       "nocms:pages"
     end

data/lib/generators/nocms/templates/config/initializers/nocms/pages.rb

@@ -1,45 +1,5 @@
 NoCms::Pages.configure do |config|
 
-  # Enable Rails fragment cache for the block templates when you call the render_block helper
-  # You can override this cache setting in any block configuration below or sending
-  # the cache option true or false when calling the block helpers
-  # e.g: render_block block, cache: true
-  # config.cache_enabled = false
-
-  # In this section we configure block layouts. It's just an array of layouts, each consisting on a hash.
-  # Each layout has a series of options
-  # E.g: config.block_layouts = {
-  #   'title-long_text' => {
-  #     template: 'title-long_text', # This is the template of this block,
-  #                                  # used as a partial both in the front
-  #                                  # and the admin (if you use the nocms-admin-pages gem)
-  #     fields: { # This is the list of fields a block with this layout would have
-  #       title: :string,
-  #       long_text: :text,
-  #       image: Image, # You may use another ActiveRecord classes of your own
-  #     }
-  #    allow_nested_blocks: true, # A block with this layout may include a list of nested blocks
-  #                               # This setting is actually used by nocms-admin-pages gem to show
-  #                               # nested forms
-  #    nest_levels: [0] # Some layout may not be nestable, or useful only in certain nesting level
-  #                     # Once again, this setting is used by nocms-admin-pages gem to hide certain
-  #                     # in nested blocks. When blank, it's assumed there's no restriction.
-  #    cache_enabled: false # When setting cache_enabled you will be **overriding** the global cache_enabled
-  #                         # setting. If you don't set a cache setting then it will use the global cache
-  #                         # setting specified above
-  #   },
-  #   'title-3_columns_text' => {
-  #     template: 'title-3_columns_text',
-  #     fields: {
-  #       title: :string,
-  #       column_1: :text,
-  #       column_2: :text,
-  #       column_3: :text
-  #     }
-  #   }
-  # }
-  # config.block_layouts = {}
-
   # By default we use blocks to create the content of the page. If we just want a big textarea to insert the content we must set use_body to true
   # config.use_body = false
 

data/lib/nocms/pages/engine.rb

@@ -1,10 +1,17 @@
 require 'globalize'
+require 'nocms-blocks'
 require 'awesome_nested_set'
 
 module NoCms
   module Pages
     class Engine < ::Rails::Engine
       isolate_namespace NoCms::Pages
+
+      config.to_prepare do
+        Dir.glob(NoCms::Pages::Engine.root + "app/decorators/**/*_decorator*.rb").each do |c|
+          require_dependency(c)
+        end
+      end
     end
   end
 end

data/lib/nocms/pages/version.rb

@@ -1,5 +1,5 @@
 module NoCms
   module Pages
-    VERSION = "0.1.0"
+    VERSION = "1.0.0"
   end
 end

data/lib/tasks/pages_tasks.rake

@@ -1,4 +1,45 @@
-# desc "Explaining what the task does"
-# task :pages do
-#   # Task goes here
-# end
+namespace :no_cms do
+
+  namespace :pages do
+
+    desc 'Task for migrating blocks from NoCms::Pages::Block model to NoCms::Blocks::Block'
+    task :migrate_blocks => :environment do
+      root_blocks = NoCms::Pages::Deprecated::Block.dump
+      root_blocks.each do |root_block|
+        import_block root_block
+      end
+    end
+
+    def import_block block
+      return if block[:page].nil?
+
+      # Due to globalize we have to send the default translation via the object attributes
+      # instead of using the translation_attributes
+      default_translation = block[:translations][I18n.locale]
+      other_translations = block[:translations].reject{|locale, _| locale == I18n.locale }
+
+      new_block = block[:page].blocks.build position: block[:position],
+        translations_attributes: other_translations.map {|locale, translation|
+          {
+            locale: locale,
+            layout: translation[:layout],
+            draft: translation[:draft],
+            fields_info: translation[:fields_info].to_hash
+          }
+        },
+        layout: default_translation[:layout],
+        draft: default_translation[:draft],
+        fields_info: default_translation[:fields_info].to_hash
+
+      new_block.translations.each(&:save!)
+
+      new_block.children = block[:children].map{|b| import_block b}
+      new_block.save!
+
+      block[:page].blocks << new_block
+      new_block
+    end
+
+  end
+
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: nocms-pages
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Simplelogica
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-09 00:00:00.000000000 Z
+date: 2015-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -34,22 +34,22 @@ dependencies:
   name: globalize
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '4.0'
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.0.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '4.0'
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.0.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.1'
 - !ruby/object:Gem::Dependency
   name: awesome_nested_set
   requirement: !ruby/object:Gem::Requirement
@@ -64,6 +64,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 3.0.0.rc.6
+- !ruby/object:Gem::Dependency
+  name: nocms-blocks
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.1
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -92,9 +106,10 @@ files:
 - app/assets/stylesheets/pages/application.css
 - app/controllers/no_cms/pages/application_controller.rb
 - app/controllers/no_cms/pages/pages_controller.rb
+- app/decorators/no_cms/blocks/block_decorator.rb
 - app/helpers/no_cms/pages/pages_helper.rb
-- app/models/no_cms/pages/block.rb
 - app/models/no_cms/pages/concerns/translation_scopes.rb
+- app/models/no_cms/pages/deprecated/block.rb
 - app/models/no_cms/pages/page.rb
 - app/views/layouts/nocms/pages/application.html.erb
 - app/views/no_cms/pages/blocks/_default.html.erb
@@ -116,6 +131,8 @@ files:
 - db/migrate/20140407083115_add_css_class_and_id_to_no_cms_pages_page_translations.rb
 - db/migrate/20140702112813_add_cache_enabled_to_no_cms_pages_page_translations.rb
 - db/migrate/20140806074811_add_layout_to_no_cms_pages_page.rb
+- db/migrate/20141211224243_create_no_cms_blocks_no_cms_pages_relation.rb
+- db/migrate/20141211224348_destroy_no_cms_pages_blocks.rb
 - lib/generators/nocms/pages_generator.rb
 - lib/generators/nocms/templates/config/initializers/nocms/pages.rb
 - lib/nocms-pages.rb

data/app/models/no_cms/pages/block.rb

@@ -1,174 +0,0 @@
-module NoCms::Pages
-  class Block < ActiveRecord::Base
-
-    include NoCms::Pages::Concerns::TranslationScopes
-
-    scope :drafts, ->() { where_with_locale(draft: true) }
-    scope :no_drafts, ->() { where_with_locale(draft: false) }
-    scope :roots, ->() { where parent_id: nil }
-    belongs_to :page
-
-    belongs_to :parent, class_name: "NoCms::Pages::Block"
-    has_many :children, class_name: "NoCms::Pages::Block", foreign_key: 'parent_id', inverse_of: :parent, dependent: :destroy
-    accepts_nested_attributes_for :children, allow_destroy: true
-
-    attr_reader :cached_objects
-
-    translates :layout, :fields_info, :draft
-
-    class Translation
-      serialize :fields_info, Hash
-    end
-
-    after_initialize :set_blank_fields
-    after_create :set_default_position
-    before_save :save_related_objects
-    before_validation :copy_parent_page
-
-    validates :fields_info, presence: { allow_blank: true }
-    validates :page, :layout, presence: true
-
-    def position
-      self[:position] || 0
-    end
-
-    def layout_config
-      NoCms::Pages.block_layouts.stringify_keys[layout]
-    end
-
-    def template
-      layout_config[:template] if layout_config
-    end
-
-    def cache_enabled
-      return NoCms::Pages.cache_enabled unless layout_config
-      layout_config.has_key?(:cache_enabled) ? layout_config[:cache_enabled] : NoCms::Pages.cache_enabled
-    end
-
-    def has_field? field
-      # We have the field if...
-      !layout_config.nil? && # We have a layout configuration AND
-        (
-          !layout_config[:fields].symbolize_keys[field.to_sym].nil? || # We have this field OR
-          !layout_config[:fields].symbolize_keys[field.to_s.gsub(/\_id$/, '').to_sym].nil? # we remove the final _id and then we have the field
-        )
-    end
-
-    def field_type field
-      return nil unless has_field?(field)
-      layout_config[:fields].symbolize_keys[field.to_sym]
-    end
-
-    def read_field field
-      return nil unless has_field?(field)
-
-      value = fields_info[field.to_sym] || # first, we get the value
-                @cached_objects[field.to_sym] # or we get it from the cached objects
-
-      # If value is still nil, but the field exists we must get the object from the database
-      if value.nil?
-        field_type = field_type(field)
-        field_id = fields_info["#{field}_id".to_sym]
-        value = @cached_objects[field.to_sym] = field_type.find(field_id) unless field_id.nil?
-      end
-
-      # If value is still nil, and the field_type is an ActiveRecord class, then we
-      if value.nil? && field_type.is_a?(Class)
-        value = @cached_objects[field.to_sym] = field_type.new
-      end
-      value
-    end
-
-    def write_field field, value
-      return nil unless has_field?(field)
-      field_type = field_type field
-      # If field type is a model then we update the cached object
-      if field_type.is_a?(Class) && field_type < ActiveRecord::Base
-        # First, we initialize the object if we don't read the object (it loads it into the cached objects)
-        @cached_objects[field.to_sym] = field_type.new if read_field(field).nil?
-        # Then, assign attributes
-        @cached_objects[field.to_sym].assign_attributes value
-      else # If it's a model then  a new object or update the previous one
-        self.fields_info = fields_info.merge field.to_sym => value # when updating through an object (i.e. the page updates through nested attributes) fields_info[field.to_sym] = value doesn't work. Kudos to Rubo for this fix
-      end
-    end
-
-    # In this missing method we check wether we're asking for one field
-    # in which case we will read or write ir
-    def method_missing(m, *args, &block)
-      # We get the name of the field stripping out the '=' for writers
-      field = m.to_s
-      write_accessor = field.ends_with? '='
-      field.gsub!(/\=$/, '')
-
-      # If this field actually exists, then we write it or read it.
-      if has_field?(field)
-        write_accessor ?
-          write_field(field, args.first) :
-          read_field(field.to_sym)
-      else
-        super
-      end
-    end
-
-    # When we are assigning attributes (this method is called in new, create...)
-    # we must split those fields from our current layout and those who are not
-    # (they must be attributes).
-    # Attributes are processed the usual way and fields are written later
-    def assign_attributes new_attributes
-      fields = []
-
-      set_blank_fields
-
-      # We get the layout
-      new_layout = new_attributes[:layout] || new_attributes['layout']
-      self.layout = new_layout unless new_layout.nil?
-
-      Rails.logger.info "Searching #{new_attributes.keys.inspect} fields in #{self.layout} layout"
-
-      # And now separate fields and attributes
-      fields = new_attributes.select{|k, _| has_field? k }.symbolize_keys
-      new_attributes.reject!{|k, _| has_field? k }
-
-      super(new_attributes)
-
-      Rails.logger.info "Writing #{fields.inspect} to #{self.layout} block"
-
-      fields.each do |field_name, value|
-        self.write_field field_name, value
-      end
-    end
-
-    def reload
-      @cached_objects = {}
-      super
-    end
-
-    def copy_parent_page
-      self.page = parent.page unless parent.nil?
-    end
-
-    private
-
-    def set_blank_fields
-      self.fields_info ||= {}
-      @cached_objects ||= {}
-    end
-
-    def set_default_position
-      self.update_attribute :position, ((page.blocks.pluck(:position).compact.max || 0) + 1) if self[:position].blank?
-    end
-
-    def save_related_objects
-      # Now we save each activerecord related object
-      cached_objects.each do |field, object|
-        # Notice that we don't care if the object is actually saved
-        # We don't care because there may be some cases where no real information is sent to an object but something is sent (i.e. the locale in a new Globalize translation) and then the object is created empty
-        # When this happens if we save! the object an error is thrown and we can't leave the object blank
-        if object.is_a?(ActiveRecord::Base) && object.save
-          fields_info["#{field}_id".to_sym] = object.id
-        end
-      end
-    end
-  end
-end