a few bug fixes but mainly UI cleanup on search results page. some code cleanup as well.
BUGS
- we repeated the category filtering checkboxes on advanced search form
- we omitted a few preferences on the settings form
- we did not output snippets on the category results.
- fix syntax of watchdog() call when logging search queries
CODE
- using theme() functions to out lists of links and tables
- use comment_num_all() instead of custom query
- move excerpt highlighting to CSS
- pager_display() was out of sync with pager_query() as far as how many results to display per page
- log search results for standard form posts.
| Comment | File | Size | Author |
|---|---|---|---|
| sql_search.patch | 13.88 KB | moshe weitzman |
Comments
Comment #1
nedjoThanks, nice work, applied to HEAD (and I'll merge into 4.6 branch when I'm back at my GUI interface--unless someone has a quick tip about how to merge changes from head back to a branch?).
Comment #2
moshe weitzman commentednone of my changes caused this module to be incompatible with 4.6. so if the files for each branch were identical before, you could just copy HEAD version to a 4.6 checkout and then commit.
Comment #3
(not verified) commented