decidim-verifications 0.15.2 → 0.16.0

data/app/forms/decidim/verifications/id_documents/admin/config_form.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Verifications
+    module IdDocuments
+      # A form object to be used as the base for identity document verification
+      module Admin
+        class ConfigForm < Decidim::Form
+          include TranslatableAttributes
+          mimic :config
+
+          attribute :offline, Boolean
+          attribute :online, Boolean
+          translatable_attribute :offline_explanation, String
+
+          validates :offline_explanation, translatable_presence: true, if: :offline
+          validate :has_some_method_selected?
+
+          def map_model(model)
+            self.online = model.id_documents_methods.include?("online")
+            self.offline = model.id_documents_methods.include?("offline")
+            self.offline_explanation = model.id_documents_explanation_text
+          end
+
+          def has_some_method_selected?
+            return if online || offline
+
+            errors.add(:online, :invalid)
+            errors.add(:offline, :invalid)
+          end
+
+          def selected_methods
+            methods = []
+            methods << "offline" if offline
+            methods << "online" if online
+            methods
+          end
+        end
+      end
+    end
+  end
+end

data/app/forms/decidim/verifications/id_documents/admin/offline_confirmation_form.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Verifications
+    module IdDocuments
+      # A form object to be used as the base for identity document verification
+      module Admin
+        class OfflineConfirmationForm < InformationForm
+          attribute :email, String
+
+          validates :email, presence: true
+        end
+      end
+    end
+  end
+end

data/app/forms/decidim/verifications/id_documents/information_form.rb

@@ -11,6 +11,7 @@ module Decidim
 
         attribute :document_number, String
         attribute :document_type, String
+        attribute :verification_type, String
 
         validates :document_type,
                   inclusion: { in: DOCUMENT_TYPES },
@@ -20,6 +21,10 @@ module Decidim
                   format: { with: /\A[A-Z0-9]*\z/, message: I18n.t("errors.messages.uppercase_only_letters_numbers") },
                   presence: true
 
+        validates :verification_type,
+                  presence: true,
+                  inclusion: { in: %w(offline online) }
+
         def handler_name
           "id_documents"
         end
@@ -27,12 +32,14 @@ module Decidim
         def map_model(model)
           self.document_type = model.verification_metadata["document_type"]
           self.document_number = model.verification_metadata["document_number"]
+          self.verification_type = model.verification_metadata["verification_type"].presence || "online"
         end
 
         def verification_metadata
           {
             "document_type" => document_type,
-            "document_number" => document_number
+            "document_number" => document_number,
+            "verification_type" => verification_type
           }
         end
 
@@ -44,6 +51,10 @@ module Decidim
             ]
           end
         end
+
+        def uses_online_method?
+          verification_type == "online"
+        end
       end
     end
   end

data/app/forms/decidim/verifications/id_documents/upload_form.rb

@@ -13,7 +13,8 @@ module Decidim
         validates :verification_attachment,
                   file_size: { less_than_or_equal_to: ->(_record) { Decidim.maximum_attachment_size } },
                   file_content_type: { allow: ["image/jpeg", "image/png"] },
-                  presence: true
+                  presence: true,
+                  if: :uses_online_method?
       end
     end
   end

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Verifications
+    module Sms
+      # A form object that just holds a verification code that the user will
+      # have received by SMS.
+      class ConfirmationForm < AuthorizationHandler
+        attribute :verification_code, String
+
+        validates :verification_code, presence: true
+
+        def verification_metadata
+          { "verification_code" => verification_code }
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+require "securerandom"
+
+module Decidim
+  module Verifications
+    module Sms
+      # A form object to be used when public users want to get verified using their phone.
+      class MobilePhoneForm < AuthorizationHandler
+        attribute :mobile_phone_number, String
+
+        validates :mobile_phone_number, :verification_code, :sms_gateway, presence: true
+
+        def handler_name
+          "sms"
+        end
+
+        # A mobile phone can only be verified once but it should be private.
+        def unique_id
+          Digest::MD5.hexdigest(
+            "#{mobile_phone_number}-#{Rails.application.secrets.secret_key_base}"
+          )
+        end
+
+        # When there's a phone number, sanitize it allowing only numbers and +.
+        def mobile_phone_number
+          return unless super
+          super.gsub(/[^\+0-9]/, "")
+        end
+
+        # The verification metadata to validate in the next step.
+        def verification_metadata
+          {
+            verification_code: verification_code,
+            code_sent_at: Time.current
+          }
+        end
+
+        private
+
+        def verification_code
+          return unless sms_gateway
+          return @verification_code if defined?(@verification_code)
+
+          return unless sms_gateway.new(mobile_phone_number, generated_code).deliver_code
+          @verification_code = generated_code
+        end
+
+        def sms_gateway
+          Decidim.sms_gateway_service.to_s.safe_constantize
+        end
+
+        def generated_code
+          @generated_code ||= SecureRandom.random_number(1_000_000).to_s
+        end
+      end
+    end
+  end
+end

data/app/views/decidim/verifications/id_documents/admin/config/edit.html.erb

@@ -0,0 +1,26 @@
+<%= decidim_form_for(@form, url: { action: :update },
+                            html: { class: "form" }) do |form| %>
+  <div class="card">
+    <div class="card-divider">
+      <h2 class="card-title">
+        <%= t(".title") %>
+      </h2>
+    </div>
+
+    <div class="card-section">
+      <div class="row column">
+        <%= label_tag(:available_methods, t("activemodel.attributes.config.available_methods")) %>
+        <%= form.check_box :online %>
+        <%= form.check_box :offline %>
+      </div>
+
+      <div class="row column">
+        <%= form.translated :editor, :offline_explanation, lines: 10 %>
+      </div>
+    </div>
+  </div>
+
+  <div class="button--double form-general-submit">
+    <%= form.submit t(".update") %>
+  </div>
+<% end %>

data/app/views/decidim/verifications/id_documents/admin/confirmations/new.html.erb

@@ -16,6 +16,7 @@
       <div class="row column">
         <%= form.select :document_type, @form.document_types_for_select, prompt: true, selected: nil %>
         <%= form.text_field :document_number, value: nil %>
+        <%= form.hidden_field :verification_type, value: :online %>
       </div>
     </div>
   </div>

data/app/views/decidim/verifications/id_documents/admin/offline_confirmations/new.html.erb

@@ -0,0 +1,24 @@
+<%= decidim_form_for(@form, url: offline_confirmation_path,
+                            html: { class: "form" }) do |form| %>
+  <div class="card">
+    <div class="card-divider">
+      <h2 class="card-title">
+        <%= t(".introduce_user_data") %>
+      </h2>
+    </div>
+
+    <div class="card-section">
+      <div class="row column">
+        <%= form.text_field :email %>
+        <%= form.select :document_type, @form.document_types_for_select, prompt: true %>
+        <%= form.text_field :document_number %>
+        <%= form.hidden_field :verification_type, value: :offline %>
+      </div>
+    </div>
+  </div>
+
+  <div class="button--double form-general-submit">
+    <%= form.submit t(".verify") %>
+    <%= link_to t(".cancel"), root_path, class: "button hollow" %>
+  </div>
+<% end %>

data/app/views/decidim/verifications/id_documents/admin/pending_authorizations/index.html.erb

@@ -2,6 +2,8 @@
   <div class="card-divider">
     <h2 class="card-title">
       <%= t(".title") %>
+        <%= link_to t(".config"), edit_config_path, class: "button tiny button--title" %>
+        <%= link_to t(".offline_verification"), new_offline_confirmation_path, class: "button tiny button--title" if has_offline_method? %>
     </h2>
   </div>
   <div class="card-section">
@@ -14,7 +16,7 @@
       </thead>
       <tbody>
         <tr>
-          <% @pending_authorizations.each do |authorization| %>
+          <% @pending_online_authorizations.each do |authorization| %>
             <td>
               <%= link_to t(".verification_number", n: authorization.id),
                           new_pending_authorization_confirmation_path(authorization.id) %>

data/app/views/decidim/verifications/id_documents/authorizations/_form.html.erb

@@ -0,0 +1,17 @@
+<div class="card-section">
+  <div class="field">
+    <%= form.select :document_type, @form.document_types_for_select, prompt: true %>
+  </div>
+
+  <div class="field">
+    <%= form.text_field :document_number %>
+  </div>
+
+  <% if using_online? %>
+    <div class="field">
+      <%= form.upload :verification_attachment %>
+    </div>
+  <% end %>
+
+  <%= form.hidden_field :verification_type, value: verification_type %>
+</div>

data/app/views/decidim/verifications/id_documents/authorizations/choose.html.erb

@@ -0,0 +1,21 @@
+<div class="wrapper">
+  <div class="row collapse">
+    <div class="row collapse">
+      <div class="columns large-8 large-centered text-center page-title">
+        <h1><%= t(".title") %></h1>
+      </div>
+    </div>
+
+    <div class="row">
+      <div class="columns large-6 medium-centered">
+        <div class="card">
+          <div class="card__content">
+            <p><%= t(".choose_a_type") %></p>
+            <%= link_to t(".offline"), { action: :new, using: :offline }, class: "button button--sc expanded hollow" %>
+            <%= link_to t(".online"), { action: :new, using: :online }, class: "button button--sc expanded" %>
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+</div>

data/app/views/decidim/verifications/id_documents/authorizations/edit.html.erb

@@ -17,37 +17,28 @@
     </div>
   </div>
 
-  <% if authorization.rejected? %>
-    <div class="row">
-      <div class="columns large-6 medium-centered">
-        <div class="row column">
-          <div class="card">
-            <div class="card__content">
-              <%= decidim_form_for(@form, url: authorization_path, method: :put) do |form| %>
-              <div class="card-section">
-                <div class="field">
-                  <%= form.select :document_type, @form.document_types_for_select, prompt: true %>
-                </div>
-
-                <div class="field">
-                  <%= form.text_field :document_number %>
-                </div>
-
-                <div class="field">
-                  <%= form.upload :verification_attachment, optional: false %>
-                </div>
-              </div>
+  <div class="row">
+    <div class="columns large-6 medium-centered">
+      <div class="row column">
+        <div class="card">
+          <div class="card__content">
+            <%= translated_attribute(current_organization.id_documents_explanation_text).html_safe unless using_online? %>
+            <%= decidim_form_for(@form, url: authorization_path, method: :put) do |form| %>
+              <%= render partial: "form", locals: { form: form } %>
 
               <div class="card-section">
                 <div class="actions">
                   <%= form.submit t(".send"), class: "button expanded", "data-disable-with" => "#{t('.send')}..." %>
+                  <% if available_methods.count > 1 %>
+                    <%= link_to t(".offline"), { action: :edit, using: :offline }, class: "button expanded hollow" if using_online? %>
+                    <%= link_to t(".online"), { action: :edit, using: :online }, class: "button expanded hollow" if using_offline? %>
+                  <% end %>
                 </div>
               </div>
-              <% end %>
-            </div>
+            <% end %>
           </div>
         </div>
       </div>
     </div>
-  <% end %>
+  </div>
 </div>

data/app/views/decidim/verifications/id_documents/authorizations/new.html.erb

@@ -10,21 +10,14 @@
       <div class="columns large-6 medium-centered">
         <div class="card">
           <div class="card__content">
+            <%= translated_attribute(current_organization.id_documents_explanation_text).html_safe unless using_online? %>
             <%= decidim_form_for(@form, url: authorization_path) do |form| %>
-              <div class="field">
-                <%= form.select :document_type, @form.document_types_for_select, prompt: true %>
-              </div>
-
-              <div class="field">
-                <%= form.text_field :document_number %>
-              </div>
-
-              <div class="field">
-                <%= form.upload :verification_attachment %>
-              </div>
+              <%= render partial: "form", locals: { form: form } %>
 
-              <div class="actions">
-                <%= form.submit t(".send"), class: "button expanded", "data-disable-with" => "#{t('.send')}..." %>
+              <div class="card-section">
+                <div class="actions">
+                  <%= form.submit t(".send"), class: "button expanded", "data-disable-with" => "#{t('.send')}..." %>
+                </div>
               </div>
             <% end %>
           </div>

data/app/views/decidim/verifications/sms/authorizations/edit.html.erb

@@ -0,0 +1,27 @@
+<div class="wrapper">
+  <div class="row collapse">
+    <div class="row collapse">
+      <div class="columns large-8 large-centered text-center page-title">
+        <h1><%= t(".title") %></h1>
+      </div>
+    </div>
+
+    <div class="row">
+      <div class="columns large-6 medium-centered">
+        <div class="card">
+          <div class="card__content">
+            <%= decidim_form_for(@form, url: authorization_path, method: :put) do |form| %>
+              <div class="field">
+                <%= form.text_field :verification_code %>
+              </div>
+
+              <div class="actions">
+                <%= form.submit t(".send"), class: "button expanded", "data-disable-with" => "#{t('.send')}..." %>
+              </div>
+            <% end %>
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+</div>

data/app/views/decidim/verifications/sms/authorizations/new.html.erb

@@ -0,0 +1,27 @@
+<div class="wrapper">
+  <div class="row collapse">
+    <div class="row collapse">
+      <div class="columns large-8 large-centered text-center page-title">
+        <h1><%= t(".title") %></h1>
+      </div>
+    </div>
+
+    <div class="row">
+      <div class="columns large-6 medium-centered">
+        <div class="card">
+          <div class="card__content">
+            <%= decidim_form_for(@form, url: authorization_path) do |form| %>
+              <div class="field">
+                <%= form.text_field :mobile_phone_number %>
+              </div>
+
+              <div class="actions">
+                <%= form.submit t(".send"), class: "button expanded", "data-disable-with" => "#{t('.send')}..." %>
+              </div>
+            <% end %>
+          </div>
+        </div>
+      </div>
+    </div>
+  </div>
+</div>