Hobo / hobo

The web app builder for Rails (moved from tablatom/hobo)
http://hobocentral.net
103 stars 39 forks source link

PostgreSQL searching bug #120

Open iox opened 9 years ago

iox commented 9 years ago

https://groups.google.com/forum/#!topic/hobousers/zm7LpimXaUw

enwood commented 9 years ago

This issue I raised is very closely related, just for additional background: https://groups.google.com/d/msg/hobousers/qrnSSyzDAZQ/vBYoMGIUuVkJ

Tim

iox commented 9 years ago

Hi @enwood , I have pushed a fix to master that adds type casting when searching in PostgreSQL.

Additional testing is very welcome. If you find it's not fixed in your app, please reopen the issue :).

enwood commented 9 years ago

Ah, thank you, Ignacio!

This change is only in master (which is Hobo 2.1.x) right now, correct?

The app I originally encountered this issue on is still based on Rails 3.2.x / Hobo 2.0.1. Could this change be safely ported to Hobo 2.0.1 to make a 2.0.2 gem release?

Tim

On Sun, Sep 28, 2014 at 2:45 PM, Ignacio Huerta notifications@github.com wrote:

Hi @enwood https://github.com/enwood , I have pushed a fix to master that adds type casting when searching in PostgreSQL.

Additional testing is very welcome. If you find it's not fixed in your app, please reopen the issue :).

— Reply to this email directly or view it on GitHub https://github.com/Hobo/hobo/issues/120#issuecomment-57095472.

iox commented 9 years ago

Hi Tim, yes we can backport this change without any issue. I'll reopen this and give it a try.

enwood commented 9 years ago

Thanks, Ignacio. I know the feeling of having to reopen something already closed! But, since I'm not in a position to migrate my app to Hobo 2.1 yet, I figured this might be the most reliable method of taking advantage of this change. I wonder how many others out there are using PostgreSQL / Hobo 2.0?

T

On Mon, Sep 29, 2014 at 8:46 AM, Ignacio Huerta notifications@github.com wrote:

Reopened #120 https://github.com/Hobo/hobo/issues/120.

— Reply to this email directly or view it on GitHub https://github.com/Hobo/hobo/issues/120#event-171453021.