Mercurial > snowl
annotate extension/modules/feed.js @ 121:110b9b8aed1f
only notify observers that messages have changed if they have
| author | Myk Melez <myk@mozilla.org> |
|---|---|
| date | Sun, 08 Jun 2008 21:50:05 -0700 |
| parents | 3c16d71594e5 |
| children | 7d13369f7d01 |
| rev | line source |
|---|---|
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
1 EXPORTED_SYMBOLS = ["SnowlFeed"]; |
| 14 | 2 |
| 3 const Cc = Components.classes; | |
| 4 const Ci = Components.interfaces; | |
| 5 const Cr = Components.results; | |
| 6 const Cu = Components.utils; | |
| 7 | |
| 114 | 8 Cu.import("resource://snowl/modules/log4moz.js"); |
| 9 Cu.import("resource://snowl/modules/datastore.js"); | |
| 10 Cu.import("resource://snowl/modules/URI.js"); | |
| 11 Cu.import("resource://snowl/modules/source.js"); | |
| 12 | |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
13 // FIXME: factor this out into a common file. |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
14 const PART_TYPE_CONTENT = 1; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
15 const PART_TYPE_SUMMARY = 2; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
16 |
|
113
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
17 // nsIFeedTextConstruct::type to media type mappings. |
|
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
18 const mediaTypes = { html: "text/html", |
|
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
19 xhtml: "application/xhtml+xml", |
|
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
20 text: "text/plain" }; |
|
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
21 |
| 118 | 22 /** |
| 23 * Convert a string to an array of character codes. | |
| 24 * | |
| 25 * @param string {string} the string to convert | |
| 26 * @returns {array} the array of character codes | |
| 27 */ | |
| 28 function stringToArray(string) { | |
| 29 var array = []; | |
| 30 for (let i = 0; i < string.length; i++) | |
| 31 array.push(string.charCodeAt(i)); | |
| 32 return array; | |
| 33 } | |
| 34 | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
35 function SnowlFeed(aID, aName, aMachineURI, aHumanURI, aLastRefreshed, aImportance) { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
36 // Call the superclass's constructor to initialize the new instance. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
37 SnowlSource.call(this, aID, aName, aMachineURI, aHumanURI, aLastRefreshed, aImportance); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
38 } |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
39 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
40 SnowlFeed.prototype = { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
41 __proto__: SnowlSource.prototype, |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
42 |
|
115
9556a63bfbda
make all snowl feeds share a logger
Myk Melez <myk@mozilla.org>
parents:
114
diff
changeset
|
43 _log: Log4Moz.Service.getLogger("Snowl.Feed"), |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
44 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
45 // Observer Service |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
46 get _obsSvc() { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
47 let obsSvc = Cc["@mozilla.org/observer-service;1"]. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
48 getService(Ci.nsIObserverService); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
49 this.__defineGetter__("_obsSvc", function() { return obsSvc }); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
50 return this._obsSvc; |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
51 }, |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
52 |
| 116 | 53 refresh: function() { |
| 14 | 54 let request = Cc["@mozilla.org/xmlextras/xmlhttprequest;1"].createInstance(); |
| 55 | |
| 56 request.QueryInterface(Ci.nsIDOMEventTarget); | |
| 57 let t = this; | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
58 request.addEventListener("load", function(e) { t.onRefreshLoad(e) }, false); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
59 request.addEventListener("error", function(e) { t.onRefreshError(e) }, false); |
| 14 | 60 |
| 61 request.QueryInterface(Ci.nsIXMLHttpRequest); | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
62 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
63 // The feed processor is going to parse the XML, so override the MIME type |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
64 // in order to turn off parsing by XMLHttpRequest itself. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
65 request.overrideMimeType("text/plain"); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
66 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
67 request.open("GET", this.machineURI.spec, true); |
| 14 | 68 request.send(null); |
| 69 }, | |
| 70 | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
71 onRefreshLoad: function(aEvent) { |
| 14 | 72 let request = aEvent.target; |
| 73 | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
74 // XXX What's the right way to handle this? |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
75 if (request.responseText.length == 0) |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
76 throw("feed contains no data"); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
77 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
78 let parser = Cc["@mozilla.org/feed-processor;1"]. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
79 createInstance(Ci.nsIFeedProcessor); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
80 parser.listener = { t: this, handleResult: function(r) { this.t.onRefreshResult(r) } }; |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
81 parser.parseFromString(request.responseText, request.channel.URI); |
| 14 | 82 }, |
| 83 | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
84 onRefreshError: function(aEvent) { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
85 this._log.error("onRefreshError: " + aEvent.target.status + " " + |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
86 aEvent.target.statusText + " " + aEvent.target.responseText.length); |
| 14 | 87 }, |
| 88 | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
89 onRefreshResult: function(aResult) { |
| 14 | 90 // Now that we know we successfully downloaded the feed and obtained |
| 91 // a result from it, update the "last refreshed" timestamp. | |
|
117
31fedd0b1466
refactor resetLastRefreshed into a lastRefreshed getter
Myk Melez <myk@mozilla.org>
parents:
116
diff
changeset
|
92 this.lastRefreshed = new Date(); |
| 14 | 93 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
94 let feed = aResult.doc.QueryInterface(Components.interfaces.nsIFeed); |
| 14 | 95 |
| 118 | 96 let currentMessageIDs = []; |
|
121
110b9b8aed1f
only notify observers that messages have changed if they have
Myk Melez <myk@mozilla.org>
parents:
118
diff
changeset
|
97 let messagesChanged = false; |
| 14 | 98 |
| 99 SnowlDatastore.dbConnection.beginTransaction(); | |
| 100 try { | |
| 101 for (let i = 0; i < feed.items.length; i++) { | |
| 102 let entry = feed.items.queryElementAt(i, Ci.nsIFeedEntry); | |
| 103 //entry.QueryInterface(Ci.nsIFeedContainer); | |
| 104 | |
| 105 // Figure out the ID for the entry, then check if the entry has already | |
| 106 // been retrieved. If we can't figure out the entry's ID, then we skip | |
| 107 // the entry, since its ID is the only way for us to know whether or not | |
| 108 // it has already been retrieved. | |
|
84
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
109 let externalID; |
| 14 | 110 try { |
| 118 | 111 externalID = entry.id || this._generateID(entry); |
| 14 | 112 } |
| 113 catch(ex) { | |
| 118 | 114 this._log.warn("couldn't retrieve a message: " + ex); |
| 14 | 115 continue; |
| 116 } | |
| 117 | |
| 118 | 118 let internalID = this._getInternalIDForExternalID(externalID); |
| 119 if (internalID) | |
| 120 continue; | |
| 14 | 121 |
|
121
110b9b8aed1f
only notify observers that messages have changed if they have
Myk Melez <myk@mozilla.org>
parents:
118
diff
changeset
|
122 messagesChanged = true; |
| 118 | 123 this._log.info(this.name + " adding message " + externalID); |
| 124 internalID = this._addMessage(entry, externalID); | |
| 125 currentMessageIDs.push(internalID); | |
| 14 | 126 } |
| 127 | |
| 128 // Update the current flag. | |
|
121
110b9b8aed1f
only notify observers that messages have changed if they have
Myk Melez <myk@mozilla.org>
parents:
118
diff
changeset
|
129 // XXX Should this affect whether or not messages have changed? |
| 14 | 130 SnowlDatastore.dbConnection.executeSimpleSQL("UPDATE messages SET current = 0 WHERE sourceID = " + this.id); |
| 118 | 131 SnowlDatastore.dbConnection.executeSimpleSQL("UPDATE messages SET current = 1 WHERE id IN (" + currentMessageIDs.join(", ") + ")"); |
| 14 | 132 |
| 133 SnowlDatastore.dbConnection.commitTransaction(); | |
| 134 } | |
| 135 catch(ex) { | |
| 136 SnowlDatastore.dbConnection.rollbackTransaction(); | |
| 137 throw ex; | |
| 138 } | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
139 |
|
121
110b9b8aed1f
only notify observers that messages have changed if they have
Myk Melez <myk@mozilla.org>
parents:
118
diff
changeset
|
140 if (messagesChanged) |
|
110b9b8aed1f
only notify observers that messages have changed if they have
Myk Melez <myk@mozilla.org>
parents:
118
diff
changeset
|
141 this._obsSvc.notifyObservers(null, "messages:changed", null); |
| 14 | 142 }, |
| 143 | |
| 144 /** | |
| 145 * Add a message to the datastore for the given feed entry. | |
| 146 * | |
| 147 * @param aEntry {nsIFeedEntry} the feed entry | |
|
84
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
148 * @param aExternalID {string} the external ID of the feed entry |
| 14 | 149 */ |
| 118 | 150 _addMessage: function(aEntry, aExternalID) { |
| 14 | 151 // Combine the first author's name and email address into a single string |
| 152 // that we'll use as the author of the message. | |
| 153 let author = null; | |
| 154 if (aEntry.authors.length > 0) { | |
| 155 let firstAuthor = aEntry.authors.queryElementAt(0, Ci.nsIFeedPerson); | |
| 156 let name = firstAuthor.name; | |
| 157 let email = firstAuthor.email; | |
| 158 if (name) { | |
| 159 author = name; | |
| 160 if (email) | |
| 161 author += " <" + email + ">"; | |
| 162 } | |
| 163 else if (email) | |
| 164 author = email; | |
| 165 } | |
| 166 | |
| 167 // Convert the publication date/time string into a JavaScript Date object. | |
| 168 let timestamp = aEntry.published ? new Date(aEntry.published) : null; | |
| 169 | |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
170 // FIXME: handle titles that contain markup or are missing. |
|
84
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
171 let messageID = this.addSimpleMessage(this.id, aExternalID, |
|
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
172 aEntry.title.text, author, |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
173 timestamp, aEntry.link); |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
174 |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
175 // Add parts |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
176 if (aEntry.content) { |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
177 this.addPart(messageID, PART_TYPE_CONTENT, aEntry.content.text, |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
178 (aEntry.content.base ? aEntry.content.base.spec : null), |
|
113
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
179 aEntry.content.lang, mediaTypes[aEntry.content.type]); |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
180 } |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
181 |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
182 if (aEntry.summary) { |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
183 this.addPart(messageID, PART_TYPE_SUMMARY, aEntry.summary.text, |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
184 (aEntry.summary.base ? aEntry.summary.base.spec : null), |
|
113
f633d4f1fefa
make mediaTypes a constant rather than a property of the SnowlFeed object
Myk Melez <myk@mozilla.org>
parents:
112
diff
changeset
|
185 aEntry.summary.lang, mediaTypes[aEntry.summary.type]); |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
186 } |
| 14 | 187 |
| 188 // Add metadata. | |
| 189 let fields = aEntry.QueryInterface(Ci.nsIFeedContainer). | |
| 190 fields.QueryInterface(Ci.nsIPropertyBag).enumerator; | |
| 191 while (fields.hasMoreElements()) { | |
| 192 let field = fields.getNext().QueryInterface(Ci.nsIProperty); | |
| 193 | |
| 194 if (field.name == "authors") { | |
| 195 let values = field.value.QueryInterface(Ci.nsIArray).enumerate(); | |
| 196 while (values.hasMoreElements()) { | |
| 197 let value = values.getNext().QueryInterface(Ci.nsIFeedPerson); | |
| 198 // FIXME: store people records in a separate table with individual | |
| 199 // columns for each person attribute (i.e. name, email, url)? | |
| 118 | 200 this._addMetadatum(messageID, |
| 201 "atom:author", | |
| 202 value.name && value.email ? value.name + "<" + value.email + ">" | |
| 203 : value.name ? value.name : value.email); | |
| 14 | 204 } |
| 205 } | |
| 206 | |
| 207 else if (field.name == "links") { | |
| 208 let values = field.value.QueryInterface(Ci.nsIArray).enumerate(); | |
| 209 while (values.hasMoreElements()) { | |
| 210 let value = values.getNext().QueryInterface(Ci.nsIPropertyBag2); | |
| 211 // FIXME: store link records in a separate table with individual | |
| 212 // colums for each link attribute (i.e. href, type, rel, title)? | |
| 118 | 213 this._addMetadatum(messageID, |
| 214 "atom:link_" + value.get("rel"), | |
| 215 value.get("href")); | |
| 14 | 216 } |
| 217 } | |
| 218 | |
| 219 // For some reason, the values of certain simple fields (like RSS2 guid) | |
| 220 // are property bags containing the value instead of the value itself. | |
| 221 // For those, we need to unwrap the extra layer. This strange behavior | |
| 222 // has been filed as bug 427907. | |
| 223 else if (typeof field.value == "object") { | |
| 224 if (field.value instanceof Ci.nsIPropertyBag2) { | |
| 225 let value = field.value.QueryInterface(Ci.nsIPropertyBag2).get(field.name); | |
| 118 | 226 this._addMetadatum(messageID, field.name, value); |
| 14 | 227 } |
| 228 else if (field.value instanceof Ci.nsIArray) { | |
| 229 let values = field.value.QueryInterface(Ci.nsIArray).enumerate(); | |
| 230 while (values.hasMoreElements()) { | |
| 231 let value = values.getNext().QueryInterface(Ci.nsIPropertyBag2); | |
| 118 | 232 this._addMetadatum(messageID, field.name, value.get(field.name)); |
| 14 | 233 } |
| 234 } | |
| 235 } | |
| 236 | |
| 237 else | |
| 118 | 238 this._addMetadatum(messageID, field.name, field.value); |
| 14 | 239 } |
| 240 | |
| 241 return messageID; | |
| 242 }, | |
| 243 | |
| 244 /** | |
| 245 * Given an entry, generate an ID for it based on a hash of its link, | |
| 246 * published, and title attributes. Useful for uniquely identifying entries | |
| 247 * that don't provide their own IDs. | |
| 248 * | |
| 249 * @param entry {nsIFeedEntry} the entry for which to generate an ID | |
| 250 * @returns {string} an ID for the entry | |
| 251 */ | |
| 118 | 252 _generateID: function(entry) { |
| 14 | 253 let hasher = Cc["@mozilla.org/security/hash;1"]. |
| 254 createInstance(Ci.nsICryptoHash); | |
| 255 hasher.init(Ci.nsICryptoHash.SHA1); | |
| 118 | 256 let identity = stringToArray(entry.link.spec + entry.published + entry.title.text); |
| 14 | 257 hasher.update(identity, identity.length); |
| 258 return "urn:" + hasher.finish(true); | |
| 259 }, | |
| 260 | |
| 261 // FIXME: Make the rest of this stuff be part of a superclass from which | |
| 262 // this class is derived. | |
| 263 | |
| 264 /** | |
| 265 * Get the internal ID of the message with the given external ID. | |
| 266 * | |
| 267 * @param aExternalID {string} | |
| 268 * the external ID of the message | |
| 269 * | |
| 270 * @returns {number} | |
| 271 * the internal ID of the message, or undefined if the message | |
| 272 * doesn't exist | |
| 273 */ | |
| 118 | 274 _getInternalIDForExternalID: function(aExternalID) { |
| 14 | 275 return SnowlDatastore.selectInternalIDForExternalID(aExternalID); |
| 276 }, | |
| 277 | |
| 278 /** | |
| 279 * Add a message with a single part to the datastore. | |
| 280 * | |
| 281 * @param aSourceID {integer} the record ID of the message source | |
|
84
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
282 * @param aExternalID {string} the external ID of the message |
| 14 | 283 * @param aSubject {string} the title of the message |
| 284 * @param aAuthor {string} the author of the message | |
| 285 * @param aTimestamp {Date} the date/time at which the message was sent | |
| 286 * @param aLink {nsIURI} a link to the content of the message, | |
| 287 * if the content is hosted on a server | |
| 288 * | |
| 289 * @returns {integer} the internal ID of the newly-created message | |
| 290 */ | |
|
84
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
291 addSimpleMessage: function(aSourceID, aExternalID, aSubject, aAuthor, |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
292 aTimestamp, aLink) { |
| 14 | 293 // Convert the timestamp to milliseconds-since-epoch, which is how we store |
| 294 // it in the datastore. | |
| 295 let timestamp = aTimestamp ? aTimestamp.getTime() : null; | |
| 296 | |
| 297 // Convert the link to its string spec, which is how we store it | |
| 298 // in the datastore. | |
| 299 let link = aLink ? aLink.spec : null; | |
| 300 | |
| 301 let messageID = | |
|
84
915e41848f6d
rename universalID to externalID
Myk Melez <myk@mozilla.org>
parents:
76
diff
changeset
|
302 SnowlDatastore.insertMessage(aSourceID, aExternalID, aSubject, aAuthor, |
| 14 | 303 timestamp, link); |
| 304 | |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
305 return messageID; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
306 }, |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
307 |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
308 get _addPartStatement() { |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
309 let statement = SnowlDatastore.createStatement( |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
310 "INSERT INTO parts(messageID, partType, content, baseURI, languageCode, mediaType) \ |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
311 VALUES (:messageID, :partType, :content, :baseURI, :languageCode, :mediaType)" |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
312 ); |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
313 this.__defineGetter__("_addPartStatement", function() { return statement }); |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
314 return this._addPartStatement; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
315 }, |
| 14 | 316 |
|
85
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
317 addPart: function(aMessageID, aPartType, aContent, aBaseURI, aLanguageCode, |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
318 aMediaType) { |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
319 this._addPartStatement.params.messageID = aMessageID; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
320 this._addPartStatement.params.partType = aPartType; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
321 this._addPartStatement.params.content = aContent; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
322 this._addPartStatement.params.baseURI = aBaseURI; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
323 this._addPartStatement.params.languageCode = aLanguageCode; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
324 this._addPartStatement.params.mediaType = aMediaType; |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
325 this._addPartStatement.execute(); |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
326 |
|
f5161c834622
store summaries in addition to content and display them in the river view
Myk Melez <myk@mozilla.org>
parents:
84
diff
changeset
|
327 return SnowlDatastore.dbConnection.lastInsertRowID; |
| 14 | 328 }, |
| 329 | |
| 118 | 330 _addMetadatum: function(aMessageID, aAttributeName, aValue) { |
| 14 | 331 // FIXME: speed this up by caching the list of known attributes. |
| 332 let attributeID = SnowlDatastore.selectAttributeID(aAttributeName) | |
| 333 || SnowlDatastore.insertAttribute(aAttributeName); | |
| 334 SnowlDatastore.insertMetadatum(aMessageID, attributeID, aValue); | |
| 335 }, | |
| 336 | |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
337 // FIXME: make this accept a callback to which it reports on its progress |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
338 // so we can provide feedback to the user in subscription interfaces. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
339 subscribe: function() { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
340 this._log.info("subscribing to " + this.name + " <" + this.machineURI.spec + ">"); |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
341 |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
342 let request = Cc["@mozilla.org/xmlextras/xmlhttprequest;1"].createInstance(); |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
343 |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
344 request = request.QueryInterface(Ci.nsIDOMEventTarget); |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
345 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
346 let t = this; |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
347 request.addEventListener("load", function(e) { t.onSubscribeLoad(e) }, false); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
348 request.addEventListener("error", function(e) { t.onSubscribeError(e) }, false); |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
349 |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
350 request = request.QueryInterface(Ci.nsIXMLHttpRequest); |
|
106
2a08b4a82802
integrate OPML import into the sidebar
Myk Melez <myk@mozilla.org>
parents:
104
diff
changeset
|
351 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
352 // The feed processor is going to parse the XML, so override the MIME type |
|
106
2a08b4a82802
integrate OPML import into the sidebar
Myk Melez <myk@mozilla.org>
parents:
104
diff
changeset
|
353 // in order to turn off parsing by XMLHttpRequest itself. |
|
2a08b4a82802
integrate OPML import into the sidebar
Myk Melez <myk@mozilla.org>
parents:
104
diff
changeset
|
354 request.overrideMimeType("text/plain"); |
|
2a08b4a82802
integrate OPML import into the sidebar
Myk Melez <myk@mozilla.org>
parents:
104
diff
changeset
|
355 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
356 request.open("GET", this.machineURI.spec, true); |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
357 request.send(null); |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
358 }, |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
359 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
360 onSubscribeLoad: function(aEvent) { |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
361 let request = aEvent.target; |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
362 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
363 // XXX What's the right way to handle this? |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
364 if (request.responseText.length == 0) |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
365 throw("feed contains no data"); |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
366 |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
367 let parser = Cc["@mozilla.org/feed-processor;1"]. |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
368 createInstance(Ci.nsIFeedProcessor); |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
369 parser.listener = { t: this, handleResult: function(r) { this.t.onSubscribeResult(r) } }; |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
370 parser.parseFromString(request.responseText, request.channel.URI); |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
371 }, |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
372 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
373 onSubscribeError: function(aEvent) { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
374 this._log.error("onSubscribeError: " + aEvent.target.status + " " + |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
375 aEvent.target.statusText + " " + aEvent.target.responseText.length); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
376 }, |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
377 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
378 onSubscribeResult: function(aResult) { |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
379 let feed = aResult.doc.QueryInterface(Components.interfaces.nsIFeed); |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
380 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
381 // Extract the name (if we don't already have one) and human URI from the feed. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
382 if (!this.name) |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
383 this.name = feed.title.plainText(); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
384 this.humanURI = feed.link; |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
385 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
386 // Add the source to the database. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
387 let statement = |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
388 SnowlDatastore.createStatement("INSERT INTO sources (name, machineURI, humanURI) " + |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
389 "VALUES (:name, :machineURI, :humanURI)"); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
390 try { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
391 statement.params.name = this.name; |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
392 statement.params.machineURI = this.machineURI.spec; |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
393 statement.params.humanURI = this.humanURI.spec; |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
394 statement.step(); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
395 } |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
396 finally { |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
397 statement.reset(); |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
398 } |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
399 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
400 // Extract the ID of the source from the newly-created database record. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
401 this.id = SnowlDatastore.dbConnection.lastInsertRowID; |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
402 |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
403 // Let observers know about the new source. |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
404 this._obsSvc.notifyObservers(null, "sources:changed", null); |
|
112
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
405 |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
406 // Refresh the feed to import all its items. |
|
754441ddeaa3
combine the SnowlFeed and SnowlFeedSubscriber objects, and make it inherit from SnowlSource; also, remove the obsolete SnowlFeedClient
Myk Melez <myk@mozilla.org>
parents:
108
diff
changeset
|
407 this.onRefreshResult(aResult); |
|
76
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
408 } |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
409 |
|
1cbd4c5a511b
link the source name to the human URI if available; add a new SnowlFeedSubscriber object that can handle subscribing to feeds from various interfaces and which sets the human URI if available; make the OPML importer use the new SnowlFeedSubscriber object
Myk Melez <myk@mozilla.org>
parents:
75
diff
changeset
|
410 }; |