[aur-dev] [PATCH] add search only by name and use it as default
This add the ability to search packages only by name and set it as default. The possibility to search by both is kept using Advanced Search. -- Andrea
On Wednesday 10 March 2010 19:36:37 Andrea Scarpino wrote:
This add the ability to search packages only by name and set it as default. The possibility to search by both is kept using Advanced Search. s/both/"name and description"
-- Andrea
On 10/03/10 18:36, Andrea Scarpino wrote:
This add the ability to search packages only by name and set it as default. The possibility to search by both is kept using Advanced Search.
Please don't make it name-only by default, the search is crappy enough already without the extra effort to make it kinda-work-sorta.
On Wednesday 10 March 2010 19:50:25 Nathan Wayde wrote:
Please don't make it name-only by default, the search is crappy enough already without the extra effort to make it kinda-work-sorta. I think is better to search by name as default. Anyway, I am attaching a patch that only add search by name, so this is up to Loui.
-- Andrea
On Wed, Mar 10, 2010 at 8:01 PM, Andrea Scarpino
On Wednesday 10 March 2010 19:50:25 Nathan Wayde wrote:
Please don't make it name-only by default, the search is crappy enough already without the extra effort to make it kinda-work-sorta. I think is better to search by name as default. Anyway, I am attaching a patch that only add search by name, so this is up to Loui.
-- Andrea
I agree with Nathan, it would be a restriction that I don't know why it's good, and it's really 'crappy enough' to search, and with name only... Best Regards, Laszlo Papp
On Wed 10 Mar 2010 20:01 +0100, Andrea Scarpino wrote:
On Wednesday 10 March 2010 19:50:25 Nathan Wayde wrote:
Please don't make it name-only by default, the search is crappy enough already without the extra effort to make it kinda-work-sorta.
I think is better to search by name as default.
Please test your patches! Missing a closing parenthesis: + $q.= "AND (Name LIKE '%".$_GET['K']."%' "; Also, the default should remain name and desc even when opening up the advanced search, so it should be the first option in the combobox. Thanks for the patch. I've applied those changes.
participants (4)
-
Andrea Scarpino
-
Laszlo Papp
-
Loui Chang
-
Nathan Wayde