644079 - QuickFilterManager.killFilter always removes the last filter

Status: RESOLVED FIXED

(Thunderbird :: Search, defect)

Description

[opera wang]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.15) Gecko/20110303 Lightning/1.0b2 Thunderbird/3.1.9

QuickFilterManager.killFilter always remove the last one filter

Reproducible: Always

Steps to Reproduce:
1.add two new filters to QuickFilterManager by defineFilter, say filter1,filter2
2.remove filter1 by killFilter(filter1)
3.
Actual Results:  
filter2 was removed

Expected Results:  
filter1 was removed

typo?

current code is:
  killFilter: function MFM_killFilter(aName) {
    let filterDef = this.filterDefsByName[aName];
    this.filterDefs.splice(this.filterDefs.indexOf(aName), 1); // ERROR
    delete this.filterDefsByName[aName];
  },

The ERROR line should be:
this.filterDefs.splice(this.filterDefs.indexOf(filterDef), 1);

note that aName changed to filterDef

Comment 1

[opera wang]

see both in 3.3a3 and 3.1.9

Comment 2

[opera wang]

Attachment: fix the bug in killFilter

patch just for the code, don't know how to add test cases.

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 521133 [details] [diff] [review]
fix the bug in killFilter

Oops; that was a pretty dumb bug.  Thanks very much for the patch!  Because this is not a user-facing bug, the bug was typo-class, the specific bug is unlikely to regress, and we have no unit tests for the extension API for quick filters, I'm going to say we don't need a test for this.

The tree is currently closed, but we'll land this on comm-central for release in 3.3.  Because this is an extension API and extensions targeted at 3.1 probably already have to work around this anyways, I don't think it makes sense to land it in 3.1.

Comment 4

[opera wang]

Yes, I'll make work around in my extension (Expression Search) and later after I re-org it, I needn't to call killFilter.

Comment 5

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/d96b3ce7dbed