decidim-verifications 0.31.5 → 0.32.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ab23660af67a5437d887ae8dfd0e774ab25c7d3569179f02015849841c17ad86
-  data.tar.gz: 06c6dc46eb3d1f008232532da895891d28d16350c5b0f65866bb33d2264f4567
+  metadata.gz: 18eeb490e000d6bdf84f6a1745e60fd21d005e88c2a9bb16a8e660b6b090ea2b
+  data.tar.gz: 82511430b2884829f35dae429c9bc611bf14c88f16f4328fa9d3a87e87dc560e
 SHA512:
-  metadata.gz: 74064a45c98e7c14df37af2f9df691d639aeadb07d0320ae661beef1f5e2269d453dcb2bf4b4101eea1d9877f402c9fd8139c4b984da147c8d514644b72acc93
-  data.tar.gz: e4035eb218a812d69cefce1af846e59b7f774f2faec189cd95a9223543a390a3faeba173f07bf4b6c218d52b0e23c42c3f79ec5acc5b7fdc6277a0840577d551
+  metadata.gz: '096de425a399c348ec3f0964d53a484310686280b3ed1d28de458566585b7f65ea0b9bf306eb9706cbd37062c498661bcdfe0080f1a0875c0304d7c946cfeb34'
+  data.tar.gz: 5a7f737b1c79c194e1c859c5ef83c5fb07bf8fc2fae183ec3994fc60b27474f36e7590c8f0b188f707d1c7e29f1a0fb8e5cea4d9f765ea7ebdaf31de7cc22426

data/README.md

@@ -1,4 +1,4 @@
-# Decidim::Verifications
+# decidim-verifications
 
 Decidim offers several methods for allowing participants to get authorization to
 perform certain privileged actions. This module implements several of those methods
@@ -23,6 +23,10 @@ There are several use cases for this, such as
 Right now Decidim supports only a few of these cases, but we have an internal
 API where you can program your own kind of authorizations.
 
+## Usage
+
+This is a module oriented for developers, as a building block to be used by other modules.
+
 ## Introduction
 
 Each decidim instance is in full control of its authorizations, and can customize:
@@ -151,7 +155,7 @@ In order to setup Decidim with SMS verification you need to:
 Keep in mind that Decidim will not store a free text version of the mobile phone, only a hashed
 version so we can avoid duplicates and guarantee the users' privacy.
 
-You can find an example [here][example SMS gateway].
+You can find an example in the [example SMS gateway implementation][example SMS gateway].
 
 ## Authorization options
 
@@ -243,17 +247,7 @@ authorization.
 
 ## Installation
 
-Add this line to your application's Gemfile:
-
-```ruby
-gem 'decidim-verifications'
-```
-
-And then execute:
-
-```bash
-bundle
-```
+This is on the default Decidim installation so you should not change anything to use this feature.
 
 ## Contributing
 

data/app/controllers/concerns/decidim/verifications/renewable.rb

@@ -7,6 +7,7 @@ module Decidim
     # Common logic to renew authorizations
     module Renewable
       extend ActiveSupport::Concern
+
       included do
         def renew
           enforce_permission_to(:renew, :authorization, authorization:)

data/app/controllers/decidim/verifications/application_controller.rb

@@ -27,9 +27,8 @@ module Decidim
       def confirmed_user
         return true if !current_user || (current_user && current_user.verifiable?)
 
-        redirect_back(
-          fallback_location: root_path,
-          alert: t(
+        redirect_back_or_to(
+          root_path, alert: t(
             "authorizations.create.unconfirmed",
             scope: "decidim.verifications"
           )

data/app/controllers/decidim/verifications/authorizations_controller.rb

@@ -14,6 +14,7 @@ module Decidim
       include Decidim::UserProfile
       include Decidim::HtmlSafeFlash
       include Decidim::Verifications::Renewable
+
       helper Decidim::DecidimFormHelper
       helper Decidim::AuthorizationFormHelper
       helper Decidim::TranslationsHelper
@@ -25,7 +26,7 @@ module Decidim
       def index; end
 
       def onboarding_pending
-        return redirect_back(fallback_location: authorizations_path) unless onboarding_manager.valid?
+        return redirect_back_or_to(authorizations_path) unless onboarding_manager.valid?
 
         authorizations = action_authorized_to(onboarding_manager.action, **onboarding_manager.action_authorized_resources)
 
@@ -86,7 +87,7 @@ module Decidim
 
           on(:invalid) do
             flash[:alert] = t("authorizations.create.error", scope: "decidim.verifications")
-            render action: :new, status: :unprocessable_entity
+            render action: :new, status: :unprocessable_content
           end
         end
       end
@@ -151,7 +152,7 @@ module Decidim
         authorizations = action_authorized_to(onboarding_manager.action, **onboarding_manager.action_authorized_resources)
         return unless authorizations.ephemeral?
 
-        form = Decidim::EphemeralUserForm.new(organization: current_organization, locale: current_locale)
+        form = Decidim::EphemeralUserForm.new(organization: current_organization)
         CreateEphemeralUser.call(form) do
           on(:ok) do |ephemeral_user|
             sign_in(ephemeral_user)

data/app/controllers/decidim/verifications/csv_census/admin/census_controller.rb

@@ -18,13 +18,9 @@ module Decidim
 
           helper_method :csv_census_data, :last_login
 
-          def index
-            enforce_permission_to :index, :authorization
-          end
+          def index; end
 
           def destroy
-            enforce_permission_to :destroy, :authorization
-
             Decidim::Commands::DestroyResource.call(census_data, current_user) do
               on(:ok) do
                 flash[:notice] = I18n.t("census.destroy.success", scope: "decidim.verifications.csv_census.admin")
@@ -34,15 +30,12 @@ module Decidim
           end
 
           def new_import
-            enforce_permission_to :create, :authorization
-
             @form = form(CensusDataForm).from_params(params)
             @status = Status.new(current_organization)
           end
 
           def create_import
             enforce_permission_to :create, :authorization
-
             @form = form(CensusDataForm).from_params(params)
             @status = Status.new(current_organization)
 
@@ -83,7 +76,6 @@ module Decidim
 
           def show_instructions
             enforce_permission_to :index, :authorization
-
             render :instructions
           end
 

data/app/controllers/decidim/verifications/csv_census/admin/census_records_controller.rb

@@ -10,14 +10,10 @@ module Decidim
           helper_method :csv_census_data
 
           def new_record
-            enforce_permission_to :create, :authorization
-
             @form = form(Admin::CensusForm).instance
           end
 
           def create_record
-            enforce_permission_to :create, :authorization
-
             @form = form(Admin::CensusForm).from_params(params)
             Admin::CreateCensusRecord.call(@form) do
               on(:ok) do
@@ -26,20 +22,16 @@ module Decidim
               end
 
               on(:invalid) do
-                render :new_record, status: :unprocessable_entity
+                render :new_record, status: :unprocessable_content
               end
             end
           end
 
           def edit_record
-            enforce_permission_to :update, :authorization
-
             @form = form(Admin::CensusForm).from_model(census_data)
           end
 
           def update_record
-            enforce_permission_to :update, :authorization
-
             @form = form(Admin::CensusForm).from_params(params)
 
             Admin::UpdateCensusRecord.call(@form, census_data) do
@@ -50,7 +42,7 @@ module Decidim
 
               on(:invalid) do
                 flash.now[:alert] = I18n.t("census_records.update_record.invalid", scope: "decidim.verifications.csv_census.admin")
-                render action: "edit_record", status: :unprocessable_entity
+                render action: "edit_record", status: :unprocessable_content
               end
             end
           end

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

@@ -33,7 +33,7 @@ module Decidim
 
               on(:invalid) do
                 flash.now[:alert] = t("config.update.error", scope: "decidim.verifications.id_documents.admin")
-                render action: :edit, status: :unprocessable_entity
+                render action: :edit, status: :unprocessable_content
               end
             end
           end

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

@@ -13,7 +13,6 @@ module Decidim
           before_action :load_pending_authorization
 
           include Decidim::Admin::WorkflowsBreadcrumb
-          include Decidim::Verifications::Admin::PendingAuthorizationLoader
 
           add_breadcrumb_item_from_menu :workflows_menu
 
@@ -36,7 +35,7 @@ module Decidim
 
               on(:invalid) do
                 flash.now[:alert] = t("confirmations.create.error", scope: "decidim.verifications.id_documents.admin")
-                render action: :new, status: :unprocessable_entity
+                render action: :new, status: :unprocessable_content
               end
             end
           end
@@ -44,7 +43,7 @@ module Decidim
           private
 
           def load_pending_authorization
-            @pending_authorization = load_pending_authorization!("id_documents", params[:pending_authorization_id])
+            @pending_authorization = Authorization.find(params[:pending_authorization_id])
           end
         end
       end

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

@@ -33,7 +33,7 @@ module Decidim
 
               on(:invalid) do
                 flash.now[:alert] = t("offline_confirmations.create.error", scope: "decidim.verifications.id_documents.admin")
-                render action: :new, status: :unprocessable_entity
+                render action: :new, status: :unprocessable_content
               end
             end
           end

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

@@ -13,7 +13,6 @@ module Decidim
           before_action :load_pending_authorization
 
           include Decidim::Admin::WorkflowsBreadcrumb
-          include Decidim::Verifications::Admin::PendingAuthorizationLoader
 
           add_breadcrumb_item_from_menu :workflows_menu
 
@@ -33,7 +32,7 @@ module Decidim
           private
 
           def load_pending_authorization
-            @pending_authorization = load_pending_authorization!("id_documents", params[:pending_authorization_id])
+            @pending_authorization = Authorization.find(params[:pending_authorization_id])
           end
         end
       end

data/app/controllers/decidim/verifications/id_documents/authorizations_controller.rb

@@ -42,7 +42,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = t("authorizations.create.error", scope: "decidim.verifications.id_documents")
-              render action: :new, status: :unprocessable_entity
+              render action: :new, status: :unprocessable_content
             end
           end
         end
@@ -72,7 +72,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = t("authorizations.update.error", scope: "decidim.verifications.id_documents")
-              render action: :edit, status: :unprocessable_entity
+              render action: :edit, status: :unprocessable_content
             end
           end
         end

data/app/controllers/decidim/verifications/postal_letter/admin/postages_controller.rb

@@ -13,7 +13,6 @@ module Decidim
           before_action :load_pending_authorization
 
           include Decidim::Admin::WorkflowsBreadcrumb
-          include Decidim::Verifications::Admin::PendingAuthorizationLoader
 
           add_breadcrumb_item_from_menu :workflows_menu
 
@@ -29,7 +28,7 @@ module Decidim
               end
 
               on(:invalid) do
-                render json: { error: I18n.t("postages.create.error", scope: "decidim.verifications.postal_letter.admin") }, status: :unprocessable_entity
+                render json: { error: I18n.t("postages.create.error", scope: "decidim.verifications.postal_letter.admin") }, status: :unprocessable_content
               end
             end
           end
@@ -37,7 +36,7 @@ module Decidim
           private
 
           def load_pending_authorization
-            @pending_authorization = load_pending_authorization!("postal_letter", params[:pending_authorization_id])
+            @pending_authorization = Authorization.find(params[:pending_authorization_id])
           end
         end
       end

data/app/controllers/decidim/verifications/postal_letter/authorizations_controller.rb

@@ -29,7 +29,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = t("authorizations.create.error", scope: "decidim.verifications.postal_letter")
-              render :new, status: :unprocessable_entity
+              render :new, status: :unprocessable_content
             end
           end
         end
@@ -53,7 +53,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = t("authorizations.update.error", scope: "decidim.verifications.postal_letter")
-              render :edit, status: :unprocessable_entity
+              render :edit, status: :unprocessable_content
             end
           end
         end

data/app/controllers/decidim/verifications/sms/authorizations_controller.rb

@@ -27,7 +27,7 @@ module Decidim
             end
             on(:invalid) do
               flash.now[:alert] = t("authorizations.create.error", scope: "decidim.verifications.sms")
-              render :new, status: :unprocessable_entity
+              render :new, status: :unprocessable_content
             end
           end
         end
@@ -52,7 +52,7 @@ module Decidim
 
             on(:invalid) do
               flash.now[:alert] = t("authorizations.update.error", scope: "decidim.verifications.sms")
-              render :edit, status: :unprocessable_entity
+              render :edit, status: :unprocessable_content
             end
           end
         end

data/app/forms/decidim/verifications/csv_census/admin/census_data_form.rb

@@ -8,6 +8,7 @@ module Decidim
         class CensusDataForm < Form
           include Decidim::HasUploadValidations
           include Decidim::ProcessesFileLocally
+
           mimic :census_data
 
           attribute :file, Decidim::Attributes::Blob

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

@@ -7,6 +7,7 @@ module Decidim
       module Admin
         class ConfigForm < Decidim::Form
           include TranslatableAttributes
+
           mimic :config
 
           attribute :offline, Boolean

data/app/views/decidim/verifications/csv_census/admin/census/index.html.erb

@@ -1,9 +1,9 @@
-<% add_decidim_page_title(t("admin.index.title", scope: "decidim.verifications.csv_census")) %>
+<% add_decidim_page_title(t(".title")) %>
 
 <div class="card">
   <div class="item_show__header item_show__header--with-action-options">
     <h1 class="item_show__header-title">
-      <%= t("index.title", scope: "decidim.verifications.csv_census.admin") %>
+      <%= t(".title") %>
       <%= link_to t("index.import_csv", scope: "decidim.verifications.csv_census.admin"), new_import_census_logs_path, method: :get, class: "button button__sm button__transparent-secondary" %>
       <button type="button" data-action="csv_census-record" data-census-dialog="census_drawer" data-census-url="<%= new_record_census_logs_path %>" class="button button__sm button__secondary">
         <%= t("index.add_new_record_button", scope: "decidim.verifications.csv_census.admin") %>

data/app/views/decidim/verifications/csv_census/admin/census/instructions.html.erb

@@ -1,11 +1,12 @@
-<% add_decidim_page_title(t("admin.instructions.title", scope: "decidim.verifications.csv_census") ) %>
+<% add_decidim_page_title(t(".title")) %>
+
 <div class="card">
   <div class="item_show__header">
     <h1 class="item_show__header-title">
-      <%= t("admin.instructions.title", scope: "decidim.verifications.csv_census") %>
+      <%= t(".title") %>
     </h1>
   </div>
   <div class="card-section">
-    <p><%= t("admin.instructions.body", scope: "decidim.verifications.csv_census") %></p>
+    <p><%= t(".body") %></p>
   </div>
 </div>

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

@@ -1,8 +1,8 @@
-<% add_decidim_page_title(t(".introduce_user_data")) %>
+<% add_decidim_page_title(t(".title")) %>
 
 <div class="item_show__header">
   <h1 class="item_show__header-title">
-    <%= t(".introduce_user_data") %>
+    <%= t(".title") %>
     <%= link_to t(".reject"), pending_authorization_rejection_path(@pending_authorization), method: :post, class: "button button__sm button__secondary alert button--title destroy" %>
   </h1>
 </div>
@@ -17,7 +17,7 @@
 
           <div class="card-section">
             <div class="row column">
-              <%= image_tag decidim.private_download_path(Decidim::PrivateDownload.for(@pending_authorization, attachment_name: :verification_attachment).token), class: "thumbnail" %>
+              <%= image_tag @pending_authorization.attached_uploader(:verification_attachment).variant_url(:big), class: "thumbnail" %>
             </div>
 
             <div class="row column">

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

@@ -1,8 +1,8 @@
-<% add_decidim_page_title(t(".introduce_user_data")) %>
+<% add_decidim_page_title(t(".title")) %>
 
 <div class="item_show__header">
   <h1 class="item_show__header-title">
-    <%= t(".introduce_user_data") %>
+    <%= t(".title") %>
   </h1>
 </div>
 

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

@@ -24,7 +24,7 @@
             </td>
             <td>
               <%= link_to new_pending_authorization_confirmation_path(authorization.id) do %>
-                <%= image_tag decidim.private_download_path(Decidim::PrivateDownload.for(authorization, attachment_name: :verification_attachment).token), class: "thumbnail-list" %>
+                <%= image_tag authorization.attached_uploader(:verification_attachment).variant_url(:thumbnail), class: "thumbnail-list" %>
               <% end %>
             </td>
           </tr>

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

@@ -6,9 +6,7 @@
   <%= form.text_field :document_number %>
 
   <% if using_online? %>
-    <%= form.upload :verification_attachment,
-                    button_class: "button button__lg button__transparent-secondary verification__upload-button",
-                    attachments: [@authorization.verification_attachment] %>
+    <%= form.upload :verification_attachment, button_class: "button button__lg button__transparent-secondary verification__upload-button" %>
   <% end %>
 
   <%= form.hidden_field :verification_type, value: verification_type %>

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

@@ -1,6 +1,6 @@
-<% 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, user_postal_code: request["user_postal_code"]) %></p>
+<% if request.params["postal_codes"].present?
+  postal_codes = request.params["postal_codes"].split(",") %>
+  <p><%= t("extra_explanation.postal_codes", scope: "decidim.verifications.dummy_authorization", postal_codes: postal_codes.join(", "), count: postal_codes.count, user_postal_code: request.params["user_postal_code"]) %></p>
 <% end %>
 
 <div class="form__wrapper" data-partial-demo>

data/config/locales/ar.yml

@@ -1,4 +1,3 @@
----
 ar:
   activemodel:
     attributes:
@@ -31,16 +30,16 @@ ar:
       admin:
         id_documents:
           help:
-          - يقوم المستخدمون بملء معلومات هويتهم وتحميل نسخة من وثيقتهم.
-          - يمكنك ملء المعلومات الموجودة في الصورة التي تم تحميلها.
-          - يجب أن تتطابق المعلومات مع كل ما قام المستخدم بملئه.
-          - 
+            - يقوم المستخدمون بملء معلومات هويتهم وتحميل نسخة من وثيقتهم.
+            - يمكنك ملء المعلومات الموجودة في الصورة التي تم تحميلها.
+            - يجب أن تتطابق المعلومات مع كل ما قام المستخدم بملئه.
+            - 
         postal_letter:
           help:
-          - يطلب المشاركون إرسال رمز التحقق إلى عنوانهم.
-          - يمكنك إرسال الرسالة إلى عنوانهم مع رمز التحقق.
-          - يمكنك وضع علامة على الرسالة كما تم إرسالها.
-          - بمجرد وضع علامة على الرسالة على أنها مرسلة ، سيتمكن المشارك من تقديم الرمز والتحقق منه.
+            - يطلب المشاركون إرسال رمز التحقق إلى عنوانهم.
+            - يمكنك إرسال الرسالة إلى عنوانهم مع رمز التحقق.
+            - يمكنك وضع علامة على الرسالة كما تم إرسالها.
+            - بمجرد وضع علامة على الرسالة على أنها مرسلة ، سيتمكن المشارك من تقديم الرمز والتحقق منه.
       csv_census:
         name: تعداد المنظمة
       direct: مباشرة

data/config/locales/bg.yml

@@ -1,4 +1,3 @@
----
 bg:
   activemodel:
     attributes:
@@ -58,39 +57,39 @@ bg:
       admin:
         another_dummy_authorization_handler:
           help:
-          - Примерен манипулатор за оторизация, така че разработчиците да могат да разберат как работи една проста проверка.
-          - Той е насочен към разработчиците, за да могат да разберат как да внедрят своя собствена система за проверка.
-          - Преминете проверка, като въведете номер на паспорт започващ с "A".
+            - Примерен манипулатор за оторизация, така че разработчиците да могат да разберат как работи една проста проверка.
+            - Той е насочен към разработчиците, за да могат да разберат как да внедрят своя собствена система за проверка.
+            - Преминете проверка, като въведете номер на паспорт започващ с "A".
         csv_census:
           help:
-          - Администраторите качват CSV с имейлите на приетите участници.
-          - Само участници с имейл в този CSV файл могат да бъдат потвърдени.
+            - Администраторите качват CSV с имейлите на приетите участници.
+            - Само участници с имейл в този CSV файл могат да бъдат потвърдени.
         default:
           help:
-          - Помощния манипулатор на оторизация не е дефиниран.
-          - Разработчикът трябва да дефинира помощта в ключа за превод (i18n) „%{authorization_handler}“
+            - Помощния манипулатор на оторизация не е дефиниран.
+            - Разработчикът трябва да дефинира помощта в ключа за превод (i18n) „%{authorization_handler}“
         dummy_authorization_handler:
           help:
-          - Пример за манипулатор за оторизация, така че разработчиците да могат да разберат как работи една проста проверка.
-          - Той е насочен към разработчиците, за да могат да разберат как да внедрят своя собствена система за проверка.
-          - Преминете проверка, като въведете номер на документ завършващ с "X".
+            - Пример за манипулатор за оторизация, така че разработчиците да могат да разберат как работи една проста проверка.
+            - Той е насочен към разработчиците, за да могат да разберат как да внедрят своя собствена система за проверка.
+            - Преминете проверка, като въведете номер на документ завършващ с "X".
         id_documents:
           help:
-          - Потребителите попълват своята информация за самоличност и качват копие на своя документ.
-          - Попълнете информацията, присъстваща в каченото изображение.
-          - Информацията трябва да съответства на това, което е попълнил потребителят.
-          - Ако не можете да видите ясно информацията или не можете да я проверите, можете да отхвърлите заявката и потребителят ще има възможността да я коригира.
+            - Потребителите попълват своята информация за самоличност и качват копие на своя документ.
+            - Попълнете информацията, присъстваща в каченото изображение.
+            - Информацията трябва да съответства на това, което е попълнил потребителят.
+            - Ако не можете да видите ясно информацията или не можете да я проверите, можете да отхвърлите заявката и потребителят ще има възможността да я коригира.
         postal_letter:
           help:
-          - Участниците искат да им бъде изпратен код за потвърждение на техния адрес.
-          - Изпращате писмото до техния адрес с кода за потвърждение.
-          - Маркирате писмото като "изпратено".
-          - След като маркирате писмото като изпратено, участникът ще може да въведе кода и да бъде потвърден.
+            - Участниците искат да им бъде изпратен код за потвърждение на техния адрес.
+            - Изпращате писмото до техния адрес с кода за потвърждение.
+            - Маркирате писмото като "изпратено".
+            - След като маркирате писмото като изпратено, участникът ще може да въведе кода и да бъде потвърден.
         sms:
           help:
-          - Участниците искат код за потвърждение да бъде изпратен на техния телефон.
-          - Те трябва да въведат кода, за да бъдат потвърдени.
-          - Ако не получат кода, могат да го поискат отново.
+            - Участниците искат код за потвърждение да бъде изпратен на техния телефон.
+            - Те трябва да въведат кода, за да бъдат потвърдени.
+            - Ако не получат кода, могат да го поискат отново.
       csv_census:
         explanation: Потвърждение чрез преброяването на участниците в организацията.
         name: Преброяване на участниците в организацията