decidim-spam_detection 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a68cdcfc4bee9d6fe8d391eb22a121b2f7cfb2a47557c1081b4ae47e649fc7f
-  data.tar.gz: b7ee10f5d795d9044c7bd0d6270d899f3ce90e72496221739dad6715cd287de8
+  metadata.gz: 38cc3310b2ed4c7e48cadea2c1bf9f0ff86d8f28bad7b3e04398d5922598442d
+  data.tar.gz: be191b16b6827c2e6963b72578fe899d1feace76156b87cb246928dc869619d3
 SHA512:
-  metadata.gz: 921412beab2eb7bbf993bb1f6f02f95556ebdd7bb61ac18695764176762a870e48ac572a9f86b0cc4dcdc91a2ed09a6786d3e54b7f8b14f4b56e912992c8a9ba
-  data.tar.gz: d46133286084b2a2c353a8fb50a0fc0460f299925e8335c791f6f2472d66c9cf2ea70ef647f616645105584630f798573ef988ae4c862c1d3a2dc713e87b4d8d
+  metadata.gz: ef0c0fcc58dfdb427725cc53ad5a3fa2829b0482180e4b311427a483345b97f09ec90d55ffdcfaf778903ed00913bf17976ae56e2a9fb13deb17605cf45c0425
+  data.tar.gz: 9d6b782f0b236cd000b1bf361b5d83524c037da23fa22aa779321f5c09fafbc849d45d713c965b2e9a6eb2a37d465565e91b33bfcff0a1e73436065ee76265b6

data/app/jobs/decidim/spam_detection/notify_admins.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module SpamDetection
+    class NotifyAdmins < ApplicationJob
+      queue_as :default
+
+      def perform(results_hash)
+        results_hash.each do |id, result|
+          next if result.keys == [:nothing]
+
+          Decidim::Organization.find(id).admins.each do |admin|
+            Decidim::SpamDetection::SpamDetectionMailer.notify_detection(admin, result).deliver_later
+          end
+        end
+      end
+    end
+  end
+end

data/app/mailers/decidim/spam_detection/spam_detection_mailer.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module SpamDetection
+    class SpamDetectionMailer < Decidim::ApplicationMailer
+      def notify_detection(user, results)
+        with_user(user) do
+          @reported_count = results[:reported_user]
+          @blocked_count = results[:blocked_user]
+          @organization = user.organization
+          @user = user
+
+          subject = I18n.t("notify_detection.subject", scope: "decidim.spam_detection_mailer")
+          mail(to: @user.email, subject: subject)
+        end
+      end
+    end
+  end
+end

data/app/services/decidim/spam_detection/mark_users_service.rb

@@ -27,7 +27,7 @@ module Decidim
                               .where(admin: false, blocked: false, deleted_at: nil)
                               .where("(extended_data #> '{spam_detection, unreported_at}') is null")
                               .where("(extended_data #> '{spam_detection, unblocked_at}') is null")
-        @results = []
+        @results = {}
       end
 
       def self.call
@@ -38,11 +38,15 @@ module Decidim
         spam_probability_array = Decidim::SpamDetection::ApiProxy.request(cleaned_users)
 
         mark_spam_users(merge_response_with_users(spam_probability_array))
+        notify_admins!
       end
 
       def mark_spam_users(probability_array)
         probability_array.each do |probability_hash|
-          @results << Decidim::SpamDetection::SpamUserCommandAdapter.call(probability_hash).result
+          result = Decidim::SpamDetection::SpamUserCommandAdapter.call(probability_hash).result
+          organization_id = probability_hash["decidim_organization_id"]
+
+          add_to_results(organization_id.to_s, result)
         end
       end
 
@@ -56,7 +60,23 @@ module Decidim
       end
 
       def status
-        @results.tally
+        @results.each_with_object({}) do |result, hash|
+          hash[result[0]] = result[1].tally
+        end
+      end
+
+      def notify_admins!
+        Decidim::SpamDetection::NotifyAdmins.perform_later(status)
+      end
+
+      private
+
+      def add_to_results(organization_id, result)
+        if @results[organization_id]
+          @results[organization_id] << result
+        else
+          @results[organization_id] = [result]
+        end
       end
     end
   end

data/app/views/decidim/spam_detection/spam_detection_mailer/notify_detection.html.erb

@@ -0,0 +1,14 @@
+<p class="email-greeting"><%= t(".hello", name: @user.name) %></p>
+<p><%= t(".intro") %></p>
+
+<% if @reported_count.present? %>
+  <p>
+    <%= t(".reported_count", count: @reported_count) %>
+  </p>
+<% end %>
+
+<% if @blocked_count.present? %>
+  <p>
+    <%= t(".blocked_count", count: @blocked_count) %>
+  </p>
+<% end %>

data/config/i18n-tasks.yml

@@ -1,7 +1,7 @@
 ---
 
 base_locale: en
-locales: [en]
+locales: [en, fr, es, ca]
 
 ignore_unused:
   - "decidim.components.spam_detection.name"

data/config/locales/ca.yml

@@ -0,0 +1,16 @@
+---
+ca:
+  decidim:
+    components:
+      spam_detection:
+        name: SpamDetection
+    spam_detection:
+      spam_detection_mailer:
+        notify_detection:
+          blocked_count: blocked_count %{count}
+          hello: Hello %{name}
+          intro: Aquí teniu l'informe de la tasca de detecció de correu brossa
+          reported_count: reported_count %{count}
+    spam_detection_mailer:
+      notify_detection:
+        subject: Resum de detecció de correu brossa

data/config/locales/en.yml

@@ -4,3 +4,13 @@ en:
     components:
       spam_detection:
         name: SpamDetection
+    spam_detection:
+      spam_detection_mailer:
+        notify_detection:
+          blocked_count: 'Blocked users count: %{count}'
+          hello: Hello %{name}
+          intro: Here is the report of the spam detection task
+          reported_count: 'Reported users count: %{count}'
+    spam_detection_mailer:
+      notify_detection:
+        subject: Spam detection digest

data/config/locales/es.yml

@@ -0,0 +1,16 @@
+---
+es:
+  decidim:
+    components:
+      spam_detection:
+        name: SpamDetection
+    spam_detection:
+      spam_detection_mailer:
+        notify_detection:
+          blocked_count: blocked_count %{count}
+          hello: Hello %{name}
+          intro: Here is the report of the spam detection task
+          reported_count: reported_count %{count}
+    spam_detection_mailer:
+      notify_detection:
+        subject: Subject

data/config/locales/fr.yml

@@ -0,0 +1,16 @@
+---
+fr:
+  decidim:
+    components:
+      spam_detection:
+        name: SpamDetection
+    spam_detection:
+      spam_detection_mailer:
+        notify_detection:
+          blocked_count: blocked_count %{count}
+          hello: Hello %{name}
+          intro: Here is the report of the spam detection task
+          reported_count: reported_count %{count}
+    spam_detection_mailer:
+      notify_detection:
+        subject: Subject

data/lib/decidim/spam_detection/abstract_spam_user_command.rb

@@ -24,6 +24,14 @@ module Decidim
         raise NotImplementedError
       end
 
+      def reason
+        raise NotImplementedError
+      end
+
+      def details
+        raise NotImplementedError
+      end
+
       def moderation_user
         moderation_admin_params = {
           name: SPAM_USER[:name],
@@ -56,9 +64,10 @@ module Decidim
       end
 
       def add_spam_detection_metadata!(metadata)
-        @user.update!(extended_data: @user.extended_data
-                                        .dup
-                                        .deep_merge("spam_detection" => metadata))
+        @user.extended_data = @user.extended_data
+                                   .dup
+                                   .deep_merge("spam_detection" => metadata)
+        @user.save(validate: false)
       end
     end
   end

data/lib/decidim/spam_detection/block_spam_user_command.rb

@@ -9,28 +9,60 @@ module Decidim
       prepend Decidim::SpamDetection::Command
 
       def call
-        form = form(Decidim::Admin::BlockUserForm).from_params(
-          justification: "The user was blocked because of a high spam probability by Decidim spam detection bot"
-        )
+        ActiveRecord::Base.transaction do
+          create_user_moderation
+          block!
+          register_justification!
+          notify_user!
+          add_spam_detection_metadata!({ "blocked_at" => Time.current, "spam_probability" => @probability })
+        end
 
-        moderator = @moderator
-        user = @user
+        Rails.logger.info("User with id #{@user["id"]} was blocked for spam with a probability of #{@probability}%")
 
-        form.define_singleton_method(:user) { user }
-        form.define_singleton_method(:current_user) { moderator }
-        form.define_singleton_method(:blocking_user) { moderator }
-
-        Decidim::Admin::BlockUser.call(form)
+        :ok
+      end
 
-        add_spam_detection_metadata!({
-                                       "blocked_at" => Time.current,
-                                       "spam_probability" => @probability
-                                     })
+      private
 
+      def create_user_moderation
         @user.create_user_moderation
-        Rails.logger.info("User with id #{@user["id"]} was blocked for spam")
+      end
 
-        :ok
+      def register_justification!
+        UserBlock.create!(justification: reason, user: @user, blocking_user: @moderator)
+      end
+
+      def notify_user!
+        Decidim::BlockUserJob.perform_later(@user, reason)
+      end
+
+      def block!
+        Decidim.traceability.perform_action!(
+          "block",
+          @user,
+          @moderator,
+          extra: {
+            reportable_type: @user.class.name,
+            current_justification: reason
+          },
+          resource: {
+            # Make sure the action log entry gets the original user name instead
+            # of "Blocked user". Otherwise the log entries would show funny
+            # messages such as "Mr. Admin blocked user Blocked user"-
+            title: @user.name
+          }
+        ) do
+          @user.blocked = true
+          @user.blocked_at = Time.current
+          @user.blocking = @current_blocking
+          @user.extended_data["user_name"] = @user.name
+          @user.name = "Blocked user"
+          @user.save!
+        end
+      end
+
+      def reason
+        "The user was blocked because of a high spam probability by Decidim spam detection bot with a probability of #{@probability}%"
       end
     end
   end

data/lib/decidim/spam_detection/report_spam_user_command.rb

@@ -9,29 +9,43 @@ module Decidim
       prepend Decidim::SpamDetection::Command
 
       def call
-        form = form(Decidim::ReportForm).from_params(
-          reason: "spam",
-          details: "The user was marked as spam by Decidim spam detection bot"
-        )
+        ActiveRecord::Base.transaction do
+          find_or_create_moderation!
+          create_report!
+          update_report_count!
+          add_spam_detection_metadata!({ "reported_at" => Time.current, "spam_probability" => @probability })
+        end
 
-        current_organization = @user.organization
-        moderator = @moderator
-        user = @user
+        Rails.logger.info("User with id #{@user.id} was reported for spam with a probability of #{@probability}%")
 
-        report = Decidim::CreateUserReport.new(form, user, moderator)
-        report.define_singleton_method(:current_organization) { current_organization }
-        report.define_singleton_method(:current_user) { moderator }
-        report.define_singleton_method(:reportable) { user }
-        report.call
+        :ok
+      end
 
-        add_spam_detection_metadata!({
-                                       "reported_at" => Time.current,
-                                       "spam_probability" => @probability
-                                     })
+      private
 
-        Rails.logger.info("User with id #{user.id} was reported for spam")
+      def reason
+        "spam"
+      end
 
-        :ok
+      def details
+        "The user was marked as spam by Decidim spam detection bot with a probability of #{@probability}%"
+      end
+
+      def find_or_create_moderation!
+        @moderation = UserModeration.find_or_create_by!(user: @user)
+      end
+
+      def create_report!
+        @report = UserReport.create!(
+          moderation: @moderation,
+          user: @moderator,
+          reason: reason,
+          details: details
+        )
+      end
+
+      def update_report_count!
+        @moderation.update!(report_count: @moderation.report_count + 1)
       end
     end
   end

data/lib/decidim/spam_detection/version.rb

@@ -5,7 +5,7 @@ module Decidim
   # This holds the decidim-spam_detection version.
   module SpamDetection
     def self.version
-      "1.0.0"
+      "1.1.0"
     end
 
     def self.decidim_version

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: decidim-spam_detection
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Armand Fardeau
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-01-08 00:00:00.000000000 Z
+date: 2022-02-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: decidim-core
@@ -40,10 +40,15 @@ files:
 - app/commands/decidim/admin/unblock_user.rb
 - app/commands/decidim/admin/unreport_user.rb
 - app/jobs/decidim/spam_detection/mark_users_job.rb
+- app/jobs/decidim/spam_detection/notify_admins.rb
+- app/mailers/decidim/spam_detection/spam_detection_mailer.rb
 - app/services/decidim/spam_detection/mark_users_service.rb
-- config/assets.rb
+- app/views/decidim/spam_detection/spam_detection_mailer/notify_detection.html.erb
 - config/i18n-tasks.yml
+- config/locales/ca.yml
 - config/locales/en.yml
+- config/locales/es.yml
+- config/locales/fr.yml
 - lib/decidim/spam_detection.rb
 - lib/decidim/spam_detection/abstract_spam_user_command.rb
 - lib/decidim/spam_detection/admin.rb

data/config/assets.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-base_path = File.expand_path("..", __dir__)
-
-Decidim::Webpacker.register_path("#{base_path}/app/packs")
-Decidim::Webpacker.register_entrypoints(
-  decidim_spam_detection: "#{base_path}/app/packs/entrypoints/decidim_spam_detection.js"
-)
-Decidim::Webpacker.register_stylesheet_import("stylesheets/decidim/spam_detection/spam_detection")