tree.database.RemoveDatasource --> tree.database.RemoveDataSource

VERIFIED FIXED

Status

()

Core
XUL
VERIFIED FIXED
17 years ago
6 years ago

People

(Reporter: André Dahlqvist, Assigned: dougt)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: would like for 0.9)

(Reporter)

Description

17 years ago
The 's' in RemoveDatasource' should be uppercase. The below patch fixes that.

Index: xpfe/components/directory/directory.js
===================================================================
RCS file: /cvsroot/mozilla/xpfe/components/directory/directory.js,v
retrieving revision 1.41
diff -u -r1.41 directory.js
--- xpfe/components/directory/directory.js      2001/04/14 21:10:13     1.41
+++ xpfe/components/directory/directory.js      2001/04/22 13:55:09
@@ -225,7 +225,7 @@
        var tree = document.getElementById("tree");
        if (tree)
        {
-               tree.database.RemoveDatasource(HTTPIndex);
+               tree.database.RemoveDataSource(HTTPIndex);
                tree.database.RemoveObserver(RDF_observer);
            debug("Directory: removed observer\n");
        }

Comment 1

17 years ago
the 1.40 revision to that file by dougt introduced that error.

bbaetz: since you r='ed that patch,can you r= this?

nominating for 0.9
Keywords: mozilla0.9

Comment 2

17 years ago
r=bbaetz
Whiteboard: would like for 0.9
sr=shaver.

Comment 4

17 years ago
a= asa@mozilla.org for checkin to 0.9
(Reporter)

Comment 5

17 years ago
Fix has been checked in by Bradley Baetz. I'm marking this one fixed.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

17 years ago
Thank you for catching this André!!
(Reporter)

Comment 7

16 years ago
Verified fixed with CVS source as of 2001-12-20-08.
Status: RESOLVED → VERIFIED

Updated

10 years ago
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.