add test to ensure RESULTS_AS_TAG_CONTENTS queries do liveupdate

RESOLVED FIXED in Firefox 3.7a1

Status

()

defect
RESOLVED FIXED
12 years ago
10 years ago

People

(Reporter: cmtalbert, Assigned: mak)

Tracking

Trunk
Firefox 3.7a1
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

Attachments

(2 attachments)

If you have a result set generated from a RESULTS_AS_TAG_CONTENTS query, then it will not respond to live update actions to add additional items to the query set.  Both outright additions and updates to existing items (such that they now match the query standards) do not work.

I will attach a test for this behavior.  To see the update case, simply comment out the "add" case's checks and re-run.  The test can also be used to test removing an item from the query set via Live Update too.
i don't undestand this
   var options = histsvc.getNewQueryOptions();
   options.sortingMode = options.SORT_BY_TAGS_ASCENDING;
   options.resultType = options.RESULTS_AS_TAG_CONTENTS;

   // Results
   var result = histsvc.executeQuery(query, options);
   var root = result.root;
   root.containerOpen = true;

RESULTS_AS_TAG_CONTENTS requires that the query specify a &folder=TAG_ID
Blocks: 509868
So, the test as it is is invalid because RESULTS_AS_TAG_CONTENTS is a special remapping query, that is on place till we reimplement tags in a saner way.
Adding support for all querying options could be hard till then.
I'm morphing bug to change the test to a basic liveupdate one, discarding advances searchTerms, annotation, ecc options to somewhere in future.
Summary: RESULTS_AS_TAG_CONTENTS queries do not add items via LiveUpdate → add test to ensure RESULTS_AS_TAG_CONTENTS queries do liveupdate
Posted patch patch v1.0Splinter Review
the new test, based on the original one.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Attachment #395847 - Flags: review?(dietrich)
Attachment #395847 - Flags: review?(dietrich) → review+
http://hg.mozilla.org/mozilla-central/rev/42460e3726ce
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a1
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.