Opened 18 years ago
Closed 15 years ago
#1183 closed enhancement (fixed)
Make Logs Searchable
| Reported by: | Owned by: | John Hampton | |
|---|---|---|---|
| Priority: | normal | Component: | IrcLogsPlugin |
| Severity: | normal | Keywords: | pyndexter search irc logs |
| Cc: | mitsuhiko | Trac Release: | 0.11 |
Description
integrated like the other things i guess
Attachments (1)
Change History (9)
comment:1 Changed 18 years ago by
| Cc: | mitsuhiko added; anonymous removed |
|---|---|
| Keywords: | pyndexter search irc logs added |
| Status: | new → assigned |
| Summary: | make the logs searchable → Make Logs Searchable |
comment:3 Changed 18 years ago by
OK, so this is done. Much thanks go to alect and his pyndexter stuff. However, I haven't actualyl committed or merged the code. It's working at http://pacopablo.com, but that's based ona slightly modified 0.10 version of the logs plugin. I hope to get it committed soon.
comment:5 Changed 17 years ago by
source:irclogsplugin/0.10 should have the initial import. Some paths are hard coded still, but it's something to test.
comment:6 Changed 17 years ago by
Here is my attempt to port the search functionality to 0.11. Tested with Trac 0.11dev-r7092 and pyndexter 0.2. The patch is against source:irclogsplugin/0.11@2591.
Changed 17 years ago by
| Attachment: | irclogsplugin-0.11-searching.patch added |
|---|
Search functionality for 0.11.
comment:7 Changed 17 years ago by
Excellent! Applied the patch and due to our setup of irc log the line
'file = hit.uri.path.lstrip(self.path)'
had to be replaced with
'file = os.path.basename(hit.uri.path)'
lstrip would not be correct here i think. After that change (and adjusting the indexer script a bit) it works.
comment:8 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
This is done in irclogsplugin/0.11
Already working on it ;)
I'm planning on using pyndexter for the searching. If you see this as a horrible impediment, let me know.