Closed
Bug 18631
Opened 25 years ago
Closed 25 years ago
datasource refresh with javascript
Categories
(Core Graveyard :: RDF, defect, P2)
Core Graveyard
RDF
Tracking
(Not tracked)
VERIFIED
FIXED
M13
People
(Reporter: janv, Assigned: waterson)
Details
Attachments
(4 files)
(deleted),
text/plain
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
unable to refresh datasource
see attached files
Reporter | ||
Comment 1•25 years ago
|
||
Reporter | ||
Comment 2•25 years ago
|
||
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
OS: Linux → All
Priority: P3 → P2
Hardware: PC → All
Target Milestone: M13
Assignee | ||
Comment 3•25 years ago
|
||
need to do two things.
1. add accessor interfaces to nsIRDFCompositeDataSource
2. possibly make nsCompositeDataSource supports nsIRDFRemoteDataSource.
varga: in the mean time, you can work around this bug by explicitly creating an
RDF/XML datasource using the component manager, initializing it to the correct
URL by hand, and then adding it to the tree's ".database" via AddDataSource().
See
http://lxr.mozilla.org/mozilla/source/xpfe/components/sidebar/resources/customi
ze.[xul|js] for examples.
Assignee | ||
Comment 4•25 years ago
|
||
scott: to do this, I think I'm going to just add a the following method to
nsIRDFCompositeDataSource:
/** @return an enumerator that enumerates all the datasources in
the composite datasource */
nsISimplEnumerator datasources();
I'll need to tweak your mini-composite datasource stuff that you use for mail
views. Sound ok?
Comment 5•25 years ago
|
||
Yes, that's fine. The datasource never gets used and I've meant to get rid of
it in the past.
Assignee | ||
Comment 6•25 years ago
|
||
Assignee | ||
Comment 7•25 years ago
|
||
rjc: can you code review this fix? I added a GetDataSources() method to
nsIRDFCompositeDataSource (cleaned up the docs in that file, as well), and to
the composite datasource implementation.
Assignee | ||
Comment 8•25 years ago
|
||
Assignee | ||
Comment 9•25 years ago
|
||
scottip: could you review the changes to nsMessageViewDataSource.cpp? thanks!
Comment 10•25 years ago
|
||
the change to nsMessageViewDataSource.cpp looks fine.
Comment 11•25 years ago
|
||
Go for it, looks good.
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 12•25 years ago
|
||
fix checked in
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•