mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix Everything search after annotations

Browse source 
And replace ancient 'annotation' search condition with
'annotationText'/'annotationComment'
This commit is contained in:
Dan Stillman 2020-09-20 02:28:44 -04:00
parent 3bd5a96560
commit ead8c6bb45
4 changed files with 47 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
chrome/content/zotero/xpcom/data/search.js
View file

@ -328,7 +328,8 @@ Zotero.Search.prototype.addCondition = function (condition, operator, value, req
this.addCondition('creator', operator, part.text, false);
if (condition == 'quicksearch-everything') {
this.addCondition('annotation', operator, part.text, false);
this.addCondition('annotationText', operator, part.text, false);
this.addCondition('annotationComment', operator, part.text, false);
if (part.inQuotes) {
this.addCondition('fulltextContent', operator, part.text, false);

16
chrome/content/zotero/xpcom/data/searchConditions.js
View file

@ -490,13 +490,23 @@ Zotero.SearchConditions = new function(){
},
{
name: 'annotation',
name: 'annotationText',
operators: {
contains: true,
doesNotContain: true
},
table: 'annotations',
field: 'text'
table: 'itemAnnotations',
field: 'text',
},
{
name: 'annotationComment',
operators: {
contains: true,
doesNotContain: true
},
table: 'itemAnnotations',
field: 'comment',
},
{

3
chrome/locale/en-US/zotero/zotero.properties
View file

@ -789,7 +789,8 @@ searchConditions.dateModified = Date Modified
searchConditions.fulltextContent = Attachment Content
searchConditions.programmingLanguage = Programming Language
searchConditions.fileTypeID = Attachment File Type
searchConditions.annotation = Annotation
searchConditions.annotationText = Annotation Text
searchConditions.annotationComment = Annotation Comment
fulltext.indexState.indexed = Indexed
fulltext.indexState.unavailable = Unknown

30
test/tests/searchTest.js
View file

@ -360,6 +360,36 @@ describe("Zotero.Search", function() {
});
});
describe("annotationText", function () {
it("should return matches for annotation text", async function () {
var attachment = await importPDFAttachment();
var annotation = await createAnnotation('highlight', attachment);
var str = annotation.annotationText.substr(0, 7);
var s = new Zotero.Search();
s.libraryID = userLibraryID;
s.addCondition('joinMode', 'any');
s.addCondition('annotationText', 'contains', str);
var matches = await s.search();
assert.sameMembers(matches, [annotation.id]);
});
});
describe("annotationComment", function () {
it("should return matches for annotation comment", async function () {
var attachment = await importPDFAttachment();
var annotation = await createAnnotation('note', attachment);
var str = annotation.annotationComment.substr(0, 7);
var s = new Zotero.Search();
s.libraryID = userLibraryID;
s.addCondition('joinMode', 'any');
s.addCondition('annotationComment', 'contains', str);
var matches = await s.search();
assert.sameMembers(matches, [annotation.id]);
});
});
describe("fulltextWord", function () {
it("should return matches with full-text conditions", function* () {
let s = new Zotero.Search();