Summary: | Self registration e-mail verification does not work | ||
---|---|---|---|
Product: | Koha | Reporter: | Josef Moravec <josef.moravec> |
Component: | OPAC | Assignee: | Josef Moravec <josef.moravec> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | jonathan.druart, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | 20287 | ||
Bug Blocks: | 20819 | ||
Attachments: |
Bug 21374: Fix self registration verification
Bug 21374: Fix self registration verification Bug 21374: Fix self registration verification |
Description
Josef Moravec
2018-09-18 20:26:51 UTC
Created attachment 79061 [details] [review] Bug 21374: Fix self registration verification Test plan: 0) Do not apply the patch 1) Set PatronSelfRegistration to Allow 2) Set PatronSelfRegistrationVerifyByEmail to Require 3) Try to create a account using self registration and verify it (use link in e-mail or take the url from table mesage_queue ;) The verificatin url return ISE - DBIx::Class::Row::store_column(): No such column 'timestamp' on Koha::Schema::Result::Borrower 4) Apply the patch 5) Try to acces the verification url again - now it should work Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Created attachment 79085 [details] [review] Bug 21374: Fix self registration verification Test plan: 0) Do not apply the patch 1) Set PatronSelfRegistration to Allow 2) Set PatronSelfRegistrationVerifyByEmail to Require 3) Try to create a account using self registration and verify it (use link in e-mail or take the url from table mesage_queue ;) The verificatin url return ISE - DBIx::Class::Row::store_column(): No such column 'timestamp' on Koha::Schema::Result::Borrower 4) Apply the patch 5) Try to acces the verification url again - now it should work Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com> Caused by commit cfbc53bb22e6f4a851a2a7cd6ede891012951410 Bug 20287: Add plain_text_password (& Remove AddMember_Opac) Created attachment 79126 [details] [review] Bug 21374: Fix self registration verification Test plan: 0) Do not apply the patch 1) Set PatronSelfRegistration to Allow 2) Set PatronSelfRegistrationVerifyByEmail to Require 3) Try to create a account using self registration and verify it (use link in e-mail or take the url from table mesage_queue ;) The verificatin url return ISE - DBIx::Class::Row::store_column(): No such column 'timestamp' on Koha::Schema::Result::Borrower 4) Apply the patch 5) Try to acces the verification url again - now it should work Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Awesome work all! Pushed to master for 18.11 Bug that caused this is not in 18.05.x series, no need to backport. |