Opened 16 years ago
Closed 16 years ago
#296 closed enhancement (fixed)
Add spelling suggestion to Omega
Reported by: | Yevgeny Binder | Owned by: | Olly Betts |
---|---|---|---|
Priority: | normal | Milestone: | 1.1.1 |
Component: | Omega | Version: | 1.0.8 |
Severity: | normal | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Operating System: | All |
Description
This is a small patch to Omega and scriptindex to enable support for spelling suggestion. To use it, in your scriptindex database definition file add the SPELL keyword to any field whose contents you want added to the spelling dictionary, reindex the database, and add the $suggestion keyword to your Omegascript output template.
A variation of this patch might instead add words to the spelling dictionary by default, and then provide a NOSPELL scriptindex keyword to prevent addition. A command-line option to scriptindex could also globally enable/disable spelling support.
Attachments (2)
Change History (11)
comment:1 by , 16 years ago
Keywords: | spell spelling suggestion correction omega scriptindex removed |
---|---|
Status: | new → assigned |
follow-up: 3 comment:2 by , 16 years ago
Hmm, the attachments are getting the wrong content type, and I can't see how to edit that - I think the ".108" is confusing things.
Can you reattach them, but with extension ".patch"?
by , 16 years ago
Attachment: | query.cc.diff added |
---|
comment:3 by , 16 years ago
Replying to olly:
Hmm, the attachments are getting the wrong content type, and I can't see how to edit that - I think the ".108" is confusing things.
Can you reattach them, but with extension ".patch"?
Oops, I did .diff instead. Seems to work ok in Trac, though. Or do you need .patch?
by , 16 years ago
Attachment: | scriptindex.cc.spelling.diff added |
---|
comment:4 by , 16 years ago
".diff" is fine too.
I think an explicit "spell" is probably best in scriptindex.
The patches look OK, apart from the lack of corresponding documentation updates. If you could update the documentation too that would be most appreciated.
comment:5 by , 16 years ago
Milestone: | 1.1.0 → 1.1.1 |
---|
Bumping milestone to 1.1.1 as this is ready to apply (no documentation) and isn't an incompatible change.
comment:7 by , 16 years ago
comment:9 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
OK, and the query template is sorted in trunk r12526. Marking as fixed.
Thanks.