[Koha-bugs] [Bug 7067] allow patron self registration via the opac

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

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

--- Comment #41 from Owen Leonard <oleonard at myacpl.org> ---
A point for discussion: I think the registration confirmation messages should
come from the email address of the library the user chose as their home
library. The counter-argument is that the user may not have chosen the correct
library, but I don't see that as much of a problem since either way the reply
would be coming from an email address which wasn't their library's.

The big issue I'm still seeing: New records are saved with NULL expiration date
and NULL registration date. Will the delete_expired_opac_registrations cron job
work with a NULL dateenrolled?

Some small issues:

- I think a note about pending patron modifications should appear on 
  the patrons page (members/members-home.pl) in addition to the staff
  client home page, just as pending tag/comment/suggestions also show
  as pending on the Tools page.

- Bug 8593 added unique IDs to the other "pending" notes. This patch
  should include one for the line it adds.

- members/members-update.pl needs a descriptive page title and page
  header. If there are no pending operations it should display a message
  saying so rather than an empty table.

- The table markup on members/members-update.pl is missing a </tr>,
  causing validation errors.

- I think the random-character verification field should be styled like 
  other required fields

- Since the datepicker widget doesn't make it easy to pick a date far in
  the past (as are birthdays for some of us), I don't think the field 
  should be read-only.

- I think opac-memberentry-update-submitted.tt should include the user
  sidebar menu.

- I see an error in the log when submitting the registration form in 
  the OPAC: 

substr outside of string at /home/oleonard/kohaclone/opac/opac-memberentry.pl
line 239

If PatronSelfRegistrationExpireTemporaryAccountsDelay is set to "0" does that
mean never? If so the description should say so.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.