search

PacketFire / goqdb

Quote Database in Go
MIT License
6 stars 2 forks source link

no form validation #6

Closed mischief closed 10 years ago

mischief commented 10 years ago

this could trigger index out of range panic on slices

mischief commented 10 years ago

http://robfig.github.io/revel/manual/validation.html

jgrar commented 10 years ago

I've done a very basic fix for now. I'll work on outputting the validation errors nicely later tonight if someone else doesn't. Leaving this open in case there's further issues with it.

Edit: My suggestion for handling errors is that we insert an error class onto the form control in the event of an error on that field.

jgrar commented 10 years ago

Woops, i was in a hurry to leave and didn't realise i broke the for input with this. i have the working version, will push asap.