decidim-comments 0.19.0 → 0.22.0

data/app/models/decidim/comments/comment.rb

@@ -28,17 +28,29 @@ module Decidim
       has_many :down_votes, -> { where(weight: -1) }, foreign_key: "decidim_comment_id", class_name: "CommentVote", dependent: :destroy
 
       validates :body, presence: true
-      validates :depth, numericality: { greater_than_or_equal_to: 0 }
+      validates :depth, numericality: { only_integer: true, greater_than_or_equal_to: 0, less_than_or_equal_to: MAX_DEPTH }
       validates :alignment, inclusion: { in: [0, 1, -1] }
 
       validates :body, length: { maximum: 1000 }
 
       validate :commentable_can_have_comments
 
-      before_save :compute_depth
+      before_validation :compute_depth
 
       delegate :organization, to: :commentable
 
+      def self.positive
+        where(alignment: 1)
+      end
+
+      def self.neutral
+        where(alignment: 0)
+      end
+
+      def self.negative
+        where(alignment: -1)
+      end
+
       def participatory_space
         return root_commentable if root_commentable.is_a?(Decidim::Participable)
 
@@ -92,7 +104,7 @@ module Decidim
 
       # Public: Returns the comment message ready to display (it is expected to include HTML)
       def formatted_body
-        @formatted_body ||= Decidim::ContentProcessor.render(sanitized_body)
+        @formatted_body ||= Decidim::ContentProcessor.render(sanitized_body, "div")
       end
 
       def self.export_serializer
@@ -125,9 +137,22 @@ module Decidim
         self.depth = commentable.depth + 1 if commentable.respond_to?(:depth)
       end
 
-      # Private: Returns the comment body sanitized, stripping HTML tags
+      # Private: Returns the comment body sanitized, sanitizing HTML tags
       def sanitized_body
-        Rails::Html::Sanitizer.full_sanitizer.new.sanitize(body)
+        Rails::Html::WhiteListSanitizer.new.sanitize(
+          render_markdown(body),
+          scrubber: Decidim::Comments::UserInputScrubber.new
+        ).try(:html_safe)
+      end
+
+      # Private: Initializes the Markdown parser
+      def markdown
+        @markdown ||= Decidim::Comments::Markdown.new
+      end
+
+      # Private: converts the string from markdown to html
+      def render_markdown(string)
+        markdown.render(string)
       end
     end
   end

data/app/queries/decidim/comments/metrics/comments_metric_manage.rb

@@ -9,9 +9,6 @@ module Decidim
         end
 
         def save
-          return @registry if @registry
-
-          @registry = []
           query.each do |key, results|
             cumulative_value = results[:cumulative]
             next if cumulative_value.zero?
@@ -23,10 +20,8 @@ module Decidim
                                                            organization: @organization, decidim_category_id: category_id,
                                                            related_object_type: related_object_type, related_object_id: related_object_id)
             record.assign_attributes(cumulative: cumulative_value, quantity: quantity_value)
-            @registry << record
+            record.save!
           end
-          @registry.each(&:save!)
-          @registry
         end
 
         private

data/app/queries/decidim/comments/sorted_comments.rb

@@ -30,8 +30,7 @@ module Decidim
       # loads comments replies. It uses Comment's MAX_DEPTH to load a maximum
       # level of nested replies.
       def query
-        scope = Comment
-                .where(commentable: commentable)
+        scope = base_scope
                 .not_hidden
                 .includes(:author, :user_group, :up_votes, :down_votes)
 
@@ -53,6 +52,13 @@ module Decidim
 
       private
 
+      def base_scope
+        id = @options[:id]
+        return Comment.where(root_commentable: commentable, id: id) if id.present?
+
+        Comment.where(commentable: commentable)
+      end
+
       def order_by_older(scope)
         scope.order(created_at: :asc)
       end

data/app/scrubbers/decidim/comments/user_input_scrubber.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Comments
+    # Use this class as a scrubber to sanitize user input.
+    # https://stackoverflow.com/a/35073814/2110884.
+    class UserInputScrubber < Rails::Html::PermitScrubber
+      def initialize
+        super
+        self.tags = custom_allowed_tags
+      end
+
+      private
+
+      def custom_allowed_tags
+        %w(p blockquote)
+      end
+    end
+  end
+end

data/app/services/decidim/comments/new_comment_notification_creator.rb

@@ -17,9 +17,12 @@ module Decidim
       # comment - the Comment from which to generate notifications.
       # mentioned_users - An ActiveRecord::Relation of the users that have been
       #   mentioned
-      def initialize(comment, mentioned_users)
+      # mentioned_groups - And ActiveRecord::Relation of the user_groups that have
+      #   been mentioned
+      def initialize(comment, mentioned_users, mentioned_groups = nil)
         @comment = comment
         @mentioned_users = mentioned_users
+        @mentioned_groups = mentioned_groups
         @already_notified_users = [comment.author]
       end
 
@@ -28,14 +31,16 @@ module Decidim
       # Returns nothing.
       def create
         notify_mentioned_users
+        notify_mentioned_groups
         notify_parent_comment_author
         notify_author_followers
+        notify_user_group_followers
         notify_commentable_recipients
       end
 
       private
 
-      attr_reader :comment, :mentioned_users, :already_notified_users
+      attr_reader :comment, :mentioned_users, :mentioned_groups, :already_notified_users
 
       def notify_mentioned_users
         affected_users = mentioned_users - already_notified_users
@@ -44,6 +49,17 @@ module Decidim
         notify(:user_mentioned, affected_users: affected_users)
       end
 
+      def notify_mentioned_groups
+        return unless mentioned_groups
+
+        mentioned_groups.each do |group|
+          affected_users = group.accepted_users - already_notified_users
+          @already_notified_users += affected_users
+
+          notify(:user_group_mentioned, affected_users: affected_users, extra: { group: group })
+        end
+      end
+
       # Notifies the author of a comment that their comment has been replied.
       # Only applies if the comment is a reply.
       def notify_parent_comment_author
@@ -62,6 +78,15 @@ module Decidim
         notify(:comment_by_followed_user, followers: followers)
       end
 
+      def notify_user_group_followers
+        return if comment.user_group.blank?
+
+        followers = comment.user_group.followers - already_notified_users
+        @already_notified_users += followers
+
+        notify(:comment_by_followed_user_group, followers: followers)
+      end
+
       # Notifies the users the `comment.commentable` resource implements as necessary.
       def notify_commentable_recipients
         followers = comment.commentable.users_to_notify_on_comment_created - already_notified_users
@@ -90,7 +115,7 @@ module Decidim
           extra: {
             comment_id: comment.id
           }
-        }.merge(users)
+        }.deep_merge(users)
 
         Decidim::EventsManager.publish(data)
       end

data/app/types/decidim/comments/commentable_interface.rb

@@ -29,9 +29,10 @@ module Decidim
         type !types[!CommentType]
 
         argument :orderBy, types.String, "Order the comments"
+        argument :singleCommentId, types.String, "ID of the single comment to look at"
 
         resolve lambda { |obj, args, _ctx|
-          SortedComments.for(obj, order_by: args[:orderBy])
+          SortedComments.for(obj, order_by: args[:orderBy], id: args[:singleCommentId])
         }
       end
 

data/app/types/decidim/comments/commentable_mutation_type.rb

@@ -16,9 +16,9 @@ module Decidim
         argument :userGroupId, types.ID, "The comment's user group id. Replaces the author."
 
         resolve lambda { |obj, args, ctx|
-          params = { "comment" => { "body" => args[:body], "alignment" => args[:alignment], "user_group_id" => args[:userGroupId] } }
+          params = { "comment" => { "body" => args[:body], "alignment" => args[:alignment], "user_group_id" => args[:userGroupId], "commentable" => obj } }
           form = Decidim::Comments::CommentForm.from_params(params).with_context(current_organization: ctx[:current_organization])
-          Decidim::Comments::CreateComment.call(form, ctx[:current_user], obj) do
+          Decidim::Comments::CreateComment.call(form, ctx[:current_user]) do
             on(:ok) do |comment|
               return comment
             end

data/config/locales/ar.yml

@@ -4,6 +4,7 @@ ar:
       decidim/comments/comment_by_followed_user_event: تعليق
       decidim/comments/comment_created_event: تعليق
       decidim/comments/reply_created_event: تعليق الرد
+      decidim/comments/user_group_mentioned_event: أشير
       decidim/comments/user_mentioned_event: أشير
   activerecord:
     models:
@@ -50,6 +51,7 @@ ar:
           against: ضد
           in_favor: لصالح
         deleted_user: مشارك محذوف
+        hide_replies: إخفاء الردود
         reply: الرد
         report:
           action: أبلغ عن
@@ -61,7 +63,8 @@ ar:
             does_not_belong: يحتوي على نشاط غير قانوني أو تهديدات انتحارية أو معلومات شخصية أو أي شيء آخر تعتقد أنه لا ينتمي إلى %{organization_name}.
             offensive: يحتوي على العنصرية والتمييز الجنسي والتشهير والهجمات الشخصية والتهديدات بالقتل أو طلبات الانتحار أو أي شكل من أشكال خطاب الكراهية.
             spam: يحتوي على clickbait أو الإعلان أو الخدع أو روبوت البرامج النصية.
-          title: الإبلاغ عن مشكلة
+        show_replies: اظهر الردود الـ %{replies_count}
+        single_comment_link_title: الحصول على رابط نحو تعليق واحد
       comment_order_selector:
         order:
           best_rated: أفضل تصنيف
@@ -74,7 +77,10 @@ ar:
       comments:
         blocked_comments_for_user_warning: لا يمكنك التعليق في هذه اللحظة ، ولكن يمكنك قراءة التعليقات السابقة.
         blocked_comments_warning: التعليقات معطلة في هذا الوقت ، لكن يمكنك قراءة التعليقات السابقة.
+        comment_details_title: تفاصيل التعليق
         loading: جارٍ تحميل التعليقات ...
+        single_comment_warning: يمكنك الإطلاع على التعليقات المتبقية الأخرى <a href="%{url}">هنا</a>.
+        single_comment_warning_title: إنك ترى تعليقا واحدا
         title: "%{count} تعليقات"
     events:
       comments:
@@ -83,6 +89,9 @@ ar:
           email_outro: لقد تلقيت هذا الإشعار لأنك تتابع %{author_name}. يمكنك إلغاء تتبع هذا المستخدم من صفحة ملفه الشخصي.
           email_subject: هناك تعليق جديد مِن %{author_name} على %{resource_title}
           notification_title: هناك تعليق جديد مِن <a href="%{author_path}">%{author_name} %{author_nickname}</a> على <a href="%{resource_path}">%{resource_title}</a>.
+        comment_by_followed_user_group:
+          email_subject: هناك تعليق جديد مِن %{author_name} على %{resource_title}
+          notification_title: هناك تعليق جديد مِن <a href="%{author_path}">%{author_name} %{author_nickname}</a> على <a href="%{resource_path}">%{resource_title}</a>.
         comment_created:
           email_intro: "%{resource_title} تم التعليق. يمكنك قراءة التعليق في هذه الصفحة:"
           email_outro: لقد تلقيت هذا الإشعار لأنك تتابع "%{resource_title}" أو مؤلفه. يمكنك إلغاء تتبعه من الرابط السابق.

data/config/locales/bg-BG.yml

@@ -0,0 +1,6 @@
+bg:
+  activemodel:
+    models:
+      decidim/comments/comment_by_followed_user_event: Коментар
+      decidim/comments/comment_created_event: Коментар
+      decidim/comments/reply_created_event: Отговор на коментар

data/config/locales/ca.yml

@@ -4,6 +4,7 @@ ca:
       decidim/comments/comment_by_followed_user_event: Comentari
       decidim/comments/comment_created_event: Comentari
       decidim/comments/reply_created_event: Resposta al comentari
+      decidim/comments/user_group_mentioned_event: Mencionar
       decidim/comments/user_mentioned_event: Mencionar
   activerecord:
     models:
@@ -16,6 +17,7 @@ ca:
   decidim:
     comments:
       comments: Comentaris
+      comments_count: Número de comentaris
       last_activity:
         new_comment_at_html: "<span>Nou comentari a %{link}</span>"
       votes:
@@ -42,6 +44,7 @@ ca:
           against: En contra
           in_favor: A favor
         deleted_user: Participant eliminada
+        hide_replies: Oculta les respostes
         reply: Respondre
         report:
           action: Denúncia
@@ -53,7 +56,9 @@ ca:
             does_not_belong: Conté activitat il·legal, amenaces de suïcidi, informació personal, o qualsevol altra cosa que creguis que no pertany a %{organization_name}.
             offensive: Conté racisme, sexisme, insults, atacs personals, amenaces de mort, peticions de suïcidi o qualsevol forma de discurs d'odi.
             spam: Conté "clickbait", publicitat o estafes.
-          title: Denúncia un problema
+          title: Notificar contingut inapropiat
+        show_replies: Mostra %{replies_count} respostes
+        single_comment_link_title: Aconsegueix l'enllaç al comentari
       comment_order_selector:
         order:
           best_rated: Més ben valorats
@@ -66,8 +71,15 @@ ca:
       comments:
         blocked_comments_for_user_warning: No pots fer comentaris en aquest moment, però pots llegir els anteriors.
         blocked_comments_warning: Els comentaris estan desactivats en aquest moment, però pots llegir els anteriors.
+        comment_details_title: Detalls del comentari
         loading: Carregant els comentaris ...
+        single_comment_warning: Pots revisar la resta de comentaris <a href="%{url}">aquí</a>.
+        single_comment_warning_title: Estàs veient un sol comentari
         title: "%{count} comentaris"
+      down_vote_button:
+        text: No estic d'acord amb aquest comentari
+      up_vote_button:
+        text: Estic d'acord amb aquest comentari
     events:
       comments:
         comment_by_followed_user:
@@ -75,6 +87,11 @@ ca:
           email_outro: Has rebut aquesta notificació perquè estàs seguint %{author_name}. Pots deixar de seguir a aquesta participant des de la seva pàgina de perfil.
           email_subject: Hi ha un nou comentari de %{author_name} en %{resource_title}
           notification_title: Hi ha un nou comentari per <a href="%{author_path}">%{author_name} %{author_nickname}</a> a <a href="%{resource_path}">%{resource_title}</a>.
+        comment_by_followed_user_group:
+          email_intro: '%{author_name} ha deixat un comentari a%{resource_title}. Pots llegir-lo en aquesta pàgina:'
+          email_outro: Reps aquesta notificació perquè segueixes a %{author_name}. Pots deixar de seguir aquest grup des de la seva pàgina de perfil.
+          email_subject: Hi ha un nou comentari de %{author_name} en %{resource_title}
+          notification_title: Hi ha un nou comentari per <a href="%{author_path}">%{author_name} %{author_nickname}</a> a <a href="%{resource_path}">%{resource_title}</a>.
         comment_created:
           email_intro: "S'ha comentat %{resource_title}. Pots llegir el comentari d'aquesta pàgina:"
           email_outro: Has rebut aquesta notificació perquè estàs seguint "%{resource_title}" o la seva autora. Pots deixar de seguir-la des de l'enllaç anterior.
@@ -85,6 +102,11 @@ ca:
           email_outro: Has rebut aquesta notificació perquè s'ha respost el teu comentari.
           email_subject: "%{author_name} ha respost el teu comentari a %{resource_title}"
           notification_title: <a href="%{author_path}">%{author_name} %{author_nickname}</a> ha respost el teu comentari a <a href="%{resource_path}">%{resource_title}</a>
+        user_group_mentioned:
+          email_intro: Un grup al qual pertanys ha estat esmentat
+          email_outro: Has rebut aquesta notificació perquè formes part del grup %{group_name} que ha estat esmentat a %{resource_title}.
+          email_subject: T'han esmentat a %{resource_title} com a membre de %{group_name}
+          notification_title: <a href="%{author_path}">%{author_name} %{author_nickname}</a> t'ha esmentat com a membre de <a href="%{group_path}">%{group_name} %{group_nickname}</a> a <a href="%{resource_path}">%{resource_title}</a>
         user_mentioned:
           email_intro: Has estat esmentada
           email_outro: Has rebut aquesta notificació perquè has estat esmentada a %{resource_title}.

data/config/locales/cs.yml

@@ -4,7 +4,8 @@ cs:
       decidim/comments/comment_by_followed_user_event: Komentář
       decidim/comments/comment_created_event: Komentář
       decidim/comments/reply_created_event: Odpověď na komentář
-      decidim/comments/user_mentioned_event: Zmínit se
+      decidim/comments/user_group_mentioned_event: Zmínka
+      decidim/comments/user_mentioned_event: Zmínka
   activerecord:
     models:
       decidim/comments/comment:
@@ -13,65 +14,76 @@ cs:
         many: Komentáře
         other: Komentáře
       decidim/comments/comment_vote:
-        one: Hlasování
+        one: Hlasovat
         few: Hlasy
         many: Hlasy
         other: Hlasy
   decidim:
     comments:
       comments: Komentáře
+      comments_count: Počet komentářů
       last_activity:
         new_comment_at_html: "<span>Nový komentář v %{link}</span>"
       votes:
         create:
-          error: Při hlasování se vyskytly chyby.
+          error: Při hlasování o komentáři došlo k chybě.
     components:
       add_comment_form:
-        account_message: <a href="%{sign_in_url}">Přihlaste se svým účtem</a> nebo <a href="%{sign_up_url}">přihlaste se</a> a přidejte svůj komentář.
+        account_message: <a href="%{sign_in_url}">Přihlaste se pomocí svého účtu</a> nebo se <a href="%{sign_up_url}">zaregistrujte</a> a přidejte svůj komentář.
         form:
           body:
             label: Komentář
             placeholder: Co si o tom myslíš?
           form_error: Text je povinný a nesmí být delší než %{length} znaků.
-          submit: Poslat
+          submit: Odeslat
           user_group_id:
             label: Komentovat jako
         opinion:
           neutral: Neutrální
         remaining_characters: "Zbývá %{count} znaků"
-        remaining_characters_1: "Zbývá %{count} znaků"
+        remaining_characters_1: "Zbývá %{count} znak"
         title: Přidejte svůj komentář
       comment:
         alignment:
           against: Proti
           in_favor: Ve prospěch
-        deleted_user: Vymazaný uživatel
+        deleted_user: Vymazaný účastník
+        hide_replies: Skrýt odpovědi
         reply: Odpověď
         report:
           action: Zpráva
-          already_reported: Tento obsah je již nahlášen a bude kontrolován administrátorem.
+          already_reported: Tento obsah je již nahlášen a bude přezkoumán administrátorem.
           close: Zavřít
           description: Je tento obsah nevhodný?
           details: Další komentáře
           reasons:
             does_not_belong: Obsahuje nezákonnou činnost, sebevražedné hrozby, osobní informace nebo něco jiného, o kterém si myslíte, že nepatří na %{organization_name}.
             offensive: Obsahuje rasismus, sexismus, podvody, osobní útoky, hrozby smrti, žádosti o sebevraždu nebo jakoukoli formu projevy nenávisti.
-            spam: Obsahuje clickbait, reklama, podvody nebo skripty.
-          title: Nahlásit problém
+            spam: Obsahuje clickbait, reklamu, podvody nebo škodlivé skripty.
+          title: Nahlásit nevhodný obsah
+        show_replies: Zobrazit %{replies_count} odpovědí
+        single_comment_link_title: Získat odkaz na jeden komentář
       comment_order_selector:
         order:
           best_rated: Nejlépe hodnocené
           most_discussed: Nejdiskutovanější
           older: Starší
-          recent: Nedávno
+          recent: Nedávné
         title: 'Seřadit podle:'
       comment_thread:
         title: Konverzace s číslem %{authorName}
       comments:
-        blocked_comments_for_user_warning: V tuto chvíli nemůžete komentovat, ale můžete si přečíst předchozí.
-        blocked_comments_warning: Komentáře jsou v tuto chvíli zakázány, ale můžete si přečíst předchozí.
+        blocked_comments_for_user_warning: Momentálně nemůžete komentovat, ale můžete si přečíst ty předchozí.
+        blocked_comments_warning: Komentáře jsou momentálně zakázány, ale můžete si přečíst ty předchozí.
+        comment_details_title: Podrobnosti komentáře
         loading: Načítání komentářů ...
+        single_comment_warning: Můžete zkontrolovat zbytek komentářů <a href="%{url}">zde</a>.
+        single_comment_warning_title: Vidíte jeden komentář
         title: "%{count} komentářů"
+      down_vote_button:
+        text: Nesouhlasím s tímto komentářem
+      up_vote_button:
+        text: Souhlasím s tímto komentářem
     events:
       comments:
         comment_by_followed_user:
@@ -79,6 +91,11 @@ cs:
           email_outro: Toto oznámení jste obdrželi, protože jste sledovali %{author_name}. Tento uživatel můžete zrušit ze své profilové stránky.
           email_subject: K dispozici je nový komentář %{author_name} v %{resource_title}
           notification_title: K dispozici je nový komentář <a href="%{author_path}">%{author_name} %{author_nickname}</a> v <a href="%{resource_path}">%{resource_title}</a>.
+        comment_by_followed_user_group:
+          email_intro: 'Skupina %{author_name} zanechala komentář v %{resource_title}. Můžete si jej přečíst na této stránce:'
+          email_outro: Obdrželi jste toto oznámení, protože sledujete %{author_name}. Můžete zrušit sledování této skupiny na její profilové stránce.
+          email_subject: K dispozici je nový komentář %{author_name} v %{resource_title}
+          notification_title: K dispozici je nový komentář <a href="%{author_path}">%{author_name} %{author_nickname}</a> v <a href="%{resource_path}">%{resource_title}</a>.
         comment_created:
           email_intro: "%{resource_title} bylo komentováno. Můžete si přečíst komentář na této stránce:"
           email_outro: Toto oznámení jste obdrželi, protože jste sledovali "%{resource_title}" nebo jeho autora. Můžete jej odhlásit od předchozího odkazu.
@@ -89,6 +106,11 @@ cs:
           email_outro: Toto oznámení jste obdrželi, protože váš komentář byl zodpovězen.
           email_subject: "%{author_name} odpověděl váš komentář v %{resource_title}"
           notification_title: <a href="%{author_path}">%{author_name} %{author_nickname}</a> odpověděl / a svůj komentář v <a href="%{resource_path}">%{resource_title}</a>
+        user_group_mentioned:
+          email_intro: Byla zmíněna skupina, do které patříte
+          email_outro: Obdrželi jste toto oznámení, protože jste členem skupiny %{group_name}, která byla zmíněna v %{resource_title}.
+          email_subject: Byl jste zmíněn v %{resource_title} jako člen %{group_name}
+          notification_title: Byli jste zmíněni v <a href="%{resource_path}">%{resource_title}</a> od <a href="%{author_path}">%{author_name} %{author_nickname}</a> jako člen <a href="%{group_path}">%{group_name} %{group_nickname}</a>
         user_mentioned:
           email_intro: Byl jste zmíněn
           email_outro: Toto oznámení jste obdrželi, protože jste byli uvedeni v %{resource_title}.