From d5fde992ef04faaa7bd3d32aec57cb76cc4672c7 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Mon, 6 Nov 2017 12:22:59 -0300
Subject: [PATCH] Bug 19514: Add client-side check

Signed-off-by: David Bourgault <david.bourgault@inlibro.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
 C4/InstallAuth.pm                              |  4 ++++
 .../en/modules/onboarding/onboardingstep3.tt   |  3 +++
 koha-tmpl/intranet-tmpl/prog/js/onboarding.js  | 18 ++++++++++++++++++
 3 files changed, 25 insertions(+)

diff --git a/C4/InstallAuth.pm b/C4/InstallAuth.pm
index 9c6795f829..6204f11405 100644
--- a/C4/InstallAuth.pm
+++ b/C4/InstallAuth.pm
@@ -149,6 +149,10 @@ sub get_template_and_user {
             $template->param( CAN_user_serials          => 1 );
             $template->param( CAN_user_reports          => 1 );
         }
+
+        my $minPasswordLength = C4::Context->preference('minPasswordLength');
+        $minPasswordLength = 3 if not $minPasswordLength or $minPasswordLength < 3;
+        $template->param(minPasswordLength => $minPasswordLength,);
     }
     return ( $template, $borrowernumber, $cookie );
 }
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep3.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep3.tt
index 67a8c9b13a..a812b089b5 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep3.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/onboarding/onboardingstep3.tt
@@ -1,3 +1,4 @@
+[% USE Koha %]
 [% INCLUDE 'doc-head-open.inc' %]
 <title>Koha &rsaquo; Web installer &rsaquo; Create Koha administrator patron</title>
 [% INCLUDE 'installer-doc-head-close.inc' %]
@@ -112,3 +113,5 @@
         </div> <!-- / .row -->
 
 [% INCLUDE 'intranet-bottom.inc' %]
+[% PROCESS 'password_check.inc' %]
+[% PROCESS 'add_password_check' new_password => 'password' %]
diff --git a/koha-tmpl/intranet-tmpl/prog/js/onboarding.js b/koha-tmpl/intranet-tmpl/prog/js/onboarding.js
index 4816dfdfd7..93cb010b0c 100644
--- a/koha-tmpl/intranet-tmpl/prog/js/onboarding.js
+++ b/koha-tmpl/intranet-tmpl/prog/js/onboarding.js
@@ -1,5 +1,17 @@
 function _(s) { return s; } // dummy function for gettext
 
+// http://stackoverflow.com/questions/1038746/equivalent-of-string-format-in-jquery/5341855#5341855
+String.prototype.format = function() { return formatstr(this, arguments); };
+function formatstr(str, col) {
+    col = typeof col === 'object' ? col : Array.prototype.slice.call(arguments, 1);
+    var idx = 0;
+    return str.replace(/%%|%s|%(\d+)\$s/g, function (m, n) {
+        if (m == "%%") { return "%"; }
+        if (m == "%s") { return col[idx++]; }
+        return col[n];
+    });
+}
+
 jQuery.validator.addMethod( "category_code_check", function(value,element){
     var patt = /^[A-Za-z0-9]{0,10}$/g;
     if (patt.test(element.value)) {
@@ -114,6 +126,12 @@ $(document).ready(function() {
                 required: true
             },
             password: {
+                required: true,
+                password_strong: true,
+                password_no_spaces: true
+            },
+            password2: {
+                required: true,
                 password_match: true
             }
         },
-- 
2.39.5