Hi ben,

>    def doBind(form: NodeSeq) = {
>      bind("peopleSearch", form,
>        "nameToSearchFor" -> search.nameToSearchFor.toForm,
>        "submit" -> submit("Search For People", performSearch),
>        "searchResults" -> renderResults(search)
>        )
>    }

You’re relying on the order of execution in your call to bind; I think
it is safe, can some guru confirm?

The only thing I can point out is in renderResults:
> if (search.nameToSearchFor != "")
You should use search.nameToSearchFor.is, but maybe some implicit is
already doing that for you.

I suppose you’re saving all searches to a database, is that why you’re
using a mapped object for PeopleSearch?

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to