decidim-comments 0.21.0 → 0.23.2

data/db/migrate/20200828101910_add_commentable_counter_cache_to_comments.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+class AddCommentableCounterCacheToComments < ActiveRecord::Migration[5.2]
+  def change
+    add_column :decidim_comments_comments, :comments_count, :integer, null: false, default: 0, index: true
+    Decidim::Comments::Comment.reset_column_information
+    Decidim::Comments::Comment.find_each(&:update_comments_count)
+  end
+end

data/lib/decidim/comments/api/comment_type.rb

@@ -25,7 +25,11 @@ module Decidim
         }
       end
 
-      field :body, !types.String, "The comment message"
+      field :body, !types.String, "The comment message" do
+        resolve lambda { |obj, _args, _ctx|
+          obj.translated_body
+        }
+      end
 
       field :formattedBody, !types.String, "The comment message ready to display (it is expected to include HTML)", property: :formatted_body
 

data/lib/decidim/comments/comment_serializer.rb

@@ -11,7 +11,8 @@ module Decidim
         {
           id: resource.id,
           created_at: resource.created_at,
-          body: resource.body,
+          body: resource.body.values.first,
+          locale: resource.body.keys.first,
           author: {
             id: resource.author.id,
             name: resource.author.name
@@ -31,7 +32,11 @@ module Decidim
       private
 
       def root_commentable_url
-        @root_commentable_url ||= Decidim::ResourceLocatorPresenter.new(resource.root_commentable).url
+        @root_commentable_url ||= if resource.root_commentable&.respond_to?(:polymorphic_resource_url)
+                                    resource.root_commentable.polymorphic_resource_url
+                                  else
+                                    ResourceLocatorPresenter.new(resource.root_commentable).url
+                                  end
       end
     end
   end

data/lib/decidim/comments/comment_vote_serializer.rb

@@ -37,7 +37,11 @@ module Decidim
       private
 
       def root_commentable_url
-        @root_commentable_url ||= Decidim::ResourceLocatorPresenter.new(resource.comment.root_commentable).url
+        @root_commentable_url ||= if resource.comment.root_commentable&.respond_to?(:polymorphic_resource_url)
+                                    resource.comment.root_commentable.polymorphic_resource_url
+                                  else
+                                    ResourceLocatorPresenter.new(resource.comment.root_commentable).url
+                                  end
       end
     end
   end

data/lib/decidim/comments/commentable.rb

@@ -51,6 +51,17 @@ module Decidim
         def user_allowed_to_comment?(_user)
           true
         end
+
+        # Public: Updates the comments counter cache. We have to do it these
+        # way in order to properly calculate the coutner with hidden
+        # comments.
+        #
+        # rubocop:disable Rails/SkipsModelValidations
+        def update_comments_count
+          comments_count = comments.not_hidden.count
+          update_columns(comments_count: comments_count, updated_at: Time.current)
+        end
+        # rubocop:enable Rails/SkipsModelValidations
       end
     end
   end

data/lib/decidim/comments/comments_helper.rb

@@ -26,9 +26,13 @@ module Decidim
         commentable_type = resource.commentable_type
         commentable_id = resource.id.to_s
         node_id = "comments-for-#{commentable_type.demodulize}-#{commentable_id}"
-        react_comments_component(node_id, commentableType: commentable_type,
-                                          commentableId: commentable_id,
-                                          locale: I18n.locale)
+        react_comments_component(
+          node_id, commentableType: commentable_type,
+                   commentableId: commentable_id,
+                   locale: I18n.locale,
+                   toggleTranslations: machine_translations_toggled?,
+                   commentsMaxLength: comments_max_length(resource)
+        )
       end
 
       # Private: Render Comments component using inline javascript
@@ -44,11 +48,31 @@ module Decidim
               {
                 commentableType: "#{props[:commentableType]}",
                 commentableId: "#{props[:commentableId]}",
-                locale: "#{props[:locale]}"
+                locale: "#{props[:locale]}",
+                toggleTranslations: #{props[:toggleTranslations]},
+                commentsMaxLength: "#{props[:commentsMaxLength]}"
               }
             );
           })
       end
+
+      def comments_max_length(resource)
+        return 1000 unless resource.respond_to?(:component)
+        return component_comments_max_length(resource) if component_comments_max_length(resource)
+        return organization_comments_max_length(resource) if organization_comments_max_length(resource)
+
+        1000
+      end
+
+      def component_comments_max_length(resource)
+        return unless resource.component&.settings.respond_to?(:comments_max_length)
+
+        resource.component.settings.comments_max_length if resource.component.settings.comments_max_length.positive?
+      end
+
+      def organization_comments_max_length(resource)
+        resource.component.organization.comments_max_length if resource.component.organization.comments_max_length.positive?
+      end
     end
   end
 end

data/lib/decidim/comments/engine.rb

@@ -59,6 +59,19 @@ module Decidim
           metric_operation.manager_class = "Decidim::Comments::Metrics::CommentParticipantsMetricMeasure"
         end
       end
+
+      initializer "decidim_comments.register_resources" do
+        Decidim.register_resource(:comment) do |resource|
+          resource.model_class_name = "Decidim::Comments::Comment"
+          resource.card = "decidim/comments/comment"
+          resource.searchable = true
+        end
+      end
+
+      initializer "decidim_comments.add_cells_view_paths" do
+        Cell::ViewModel.view_paths << File.expand_path("#{Decidim::Comments::Engine.root}/app/cells")
+        Cell::ViewModel.view_paths << File.expand_path("#{Decidim::Comments::Engine.root}/app/views") # for partials
+      end
     end
   end
 end

data/lib/decidim/comments/mutation_extensions.rb

@@ -16,8 +16,12 @@ module Decidim
 
           argument :id, !types.String, "The commentable's ID"
           argument :type, !types.String, "The commentable's class name. i.e. `Decidim::ParticipatoryProcess`"
+          argument :locale, !types.String, "The locale for which to get the comments text"
+          argument :toggleTranslations, !types.Boolean, "Whether the user asked to toggle the machine translations or not."
 
           resolve lambda { |_obj, args, _ctx|
+            I18n.locale = args[:locale].presence
+            RequestStore.store[:toggle_machine_translations] = args[:toggleTranslations]
             args[:type].constantize.find(args[:id])
           }
         end
@@ -26,8 +30,12 @@ module Decidim
           description "A comment"
 
           argument :id, !types.ID, "The comment's id"
+          argument :locale, !types.String, "The locale for which to get the comments text"
+          argument :toggleTranslations, !types.Boolean, "Whether the user asked to toggle the machine translations or not."
 
           resolve lambda { |_obj, args, _ctx|
+            I18n.locale = args[:locale].presence
+            RequestStore.store[:toggle_machine_translations] = args[:toggleTranslations]
             Comment.find(args["id"])
           }
         end

data/lib/decidim/comments/query_extensions.rb

@@ -16,8 +16,12 @@ module Decidim
 
           argument :id, !types.String, "The commentable's ID"
           argument :type, !types.String, "The commentable's class name. i.e. `Decidim::ParticipatoryProcess`"
+          argument :locale, !types.String, "The locale for which to get the comments text"
+          argument :toggleTranslations, !types.Boolean, "Whether the user asked to toggle the machine translations or not."
 
           resolve lambda { |_obj, args, _ctx|
+            I18n.locale = args[:locale].presence
+            RequestStore.store[:toggle_machine_translations] = args[:toggleTranslations]
             args[:type].constantize.find(args[:id])
           }
         end

data/lib/decidim/comments/test/factories.rb

@@ -7,7 +7,16 @@ FactoryBot.define do
     author { build(:user, organization: commentable.organization) }
     commentable { build(:dummy_resource) }
     root_commentable { commentable }
-    body { Faker::Lorem.paragraph }
+    body { Decidim::Faker::Localized.paragraph }
+
+    after(:build) do |comment, evaluator|
+      comment.body = if evaluator.body.is_a?(String)
+                       { comment.root_commentable.organization.default_locale || "en" => evaluator.body }
+                     else
+                       evaluator.body
+                     end
+      comment.body = Decidim::ContentProcessor.parse_with_processor(:hashtag, comment.body, current_organization: comment.root_commentable.organization).rewrite
+    end
 
     trait :comment_on_comment do
       author { build(:user, organization: root_commentable.organization) }

data/lib/decidim/comments/test/shared_examples/comment_event.rb

@@ -12,10 +12,16 @@ shared_context "when it's a comment event" do
 
   let(:comment) { create :comment }
   let(:comment_author) { comment.author }
+  let(:normalized_comment_author) { comment.author }
   let(:comment_author_name) { decidim_html_escape comment.author.name }
 
   let(:extra) { { comment_id: comment.id } }
-  let(:resource_title) { decidim_html_escape resource.title }
+  let(:resource_title) { decidim_html_escape(translated(resource.title)) }
+  let(:user_group) do
+    user_group = create(:user_group, :verified, organization: organization, users: [comment_author])
+    comment.update!(user_group: user_group)
+    user_group
+  end
 end
 
 shared_examples_for "a comment event" do
@@ -23,7 +29,11 @@ shared_examples_for "a comment event" do
 
   describe "author" do
     it "returns the comment author" do
-      expect(subject.author).to eq(comment_author)
+      if defined? user_group_author
+        expect(subject.author).to eq(user_group_author)
+      else
+        expect(subject.author).to eq(comment_author)
+      end
     end
   end
 

data/lib/decidim/comments/test/shared_examples/create_comment_context.rb

@@ -16,7 +16,8 @@ RSpec.shared_context "when creating a comment" do
       "comment" => {
         "body" => body,
         "alignment" => alignment,
-        "user_group_id" => user_group_id
+        "user_group_id" => user_group_id,
+        "commentable" => commentable
       }
     }
   end
@@ -27,5 +28,5 @@ RSpec.shared_context "when creating a comment" do
       current_organization: organization
     )
   end
-  let(:command) { described_class.new(form, author, commentable) }
+  let(:command) { described_class.new(form, author) }
 end

data/lib/decidim/comments/version.rb

@@ -4,7 +4,7 @@ module Decidim
   # This holds the decidim-comments version.
   module Comments
     def self.version
-      "0.21.0"
+      "0.23.2"
     end
   end
 end

metadata

@@ -1,16 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: decidim-comments
 version: !ruby/object:Gem::Version
-  version: 0.21.0
+  version: 0.23.2
 platform: ruby
 authors:
 - Josep Jaume Rey Peroy
 - Marc Riera Casals
 - Oriol Gual Oliva
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-08 00:00:00.000000000 Z
+date: 2021-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: decidim-core
@@ -18,14 +18,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.21.0
+        version: 0.23.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.21.0
+        version: 0.23.2
 - !ruby/object:Gem::Dependency
   name: jquery-rails
   requirement: !ruby/object:Gem::Requirement
@@ -60,28 +60,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.21.0
+        version: 0.23.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.21.0
+        version: 0.23.2
 - !ruby/object:Gem::Dependency
   name: decidim-dev
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.21.0
+        version: 0.23.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.21.0
+        version: 0.23.2
 description: Pluggable comments system for some components.
 email:
 - josepjaume@gmail.com
@@ -98,12 +98,18 @@ files:
 - app/assets/javascripts/decidim/comments/bundle.js.map
 - app/assets/javascripts/decidim/comments/comments.js.erb
 - app/cells/decidim/comments/comment_activity_cell.rb
+- app/cells/decidim/comments/comment_cell.rb
+- app/cells/decidim/comments/comment_m/footer.erb
+- app/cells/decidim/comments/comment_m/top.erb
+- app/cells/decidim/comments/comment_m_cell.rb
 - app/commands/decidim/comments/create_comment.rb
 - app/commands/decidim/comments/vote_comment.rb
 - app/events/decidim/comments/comment_by_followed_user_event.rb
+- app/events/decidim/comments/comment_by_followed_user_group_event.rb
 - app/events/decidim/comments/comment_created_event.rb
 - app/events/decidim/comments/comment_event.rb
 - app/events/decidim/comments/reply_created_event.rb
+- app/events/decidim/comments/user_group_mentioned_event.rb
 - app/events/decidim/comments/user_mentioned_event.rb
 - app/forms/decidim/comments/comment_form.rb
 - app/frontend/application/apollo_client.ts
@@ -149,6 +155,7 @@ files:
 - app/frontend/support/require_all.ts
 - app/frontend/support/resolve_graphql_query.ts
 - app/frontend/support/schema.ts
+- app/helpers/decidim/comments/comment_cells_helper.rb
 - app/models/decidim/comments/application_record.rb
 - app/models/decidim/comments/comment.rb
 - app/models/decidim/comments/comment_vote.rb
@@ -163,38 +170,74 @@ files:
 - app/types/decidim/comments/commentable_interface.rb
 - app/types/decidim/comments/commentable_mutation_type.rb
 - app/types/decidim/comments/commentable_type.rb
+- config/locales/am-ET.yml
 - config/locales/ar-SA.yml
 - config/locales/ar.yml
+- config/locales/bg-BG.yml
+- config/locales/bg.yml
 - config/locales/ca.yml
 - config/locales/cs-CZ.yml
 - config/locales/cs.yml
+- config/locales/da-DK.yml
+- config/locales/da.yml
 - config/locales/de.yml
 - config/locales/el-GR.yml
 - config/locales/el.yml
 - config/locales/en.yml
 - config/locales/eo-UY.yml
+- config/locales/eo.yml
 - config/locales/es-MX.yml
 - config/locales/es-PY.yml
 - config/locales/es.yml
+- config/locales/et-EE.yml
+- config/locales/et.yml
 - config/locales/eu.yml
 - config/locales/fi-pl.yml
 - config/locales/fi-plain.yml
 - config/locales/fi.yml
+- config/locales/fr-CA.yml
 - config/locales/fr.yml
+- config/locales/ga-IE.yml
 - config/locales/gl.yml
+- config/locales/hr-HR.yml
+- config/locales/hr.yml
 - config/locales/hu.yml
 - config/locales/id-ID.yml
 - config/locales/is-IS.yml
+- config/locales/is.yml
 - config/locales/it.yml
+- config/locales/ja-JP.yml
+- config/locales/ja.yml
+- config/locales/ko-KR.yml
+- config/locales/ko.yml
+- config/locales/lt-LT.yml
+- config/locales/lt.yml
+- config/locales/lv.yml
+- config/locales/mt-MT.yml
+- config/locales/mt.yml
 - config/locales/nl.yml
 - config/locales/no.yml
+- config/locales/om-ET.yml
 - config/locales/pl.yml
 - config/locales/pt-BR.yml
 - config/locales/pt.yml
+- config/locales/ro-RO.yml
 - config/locales/ru.yml
+- config/locales/si-LK.yml
+- config/locales/sk-SK.yml
+- config/locales/sk.yml
+- config/locales/sl.yml
+- config/locales/so-SO.yml
+- config/locales/sr-CS.yml
 - config/locales/sv.yml
+- config/locales/sw-KE.yml
+- config/locales/ti-ER.yml
 - config/locales/tr-TR.yml
 - config/locales/uk.yml
+- config/locales/vi-VN.yml
+- config/locales/vi.yml
+- config/locales/zh-CN.yml
+- config/locales/zh-TW.yml
 - db/migrate/20161130143508_create_comments.rb
 - db/migrate/20161214082645_add_depth_to_comments.rb
 - db/migrate/20161216102820_add_alignment_to_comments.rb
@@ -206,6 +249,9 @@ files:
 - db/migrate/20181003080320_fix_user_groups_ids_in_comments.rb
 - db/migrate/20181016142511_make_authors_polymorphic_for_comments.rb
 - db/migrate/20181019092928_make_author_polymorphic_for_comment_votes.rb
+- db/migrate/20200320105911_index_foreign_keys_in_decidim_comments_comments.rb
+- db/migrate/20200706123136_make_comments_handle_i18n.rb
+- db/migrate/20200828101910_add_commentable_counter_cache_to_comments.rb
 - db/seeds.rb
 - lib/decidim/comments.rb
 - lib/decidim/comments/admin.rb
@@ -231,7 +277,7 @@ homepage: https://github.com/decidim/decidim
 licenses:
 - AGPL-3.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -239,15 +285,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Decidim comments module
 test_files: []