javaeetutorial
  1. javaeetutorial
  2. JAVAEETUTORIAL-94

Problems with Duke's Tutoring admin interface

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.0.7-3
    • Fix Version/s: 6.0.7-6
    • Component/s: examples
    • Labels:
      None

      Description

      Problem 1:

      When I try to add a student (createStudent.xhtml) and I enter the phone number

      111-222-3333

      I get an error message:

      Phone numbers should be of the form (xxx) xxx-xxxx or xxx-xxx-xxxx.

      It appears that in Person.java the only allowed format is (xxx) xxx-xxxx. Either Person.java or the error message needs to be fixed.

      Problem 2:

      When I try to create a new guardian, it doesn't "take" – I click Submit, but the new person is not added to the selection list, or to the database. What is supposed to happen here? Creating a new student works all right (except for the phone glitch).

      Interestingly, I don't get the phone validation error on the createGuardian page, only on the createStudent page. It's as if it doesn't even get as far as validating it against the Person entity.

        Activity

        Hide
        Ian Evans added a comment -

        Guardian creation works now.

        Figuring out the phone number issue is complicated, due to the interaction between validators and converters. The easiest thing to do is to just update the validation message to only allow (xxx) xxx-xxxx phone numbers.

        Show
        Ian Evans added a comment - Guardian creation works now. Figuring out the phone number issue is complicated, due to the interaction between validators and converters. The easiest thing to do is to just update the validation message to only allow (xxx) xxx-xxxx phone numbers.
        Hide
        Ian Evans added a comment -

        Revision: 986
        Author: ievans
        Date: 2012-08-15 10:59
        Message: Modified validation messages to remove xxx-xxx-xxxx phone numbers as valid patterns. Still need to update the Chinese localized string.
        Issue #JAVAEETUTORIAL-94 - Problems with Duke's Tutoring admin interface

        Show
        Ian Evans added a comment - Revision: 986 Author: ievans Date: 2012-08-15 10:59 Message: Modified validation messages to remove xxx-xxx-xxxx phone numbers as valid patterns. Still need to update the Chinese localized string. Issue # JAVAEETUTORIAL-94 - Problems with Duke's Tutoring admin interface
        Hide
        Ian Evans added a comment -

        Reassigning to Eric for checking in the Chinese locale validation messages.

        Show
        Ian Evans added a comment - Reassigning to Eric for checking in the Chinese locale validation messages.
        Hide
        jendrock added a comment -

        Checked in the updated Simplified Chinese validation messages properties file (Revision 991) 2012-08-16 18:58:42 UTC.

        Show
        jendrock added a comment - Checked in the updated Simplified Chinese validation messages properties file (Revision 991) 2012-08-16 18:58:42 UTC.
        Hide
        jendrock added a comment -

        All the fixes for this issue are now checked in. The fixed case study will be included in the 6.0.7-6 release.

        Show
        jendrock added a comment - All the fixes for this issue are now checked in. The fixed case study will be included in the 6.0.7-6 release.

          People

          • Assignee:
            jendrock
            Reporter:
            Kim Haase
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: