This is a snapshot of Indico's old Trac site. Any information contained herein is most probably outdated. Access our new GitHub site here.

Opened 14 months ago

Last modified 13 months ago

#1588 assigned defect

Add validation to fields in registration form

Reported by: pferreir Owned by: aavilesd
Priority: high Milestone: v1.9
Component: Registration form Version: 1.9
Keywords: Cc:

Description

Date fields, telephone numbers and "number" fields are not properly validated.
E.g., setting a date to a letter results in it being ignored (no warning), same for a "number" field. The "phone number" case is even worse, as it results in a server-side error with a "go back" button which will then result in an empty registration form (the user loses all fields filled till then).

Change History (3)

comment:1 Changed 13 months ago by pferreir

  • Owner set to aavilesd
  • Status changed from new to assigned

comment:2 Changed 13 months ago by aavilesd

  • Status changed from assigned to in_work

comment:3 Changed 13 months ago by aavilesd

  • Status changed from in_work to assigned

Phone validation has been fixed: https://github.com/OmeGak/indico/compare/1588-fix-regform-validation

Number validation when it involves non-numeric characters seems to be a bug in AngularJS. It hasn't been fixed until version 1.3. The problem should be fixed when we take care of ticket http://indico-software.org/ticket/1462.

Date validation occurs only when the field is not mandatory. I've spent a great deal of time and I only achieved half-working solutions and using not very pretty hacks. I would suggest deprecating the legacy datepicker directive.

Note: See TracTickets for help on using tickets.