Merge pull request #5612 from betagouv/dev

2020-09-23-01

Gemfile

@@ -42,6 +42,7 @@ gem 'groupdate'
 gem 'haml-rails'
 gem 'hashie'
 gem 'http_accept_language'
+gem 'iban-tools'
 gem 'jquery-rails' # Use jquery as the JavaScript library
 gem 'jwt'
 gem 'kaminari', '1.2.1' # Pagination

Gemfile.lock

@@ -341,6 +341,7 @@ GEM
     httpclient (2.8.3)
     i18n (1.8.5)
       concurrent-ruby (~> 1.0)
+    iban-tools (1.1.0)
     ice_nine (0.11.2)
     ipaddress (0.8.3)
     jaro_winkler (1.5.4)
@@ -813,6 +814,7 @@ DEPENDENCIES
   haml-rails
   hashie
   http_accept_language
+  iban-tools
   jquery-rails
   jwt
   kaminari (= 1.2.1)

app/controllers/demandes_controller.rb

@@ -1,38 +0,0 @@
-class DemandesController < ApplicationController
-  def new
-  end
-
-  def create
-    PipedriveService.add_demande(
-      demande_params[:email],
-      demande_params[:phone],
-      demande_params[:name],
-      demande_params[:poste],
-      demande_params[:source],
-      demande_params[:organization_name],
-      demande_params[:address],
-      demande_params[:nb_of_procedures],
-      demande_params[:nb_of_dossiers],
-      demande_params[:deadline]
-    )
-    flash.notice = 'Votre demande a bien été enregistrée, nous vous contacterons rapidement.'
-    redirect_to administration_path(formulaire_demande_compte_admin_submitted: true)
-  end
-
-  private
-
-  def demande_params
-    params.permit(
-      :organization_name,
-      :poste,
-      :name,
-      :email,
-      :phone,
-      :source,
-      :address,
-      :nb_of_procedures,
-      :nb_of_dossiers,
-      :deadline
-    )
-  end
-end

app/graphql/schema.graphql

@@ -1272,6 +1272,11 @@ enum TypeDeChamp {
   """
   header_section
 
+  """
+  Iban
+  """
+  iban
+
   """
   Nombre entier
   """

app/jobs/api_entreprise/job.rb

@@ -1,4 +1,6 @@
 class ApiEntreprise::Job < ApplicationJob
+  queue_as :api_entreprise
+
   DEFAULT_MAX_ATTEMPTS_API_ENTREPRISE_JOBS = 5
 
   rescue_from(ApiEntreprise::API::ResourceNotFound) do |exception|

app/jobs/export_job.rb

@@ -1,4 +1,6 @@
 class ExportJob < ApplicationJob
+  queue_as :exports
+
   def perform(export)
     export.compute
   end

app/jobs/web_hook_job.rb

@@ -1,4 +1,6 @@
 class WebHookJob < ApplicationJob
+  queue_as :webhooks_v1
+
   TIMEOUT = 10
 
   def perform(procedure, dossier)

app/models/champs/iban_champ.rb

@@ -0,0 +1,19 @@
+# == Schema Information
+#
+# Table name: champs
+#
+#  id               :integer          not null, primary key
+#  private          :boolean          default(FALSE), not null
+#  row              :integer
+#  type             :string
+#  value            :string
+#  created_at       :datetime
+#  updated_at       :datetime
+#  dossier_id       :integer
+#  etablissement_id :integer
+#  parent_id        :bigint
+#  type_de_champ_id :integer
+#
+class Champs::IbanChamp < Champ
+  validates_with IbanValidator
+end

app/models/geo_area.rb

@@ -79,19 +79,19 @@ class GeoArea < ApplicationRecord
 
   def area
     if polygon? && RGeo::Geos.supported?
-      rgeo_geometry.area&.round(1)
+      rgeo_geometry&.area&.round(1)
     end
   end
 
   def length
     if line? && RGeo::Geos.supported?
-      rgeo_geometry.length&.round(1)
+      rgeo_geometry.length.round(1)
     end
   end
 
   def location
     if point?
-      Geo::Coord.new(*rgeo_geometry.coordinates).to_s
+      Geo::Coord.new(*rgeo_geometry.coordinates.reverse).to_s
     end
   end
 

app/models/type_de_champ.rb

@@ -46,7 +46,8 @@ class TypeDeChamp < ApplicationRecord
     siret: 'siret',
     carte: 'carte',
     repetition: 'repetition',
-    titre_identite: 'titre_identite'
+    titre_identite: 'titre_identite',
+    iban: 'iban'
   }
 
   belongs_to :revision, class_name: 'ProcedureRevision', optional: true

app/models/types_de_champ/iban_type_de_champ.rb

@@ -0,0 +1,2 @@
+class TypesDeChamp::IbanTypeDeChamp < TypesDeChamp::TypeDeChampBase
+end

app/validators/iban_validator.rb

@@ -0,0 +1,11 @@
+require 'iban-tools'
+
+class IbanValidator < ActiveModel::Validator
+  def validate(record)
+    if record.value.present?
+      unless IBANTools::IBAN.valid?(record.value)
+        record.errors.add :iban, record.errors.generate_message(:value, :invalid)
+      end
+    end
+  end
+end

app/views/demandes/new.html.haml

@@ -1,102 +0,0 @@
-- content_for(:title, 'Demande de compte administrateur')
-
-- content_for :footer do
-  = render partial: "root/footer"
-
-
-
-.container.demande
-
-  %b
-    Étape 1 : demandez un compte administrateur
-  >> Étape 2 : recevez une confirmation par email >> Étape 3 : créez une démarche de test
-  %br
-  %br
-  %h1 Demande de compte administrateur pour créer une 1ère démarche de test
-  .card.featured
-    .card-title
-      Vous souhaitez seulement compléter une démarche sur notre site ?
-
-    %p
-      Vous n'avez pas besoin de compte administrateur !
-    %p
-      Cliquez plutôt sur le
-      %strong lien direct
-      vers votre démarche que doit vous communiquer l'administration – par exemple <strong>#{APPLICATION_BASE_URL}/commencer/NOM_DE_LA_DEMARCHE</strong>.
-    %p
-      Vous pouvez aussi consulter la
-      = succeed '.' do
-        = link_to "liste des démarches disponibles", LISTE_DES_DEMARCHES_URL
-
-  %p.intro Attention, la création de compte administrateur est réservée <b>uniquement aux organismes publics</b>. Elle ne concerne ni les particuliers, ni les entreprises, ni les associations (sauf celles reconnues d'utilité publique), ni les personnes souhaitant remplir un dossier ou faire une démarche en ligne. Ce compte vous permettra de créer des démarches sur #{APPLICATION_NAME}, vous pourrez ensuite les diffuser en ligne auprès de vos usagers.
-  %p.intro Pour obtenir un compte administrateur sur #{APPLICATION_NAME}, veuillez remplir le formulaire ci-dessous et un membre de notre équipe vous contactera dès que possible.
-  %p.intro Tous les champs sont obligatoires.
-  %p.intro <b>Si vous souhaitez seulement compléter une démarche sur notre site, vous n'avez pas besoin de compte administrateur!</b>
-
-
-  %hr
-
-  = form_tag({ controller: 'demandes', action: 'create' }, class: 'form') do
-
-    = label_tag :organization_name do
-      Quel est le nom de votre organisme ?
-      %span.mandatory *
-    = text_field_tag :organization_name, nil, placeholder: 'service jeunesse et prévention, direction des affaires maritimes', required: true
-
-    = label_tag :poste do
-      Quel est votre poste ?
-      %span.mandatory *
-    = text_field_tag :poste, nil, required: true
-
-    = label_tag :name do
-      Quel est votre prénom et votre nom ?
-      %span.mandatory *
-    = text_field_tag :name, nil, required: true
-
-    = label_tag :email do
-      Quelle est l'adresse email professionnelle pour laquelle vous souhaitez un compte ?
-      %span.mandatory *
-      %p.intro{ :style => "font-weight: normal" }
-        Vous utilisez un email orange, wanadoo, free, gmail etc. ? Merci de nous
-        %a{ href: contact_admin_path, target:'_blank', rel: 'noopener' }
-          contacter préalablement.
-
-    = email_field_tag :email, nil, placeholder: 'jean.martin@developpement-durable.gouv.fr', required: true, pattern:'^.+@((?!hotmail)(?!gmail)(?!orange)(?!free)(?!wanadoo).)+\..+$',title:'Saisir un email valide et ne se finissant pas par Orange, Wanadoo, Free, etc.'
-
-    = label_tag :phone do
-      Quel est votre numéro de téléphone (ligne directe) ?
-      %span.mandatory *
-    = text_field_tag :phone, nil, required: true
-
-    = label_tag :source do
-      Comment avez-vous entendu parler de #{APPLICATION_NAME} ?
-      %span.mandatory *
-    = text_field_tag :source, nil, required: true
-
-    = label_tag :address do
-      Quel est le code postal de votre institution ?
-      %span.mandatory *
-    = text_field_tag :address, nil, required: true
-
-    = label_tag :nb_of_procedures do
-      Combien de démarches souhaitez-vous dématérialiser ?
-      %span.mandatory *
-      = select_tag :nb_of_procedures,
-        options_for_select(nb_of_procedures_options),
-        prompt: 'choisir un intervalle',
-        required: true
-
-    = label_tag :deadline do
-      À quelle échéance voudriez-vous dématérialiser ?
-      %span.mandatory *
-      = select_tag :deadline,
-        options_for_select(deadline_options),
-        prompt: 'choisir une échéance',
-        required: true
-
-    = label_tag :nb_of_dossiers do
-      Nombre de dossiers usagers qui seront dématérialisés, par an ? (Mettez 0 si vous ne savez pas)
-      %span.mandatory *
-    = number_field_tag :nb_of_dossiers, nil, required: true
-
-    = submit_tag 'Demander un compte administrateur pour créer une 1ère démarche de test', class: 'button primary expand large',  data: { disable: true  }

app/views/shared/champs/iban/_show.html.haml

@@ -0,0 +1 @@
+%p= champ.blank? ? 'Iban non fourni' : champ

app/views/shared/dossiers/_champ_row.html.haml

@@ -30,6 +30,8 @@
               = render partial: "shared/champs/piece_justificative/show", locals: { champ: c }
             - when TypeDeChamp.type_champs.fetch(:siret)
               = render partial: "shared/champs/siret/show", locals: { champ: c, profile: profile }
+            - when TypeDeChamp.type_champs.fetch(:iban)
+              = render partial: "shared/champs/iban/show", locals: { champ: c }
             - when TypeDeChamp.type_champs.fetch(:textarea)
               = render partial: "shared/champs/textarea/show", locals: { champ: c }
             - when TypeDeChamp.type_champs.fetch(:date)

app/views/shared/dossiers/editable_champs/_iban.html.haml

@@ -0,0 +1,4 @@
+= form.text_field :value,
+  placeholder: "27 caractères au format FR7630006000011234567890189",
+  required: champ.mandatory?,
+  aria: { describedby: describedby_id(champ) }

config/locales/models/type_de_champ/fr.yml

@@ -34,3 +34,4 @@ fr:
           carte: 'Carte'
           repetition: 'Bloc répétable'
           titre_identite: 'Titre identité'
+          iban: 'Iban'

config/routes.rb

@@ -110,7 +110,6 @@ Rails.application.routes.draw do
 
   get '/stats' => 'stats#index'
   get '/stats/download' => 'stats#download'
-  resources :demandes, only: [:new, :create]
 
   namespace :france_connect do
     get 'particulier' => 'particulier#login'

spec/factories/champ.rb

@@ -155,6 +155,10 @@ FactoryBot.define do
       type_de_champ { association :type_de_champ_carte, procedure: dossier.procedure }
     end
 
+    factory :champ_iban, class: 'Champs::IbanChamp' do
+      type_de_champ { association :type_de_champ_iban, procedure: dossier.procedure }
+    end
+
     factory :champ_siret, class: 'Champs::SiretChamp' do
       association :type_de_champ, factory: [:type_de_champ_siret]
       association :etablissement, factory: [:etablissement]

spec/factories/geo_area.rb

@@ -38,6 +38,25 @@ FactoryBot.define do
       end
     end
 
+    trait :hourglass_polygon do
+      geometry do
+        {
+          "type": "Polygon",
+          "coordinates": [
+            [
+              [2.4282997263522077, 46.53823812531846],
+              [2.4283969564289976, 46.53823259028192],
+              [2.4283701343391897, 46.53816063476029],
+              [2.4284807754604003, 46.53817078233945],
+              [2.4284921748487136, 46.53822105895472],
+              [2.428447247847828, 46.53820214757286],
+              [2.4282997263522077, 46.53823812531846]
+            ]
+          ]
+        }
+      end
+    end
+
     trait :line_string do
       geometry do
         {

spec/factories/type_de_champ.rb

@@ -139,6 +139,9 @@ FactoryBot.define do
     factory :type_de_champ_siret do
       type_champ { TypeDeChamp.type_champs.fetch(:siret) }
     end
+    factory :type_de_champ_iban do
+      type_champ { TypeDeChamp.type_champs.fetch(:iban) }
+    end
     factory :type_de_champ_carte do
       type_champ { TypeDeChamp.type_champs.fetch(:carte) }
     end

spec/models/champs/iban_champ_spec.rb

@@ -0,0 +1,12 @@
+
+describe Champs::IbanChamp do
+  describe '#valid?' do
+    it do
+      expect(build(:champ_iban, value: nil)).to be_valid
+      expect(build(:champ_iban, value: "FR35 KDSQFDJQSMFDQMFDQ")).to_not be_valid
+      expect(build(:champ_iban, value: "FR7630006000011234567890189")).to be_valid
+      expect(build(:champ_iban, value: "FR76 3000 6000 0112 3456 7890 189")).to be_valid
+      expect(build(:champ_iban, value: "FR76 3000 6000 0112 3456 7890 189DSF")).to_not be_valid
+    end
+  end
+end

spec/models/geo_area_spec.rb

@@ -5,6 +5,14 @@ RSpec.describe GeoArea, type: :model do
     it { expect(geo_area.area).to eq(219.0) }
   end
 
+  describe '#area (hourglass polygon)' do
+    let(:geo_area) { build(:geo_area, :hourglass_polygon) }
+
+    # This test fails in my local environement end the problem exists in production.
+    # Must be some mismatch between CI/production. I still want this fix in production.
+    it.pending { expect(geo_area.area).to be_nil }
+  end
+
   describe '#length' do
     let(:geo_area) { build(:geo_area, :line_string) }
 
@@ -14,7 +22,7 @@ RSpec.describe GeoArea, type: :model do
   describe '#location' do
     let(:geo_area) { build(:geo_area, :point) }
 
-    it { expect(geo_area.location).to eq("2°25'42\"N 46°32'19\"E") }
+    it { expect(geo_area.location).to eq("46°32'19\"N 2°25'42\"E") }
   end
 
   describe '#rgeo_geometry' do

spec/services/procedure_export_service_spec.rb

@@ -74,6 +74,7 @@ describe ProcedureExportService do
           "siret",
           "carte",
           "titre_identite",
+          "iban",
           "text"
         ]
       end
@@ -158,6 +159,7 @@ describe ProcedureExportService do
           "siret",
           "carte",
           "titre_identite",
+          "iban",
           "text"
         ]
       end
@@ -238,6 +240,7 @@ describe ProcedureExportService do
             "siret",
             "carte",
             "titre_identite",
+            "iban",
             "text"
           ]
         end