[Koha-bugs] [Bug 12371] New: Links in every patron self-registration email points to a single borrower

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12371

            Bug ID: 12371
           Summary: Links in every patron self-registration email points
                    to a single borrower
 Change sponsored?: ---
           Product: Koha
           Version: 3.14
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P5 - low
         Component: OPAC
          Assignee: oleonard at myacpl.org
          Reporter: barton at bywatersolutions.com
        QA Contact: testopia at bugs.koha-community.org

Library has the following system preferences set for Patron Self Registration:

*************************** 1. row ***************************
   variable: PatronSelfRegistration
      value: 1
    options: NULL
explanation: If enabled, patrons will be able to register themselves via the
OPAC.
       type: YesNo
*************************** 2. row ***************************
   variable: PatronSelfRegistrationAdditionalInstructions
      value: Thank you for registering for a library card.<br>

To complete your registration, please come to your library (Cook, Jackson, or
Madison) with identification documents, that show:
<ul> 
<li>proof of residency (physical address) </li>
<li>mailing address</li>
<li>photo ID</li>
</ul>
<br>
We will save your information for 30 days, please stop in soon!
    options: NULL
explanation: A free text field to display additional instructions to newly self
registered patrons.
       type: free
*************************** 3. row ***************************
   variable: PatronSelfRegistrationBorrowerMandatoryField
      value: surname|firstname|email|phone
    options: NULL
explanation: Choose the mandatory fields for a patron's account, when
registering via the OPAC.
       type: free
*************************** 4. row ***************************
   variable: PatronSelfRegistrationBorrowerUnwantedField
      value: sex
    options: NULL
explanation: Name the fields you don't want to display when registering a new
patron via the OPAC.
       type: free
*************************** 5. row ***************************
   variable: PatronSelfRegistrationDefaultCategory
      value: ONLINEREG
    options: 
explanation: A patron registered via the OPAC will receive a borrower category
code set in this system preference.
       type: free
*************************** 6. row ***************************
   variable: PatronSelfRegistrationExpireTemporaryAccountsDelay
      value: 30
    options: NULL
explanation: If PatronSelfRegistrationDefaultCategory is enabled, this system
preference controls how long a patron can have a temporary status before the
acc
ount is deleted automatically. It is an integer value representing a number of
days to wait before deleting a temporary patron account. Setting it to 0 disabl
es the deleting of temporary accounts.
       type: Integer
*************************** 7. row ***************************
   variable: PatronSelfRegistrationVerifyByEmail
      value: 1
    options: NULL
explanation: If enabled, any patron attempting to register themselves via the
OPAC will be required to verify themselves via email to activate his or her
acco
unt.
       type: YesNo

---

The self registration email is sent emails contining unique tokens, but all of
the tokens are associated with borrowernumber 0:

select verification_token, count(verification_token), borrowernumber from 
borrower_modifications where verification_token is not null group by
verification_token;
+----------------------------------+---------------------------+----------------+
| verification_token               | count(verification_token) | borrowernumber
|
+----------------------------------+---------------------------+----------------+
| 8748c0cf981f3fadd6ec78433a5beddb |                         1 |              0
|
| 96738d8a27251326a7eb3971b1a03eb5 |                         1 |              0
|
| a5de1159d20fd3ed3f4a8d189db3fb32 |                         1 |              0
|
| aa131a40d93057e676c363a3e11be060 |                         1 |              0
|
| adf1bbfd6715714e9e8642e38d0c4462 |                         1 |              0
|
| b28898063358455c350e7eff9dc4cb94 |                         1 |              0
|
| ba24d496a711aa07f4a20a99c10c6d58 |                         1 |              0
|
| e035486fb47e2b6f3af641550098f6ea |                         1 |              0
|
| ec09849390835652149736252010b66e |                         1 |              0
|
| fdf76c060abb49b2ad1dc7aa2bdea7fb |                         1 |              0
|
+----------------------------------+---------------------------+----------------+
10 rows in set (0.00 sec)

This means that when a patron clicks on the link to verify their identity, they
will all connect to the same patron record.

-- 
You are receiving this mail because:
You are watching all bug changes.