Street type needs to be added to the alternate address block in the OPAC (see bug 18112).
Created attachment 120813 [details] [review] Bug 28313: Add street type to alternate address in OPAC 1. Add some values to the ROADTYPE authorized value 2. Go to PatronSelfRegistrationBorrowerUnwantedField and make sure B_streettype is NOT checked 3. Go the self reg form, no alternate address street type field 4. Apply patch 5. Check the self reg form, B_streettype should be there with all the options defined in ROADTYPE a.v. 6. Check the self modification form, B_street type should be there. 7. Make sure PatronSelfRegistrationBorrowerUnwantedField and PatronSelfModificationBorrowerUnwantedField hide the field correctly. 8. Make sure PatronSelfRegistrationBorrowerMandatoryField properly makes the field required.
That was pretty quick!
Created attachment 120919 [details] [review] Bug 28313: Add street type to alternate address in OPAC 1. Add some values to the ROADTYPE authorized value 2. Go to PatronSelfRegistrationBorrowerUnwantedField and make sure B_streettype is NOT checked 3. Go the self reg form, no alternate address street type field 4. Apply patch 5. Check the self reg form, B_streettype should be there with all the options defined in ROADTYPE a.v. 6. Check the self modification form, B_street type should be there. 7. Make sure PatronSelfRegistrationBorrowerUnwantedField and PatronSelfModificationBorrowerUnwantedField hide the field correctly. 8. Make sure PatronSelfRegistrationBorrowerMandatoryField properly makes the field required. Signed-off-by: David Nind <david@davidnind.com>
This includes strings, so probably will have to wait until after release.
Created attachment 121592 [details] [review] Bug 28313: Add street type to alternate address in OPAC 1. Add some values to the ROADTYPE authorized value 2. Go to PatronSelfRegistrationBorrowerUnwantedField and make sure B_streettype is NOT checked 3. Go the self reg form, no alternate address street type field 4. Apply patch 5. Check the self reg form, B_streettype should be there with all the options defined in ROADTYPE a.v. 6. Check the self modification form, B_street type should be there. 7. Make sure PatronSelfRegistrationBorrowerUnwantedField and PatronSelfModificationBorrowerUnwantedField hide the field correctly. 8. Make sure PatronSelfRegistrationBorrowerMandatoryField properly makes the field required. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
*** Bug 12936 has been marked as a duplicate of this bug. ***
Created attachment 122213 [details] [review] Bug 28313: (follow-up) Define roadtypes It's not necessarily defined at this point
Pushed to master for 21.11, thanks to everybody involved!
Pushed to 21.05.x for 21.05.02
Not pushed to 20.11.x since Bug 18112 is not