noosfero | Search infra improvements (#727)

Antonio Terceiro gitlab at mg.gitlab.com
Wed Nov 18 13:44:43 BRST 2015


New comment for Merge Request 727

https://gitlab.com/noosfero/noosfero/merge_requests/727#note_2694869


Antonio Terceiro

no ... clearly anything that comes from `params` is either nil or a string, so the case where it does not respond to `#to_i` should never happen. What I meant is what happens if `params[:limit]` is a string for which `#to_i` will return 0? Is 0 a valid limit here?

Also, what if `params[:limit]` is a _very large number_?

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listas.softwarelivre.org/pipermail/noosfero-dev/attachments/20151118/fe26dcf4/attachment.html>


More information about the Noosfero-dev mailing list