Bug #46881 for Search-QueryParser-SQL: INTs are being fuzzified

>> Queue: Search-QueryParser-SQL >> Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=46881 > >> >> Peter, >> >> Thank you for the quick response. I apologize for my delay. I would >> be more than happy to test your changes, but I have hacked another >> feature into my copy. I needed to have the ability to have aliases >> for field names. For example, I use DBIx::Class and have a query >> that >> queries Inventory, InventoryXArtists and Artists tables. If I had >> inventory_id in my field list, the query would fail because >> inventory_id is in both inventory and inventoryxartist tables. So >> what I did was add an extra parameter to the constructor called >> "aliases" that consists of a hash of aliases. (i.e. >> { 'inventory_id' >> => 'me.inventory_id', 'name' => 'artist.name' }). I then added code >> that in your column loop after the "TODO better operator selection" >> comment that checks aliases and renames columns if they exists. >> >> I also want to add some type of call back functionality that handles >> things such as the following situation. >> >> The inventory table can be queried looking for entries added in the >> last x days. So when someone enters recent:30, the query needs to >> end >> up being, 'to_days(now()) - to_days(me.date_created) <= $recent'. >> Now >> that I'm thinking about it, the aliases feature handles everything >> but >> changing the operator. >> >> I would love to change columns to be something like: >> >> $columns = { >> 'column_name' => { >> 'type' => 'int|char|etc", >> 'alias' => 'name to use for sql query', >> 'op' => 'name of op to force', >> 'callback' => 'call to function that takes $column, $op, and $value >> and returns what's pushed on to buf' >> } >> } >> >> unfortunately I think that would break backwards compatibility. >> >> Maybe I could put something that loops through the columns hash, >> checking if the members are a string or a hash and convert them if >> needed. >> >> What are your thoughts? I'm more than willing to do the code, all >> you >> have to do is review it and submit it. >>