archive-gh-me/offline-editor-js
1
0
Fork 0
mirror of https://github.com/Esri/offline-editor-js.git synced 2025-12-15 15:20:05 +00:00
Code Issues Projects Releases Wiki Activity

updates resetLimitedPhantomGraphicsQueue()

Browse Source
This commit is contained in:
Andy Gup 2015-03-25 14:16:32 -06:00
parent ca7054b9b2
commit e75ee0cb8e
2 changed files with 64 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
lib/edit/editsStore.js
View File

@ -425,7 +425,17 @@ O.esri.Edit.EditStore = function () {
};
/**
* Removes some phantom graphics from database
* Removes some phantom graphics from database.
* The responseObject contains {id,layer,tempId,addResults,updateResults,deleteResults}.
* IF there are no results.success then nothing will be deleted.
*
* WARNING: Can generate false positives. IndexedDB will always return success
* even if you attempt to delete a non-existent id.
*
* CAUTION: This should always be used in conjunction with deleting the phantom graphic's
* associated edit entry in the database.
*
* @param responseObject
* @param callback boolean
*/
this.resetLimitedPhantomGraphicsQueue = function (responseObject, callback) {
@ -454,30 +464,18 @@ O.esri.Edit.EditStore = function () {
// CAUTION:
// TO-DO we do NOT match the edit.id with edit's objectId
if (edit.updateResults.length > 0) {
if (edit.updateResults[0].success) {
objectStore.delete(id);
}
}
if (edit.deleteResults.length > 0) {
if (edit.deleteResults[0].success) {
objectStore.delete(id);
}
}
if (edit.addResults.length > 0) {
if (edit.addResults[0].success) {
// If we have an add, update or delete success then delete the entry, otherwise we skip it.
if (edit.updateResults.length > 0 || edit.deleteResults.length > 0 || edit.addResults.length > 0) {
if (edit.updateResults[0].success || edit.deleteResults[0].success || edit.addResults[0].success) {
objectStore.delete(id);
}
}
}
}
callback(true);
}
else {
callback(true);
}
};

52
test/spec/offlineFeaturesManagerSpec.js
View File

@ -580,7 +580,7 @@ describe("Offline Editing", function()
// //TO-DO
//});
describe("Test PhantomGraphicsLayer", function()
describe("Test PhantomGraphicsLayer using editsStore directly", function()
{
async.it("Get PhantomLayerGraphics array via editsStore", function(done){
g_editsStore.getPhantomGraphicsArray(function(results,errors){
@ -721,6 +721,54 @@ describe("Offline Editing", function()
// });
//});
async.it("Delete one of the phantom graphics using resetLimitedPhantomGraphicsQueue()", function(done){
var responseObject = {
0:{
"id":-1,
"updateResults":[
{
"success":false
}
],
"addResults":[
{
"success":true
}
],
"deleteResults":[
{
"success":false
}
]
}
};
g_editsStore.resetLimitedPhantomGraphicsQueue(responseObject,function(success){
expect(success).toBe(true);
done();
});
});
async.it("Reverify total number of phantom graphics", function(done){
g_editsStore._getPhantomGraphicsArraySimple(function(results,errors){
// We remove one graphic from the previous result and should now be @ 9
expect(results.length).toBe(9);
//expect(results[0]).toBe("phantom-layer|@|-1");
expect(results[0]).toBe("phantom-layer|@|-3");
expect((results[1]).indexOf(g_editsStore.PHANTOM_GRAPHIC_PREFIX)).toBe(0);
expect((results[2]).indexOf(g_editsStore.PHANTOM_GRAPHIC_PREFIX)).toBe(0);
expect((results[3]).indexOf(g_editsStore.PHANTOM_GRAPHIC_PREFIX)).toBe(0);
expect((results[4]).indexOf(g_editsStore.PHANTOM_GRAPHIC_PREFIX)).toBe(0);
expect((results[5]).indexOf(g_editsStore.PHANTOM_GRAPHIC_PREFIX)).toBe(0);
expect((results[6]).indexOf(g_editsStore.PHANTOM_GRAPHIC_PREFIX)).toBe(0);
expect(results[7]).toBe("phantom-layer|@|test002");
expect(results[8]).toBe("phantom-layer|@|test003");
expect(errors).toBe("end");
done();
})
});
async.it("Delete all PhantomLayerGraphics", function(done){
g_editsStore.resetPhantomGraphicsQueue(function(result){
@ -751,7 +799,7 @@ describe("Offline Editing", function()
})
});
async.it("store FeatureLayerJSON data", function(done){
async.it("store FeatureLayerJSON data using editsStore directly", function(done){
var dataObject = {
graphics: {test:1},
renderer: {test:2}