[Koha-bugs] [Bug 32500] OPAC_REG_VERIFY has too many options for "Message body"

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

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32500

David Cook <dcook at prosentient.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #4 from David Cook <dcook at prosentient.com.au> ---
(In reply to Marcel de Rooy from comment #3)
> Why not add_fields on borrower modifications? Maybe you want to change the
> notice and show some of the fields that have been entered? This seems too
> drastic?

Upon reviewing the borrower modifications table, I'm not sure why I thought
verification_token should be the only field. It doesn't really make much sense.

My concern about add_fields() is that it would have access to the "password"
field. It looks like it's the hashed password rather than the clear text
password, but it should probably be excluded. But it looks like we could modify
the "get_columns_for" function in letter.pl to exclude that field.

I'll have another think about this one another time!

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