only run this.search() when a query is set

[joshbedo]

No description provided.

[YousefED]

1. Seems you've added some code changes in demo.html which weren't intential
2. can't we move the null checks to a more centralized place, instead of adding 5+ of them all around?

Thanks,

[joshbedo]

Cool i can make those changes. Everything else make sense? I'll create some method like isQuerySet to check the value.

[joshbedo]

Just made changes let me know if everything looks alright.

[YousefED]

Hi, I didn't mean moving the null checks to a separate method. I meant can't we have 1 check further down / up in the call stack and thus centralize the logic?

[joshbedo]

ahh okay, sorry i misunderstood.