Needs review
Project:
Bot
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
8 Jun 2009 at 19:23 UTC
Updated:
5 Oct 2011 at 21:49 UTC
Jump to comment: Most recent file
We would love to be able to search the bot logs along with the rest of Drupal content-- maybe as a separate tab on the /search page would be a much easier implementation though.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 485514-bot-search-logs-7.patch | 1.08 KB | jeffschuler |
| #5 | bot-log-search-485514.patch | 2.49 KB | Shiny |
| #3 | bot_log_search.patch | 5.95 KB | snufkin |
Comments
Comment #1
binford2k commentedWe would like this feature too. I think this is *the* reason we'd want to use the drupal bot instead of one of the many others.
Comment #2
mlncn commentedAnyone have a copy of Pro Drupal Development handy and want a good learning Drupal exercise? PDD has a chapter on doing something very similar!
Comment #3
snufkin commentedHere is a patch, includes some CSS and form structure changes to the filter form on the bot_log interface. The filter adds two more elements to the url: a keyword identifying the table field that is used, and the search term which is then used in the query as LIKE "%keyword%". The keywords persist on paging between days.
Also cleaned up the filter form a little bit, removed the wrapping div, instead packed the date filter and the search into its own fieldset, inline display is applied from CSS. Margins had to be redone as well, but I think it looks quite nice with this change.
Comment #4
Shiny commentedi've tested the patch by snufkin at #3 - it does as s/he describes, but doesn't do the original feature request, which is to include the irc logs in the indexed content for drupal searches.
Comment #5
Shiny commentedsimple patch that adds searching of the logs.
uses SQL and LIKE '%keyword%'; --- won't scale without some indexes and such what.
Comment #6
dafederA search that just searches within one day isn't that useful; I could do that with my browser search. Whether or not it uses drupal's index, a universal search i needed, I think...
Comment #7
jeffschulerRe-rolled patch from #5. The first [unrelated] change in the patch was removed, as it has since been fixed. Otherwise no changes.
dafeder: looks like you were trying the patch from #3?
This implementation is simple and won't scale well, as Shiny suggests, but it searches logs over all time.