Review Request 122643: Fix support for LIKE operator in Kexi Query Designer

[Radosław Wicik]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122643/#review76338
-----------------------------------------------------------

Ship it!


Ship It!

- Radosław Wicik


On Lut 19, 2015, 10:30 po południu, Jarosław Staniek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122643/
> -----------------------------------------------------------
> 
> (Updated Lut 19, 2015, 10:30 po południu)
> 
> 
> Review request for Calligra, Andrius da Costa Ribas, Friedrich W. H. Kossebau, Adam Pigg, Roman Shtemberko, Radosław Wicik, and Wojciech Kosowicz.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> Fix support for LIKE operator in Kexi Query Designer
> 
> - support both LIKE and NOT LIKE
> - works well with strings and query parameters [...]
> - properly interacts with the SQL view
> - existing code refactored so LIKE can be easily added
> 
> 
> Diffs
> -----
> 
>   kexi/plugins/queries/kexiquerydesignerguieditor.cpp 86bc6de6afb604a93ac01425a5cc2c0bc821e2a2 
> 
> Diff: https://git.reviewboard.kde.org/r/122643/diff/
> 
> 
> Testing
> -------
> 
> Queries with LIKE and NOT LIKE work in all views.
> 
> Example test:
> For table foo with column c of text type, try:
> 
> SELECT * FROM foo WHERE c LIKE '%a%'
> 
> (should show all rows where c contains 'a')
> 
> and 
> 
> SELECT * FROM foo WHERE c NOT LIKE '%a%'
> 
> (should show all rows where c does not contain 'a')
> 
> and 
> 
> SELECT * FROM foo WHERE c LIKE [Enter filter]
> 
> (should show all rows where c mathes the filter specified interactively -- don't forget to prepend/append %)
> 
> 
> Thanks,
> 
> Jarosław Staniek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150220/cbcbc8fe/attachment.htm>