decidim-blogs 0.26.2 → 0.27.0.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b09fd77f2afed7d1e0c8c6e2d4a76d07e5e9487f96fa0618912dbdad63ec152e
-  data.tar.gz: 0c2fe3a5dd93723960dae9487d67bf76dc49823d94ccc8ab64e872267d4b0eb8
+  metadata.gz: bfc72d278f0359e9c5f9939d2664139b49cb646e5dd0fb3cee8b9dd1a46e4dfa
+  data.tar.gz: 378e6705ce51faa371d6cf33beae2cd4425d4eeee461fd827bf49aee204cffb3
 SHA512:
-  metadata.gz: cf119b7dc3e6e366f51a402697882cccd0cf1afdf6957522724c61b52d7eb9981da7b3966fde57e9f8676f99034643be174c98d75f376ecc73fb31a17b3c69f4
-  data.tar.gz: 443374c6f71405e9177bc2b31c0570cc114211ccfc1eae65901bffb4f699d08e5751b2750cec30cc9658ed9d3e393e94ca1a61af573713818f4d1944eebaf22b
+  metadata.gz: a2cd5ecb7a1e9d1f7cec12fa5d20aef440954b23c75a781fc9f22278f104f4252d32682356c89a61534a563c176bab140c08b90cb43a2feed54401837151cee4
+  data.tar.gz: ca92b63f317abdbfb05451989908395af42e6a961b27974e9258e3aff23447d68e29a5073ed004c634b387a79fea7ac64395161e086cd5fed9d0a202e10451dd

data/app/commands/decidim/blogs/admin/create_post.rb

@@ -5,7 +5,7 @@ module Decidim
     module Admin
       # This command is executed when the user creates a Post from the admin
       # panel.
-      class CreatePost < Rectify::Command
+      class CreatePost < Decidim::Command
         def initialize(form, current_user)
           @form = form
           @current_user = current_user

data/app/commands/decidim/blogs/admin/update_post.rb

@@ -5,14 +5,15 @@ module Decidim
     module Admin
       # This command is executed when the user changes a Blog from the admin
       # panel.
-      class UpdatePost < Rectify::Command
+      class UpdatePost < Decidim::Command
         # Initializes a UpdateBlog Command.
         #
         # form - The form from which to get the data.
         # blog - The current instance of the page to be updated.
-        def initialize(form, post)
+        def initialize(form, post, user)
           @form = form
           @post = post
+          @user = user
         end
 
         # Updates the blog if valid.
@@ -33,7 +34,9 @@ module Decidim
         attr_reader :form, :post
 
         def update_post!
-          post.update!(
+          Decidim.traceability.update!(
+            post,
+            @user,
             title: form.title,
             body: form.body,
             author: form.author

data/app/controllers/decidim/blogs/admin/posts_controller.rb

@@ -6,6 +6,7 @@ module Decidim
       # This controller allows the create or update a blog.
       class PostsController < Admin::ApplicationController
         helper UserGroupHelper
+        helper PostsHelper
 
         def new
           enforce_permission_to :create, :blogpost
@@ -38,7 +39,7 @@ module Decidim
           enforce_permission_to :update, :blogpost, blogpost: post
           @form = form(PostForm).from_params(params, current_component: current_component)
 
-          UpdatePost.call(@form, post) do
+          UpdatePost.call(@form, post, current_user) do
             on(:ok) do
               flash[:notice] = I18n.t("posts.update.success", scope: "decidim.blogs.admin")
               redirect_to posts_path
@@ -53,7 +54,10 @@ module Decidim
 
         def destroy
           enforce_permission_to :destroy, :blogpost, blogpost: post
-          post.destroy!
+
+          Decidim.traceability.perform_action!("delete", post, current_user) do
+            post.destroy!
+          end
 
           flash[:notice] = I18n.t("posts.destroy.success", scope: "decidim.blogs.admin")
 

data/app/forms/decidim/blogs/admin/post_form.rb

@@ -10,24 +10,44 @@ module Decidim
         translatable_attribute :title, String
         translatable_attribute :body, String
 
-        attribute :user_group_id, Integer
+        attribute :decidim_author_id, Integer
 
         validates :title, translatable_presence: true
         validates :body, translatable_presence: true
+        validate :can_set_author
 
-        def map_model(post)
-          self.user_group_id = post.author.id if post.author.is_a?(Decidim::UserGroup)
+        def map_model(model)
+          self.decidim_author_id = nil if model.author.is_a? Decidim::Organization
         end
 
-        def user_group
-          @user_group ||= Decidim::UserGroup.find_by(
+        def user_or_group
+          @user_or_group ||= Decidim::UserBaseEntity.find_by(
             organization: current_organization,
-            id: user_group_id.to_i
+            id: decidim_author_id
           )
         end
 
         def author
-          user_group || current_user
+          user_or_group || current_organization
+        end
+
+        private
+
+        def can_set_author
+          return if author == current_user.organization
+          return if author == current_user
+          return if user_groups.include? author
+          return if author == post&.author
+
+          errors.add(:decidim_author_id, :invalid)
+        end
+
+        def post
+          @post ||= Post.find_by(id: id)
+        end
+
+        def user_groups
+          @user_groups ||= Decidim::UserGroups::ManageableUserGroups.for(current_user).verified
         end
       end
     end

data/app/helpers/decidim/blogs/admin/posts_helper.rb

@@ -18,6 +18,21 @@ module Decidim
           body = translated_attribute(post.body)
           CGI.unescapeHTML html_truncate(body, max_length: max_length)
         end
+
+        def post_author_select_field(form, name, _options = {})
+          select_options = [
+            [current_organization.name, ""],
+            [current_user.name, current_user.id]
+          ]
+          current_user_groups = Decidim::UserGroups::ManageableUserGroups.for(current_user).verified
+
+          select_options += current_user_groups.map { |g| [g.name, g.id] } if current_organization.user_groups_enabled? && current_user_groups.any?
+          unless form.object.author.is_a?(Organization) || select_options.pluck(1).include?(form.object.author.id)
+            select_options << [form.object.author.name, form.object.author.id]
+          end
+
+          form.select(name, select_options)
+        end
       end
     end
   end

data/app/models/decidim/blogs/post.rb

@@ -39,6 +39,10 @@ module Decidim
         participatory_space.try(:visible?) && component.try(:published?)
       end
 
+      def self.log_presenter_class_for(_log)
+        Decidim::Blogs::AdminLog::PostPresenter
+      end
+
       # Public: Overrides the `comments_have_alignment?` Commentable concern method.
       def comments_have_alignment?
         true
@@ -55,7 +59,11 @@ module Decidim
       end
 
       def official?
-        author.nil?
+        author.is_a?(Decidim::Organization)
+      end
+
+      def user_group?
+        author.is_a?(Decidim::UserGroup)
       end
 
       def users_to_notify_on_comment_created

data/app/presenters/decidim/blogs/admin_log/post_presenter.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Blogs
+    module AdminLog
+      # This class holds the logic to present a `Decidim::Blogs::Post`
+      # for the `AdminLog` log.
+      #
+      # Usage should be automatic and you shouldn't need to call this class
+      # directly, but here's an example:
+      #
+      #    action_log = Decidim::ActionLog.last
+      #    view_helpers # => this comes from the views
+      #    PostPresenter.new(action_log, view_helpers).present
+      class PostPresenter < Decidim::Log::BasePresenter
+        private
+
+        def action_string
+          case action
+          when "create", "delete", "update"
+            "decidim.blogs.admin_log.post.#{action}"
+          else
+            super
+          end
+        end
+
+        def diff_fields_mapping
+          {
+            title: :i18n,
+            body: :i18n
+          }
+        end
+      end
+    end
+  end
+end

data/app/presenters/decidim/blogs/official_author_presenter.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Blogs
+    #
+    # A dummy presenter to abstract out the author of an official post.
+    #
+    class OfficialAuthorPresenter < Decidim::OfficialAuthorPresenter
+      def name
+        I18n.t("decidim.blogs.models.post.fields.official_blog_post")
+      end
+    end
+  end
+end

data/app/presenters/decidim/blogs/post_presenter.rb

@@ -7,7 +7,13 @@ module Decidim
     #
     class PostPresenter < SimpleDelegator
       def author
-        @author ||= Decidim::UserPresenter.new(super)
+        @author ||= if official?
+                      Decidim::Blogs::OfficialAuthorPresenter.new
+                    elsif user_group?
+                      Decidim::UserGroupPresenter.new(super)
+                    else
+                      Decidim::UserPresenter.new(super)
+                    end
       end
     end
   end

data/app/views/decidim/blogs/admin/posts/_form.html.erb

@@ -3,11 +3,11 @@
     <h2 class="card-title"><%= title %></h2>
   </div>
   <div class="card-section">
-    <% if current_organization.user_groups_enabled? && Decidim::UserGroups::ManageableUserGroups.for(current_user).verified.any? %>
+
       <div class="field">
-        <%= user_group_select_field form, :user_group_id, label: t("decidim.blogs.actions.user_group_id") %>
+        <%= post_author_select_field form, :decidim_author_id, label: t("decidim.blogs.actions.author_id") %>
       </div>
-    <% end %>
+
     <div class="row column">
       <%= form.translated :text_field, :title, autofocus: true %>
     </div>

data/app/views/decidim/blogs/posts/_posts.html.erb

@@ -12,7 +12,8 @@
           </h2>
           <% end %>
           <div class="card__author">
-            <%= cell "decidim/author", present(post.author), from: post, has_actions: true %>
+            <% post_presenter = Decidim::Blogs::PostPresenter.new(post) %>
+            <%= cell "decidim/author", post_presenter.author, from: post, has_actions: true %>
           </div>
         </div>
         <%= decidim_sanitize_editor post_description(post) %>

data/app/views/decidim/blogs/posts/show.html.erb

@@ -23,7 +23,9 @@
     <% end %>
   </div>
   <h2 class="heading2"><%= translated_attribute post.title %></h2>
-  <%= cell "decidim/author", present(post.author), from: post %>
+
+  <% post_presenter = Decidim::Blogs::PostPresenter.new(post) %>
+  <%= cell "decidim/author", post_presenter.author, from: post %>
 </div>
 <div class="row">
   <% if show_endorsements_card? %>

data/config/locales/ar.yml

@@ -1,3 +1,4 @@
+---
 ar:
   activemodel:
     models:
@@ -14,12 +15,12 @@ ar:
   decidim:
     blogs:
       actions:
+        author_id: إنشاء مشاركة كـ
         confirm_destroy: هل أنت متأكد أنك تريد حذف هذه المشاركة؟
         destroy: حذف
         edit: تعديل
         new: منشور جديد
         title: أفعال
-        user_group_id: إنشاء مشاركة كـ
       admin:
         models:
           components:

data/config/locales/bg.yml

@@ -1,3 +1,4 @@
+---
 bg:
   activemodel:
     models:

data/config/locales/ca.yml

@@ -1,3 +1,4 @@
+---
 ca:
   activemodel:
     models:
@@ -10,12 +11,12 @@ ca:
   decidim:
     blogs:
       actions:
+        author_id: Crear publicació com
         confirm_destroy: Estàs segura que vols eliminar aquesta publicació?
         destroy: Suprimeix
         edit: Edita
         new: Nova publicació
         title: Accions
-        user_group_id: Crear publicació com
       admin:
         models:
           components:
@@ -39,6 +40,11 @@ ca:
           update:
             invalid: S'ha produït un error en guardar la publicació.
             success: S'ha desat la publicació correctament
+      admin_log:
+        post:
+          create: "%{user_name} ha creat el post %{resource_name} al blog de %{space_name}"
+          delete: "%{user_name} ha eliminat el post %{resource_name} del blog de %{space_name}"
+          update: "%{user_name} ha actualitzat el post %{resource_name} en el blog de %{space_name}"
       last_activity:
         new_post_at_html: "<span>Nova publicació a %{link}</span>"
       models:
@@ -47,6 +53,7 @@ ca:
             author: Autora
             body: Cos
             created_at: Creat el
+            official_blog_post: Post oficial
             title: títol
       posts:
         show:

data/config/locales/cs.yml

@@ -1,3 +1,4 @@
+---
 cs:
   activemodel:
     models:
@@ -12,12 +13,12 @@ cs:
   decidim:
     blogs:
       actions:
+        author_id: Vytvořit příspěvek jako
         confirm_destroy: Opravdu chcete tento příspěvek smazat?
         destroy: Smazat
         edit: Upravit
         new: Nový příspěvek
         title: Akce
-        user_group_id: Vytvořit příspěvek jako
       admin:
         models:
           components:
@@ -41,6 +42,11 @@ cs:
           update:
             invalid: Při ukládání příspěvku došlo k chybám.
             success: Příspěvek byl úspěšně uložen
+      admin_log:
+        post:
+          create: "%{user_name} vytvořil příspěvek %{resource_name} v %{space_name}"
+          delete: "%{user_name} odstranil příspěvek %{resource_name} z %{space_name}"
+          update: "%{user_name} aktualizoval příspěvek %{resource_name} v %{space_name}"
       last_activity:
         new_post_at_html: "<span>Nový příspěvek v %{link}</span>"
       models:
@@ -49,6 +55,7 @@ cs:
             author: Autor
             body: Tělo
             created_at: Vytvořeno v
+            official_blog_post: Oficiální příspěvek
             title: titulek
       posts:
         show:

data/config/locales/de.yml

@@ -1,3 +1,4 @@
+---
 de:
   activemodel:
     models:
@@ -10,12 +11,12 @@ de:
   decidim:
     blogs:
       actions:
+        author_id: Beitrag erstellen als
         confirm_destroy: Möchten Sie diesen Beitrag wirklich löschen?
         destroy: Löschen
         edit: Bearbeiten
         new: Neuer Beitrag
         title: Aktionen
-        user_group_id: Beitrag erstellen als
       admin:
         models:
           components:
@@ -39,6 +40,11 @@ de:
           update:
             invalid: Beim Speichern des Posts sind Fehler aufgetreten.
             success: Beitrag wurde erfolgreich gespeichert
+      admin_log:
+        post:
+          create: "%{user_name} hat den Blogpost %{resource_name} in %{space_name} erstellt"
+          delete: "%{user_name} hat den Blogpost %{resource_name} in %{space_name} gelöscht"
+          update: "%{user_name} hat den Blogpost %{resource_name} in %{space_name} aktualisiert"
       last_activity:
         new_post_at_html: "<span>Neuer Beitrag bei %{link}</span>"
       models:
@@ -47,6 +53,7 @@ de:
             author: Autor
             body: Haupttext
             created_at: Hergestellt in
+            official_blog_post: Offizieller Post
             title: Titel
       posts:
         show:

data/config/locales/el.yml

@@ -1,3 +1,4 @@
+---
 el:
   activemodel:
     models:
@@ -10,12 +11,12 @@ el:
   decidim:
     blogs:
       actions:
+        author_id: Δημιουργία ανάρτησης ως
         confirm_destroy: Είστε βέβαιοι ότι θέλετε να διαγράψετε αυτήν τη δημοσίευση;
         destroy: Διαγραφή
         edit: Επεξεργασία
         new: Νέα δημοσίευση
         title: Ενέργειες
-        user_group_id: Δημιουργία ανάρτησης ως
       admin:
         models:
           components:

data/config/locales/en.yml

@@ -11,12 +11,12 @@ en:
   decidim:
     blogs:
       actions:
+        author_id: Create post as
         confirm_destroy: Are you sure you want to delete this post?
         destroy: Delete
         edit: Edit
         new: New post
         title: Actions
-        user_group_id: Create post as
       admin:
         models:
           components:
@@ -40,6 +40,11 @@ en:
           update:
             invalid: There was a problem saving the post.
             success: Post successfully saved
+      admin_log:
+        post:
+          create: "%{user_name} created the %{resource_name} blog post in %{space_name}"
+          delete: "%{user_name} deleted the %{resource_name} blog post from %{space_name}"
+          update: "%{user_name} updated the %{resource_name} blog post in %{space_name}"
       last_activity:
         new_post_at_html: "<span>New post at %{link}</span>"
       models:
@@ -48,6 +53,7 @@ en:
             author: Author
             body: Body
             created_at: Created at
+            official_blog_post: Official post
             title: title
       posts:
         show:

data/config/locales/es-MX.yml

@@ -1,3 +1,4 @@
+---
 es-MX:
   activemodel:
     models:
@@ -10,12 +11,12 @@ es-MX:
   decidim:
     blogs:
       actions:
+        author_id: Crear publicación como
         confirm_destroy: '¿Seguro que quieres eliminar esta publicación?'
         destroy: Borrar
         edit: Editar
         new: Nueva publicación
         title: Comportamiento
-        user_group_id: Crear publicación como
       admin:
         models:
           components:
@@ -39,6 +40,11 @@ es-MX:
           update:
             invalid: Ha habido errores al guardar la publicación.
             success: Publicación guardada con éxito
+      admin_log:
+        post:
+          create: "%{user_name} creó el post %{resource_name} en el blog de %{space_name}"
+          delete: "%{user_name} ha eliminado el post %{resource_name} en el blog de %{space_name}"
+          update: "%{user_name} ha actualizado el post %{resource_name} en el blog de %{space_name}"
       last_activity:
         new_post_at_html: "<span>Nueva publicación en %{link}</span>"
       models:
@@ -47,6 +53,7 @@ es-MX:
             author: Autoría
             body: Cuerpo
             created_at: Creado en
+            official_blog_post: Post oficial
             title: título
       posts:
         show:

data/config/locales/es-PY.yml

@@ -1,3 +1,4 @@
+---
 es-PY:
   activemodel:
     models:
@@ -10,12 +11,12 @@ es-PY:
   decidim:
     blogs:
       actions:
+        author_id: Crear publicación como
         confirm_destroy: '¿Seguro que quieres eliminar esta publicación?'
         destroy: Borrar
         edit: Editar
         new: Nueva publicación
         title: Comportamiento
-        user_group_id: Crear publicación como
       admin:
         models:
           components:
@@ -39,6 +40,11 @@ es-PY:
           update:
             invalid: Ha habido errores al guardar la publicación.
             success: Publicación guardada con éxito
+      admin_log:
+        post:
+          create: "%{user_name} creó el post %{resource_name} en el blog de %{space_name}"
+          delete: "%{user_name} ha eliminado el post %{resource_name} en el blog de %{space_name}"
+          update: "%{user_name} ha actualizado el post %{resource_name} en el blog de %{space_name}"
       last_activity:
         new_post_at_html: "<span>Nueva publicación en %{link}</span>"
       models:
@@ -47,6 +53,7 @@ es-PY:
             author: Autoría
             body: Cuerpo
             created_at: Creado en
+            official_blog_post: Post oficial
             title: título
       posts:
         show:

data/config/locales/es.yml

@@ -1,3 +1,4 @@
+---
 es:
   activemodel:
     models:
@@ -10,12 +11,12 @@ es:
   decidim:
     blogs:
       actions:
+        author_id: Crear publicación como
         confirm_destroy: '¿Estás segura que quieres eliminar esta publicación?'
         destroy: Borrar
         edit: Editar
         new: Nueva publicación
         title: Acciones
-        user_group_id: Crear publicación como
       admin:
         models:
           components:
@@ -39,6 +40,11 @@ es:
           update:
             invalid: Se ha producido un error al guardar la publicación.
             success: Se ha guardado la publicación correctamente
+      admin_log:
+        post:
+          create: "%{user_name} creó el post %{resource_name} en el blog de %{space_name}"
+          delete: "%{user_name} ha eliminado el post %{resource_name} en el blog de %{space_name}"
+          update: "%{user_name} ha actualizado el post %{resource_name} en el blog de %{space_name}"
       last_activity:
         new_post_at_html: "<span>Nueva publicación en %{link}</span>"
       models:
@@ -47,6 +53,7 @@ es:
             author: Autora
             body: Cuerpo
             created_at: Creado en
+            official_blog_post: Post oficial
             title: título
       posts:
         show:

data/config/locales/eu.yml

@@ -1,3 +1,4 @@
+---
 eu:
   activemodel:
     models:
@@ -10,12 +11,12 @@ eu:
   decidim:
     blogs:
       actions:
+        author_id: Sortu argitalpena hau bezala
         confirm_destroy: Ziur mezua ezabatu nahi duzula?
         destroy: Ezabatu
         edit: Editatu
         new: Post berria
         title: Ekintzak
-        user_group_id: Sortu argitalpena hau bezala
       admin:
         models:
           components:

data/config/locales/fi-plain.yml

@@ -1,3 +1,4 @@
+---
 fi-pl:
   activemodel:
     models:
@@ -10,12 +11,12 @@ fi-pl:
   decidim:
     blogs:
       actions:
+        author_id: Luo artikkeli käyttäjänä
         confirm_destroy: Haluatko varmasti poistaa tämän artikkelin?
         destroy: Poista
         edit: Muokkaa
         new: Uusi artikkeli
         title: Toiminnot
-        user_group_id: Luo artikkeli käyttäjänä
       admin:
         models:
           components:
@@ -39,6 +40,11 @@ fi-pl:
           update:
             invalid: Artikkelin tallennuksessa tapahtui virheitä.
             success: Artikkeli tallennettu onnistuneesti
+      admin_log:
+        post:
+          create: "%{user_name} loi blogikirjoituksen %{resource_name} tilassa %{space_name}"
+          delete: "%{user_name} poisti blogikirjoituksen %{resource_name} tilasta %{space_name}"
+          update: "%{user_name} päivitti blogikirjoitusta %{resource_name} tilassa %{space_name}"
       last_activity:
         new_post_at_html: "<span>Uusi artikkeli osoitteessa %{link}</span>"
       models:
@@ -47,6 +53,7 @@ fi-pl:
             author: Kirjoittaja
             body: Leipäteksti
             created_at: Luotu
+            official_blog_post: Virallinen artikkeli
             title: otsikko
       posts:
         show: