Status

()

defect
P3
normal
RESOLVED WONTFIX
14 years ago
10 years ago

People

(Reporter: bugs, Unassigned)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: swag: 3d)

Get rid of the bulky and growing query interface and simply make the query object implement a writable property bag. Different properties can have different types. 

Modifier prefixes can be used within the property names to specify special options, e.g.:

query.setProperty("foo", "*")    <-- match if first class property foo is set to 
                                     anything
query.setProperty("!foo", "*")   <-- match if first class property foo is not set
query.setProperty("A:foo", "*")  <-- match if annotation foo is set to anything
query.setProperty("!A:foo", "*") <-- match if annotation foo is not set

...

or similar.
Assignee: brettw → bryner
Priority: -- → P3
Target Milestone: --- → Firefox 2 alpha2
Assignee: bryner → brettw
Summary: Query Object is-a nsIWritablePropertyBag → Refactor places query object
Priority: P3 → P1
Target Milestone: Firefox 2 alpha2 → Firefox 2 alpha1
Blocks: 317844
Target Milestone: Firefox 2 alpha1 → Firefox 2 alpha2
Blocks: 331459
Priority: P1 → P2
Whiteboard: swag: 3d
Blocks: 331487
Blocks: 332402
Priority: P2 → P3
Target Milestone: Firefox 2 alpha2 → Firefox 3 alpha1
Assignee: brettw → nobody
Target Milestone: Firefox 3 alpha1 → ---
No longer blocks: 331487
i'm not sure that exchanging explicit method/property names for a generic api with cryptic prefixing is really the way to go here.

i'm going to wontfix this; if we decide to refactor let's do it under a new bug, in the context of the recent Places changes.
No longer blocks: 331459
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
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.