decidim-comments 0.22.0 → 0.23.0

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

@@ -16,7 +16,7 @@ shared_context "when it's a comment event" do
   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)

data/lib/decidim/comments/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: decidim-comments
 version: !ruby/object:Gem::Version
-  version: 0.22.0
+  version: 0.23.0
 platform: ruby
 authors:
 - Josep Jaume Rey Peroy
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-01 00:00:00.000000000 Z
+date: 2020-11-05 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.22.0
+        version: 0.23.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
 - !ruby/object:Gem::Dependency
   name: jquery-rails
   requirement: !ruby/object:Gem::Requirement
@@ -60,28 +60,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
 - !ruby/object:Gem::Dependency
   name: decidim-dev
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.22.0
+        version: 0.23.0
 description: Pluggable comments system for some components.
 email:
 - josepjaume@gmail.com
@@ -98,6 +98,10 @@ 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
@@ -151,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
@@ -165,22 +170,27 @@ 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
@@ -190,16 +200,24 @@ files:
 - 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/lv-LV.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
@@ -208,10 +226,16 @@ files:
 - 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/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
@@ -224,6 +248,8 @@ files:
 - 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
@@ -257,14 +283,14 @@ 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
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Decidim comments module