]> WPIA git - gigi.git/commit - src/club/wpia/gigi/Gigi.java
chg: move PasswordChecker object to Gigi class
authorLucas Werkmeister <mail@lucaswerkmeister.de>
Sat, 13 Jan 2018 20:23:00 +0000 (21:23 +0100)
committerLucas Werkmeister <mail@lucaswerkmeister.de>
Sun, 10 Jun 2018 11:24:31 +0000 (13:24 +0200)
commita068c177bb6cdbc3b117038047818ae58f9aa1de
tree41728f0ff814d0186c2caea67c64e54f4d601c0a
parent1d448131d2b366927f386f5ac6778bf43a65538f
chg: move PasswordChecker object to Gigi class

This provides one centralized place where the PasswordChecker used can
be selected or changed. (setPasswordChecker() is intended for use in the
tests – in normal operation, the PasswordChecker should be set up once
during initialization and then not changed.)

I’d like to do this via dependency injection, but neither User nor
Signup seem like the right places to do this. Perhaps this kind of logic
should be moved to some kind of service where this is more feasible, but
that’s not a refactoring I want to do right now.

Change-Id: Id23d8a492a21a934085d9b91689acd723b065843
src/club/wpia/gigi/Gigi.java
src/club/wpia/gigi/dbObjects/User.java
src/club/wpia/gigi/pages/main/Signup.java
tests/club/wpia/gigi/testUtils/ConfiguredTest.java