зеркало из https://github.com/mozilla/gecko-dev.git
Bug #250768 --> Removing a feed from the subscriptions dialog doesn't really remove it
This commit is contained in:
Родитель
daa817f8cf
Коммит
4291ad9ba4
|
@ -74,7 +74,8 @@ function doAdd() {
|
||||||
|
|
||||||
// XXX This should be something like "subscribe to feed".
|
// XXX This should be something like "subscribe to feed".
|
||||||
dump ("feed name = " + feed.name + "\n");
|
dump ("feed name = " + feed.name + "\n");
|
||||||
addFeed(feedProperties.feedLocation, feed.name, null, folder);
|
addFeed(feedProperties.feedLocation, feed.name, null, folder); // add feed flushes the subscription database
|
||||||
|
|
||||||
// now download it for real, now that we have a folder.
|
// now download it for real, now that we have a folder.
|
||||||
feed.download();
|
feed.download();
|
||||||
}
|
}
|
||||||
|
@ -234,8 +235,8 @@ function doRemove() {
|
||||||
feeds.RemoveElementAt(index, true);
|
feeds.RemoveElementAt(index, true);
|
||||||
}
|
}
|
||||||
// Remove all assertions about the feed from the subscriptions database.
|
// Remove all assertions about the feed from the subscriptions database.
|
||||||
var ds = getSubscriptionsDS();
|
|
||||||
removeAssertions(ds, feed);
|
removeAssertions(ds, feed);
|
||||||
|
ds.QueryInterface(Components.interfaces.nsIRDFRemoteDataSource).Flush(); // flush any changes
|
||||||
|
|
||||||
// Remove all assertions about items in the feed from the items database.
|
// Remove all assertions about items in the feed from the items database.
|
||||||
ds = getItemsDS();
|
ds = getItemsDS();
|
||||||
|
@ -249,8 +250,6 @@ function doRemove() {
|
||||||
else
|
else
|
||||||
removeAssertions(ds, item);
|
removeAssertions(ds, item);
|
||||||
}
|
}
|
||||||
|
|
||||||
//tree.builder.rebuild();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|
Загрузка…
Ссылка в новой задаче