From 67a99bce1bc7b35518b633c7469016791bd8924e Mon Sep 17 00:00:00 2001 From: f Date: Thu, 8 Aug 2019 15:28:23 -0300 Subject: [PATCH] renombrar archivos --- .rubocop.yml | 2 + .../core_extensions/jekyll/document/path.rb | 22 +++++++ app/models/metadata_date.rb | 3 + app/models/metadata_document_date.rb | 2 + app/models/metadata_path.rb | 35 +++++++++++ app/models/metadata_slug.rb | 2 + app/models/metadata_template.rb | 2 +- app/models/post.rb | 61 +++++++++++-------- config/initializers/core_extensions.rb | 3 + test/models/post_test.rb | 48 ++++++++++++--- 10 files changed, 145 insertions(+), 35 deletions(-) create mode 100644 app/lib/core_extensions/jekyll/document/path.rb create mode 100644 app/models/metadata_date.rb create mode 100644 app/models/metadata_path.rb diff --git a/.rubocop.yml b/.rubocop.yml index fad6db48..5e07d37b 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -33,6 +33,7 @@ Metrics/MethodLength: - 'app/controllers/i18n_controller.rb' - 'app/controllers/collaborations_controller.rb' - 'app/controllers/usuaries_controller.rb' + - 'app/models/post.rb' Metrics/BlockLength: Exclude: @@ -47,6 +48,7 @@ Metrics/ClassLength: - 'app/models/site.rb' - 'app/controllers/posts_controller.rb' - 'app/controllers/sites_controller.rb' + - 'test/models/post_test.rb' Lint/HandleExceptions: Exclude: diff --git a/app/lib/core_extensions/jekyll/document/path.rb b/app/lib/core_extensions/jekyll/document/path.rb new file mode 100644 index 00000000..fb90d5f3 --- /dev/null +++ b/app/lib/core_extensions/jekyll/document/path.rb @@ -0,0 +1,22 @@ +# frozen_string_literal: true + +module CoreExtensions + module Jekyll + module Document + # Permite cambiar la ubicación del archivo para que podamos leerlo + # sin tener que instanciar uno nuevo. + module Path + def path=(new_path) + @path = new_path + # Dejar que lo recalcule al releer + @cleaned_relative_path = + @extname = + @basename = + @basename_without_ext = + @output_ext = + @relative_path = nil + end + end + end + end +end diff --git a/app/models/metadata_date.rb b/app/models/metadata_date.rb new file mode 100644 index 00000000..2c4741f3 --- /dev/null +++ b/app/models/metadata_date.rb @@ -0,0 +1,3 @@ +class MetadataDate < MetadataTemplate + +end diff --git a/app/models/metadata_document_date.rb b/app/models/metadata_document_date.rb index 425e8935..3d98885c 100644 --- a/app/models/metadata_document_date.rb +++ b/app/models/metadata_document_date.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + # Maneja la fecha del document class MetadataDocumentDate < MetadataTemplate # La fecha por defecto es ahora! diff --git a/app/models/metadata_path.rb b/app/models/metadata_path.rb new file mode 100644 index 00000000..ec0467e5 --- /dev/null +++ b/app/models/metadata_path.rb @@ -0,0 +1,35 @@ +# Este campo representa el archivo donde se almacenan los datos +class MetadataPath < MetadataTemplate + # :label en este caso es el idioma/colección + def default_value + File.join(site.path, "_#{lang}", "#{date}-#{slug}#{ext}") + end + + def value + default_value + end + alias absolute value + alias to_s value + + def relative + value.sub(site.path, '').sub(%r{^/}, '') + end + + private + + def ext + document.extname || '.markdown' + end + + def lang + post.lang + end + + def slug + post.slug.value + end + + def date + post.date.value.strftime('%F') + end +end diff --git a/app/models/metadata_slug.rb b/app/models/metadata_slug.rb index 905ba0c4..19206fc2 100644 --- a/app/models/metadata_slug.rb +++ b/app/models/metadata_slug.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'jekyll/utils' # El slug es el nombre del archivo sin la fecha ni la extensión y se diff --git a/app/models/metadata_template.rb b/app/models/metadata_template.rb index 96bc7b5c..2a48983e 100644 --- a/app/models/metadata_template.rb +++ b/app/models/metadata_template.rb @@ -4,7 +4,7 @@ # # TODO: Validar el tipo de valor pasado a value= según el :type MetadataTemplate = Struct.new(:site, :document, :name, :label, :type, - :value, :help, :required, :errors, + :value, :help, :required, :errors, :post, :layout, keyword_init: true) do # El valor por defecto def default_value diff --git a/app/models/post.rb b/app/models/post.rb index 59580d29..af88d370 100644 --- a/app/models/post.rb +++ b/app/models/post.rb @@ -13,7 +13,7 @@ class Post < OpenStruct # XXX: Volver document opcional cuando estemos creando DEFAULT_ATTRIBUTES = %i[site document layout].freeze # Otros atributos que no vienen en los metadatos - ATTRIBUTES = %i[content lang date slug attributes errors].freeze + ATTRIBUTES = %i[content lang path date slug attributes errors].freeze # Redefinir el inicializador de OpenStruct # @@ -22,7 +22,6 @@ class Post < OpenStruct # @param layout: [Layout] la plantilla # # rubocop:disable Metrics/AbcSize - # rubocop:disable Metrics/MethodLength def initialize(**args) default_attributes_missing(args) super(args) @@ -45,6 +44,7 @@ class Post < OpenStruct layout.metadata.each_pair do |name, template| send "#{name}=".to_sym, MetadataFactory.build(document: document, + post: self, site: site, name: name, layout: layout, @@ -56,12 +56,12 @@ class Post < OpenStruct load_slug! load_date! + load_path! # Leer el documento read end # rubocop:enable Metrics/AbcSize - # rubocop:enable Metrics/MethodLength # Levanta un error si al construir el artículo no pasamos un atributo. def default_attributes_missing(**args) @@ -127,13 +127,13 @@ class Post < OpenStruct # # XXX Commit def destroy - FileUtils.rm_f path + FileUtils.rm_f path.absolute site.posts(lang: lang).delete_if do |post| - post.path == path + post.path.absolute == path.absolute end - !File.exist?(path) && !site.posts(lang: lang).include?(self) + !File.exist?(path.absolute) && !site.posts(lang: lang).include?(self) end alias destroy! destroy # rubocop:enable Metrics/AbcSize @@ -141,10 +141,12 @@ class Post < OpenStruct # Guarda los cambios def save return false unless valid? + # Salir si tenemos que cambiar el nombre del archivo y no pudimos + return false if path_changed? && !update_path! return false unless write # Vuelve a leer el post para tomar los cambios - document.read + read true end @@ -152,20 +154,15 @@ class Post < OpenStruct # Lee el documento def read - Dir.chdir(site.path) do - document.read - end + document.read end - # Devuelve la ruta del post, si se cambió alguno de los datos, - # generamos una ruta nueva para tener siempre la ruta actualizada. - def path - document.path - end - alias relative_path path - - def absolute_path - File.join site.path, path + # Actualizar la ubicación del archivo si cambió de lugar y si no + # existe el destino + def update_path! + !File.exist?(path.absolute) && + FileUtils.mv(path_was, path.absolute) && + document.path = path.absolute end # Detecta si el artículo es válido para guardar @@ -224,7 +221,7 @@ class Post < OpenStruct def write return true if persisted? - Site::Writer.new(site: site, file: path, + Site::Writer.new(site: site, file: path.absolute, content: full_content, usuarie: usuarie, message: title.value).save end @@ -241,25 +238,27 @@ class Post < OpenStruct # Verifica si hace falta escribir cambios def persisted? - File.exist?(absolute_path) && full_content == File.read(absolute_path) + File.exist?(path.absolute) && full_content == File.read(path.absolute) end private + # rubocop:disable Metrics/AbcSize def new_attribute_was(method) attr_was = (attribute_name(method).to_s + '_was').to_sym return attr_was if singleton_class.method_defined? attr_was define_singleton_method(attr_was) do name = attribute_name(attr_was) - name == :content ? document.content : document.data[name.to_s] + if document.respond_to?(name) + document.send(name) + else + document.data[name.to_s] + end end - - attr_was end # Pregunta si el atributo cambió - # rubocop:disable Metrics/AbcSize def new_attribute_changed(method) attr_changed = (attribute_name(method).to_s + '_changed?').to_sym @@ -284,15 +283,25 @@ class Post < OpenStruct def load_slug! self.slug = MetadataSlug.new(document: document, site: site, - layout: layout, name: :slug, + layout: layout, name: :slug, type: :slug, + post: self, required: true) end def load_date! self.date = MetadataDocumentDate.new(document: document, site: site, layout: layout, name: :date, + type: :document_date, + post: self, required: true) end + + def load_path! + self.path = MetadataPath.new(document: document, site: site, + layout: layout, name: :path, + type: :path, post: self, + required: true) + end end # rubocop:enable Metrics/ClassLength # rubocop:enable Style/MethodMissingSuper diff --git a/config/initializers/core_extensions.rb b/config/initializers/core_extensions.rb index 2207fe85..95cf9571 100644 --- a/config/initializers/core_extensions.rb +++ b/config/initializers/core_extensions.rb @@ -1,3 +1,6 @@ # frozen_string_literal: true +require 'jekyll/document' + String.include CoreExtensions::String::StripTags +Jekyll::Document.include CoreExtensions::Jekyll::Document::Path diff --git a/test/models/post_test.rb b/test/models/post_test.rb index 4d0cb651..2d01c84f 100644 --- a/test/models/post_test.rb +++ b/test/models/post_test.rb @@ -34,18 +34,18 @@ class PostTest < ActiveSupport::TestCase test 'se pueden eliminar' do # TODO: cuando esté con git, solo aplicar git reset tmp = File.join(Rails.root, 'tmp', 'eliminar.md') - FileUtils.cp @post.path, tmp + FileUtils.cp @post.path.absolute, tmp assert @post.destroy - assert_not File.exist?(@post.path) + assert_not File.exist?(@post.path.absolute) assert_not @site.posts.include?(@post) - FileUtils.mv tmp, @post.path + FileUtils.mv tmp, @post.path.absolute end test 'se puede ver el contenido completo después de guardar' do assert @post.save - @post.document.read + @post.read # Queremos saber si todos los atributos del post terminaron en el # archivo @@ -56,10 +56,10 @@ class PostTest < ActiveSupport::TestCase if metadata.empty? assert_not @post.document.data[attr.to_s].present? - elsif attr == :date - assert_equal metadata.value, @post.document.date + elsif @post.document.respond_to? attr + assert_equal metadata.value, @post.document.send(attr), attr else - assert_equal metadata.value, @post.document.data[attr.to_s] + assert_equal metadata.value, @post.document.data[attr.to_s], attr end end end @@ -83,7 +83,7 @@ class PostTest < ActiveSupport::TestCase Dir.chdir(@site.path) do collection = Jekyll::Collection.new(@site.jekyll, I18n.locale.to_s) - document = Jekyll::Document.new(@post.path, + document = Jekyll::Document.new(@post.path.value, site: @site.jekyll, collection: collection) document.read @@ -128,4 +128,36 @@ class PostTest < ActiveSupport::TestCase assert @post.date_changed? assert @post.date.valid? end + + test 'al cambiar slug o fecha cambia el archivo de ubicacion' do + hoy = Date.today.to_time + path_was = @post.path.absolute + @post.slug.value = 'test' + @post.date.value = hoy + + assert @post.path_changed? + assert_equal "_es/#{hoy.strftime('%F')}-test.markdown", + @post.path.relative + + assert @post.save + assert_equal "_es/#{hoy.strftime('%F')}-test.markdown", + @post.document.relative_path + assert_not File.exist?(path_was) + assert File.exist?(@post.path.absolute) + end + + test 'no podemos pisar otros archivos' do + # Elegir un post al azar que no sea el que usamos + loop do + post = @site.posts.sample + break if post != @post + end + + @post.slug.value = post.slug.value + @post.date.value = post.date.value + + assert_not @post.save + assert File.exist?(@post.path.absolute) + assert File.exist?(@post.path_was) + end end