https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32500
David Cook <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #4 from David Cook <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
