diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index ec462521..72679cff 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -22,11 +22,15 @@ class ApplicationController < ActionController::Base def find_post(site) id = params[:post_id] || params[:id] - lang = params.fetch(:lang, I18n.locale.to_s) + lang = find_lang posts = site.posts_for(lang) posts.find do |p| p.id == id end end + + def find_lang + params.fetch(:lang, I18n.locale.to_s) + end end diff --git a/app/controllers/posts_controller.rb b/app/controllers/posts_controller.rb index 7c099311..26b7b048 100644 --- a/app/controllers/posts_controller.rb +++ b/app/controllers/posts_controller.rb @@ -2,23 +2,26 @@ class PostsController < ApplicationController before_action :authenticate! def index + @lang = find_lang @site = find_site @category = session[:category] = params.dig(:category) - @lang = params.fetch(:lang, I18n.locale.to_s) @posts = @site.posts_for(@lang) end def show + @lang = find_lang @site = find_site @post = find_post(@site) end def new + @lang = find_lang @site = find_site @post = Post.new(site: @site, front_matter: { date: Time.now }) end def create + @lang = find_lang @site = find_site @post = Post.new(site: @site, front_matter: post_params.to_hash) @@ -30,12 +33,14 @@ class PostsController < ApplicationController end def edit + @lang = find_lang @site = find_site @post = find_post(@site) end def update p = post_params + @lang = find_lang @site = find_site @post = find_post(@site) diff --git a/app/models/post.rb b/app/models/post.rb index 203303c1..63b9a166 100644 --- a/app/models/post.rb +++ b/app/models/post.rb @@ -36,13 +36,20 @@ class Post class: Jekyll::Document) end - @lang = lang || I18n.locale.to_s @site = site @post = post # los errores tienen que ser un hash para que # ActiveModel pueda traer los errores normalmente @errors = {} + # Si el sitio está traducido, trabajamos con la colección del + # idioma, sino, con posts + if @site.i18n? + @collection = @lang = lang || I18n.locale.to_s + else + @collection = 'posts' + end + # sincronizar los datos del document if new? @front_matter = {} @@ -66,7 +73,7 @@ class Post # Determina si fue traducido, buscando los slugs de su front_matter # lang en otras colecciones def translated? - get_front_matter('lang').present? + @site.i18n? && get_front_matter('lang').present? end def translations @@ -84,6 +91,7 @@ class Post # todos los datos para escribir el archivo, que es la condición # necesaria para poder crearlo :P def save + binding.pry cleanup! return false unless valid? @@ -120,7 +128,11 @@ class Post # Devuelve la ruta del post, si se cambió alguno de los datos, # generamos una ruta nueva para tener siempre la ruta actualizada. def path - basename_changed? ? File.join(@site.path, '_posts', basename_from_front_matter) : @post.try(:path) + if basename_changed? + File.join(@site.path, "_#{@collection}", basename_from_front_matter) + else + @post.try(:path) + end end # TODO los slugs se pueden repetir, el identificador real sería @@ -174,7 +186,7 @@ class Post # Genera un post nuevo y lo agrega a la colección del sitio. def new_post - opts = { site: @site.jekyll, collection: @site.jekyll.posts } + opts = { site: @site.jekyll, collection: @site.jekyll.collections[@collection] } @post = Jekyll::Document.new(path, opts) end @@ -182,11 +194,13 @@ class Post # # TODO no sería la forma correcta de hacerlo en Rails def add_post_to_site! - @site.jekyll.posts.docs << @post - @site.jekyll.posts.docs.sort! + @site.jekyll.collections[@collection].docs << @post + @site.jekyll.collections[@collection].docs.sort! - @site.posts << self unless @site.posts.include? self - @site.posts.sort! + unless @site.collections[@collection].include? self + @site.collections[@collection] << self + @site.collections[@collection].sort! + end end # Los define, asegurandose que las llaves siempre son strings, para no @@ -210,7 +224,7 @@ class Post # Reemplaza el post en el sitio por uno nuevo def replace_post! - @old_post = @site.jekyll.posts.docs.delete @post + @old_post = @site.jekyll.collections[@lang].docs.delete @post new_post end diff --git a/app/models/site.rb b/app/models/site.rb index 2f51e8b1..97f37b1d 100644 --- a/app/models/site.rb +++ b/app/models/site.rb @@ -2,12 +2,13 @@ # Usuaria class Site - attr_accessor :jekyll + attr_accessor :jekyll, :collections attr_reader :path def initialize(jekyll:, path: nil) @jekyll = jekyll @path = path || @jekyll.config['source'] + @collections = {} end # Determina si el sitio está en varios idiomas @@ -61,6 +62,10 @@ class Site @jekyll.config end + def collections + @collections + end + # Los posts de este sitio, si el sitio está traducido, trae los posts # del idioma actual, porque _posts va a estar vacío def posts @@ -77,9 +82,8 @@ class Site def posts_for(collection) return unless @jekyll.config['collections'].key? collection - # TODO tal vez no necesitamos esta magia - instance = instance_variable_get("@#{collection}") - return instance if instance + _collection = @collections[collection] + return _collection if _collection Rails.logger.info "Procesando #{collection}" @@ -92,9 +96,9 @@ class Site # Los convertimos a una clase intermedia capaz de acceder a sus # datos y modificarlos - instance_variable_set("@#{collection}", col.map do |post| + @collections[collection] = col.map do |post| Post.new(site: self, post: post, lang: collection) - end) + end end def categories diff --git a/app/views/posts/_form.haml b/app/views/posts/_form.haml index 44818631..9696bf06 100644 --- a/app/views/posts/_form.haml +++ b/app/views/posts/_form.haml @@ -10,10 +10,10 @@ - field_class = "form-control #{direction}" -# TODO habilitar form_for - if @post.new? - - url = site_posts_path(@site) + - url = site_posts_path(@site, lang: @lang) - method = :post - else - - url = site_post_path(@site, @post) + - url = site_post_path(@site, @post, lang: @lang) - method = :patch = form_tag url, method: method, class: 'form' do .form-group diff --git a/app/views/posts/show.haml b/app/views/posts/show.haml index 165df242..8d9eef23 100644 --- a/app/views/posts/show.haml +++ b/app/views/posts/show.haml @@ -13,7 +13,8 @@ .row .col - = link_to t('posts.edit'), edit_site_post_path(@site, @post), + = link_to t('posts.edit'), + edit_site_post_path(@site, @post, lang: @lang), class: 'btn btn-info' .row