decidim-comments 0.26.4 → 0.27.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0afed8636ad281b3e5cb440eb37b2080d40cb4386a9561125d209887feadba1
-  data.tar.gz: 6fb8da27ed3e66447361539cdafa40123e2835152d450f5fc5602ea35b5a29d7
+  metadata.gz: 15e0e5dd8fbb2fc7171061fa36077e83f956f3ada42b41dfa32cd8e8f60cdd0a
+  data.tar.gz: 6cb4e67d56972f657983cc174806b8766d0916fb55b7ca783ee2e8a684859283
 SHA512:
-  metadata.gz: 3de3452f8cbaa4b35f624a9ea896a0a3d9c7a0788eadd77e7c7e053c37b3947f586b456dfef2c7bda7785eb395b7fe9233d919b9873e0523c6ae101b3ba91581
-  data.tar.gz: 5498dd2599639550040190fe261fec3e790a078be78deb46c699e77f60782bec7e3d366e57856ad3d8d7d374727a96eab8cc6454e618e5547fcfc3d82a6356f7
+  metadata.gz: a3a7482658e91430342214a3d668979c0450b1203c28725ab083c46773b6c39474f148a4489238c90187cda2407a81edbc1c7a1b9bd26944081ce36261f86e7e
+  data.tar.gz: ee42d7e4922a5cbc826a1a98f73b1d1056d4ce6879a398da32ad6962d9987323f7b37b2e1fc57111251baf2185301cbdc0e370621107778afd2fb47263dc285b

data/app/cells/decidim/comments/comment_cell.rb

@@ -186,11 +186,11 @@ module Decidim
       end
 
       def has_replies?
-        model.comment_threads.includes(:moderation).collect { |c| !c.deleted? && !c.hidden? }.any?
+        model.comment_threads.not_hidden.not_deleted.exists?
       end
 
       def has_replies_in_children?
-        has_replies? || model.comment_threads.includes(:moderation).collect { |t| t.comment_threads.includes(:moderation).collect { |c| !c.deleted? && !c.hidden? }.any? }.any?
+        model.descendants.where(decidim_commentable_type: "Decidim::Comments::Comment").not_hidden.not_deleted.exists?
       end
 
       # action_authorization_button expects current_component to be available

data/app/cells/decidim/comments/comments/comments_loading.erb

@@ -0,0 +1,3 @@
+<div class="callout primary loading-comments">
+  <p><%= t("decidim.components.comments.loading") %></p>
+</div>

data/app/cells/decidim/comments/comments/show.erb

@@ -16,6 +16,7 @@
       <%= single_comment_warning %>
       <%= blocked_comments_warning %>
       <div class="comment-threads">
+        <%= comments_loading %>
         <% comments.each do |comment| %>
           <%= cell("decidim/comments/comment_thread", comment, order: order) %>
         <% end %>

data/app/cells/decidim/comments/comments_cell.rb

@@ -22,6 +22,12 @@ module Decidim
         render :single_comment_warning
       end
 
+      def comments_loading
+        return if single_comment?
+
+        render :comments_loading
+      end
+
       def blocked_comments_warning
         return unless comments_blocked?
         return unless user_comments_blocked?
@@ -43,11 +49,7 @@ module Decidim
       end
 
       def comments
-        if single_comment?
-          [single_comment]
-        else
-          SortedComments.for(model, order_by: order)
-        end
+        single_comment? ? [single_comment] : []
       end
 
       def comments_count
@@ -97,17 +99,10 @@ module Decidim
           commentableGid: model.to_signed_global_id.to_s,
           commentsUrl: decidim_comments.comments_path,
           rootDepth: root_depth,
-          lastCommentId: last_comment_id,
           order: order
         }
       end
 
-      def last_comment_id
-        Decidim::Comments::Comment.where(
-          root_commentable: model
-        ).order(:id).pluck(:id).last
-      end
-
       def single_comment?
         single_comment.present?
       end

data/app/cells/decidim/comments/edit_comment_modal_form_cell.rb

@@ -2,7 +2,7 @@
 
 module Decidim
   module Comments
-    # A cell to display a form for editing a comment.
+    # A cell to display a form for edditing a comment.
     class EditCommentModalFormCell < Decidim::ViewModel
       delegate :current_user, :user_signed_in?, to: :controller
       alias comment model

data/app/commands/decidim/comments/create_comment.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Comments
     # A command with all the business logic to create a new comment
-    class CreateComment < Rectify::Command
+    class CreateComment < Decidim::Command
       # Public: Initializes the command.
       #
       # form - A form object with the params.

data/app/commands/decidim/comments/delete_comment.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Comments
     # A command with all the business logic to delete a comment
-    class DeleteComment < Rectify::Command
+    class DeleteComment < Decidim::Command
       # Public: Initializes the command.
       #
       # comment - The comment to delete.

data/app/commands/decidim/comments/update_comment.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Comments
     # A command with all the business logic to update an existing comment
-    class UpdateComment < Rectify::Command
+    class UpdateComment < Decidim::Command
       # Public: Initializes the command.
       #
       # comment - Decidim::Comments::Comment
@@ -48,7 +48,14 @@ module Decidim
           edit: true
         )
 
+        mentioned_users = parsed.metadata[:user].users
+        mentioned_groups = parsed.metadata[:user_group].groups
         CommentCreation.publish(@comment, parsed.metadata)
+        send_notifications(mentioned_users, mentioned_groups)
+      end
+
+      def send_notifications(mentioned_users, mentioned_groups)
+        NewCommentNotificationCreator.new(comment, mentioned_users, mentioned_groups).create
       end
     end
   end

data/app/commands/decidim/comments/vote_comment.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Comments
     # A command with all the business logic to upvote a comment
-    class VoteComment < Rectify::Command
+    class VoteComment < Decidim::Command
       # Public: Initializes the command.
       #
       # comment - A comment

data/app/controllers/decidim/comments/comments_controller.rb

@@ -23,6 +23,12 @@ module Decidim
           order_by: order,
           after: params.fetch(:after, 0).to_i
         )
+        @comments = @comments.reject do |comment|
+          next if comment.depth < 1
+          next if !comment.deleted? && !comment.hidden?
+
+          comment.commentable.descendants.where(decidim_commentable_type: "Decidim::Comments::Comment").not_hidden.not_deleted.blank?
+        end
         @comments_count = commentable.comments_count
 
         respond_to do |format|
@@ -136,14 +142,12 @@ module Decidim
 
       def handle_success(comment)
         @comment = comment
-        @comments_count = begin
-          case commentable
-          when Decidim::Comments::Comment
-            commentable.root_commentable.comments_count
-          else
-            commentable.comments_count
-          end
-        end
+        @comments_count = case commentable
+                          when Decidim::Comments::Comment
+                            commentable.root_commentable.comments_count
+                          else
+                            commentable.comments_count
+                          end
       end
 
       def commentable_gid
@@ -172,7 +176,7 @@ module Decidim
       end
 
       def commentable_path
-        return commentable.polymorphic_resource_path({}) if commentable&.respond_to?(:polymorphic_resource_path)
+        return commentable.polymorphic_resource_path({}) if commentable.respond_to?(:polymorphic_resource_path)
 
         resource_locator(commentable).path
       end

data/app/helpers/decidim/comments/comment_cells_helper.rb

@@ -14,7 +14,7 @@ module Decidim
       end
 
       def resource_link_path
-        return root_commentable.polymorphic_resource_path(url_params) if root_commentable&.respond_to?(:polymorphic_resource_path)
+        return root_commentable.polymorphic_resource_path(url_params) if root_commentable.respond_to?(:polymorphic_resource_path)
 
         resource_locator(root_commentable).path(url_params)
       end

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

@@ -10,12 +10,13 @@ module Decidim
       include Decidim::Authorable
       include Decidim::Comments::Commentable
       include Decidim::FriendlyDates
-      include Decidim::DataPortability
+      include Decidim::DownloadYourData
       include Decidim::Traceable
       include Decidim::Loggable
       include Decidim::Searchable
       include Decidim::TranslatableResource
       include Decidim::TranslatableAttributes
+      include Decidim::ActsAsTree
 
       # Limit the max depth of a comment tree. If C is a comment and R is a reply:
       # C          (depth 0)
@@ -27,6 +28,8 @@ module Decidim
 
       translatable_fields :body
 
+      parent_item_foreign_key :decidim_commentable_id
+
       belongs_to :commentable, foreign_key: "decidim_commentable_id", foreign_type: "decidim_commentable_type", polymorphic: true
       belongs_to :root_commentable, foreign_key: "decidim_root_commentable_id", foreign_type: "decidim_root_commentable_type", polymorphic: true, touch: true
       belongs_to :participatory_space, foreign_key: "decidim_participatory_space_id", foreign_type: "decidim_participatory_space_type", polymorphic: true, optional: true
@@ -53,6 +56,8 @@ module Decidim
       validate :body_length
       validate :commentable_can_have_comments
 
+      delegate :organization, to: :commentable
+
       scope :not_deleted, -> { where(deleted_at: nil) }
 
       translatable_fields :body
@@ -76,10 +81,6 @@ module Decidim
         where(alignment: -1)
       end
 
-      def organization
-        commentable&.organization || participatory_space&.organization
-      end
-
       def visible?
         participatory_space.try(:visible?) && component.try(:published?)
       end
@@ -130,11 +131,9 @@ module Decidim
 
       # Public: Overrides the `reported_content_url` Reportable concern method.
       def reported_content_url
-        return unless root_commentable
-
         url_params = { anchor: "comment_#{id}" }
 
-        if root_commentable&.respond_to?(:polymorphic_resource_url)
+        if root_commentable.respond_to?(:polymorphic_resource_url)
           root_commentable.polymorphic_resource_url(url_params)
         else
           ResourceLocatorPresenter.new(root_commentable).url(url_params)
@@ -159,7 +158,7 @@ module Decidim
       # Expects all +resources+ to be of the same "commentable_type".
       # If the result is not `Decidim::Comments::Commentable` returns `nil`.
       def self.user_commentators_ids_in(resources)
-        if resources.first&.kind_of?(Decidim::Comments::Commentable)
+        if resources.first.is_a?(Decidim::Comments::Commentable)
           commentable_type = resources.first.class.name
           Decidim::Comments::Comment.select("DISTINCT decidim_author_id").not_hidden.not_deleted
                                     .where(decidim_commentable_id: resources.pluck(:id))
@@ -171,7 +170,7 @@ module Decidim
       end
 
       def can_participate?(user)
-        return true unless root_commentable&.respond_to?(:can_participate?)
+        return true unless root_commentable.respond_to?(:can_participate?)
 
         root_commentable.can_participate?(user)
       end

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

@@ -5,7 +5,7 @@ module Decidim
     # A comment can include user votes. A user should be able to upVote, votes with
     # weight 1 and downVote, votes with weight -1.
     class CommentVote < ApplicationRecord
-      include Decidim::DataPortability
+      include Decidim::DownloadYourData
 
       belongs_to :comment, foreign_key: "decidim_comment_id", class_name: "Comment"
       belongs_to :author, foreign_key: "decidim_author_id", foreign_type: "decidim_author_type", polymorphic: true

data/app/packs/src/decidim/comments/comments.component.js

@@ -1,4 +1,6 @@
 /* eslint id-length: ["error", { "exceptions": ["$"] }] */
+/* eslint max-lines: ["error", {"max": 350, "skipBlankLines": true}] */
+
 
 /**
  * A plain Javascript component that handles the comments.
@@ -10,6 +12,8 @@
 // This is necessary for testing purposes
 const $ = window.$;
 
+import Rails from "@rails/ujs";
+
 import { createCharacterCounter } from "src/decidim/input_character_counter"
 import ExternalLink from "src/decidim/external_link"
 import updateExternalDomainLinks from "src/decidim/external_domain_warning"
@@ -23,6 +27,8 @@ export default class CommentsComponent {
     this.order = config.order;
     this.lastCommentId = config.lastCommentId;
     this.pollingInterval = config.pollingInterval || 15000;
+    this.singleComment = config.singleComment;
+    this.toggleTranslations = config.toggleTranslations;
     this.id = this.$element.attr("id") || this._getUID();
     this.mounted = false;
   }
@@ -36,6 +42,9 @@ export default class CommentsComponent {
     if (this.$element.length > 0 && !this.mounted) {
       this.mounted = true;
       this._initializeComments(this.$element);
+      if (!this.singleComment) {
+        this._fetchComments();
+      }
 
       $(".order-by__dropdown .is-submenu-item a", this.$element).on("click.decidim-comments", () => this._onInitOrder());
     }
@@ -133,8 +142,6 @@ export default class CommentsComponent {
         $text.get(0).addEventListener("emoji.added", this._onTextInput);
       }
     });
-
-    this._pollComments();
   }
 
   /**
@@ -200,20 +207,32 @@ export default class CommentsComponent {
     this._stopPolling();
 
     this.pollTimeout = setTimeout(() => {
-      Rails.ajax({
-        url: this.commentsUrl,
-        type: "GET",
-        data: new URLSearchParams({
-          "commentable_gid": this.commentableGid,
-          "root_depth": this.rootDepth,
-          "order": this.order,
-          "after": this.lastCommentId
-        }),
-        success: this._pollComments()
-      })
+      this._fetchComments();
     }, this.pollingInterval);
   }
 
+  /**
+   * Sends an ajax request based on current
+   * params to get comments for the component
+   * @private
+   * @returns {Void} - Returns nothing
+   */
+  _fetchComments() {
+    Rails.ajax({
+      url: this.commentsUrl,
+      type: "GET",
+      data: new URLSearchParams({
+        "commentable_gid": this.commentableGid,
+        "root_depth": this.rootDepth,
+        "order": this.order,
+        "after": this.lastCommentId,
+        ...(this.toggleTranslations && { "toggle_translations": this.toggleTranslations }),
+        ...(this.lastCommentId && { "after": this.lastCommentId })
+      }),
+      success: this._pollComments()
+    })
+  }
+
   /**
    * Stops polling for new comments.
    * @private

data/app/packs/src/decidim/comments/comments.component.test.js

@@ -13,6 +13,10 @@ window.$.ajax = jest.fn().mockImplementation((...args) => $.ajax(...args));
 import Quill from "quill"
 window.Quill = Quill
 
+// Rails.ajax is used by the fetching/polling of the comments
+import Rails from "@rails/ujs";
+jest.mock("@rails/ujs");
+
 // Fake timers for testing polling
 jest.useFakeTimers();
 
@@ -383,16 +387,16 @@ describe("CommentsComponent", () => {
 
     jest.advanceTimersByTime(1000);
 
-    expect(window.$.ajax).toHaveBeenCalledWith({
+    expect(Rails.ajax).toHaveBeenCalledWith({
       url: "/comments",
-      method: "GET",
-      contentType: "application/javascript",
-      data: {
+      type: "GET",
+      data: new URLSearchParams({
         "commentable_gid": "commentable-gid",
         "root_depth": 0,
         order: "older",
         after: 456
-      }
+      }),
+      success: window.undefined
     });
   });
 

data/app/permissions/decidim/comments/permissions.rb

@@ -51,13 +51,11 @@ module Decidim
       end
 
       def commentable
-        @commentable ||= begin
-          if comment
-            comment.root_commentable
-          else
-            context.fetch(:commentable, nil)
-          end
-        end
+        @commentable ||= if comment
+                           comment.root_commentable
+                         else
+                           context.fetch(:commentable, nil)
+                         end
       end
 
       def comment

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

@@ -3,7 +3,7 @@
 module Decidim
   module Comments
     # A class used to find comments for a commentable resource
-    class SortedComments < Rectify::Query
+    class SortedComments < Decidim::Query
       attr_reader :commentable
 
       # Syntactic sugar to initialize the class and return the queried objects.
@@ -34,8 +34,6 @@ module Decidim
                 .includes(:author, :user_group, :up_votes, :down_votes)
 
         case @options[:order_by]
-        when "older"
-          order_by_older(scope)
         when "recent"
           order_by_recent(scope)
         when "best_rated"

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

@@ -117,7 +117,7 @@ module Decidim
           }
         }.deep_merge(users)
 
-        Decidim::EventsManager.publish(data)
+        Decidim::EventsManager.publish(**data)
       end
     end
   end

data/app/views/decidim/comments/comments/index.js.erb

@@ -4,6 +4,7 @@
   var $comments = $("#" + rootCommentableId);
   var component = $comments.data("comments");
 
+  $(".loading-comments").addClass("hide");
 <% @comments.each do |comment| %>
   var commentId = <%= comment.id.to_json %>;
   var commentHtml = '<%== j(render comment).strip %>';

data/app/views/decidim/comments/comments/reload.js.erb

@@ -6,6 +6,7 @@
   component.unmountComponent();
 
   var commentsHtml = '<%== j(render partial: "comments").strip %>';
+  $(".loading-comments").addClass("hide");
   $comments.replaceWith(commentsHtml);
 
   $comments = $("#" + rootCommentableId);

data/config/locales/am-ET.yml

@@ -1 +1,2 @@
+---
 am:

data/config/locales/ar.yml

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

data/config/locales/bg.yml

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

data/config/locales/ca.yml

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

data/config/locales/cs.yml

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

data/config/locales/da.yml

@@ -1 +1,2 @@
+---
 da:

data/config/locales/de.yml

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

data/config/locales/el.yml

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

data/config/locales/eo.yml

@@ -1 +1,2 @@
+---
 eo:

data/config/locales/es-MX.yml

@@ -1,3 +1,4 @@
+---
 es-MX:
   activemodel:
     models:

data/config/locales/es-PY.yml

@@ -1,3 +1,4 @@
+---
 es-PY:
   activemodel:
     models:

data/config/locales/es.yml

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

data/config/locales/et.yml

@@ -1 +1,2 @@
+---
 et:

data/config/locales/eu.yml

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

data/config/locales/fi-plain.yml

@@ -1,3 +1,4 @@
+---
 fi-pl:
   activemodel:
     models:

data/config/locales/fi.yml

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

data/config/locales/fr-CA.yml

@@ -1,3 +1,4 @@
+---
 fr-CA:
   activemodel:
     models:

data/config/locales/fr.yml

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

data/config/locales/ga-IE.yml

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

data/config/locales/gl.yml

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

data/config/locales/hr.yml

@@ -1 +1,2 @@
+---
 hr:

data/config/locales/hu.yml

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

data/config/locales/id-ID.yml

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

data/config/locales/is-IS.yml

@@ -1,4 +1,5 @@
-is-IS:
+---
+is:
   decidim:
     components:
       add_comment_form:

data/config/locales/it.yml

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

data/config/locales/ja.yml

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

data/config/locales/ko.yml

@@ -1 +1,2 @@
+---
 ko:

data/config/locales/lb.yml

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