decidim-verifications 0.23.0 → 0.24.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 900cd69a4b633ddc14fa6511b44a85d06f11705d33948ffed831ca09c295aa30
-  data.tar.gz: fb2a8d1b836f312109e79d3c76c9a9e006c1b13d12d954ac40fe1c29a2b6b6c9
+  metadata.gz: 3d5ddc2f063d130e6be5d06e33784d853028039016a23b59db919bd27af35bd1
+  data.tar.gz: 27c829745ac4f5e43b8d46832fdf44df5c64e3b7117110fd43e4f71f8099c259
 SHA512:
-  metadata.gz: 68cdf7e402bf5c6be48b43e2c9f79663497b4cc4748b8d273c24921744d9059d1ee0b8f622cf16eb62e056af9ebabe7455b31eff4dd7095be9dd9f7ba935f2f3
-  data.tar.gz: f7e4024f26baf957c646631a965bbf78eecb6ad47d8778872de5bf4b2a4264ac324faecf34a279f971ac69a778c8fa9c242beab65f9566f8683742a9fd527155
+  metadata.gz: 037b12793b4b867c5119bf2d53bd992644a7a5af2d7f852cfc8ec27a7b1a444a5511fc204eb03415627828cfcade1efae24b449d7990abc7d259d60b28e1b0b8
+  data.tar.gz: 2cd89ffdb95eea642ba2cc14b0d53aa3cb61caf1a07c8c7dcc3bc567b299247cde82afa4dcd98d65653b27563b3b0a2497e113d4d6b6fe763bcd1a83aef8ad0e

data/README.md

@@ -166,7 +166,7 @@ on the context where the authorization is being performed.
 
 For example, you can require authorization for supporting proposals in a participatory
 process, and also restrict it to users with postal codes 12345 and 12346. The
-[example authorization handler](https://github.com/decidim/decidim/blob/master/decidim-generators/lib/decidim/generators/app_templates/dummy_authorization_handler.rb)
+[example authorization handler](https://github.com/decidim/decidim/blob/develop/decidim-generators/lib/decidim/generators/app_templates/dummy_authorization_handler.rb)
 included in this module allows to do that. As an admin user, you should visit
 the proposals componenent permissions screen, choose the `Example authorization`
 as the authorization handler name for the `vote` action and enter "12345, 12346"
@@ -204,8 +204,8 @@ Decidim::Verifications.register_workflow(:my_verification) do |workflow|
 end
 ```
 
-Check the [example authorization handler](https://github.com/decidim/decidim/blob/master/decidim-generators/lib/decidim/generators/app_templates/dummy_authorization_handler.rb)
-and the [DefaultActionAuthorizer class](https://github.com/decidim/decidim/blob/master/decidim-verifications/lib/decidim/verifications/default_action_authorizer.rb)
+Check the [example authorization handler](https://github.com/decidim/decidim/blob/develop/decidim-generators/lib/decidim/generators/app_templates/dummy_authorization_handler.rb)
+and the [DefaultActionAuthorizer class](https://github.com/decidim/decidim/blob/develop/decidim-verifications/lib/decidim/verifications/default_action_authorizer.rb)
 for additional technical details.
 
 ## How Handlers work
@@ -241,8 +241,8 @@ See [Decidim](https://github.com/decidim/decidim).
 
 See [Decidim](https://github.com/decidim/decidim).
 
-[authorization handler base class]: https://github.com/decidim/decidim/blob/master/decidim-verifications/app/services/decidim/authorization_handler.rb
-[example SMS gateway]: https://github.com/decidim/decidim/blob/master/decidim-verifications/lib/decidim/verifications/sms/example_gateway.rb
+[authorization handler base class]: https://github.com/decidim/decidim/blob/develop/decidim-verifications/app/services/decidim/authorization_handler.rb
+[example SMS gateway]: https://github.com/decidim/decidim/blob/develop/decidim-verifications/lib/decidim/verifications/sms/example_gateway.rb
 
 [Decidim Barcelona]: https://github.com/AjuntamentdeBarcelona/decidim-barcelona/blob/master/app/services/census_authorization_handler.rb
 [Decidim Terrassa]: https://github.com/AjuntamentDeTerrassa/decidim-terrassa/blob/master/app/services/census_authorization_handler.rb

data/app/commands/decidim/verifications/authorize_user.rb

@@ -18,7 +18,12 @@ module Decidim
       #
       # Returns nothing.
       def call
-        return broadcast(:invalid) unless handler.valid?
+        if handler.invalid?
+          conflict = create_verification_conflict
+          notify_admins(conflict) if conflict.present?
+
+          return broadcast(:invalid)
+        end
 
         Authorization.create_or_update_from(handler)
 
@@ -28,6 +33,30 @@ module Decidim
       private
 
       attr_reader :handler
+
+      def notify_admins(conflict)
+        Decidim::EventsManager.publish(
+          event: "decidim.events.verifications.managed_user_error_event",
+          event_class: Decidim::Verifications::ManagedUserErrorEvent,
+          resource: conflict,
+          affected_users: Decidim::User.where(admin: true)
+        )
+      end
+
+      def create_verification_conflict
+        authorization = Decidim::Authorization.find_by(unique_id: handler.unique_id)
+        return if authorization.blank?
+
+        conflict = Decidim::Verifications::Conflict.find_or_initialize_by(
+          current_user: handler.user,
+          managed_user: authorization.user,
+          unique_id: handler.unique_id
+        )
+
+        conflict.update(times: conflict.times + 1)
+
+        conflict
+      end
     end
   end
 end

data/app/commands/decidim/verifications/csv_census/admin/create_census_data.rb

@@ -20,7 +20,9 @@ module Decidim
           def call
             return broadcast(:invalid) unless @form.file
 
+            # rubocop:disable Rails/SkipsModelValidations
             CsvDatum.insert_all(@organization, @form.data.values)
+            # rubocop:enable Rails/SkipsModelValidations
             RemoveDuplicatesJob.perform_later(@organization)
 
             broadcast(:ok)

data/app/commands/decidim/verifications/id_documents/admin/confirm_user_offline_authorization.rb

@@ -58,8 +58,8 @@ module Decidim
                                .new(organization: form.current_organization, name: "id_documents", granted: false)
                                .query
                                .where("verification_metadata->'rejected' IS NULL")
-                               .where("verification_metadata->>'verification_type' = 'offline'")
-                               .find_by(user: authorization_user)
+                               .where(user: authorization_user)
+                               .find { |auth| auth.verification_metadata["verification_type"] == "offline" }
           end
 
           def authorization_user

data/app/controllers/decidim/verifications/id_documents/admin/pending_authorizations_controller.rb

@@ -22,7 +22,7 @@ module Decidim
               .new(organization: current_organization, name: "id_documents", granted: false)
               .query
               .where("verification_metadata->'rejected' IS NULL")
-              .where("verification_metadata->>'verification_type' = 'online'")
+              .select { |auth| auth.verification_metadata["verification_type"] == "online" }
           end
 
           def has_offline_method?

data/app/events/decidim/verifications/managed_user_error_event.rb

@@ -0,0 +1,39 @@
+# frozen-string_literal: true
+
+module Decidim
+  module Verifications
+    class ManagedUserErrorEvent < Decidim::Events::SimpleEvent
+      delegate :profile_path, :profile_url, :name, to: :updated_user
+
+      def i18n_scope
+        "decidim.events.verifications.verify_with_managed_user"
+      end
+
+      def resource_path
+        profile_path
+      end
+
+      def resource_url
+        profile_url
+      end
+
+      def resource_title
+        updated_user.name
+      end
+
+      def default_i18n_options
+        super.merge({ managed_user_path: managed_user.profile_path, managed_user_name: managed_user.name })
+      end
+
+      private
+
+      def updated_user
+        @updated_user ||= Decidim::UserPresenter.new(resource.current_user)
+      end
+
+      def managed_user
+        @managed_user ||= Decidim::UserPresenter.new(resource.managed_user)
+      end
+    end
+  end
+end

data/app/forms/decidim/verifications/sms/mobile_phone_form.rb

@@ -26,7 +26,7 @@ module Decidim
         def mobile_phone_number
           return unless super
 
-          super.gsub(/[^\+0-9]/, "")
+          super.gsub(/[^+0-9]/, "")
         end
 
         # The verification metadata to validate in the next step.

data/app/models/decidim/verifications/conflict.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module Decidim::Verifications
+  class Conflict < ApplicationRecord
+    belongs_to :current_user, class_name: "User"
+    belongs_to :managed_user, class_name: "User"
+  end
+end

data/app/models/decidim/verifications/csv_census/data.rb

@@ -15,6 +15,7 @@ module Decidim
       # Returns nothing
       class Data
         attr_reader :errors, :values
+
         def initialize(file)
           @file = file
           @values = []

data/app/queries/decidim/verifications/authorizations.rb

@@ -26,9 +26,10 @@ module Decidim
         scope = scope.where(name: name) unless name.nil?
         scope = scope.where(user: user) unless user.nil?
 
-        if granted == true
+        case granted
+        when true
           scope = scope.where.not(granted_at: nil)
-        elsif granted == false
+        when false
           scope = scope.where(granted_at: nil)
         end
 

data/app/queries/decidim/verifications/authorizations_before_date.rb

@@ -34,13 +34,11 @@ module Decidim
 
         query = query.where("#{Decidim::Authorization.table_name}.created_at < ?", date) unless date.nil?
 
-        query = if granted
-                  query.where.not(granted_at: nil)
-                else
-                  query.where(granted_at: nil)
-                end
-
-        query
+        if granted
+          query.where.not(granted_at: nil)
+        else
+          query.where(granted_at: nil)
+        end
       end
 
       private

data/app/services/decidim/authorization_handler.rb

@@ -48,7 +48,7 @@ module Decidim
     #
     # Returns a String.
     def to_partial_path
-      handler_name.sub!(/_handler$/, "") + "/form"
+      "#{handler_name.sub!(/_handler$/, "")}/form"
     end
 
     # Any data that the developer would like to inject to the `metadata` field

data/app/views/decidim/verifications/authorizations/index.html.erb

@@ -8,7 +8,7 @@
       <div class="card card--list">
         <% @granted_authorizations.each do |authorization| %>
           <% if authorization.expired? %>
-            <%= link_to authorization_method(authorization).root_path(url_params), title: t(".expired_verification"), class: "card--list__item" do %>
+            <%= link_to authorization_method(authorization).root_path(**url_params), title: t(".expired_verification"), class: "card--list__item" do %>
               <% render partial: "granted_authorization", locals: { authorization: authorization } %>
             <% end %>
           <% elsif authorization.renewable? %>
@@ -29,7 +29,7 @@
     <% if @pending_authorizations.any? %>
       <div class="card card--list">
         <% @pending_authorizations.each do |authorization| %>
-          <%= link_to authorization_method(authorization).resume_authorization_path(url_params) do %>
+          <%= link_to authorization_method(authorization).resume_authorization_path(**url_params) do %>
             <div class="card--list__item">
               <div class="card--list__text">
                 <%= icon "reload", class: "card--list__icon", aria_label: t(".pending_verification"), role: "img" %>
@@ -56,7 +56,7 @@
     <% if unauthorized_methods.any? %>
       <div class="card card--list">
         <% unauthorized_methods.each do |unauthorized_method| %>
-          <%= link_to unauthorized_method.root_path(url_params) do %>
+          <%= link_to unauthorized_method.root_path(**url_params) do %>
             <div class="card--list__item">
               <div class="card--list__text">
                 <%= icon "lock-unlocked", class: "card--list__icon", aria_label: t(".unauthorized_verification"), role: "img" %>

data/app/views/dummy_authorization/_form.html.erb

@@ -1,10 +1,10 @@
 <% if request["postal_codes"].present?
   postal_codes = request["postal_codes"].split(",") %>
-  <p><%= t("extra_explanation.postal_codes", scope: "decidim.verifications.dummy_authorization", postal_codes: postal_codes.join(", "), count: postal_codes.count) %></p>
+  <p><%= t("extra_explanation.postal_codes", scope: "decidim.verifications.dummy_authorization", postal_codes: postal_codes.join(", "), count: postal_codes.count, user_postal_code: request["user_postal_code"]) %></p>
 <% end %>
 
 <% if request["scope"] && (scope = Decidim::Scope.find(request["scope"]&.to_i)) %>
-  <p><%= t("extra_explanation.scope", scope: "decidim.verifications.dummy_authorization", scope_name: translated_attribute(scope.name)) %></p>
+  <p><%= t("extra_explanation.scope", scope: "decidim.verifications.dummy_authorization", scope_name: translated_attribute(scope.name), user_scope_name: request["user_scope_name"]) %></p>
 <% end %>
 
 <div class="partial-demo">

data/config/locales/ar.yml

@@ -24,14 +24,6 @@ ar:
         full_address: العنوان الكامل
         verification_code: شيفرة التأكيد
   decidim:
-    admin:
-      menu:
-        authorization_workflows: التحقق
-    admin_log:
-      organization:
-        update_id_documents_config: "%{user_name} بتحديث تكوين التحقق من وثائق الهوية"
-      user:
-        grant_id_documents_offline_verification: "%{user_name} التحقق من %{resource_name} باستخدام التحقق من مستندات الهوية دون اتصال"
     authorization_handlers:
       admin:
         csv_census:
@@ -103,7 +95,6 @@ ar:
             title: بيانات التعداد الحالية
           instructions:
             body: للقيام بذلك ، يجب عليك إدخال إدارة النظام وإضافة تصاريح csv_census إلى المؤسسة
-            title: تحتاج إلى تنشيط التعداد CSV لهذه المنظمة
           new:
             file: ".csv ملف مع بيانات رسائل البريد الإلكتروني"
             submit: رفع ملف

data/config/locales/ca.yml

@@ -34,16 +34,8 @@ ca:
             confirm: La revocació d'autoritzacions anteriors a aquesta data no es pot desfer. Segur que vols continuar?
             confirm_all: La revocació de totes les autoritzacions no es pot desfer. Segur que vols continuar?
           destroy_nok: S'ha produït un error al revocar les autoritzacions.
-          destroy_ok: Totes les autoritzacions coincidents s'han revocat correctament!
           info: Hi ha un total de %{count} participants verificades.
           no_data: No hi ha participants verificades
-          title: Revocació de les autoritzacions
-        authorization_workflows: Verificacions
-    admin_log:
-      organization:
-        update_id_documents_config: "%{user_name} ha actualitzat la configuració de verificació mitjançant documents d'identitat"
-      user:
-        grant_id_documents_offline_verification: "%{user_name} ha verificat %{resource_name} utilitzant la verificació presencial mitjançant documents d'identitat"
     authorization_handlers:
       admin:
         csv_census:
@@ -131,7 +123,6 @@ ca:
             title: Dades censals actuals
           instructions:
             body: Per fer-ho, has d'entrar al panell d'administració i afegir les autoritzacions de csv_census a l'organització
-            title: Necessites activar el cens csv d'aquesta organització
           new:
             file: "fitxer .csv amb dades de correus electrònics"
             info: 'Ha de ser un document en format CSV amb una única columna indicant les adreces electròniques:'

data/config/locales/cs.yml

@@ -7,10 +7,10 @@ cs:
         offline_explanation: Pokyny pro ověření offline
         online: Online
       id_document_information:
-        document_number: Číslo dokumentu (s písmenem)
+        document_number: Číslo dokladu (jen číslice)
         document_type: Typ dokumentu
       id_document_upload:
-        document_number: Číslo dokumentu (s písmenem)
+        document_number: Číslo dokladu (jen číslice)
         document_type: Typ dokumentu
         user: Uživatel
         verification_attachment: Naskenovaná kopie dokumentu
@@ -34,16 +34,16 @@ cs:
             confirm: Zrušení před datem autorizace nelze vrátit zpět. Jste si jisti, že chcete pokračovat?
             confirm_all: Zrušení všech autorizací nelze vrátit zpět. Jste si jisti, že chcete pokračovat?
           destroy_nok: Při rušení autorizace došlo k chybě.
-          destroy_ok: Všechny odpovídající autorizace byly úspěšně zrušeny!
+          destroy_ok: Všechny odpovídající autorizace byly úspěšně zrušeny.
           info: Celkem je %{count} ověřených účastníků.
           no_data: Žádní ověření účastníci
-          title: Zrušení ověření
-        authorization_workflows: Ověření
+          title: Zrušení autorizace
+        authorization_workflows: Autorizace
     admin_log:
       organization:
-        update_id_documents_config: "%{user_name} aktualizovala konfiguraci ověření dokumentů identit"
+        update_id_documents_config: "%{user_name} aktualizoval konfiguraci autorizace dokladů totožnosti"
       user:
-        grant_id_documents_offline_verification: "%{user_name} ověřeno %{resource_name} pomocí ověření dokumentů offline"
+        grant_id_documents_offline_verification: "%{user_name} ověřil %{resource_name} pomocí offline autorizace dokladů totožnosti"
     authorization_handlers:
       admin:
         csv_census:
@@ -63,8 +63,8 @@ cs:
             - Označte písmeno jako odesláno.
             - Jakmile označíte odeslané písmeno, uživatel bude schopen kód zadat a ověřit.
       csv_census:
-        explanation: Získejte ověření pomocí sčítání organizace
-        name: Sčítání organizace
+        explanation: Získejte ověření pomocí členů organizace
+        name: Členové organizace
       direct: Přímo
       help: Pomoc
       id_documents:
@@ -74,7 +74,11 @@ cs:
       name: název
       postal_letter:
         explanation: Pošleme vám poštovní dopis s kódem, který budete muset zadat, abychom mohli ověřit vaši adresu
-        name: Kód poštovním dopisem
+        name: Kód v listovní zásilce
+    events:
+      verifications:
+        verify_with_managed_user:
+          notification_title: Účastník <a href="%{resource_path}">%{resource_title}</a> se pokusil ověřit údaji spravovaného účastníka <a href="%{managed_user_path}">%{managed_user_name}</a>
     verifications:
       authorizations:
         authorization_metadata:
@@ -90,7 +94,7 @@ cs:
         first_login:
           actions:
             another_dummy_authorization_handler: Ověřte proti jinému příkladu pověření pro autorizaci
-            csv_census: Ověřte sčítání organizace
+            csv_census: Ověřte prostřednictvím členů organizace
             dummy_authorization_handler: Ověřte proti příkladu pověření autorizace
             dummy_authorization_workflow: Ověřte proti pracovnímu postupu autorizace
             id_documents: Získejte ověření odesláním dokladu totožnosti
@@ -118,28 +122,28 @@ cs:
         admin:
           census:
             create:
-              error: Při importu sčítání došlo k chybě.
+              error: Při importu členů organizace došlo k chybě.
               success: Úspěšně importováno %{count} položek (%{errors} chyb)
             destroy_all:
-              success: Všechna data sčítání byly smazány
+              success: Všechna data členů organizace byly smazány
           destroy:
-            confirm: Odstranit veškeré sčítání nelze vrátit zpět. Jste si jistý, že chcete pokračovat?
-            title: Smazat všechna data sčítání
+            confirm: Odstranění veškerých kontaktů členů organizace nelze vrátit zpět. Jste si jistý, že chcete pokračovat?
+            title: Smazat všechna data členů organizace
           index:
             data: Celkem bylo načteno celkem %{count} záznamů. Poslední nahraje datum %{due_date}
-            empty: Neexistují údaje o sčítání. Pomocí níže uvedeného formuláře jej importujte pomocí souboru CSV.
-            title: Aktuální údaje o sčítání
+            empty: Neexistují data o členech organizace. Pomocí níže uvedeného formuláře je importujte souborem CSV.
+            title: Aktuální údaje o členech organizace
           instructions:
-            body: Chcete-li to provést, musíte vstoupit do správy systému a přidat oprávnění csv_census organizaci
-            title: Pro tuto organizaci musíte aktivovat csv sčítání
+            body: Chcete-li to provést, musíte vstoupit do správy systému a přidat oprávnění prostřednictvím souboru csv s e-maily členů organizace
+            title: Musíte aktivovat CSV s kontakty členů organizace
           new:
             file: ".csv soubor s daty e-mailů"
-            info: 'Must být souborem ve formátu CSV s pouze jedním sloupcem s e-mailovou adresou:'
+            info: 'Must být soubor ve formátu CSV s pouze jedním sloupcem s e-mailovou adresou:'
             submit: Nahrát soubor
-            title: Nahrajte nové sčítání
+            title: Nahrajte nové členy organizace
         authorizations:
           new:
-            error: Váš účet nebyl ověřen, nebo nejste v sčítání organizace.
+            error: Váš účet nebyl ověřen, nebo nejste členem organizace.
             success: Účet byl úspěšně ověřen.
       dummy_authorization:
         extra_explanation:
@@ -149,6 +153,12 @@ cs:
             many: 'Účast je omezena na uživatele s některým z následujících poštovních směrovacích čísel: %{postal_codes}.'
             other: 'Účast je omezena na uživatele s některým z následujících poštovních směrovacích čísel: %{postal_codes}.'
           scope: Účast je omezena na účastníky s rozsahem %{scope_name}.
+          user_postal_codes:
+            one: Účast je omezena na účastníky s poštovním kódem %{postal_codes} a vaše poštovní směrovací číslo je %{user_postal_code}.
+            few: 'Účast je omezena na účastníky s poštovním kódem %{postal_codes} a vaše poštovní směrovací číslo je %{user_postal_code}.'
+            many: 'Účast je omezena na účastníky s poštovními kódy %{postal_codes} a vaše poštovní směrovací číslo je %{user_postal_code}.'
+            other: 'Účast je omezena na účastníky s kterýmkoli z následujících poštovních kódů: %{postal_codes}. Vaše poštovní směrovací číslo je %{user_postal_code}.'
+          user_scope: Účast je omezena na účastníky s rozsahem %{scope_name} a váš rozsah je %{user_scope_name}.
       id_documents:
         admin:
           config:
@@ -168,7 +178,7 @@ cs:
               verify: Ověřte
           offline_confirmations:
             create:
-              error: Ověření neodpovídá. Zkuste to znovu nebo řekněte uživateli, že ho změní
+              error: Ověření neodpovídá vloženým údajům. Zkuste to znovu, nebo řekněte uživateli, ať si je změní
               success: Uživatel úspěšně ověřen
             new:
               cancel: zrušení
@@ -176,7 +186,7 @@ cs:
               verify: Ověřte
           pending_authorizations:
             index:
-              config: Config
+              config: Konfigurace
               offline_verification: Ověření offline
               title: Čekající ověření online
               verification_number: 'Ověření #%{n}'
@@ -207,7 +217,7 @@ cs:
             error: Při načítání dokumentu došlo k problému
             success: Dokument byl úspěšně obnoven
         dni: DNI
-        nie: NIE
+        nie: Občanský průkaz
         passport: Cestovní pas
       postal_letter:
         admin: