search

angeliklev / alis-ef-code-1st

3 stars 0 forks source link

Lms borrowers controller #30

Closed Moggers closed 11 years ago

Moggers commented 11 years ago

Fixes up some validation, moved validation function for the email's unique constraint to the repository, and more importantly datepickers are made readonly, they can only be interacted with using the calender control.

Moggers commented 11 years ago

If someone could have a mess around with the the datepicker @ /LMS/Borrowers/Edit/1 and see if you can get it to crash, that'd be cool.

Thanks, Matthew

angeliklev commented 11 years ago

DatePicker works ok since it has default value. Barcode now is accepted empty string and notunique which is not what required I guess. Email - once accepted empty string, then was checking for unique values.

Moggers commented 11 years ago

Thanks Anzhelika, I'll fix those.

On Wed, May 22, 2013 at 5:22 PM, AnzhelikaKlev notifications@github.comwrote:

DatePicker works ok since it has default value. Barcode now is accepted empty string and notunique which is not what required I guess. Email - once accepted empty string, then was checking for unique values.

— Reply to this email directly or view it on GitHubhttps://github.com/AnzhelikaKlev/alis-ef-code-1st/pull/30#issuecomment-18266545 .

Moggers commented 11 years ago

Oky doky, she should be good now.

Moggers commented 11 years ago

Sure thing Bill, cheers, you mean the string == "" comparisons?