Advanced Search misses all search fields

RESOLVED FIXED in seamonkey2.7

Status

defect
--
major
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Hb, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.7
Dependency tree / graph

SeaMonkey Tracking Flags

(seamonkey2.3 wontfix, seamonkey2.4 wontfix, seamonkey2.5 fixed, seamonkey2.6 fixed, seamonkey2.7 fixed)

Details

Attachments

(1 attachment)

The Advanced Address Book Search dialog has two dropdowns in the criteria list.  The first one should have the target column names and the second one the condition operators. Both dropdowns are empty. 

Advanced Search is therefore completely broken.

Confirmed on Version 2.4 and on Trunk, perhaps a bug since 2.x.
Error: Component returned failure code: 0x80004002 (NS_NOINTERFACE) [nsISupports.QueryInterface]
Source file: chrome://messenger/content/ABSearchDialog.js
Line: 155

Probably fallout from deRDF addressbook bug.

function GetScopeForDirectoryURI(aURI)
{
  var directory = gRDF.GetResource(aURI).QueryInterface(nsIAbDirectory);
Should be:
  var directory = MailServices.ab.getDirectory(aURI);

See Bug 652855 and http://hg.mozilla.org/comm-central/rev/e3f147cb26aa#l1.1
Depends on: 652855
OS: Windows 7 → All
Hardware: x86_64 → All
Bah, seems I missed this in bug 654864 because I only referred to abCommon.js in my bug 652855 comment. :-(
Assignee: nobody → jh
Status: NEW → ASSIGNED
Going for MOA
Attachment #565305 - Flags: superreview?(mnyromyr)
Attachment #565305 - Flags: review?(mnyromyr)
Attachment #565305 - Flags: approval-comm-beta?
Attachment #565305 - Flags: approval-comm-aurora?
I tested the patch with a homebrew non optimized omni.jar. Target columns and conditions appear now. Search works fine when started with the Search button or typing Alt+S.

But when the search is started by Enter key after typing in the condition text the dialog window closes completely without result. This behaviour may be another bug.
Attachment #565305 - Flags: superreview?(mnyromyr)
Attachment #565305 - Flags: superreview+
Attachment #565305 - Flags: approval-comm-beta?
Attachment #565305 - Flags: approval-comm-beta+
Attachment #565305 - Flags: approval-comm-aurora?
Attachment #565305 - Flags: approval-comm-aurora+
Attachment #565305 - Flags: review?(mnyromyr) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.7
You need to log in before you can comment on or make changes to this bug.