Skip to content

Commit 53bf1c6

Browse files
committed
Merge pull request #105 from killme2008/feature/2015.07.09
Fixed fetchWhenSave can't work with new object.
2 parents bce3814 + 03a35c8 commit 53bf1c6

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

lib/object.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -911,7 +911,7 @@
911911

912912
var json = model._getSaveJSON();
913913

914-
if(method === 'PUT' && model._fetchWhenSave){
914+
if(model._fetchWhenSave){
915915
//Sepcial-case fetchWhenSave when updating object.
916916
json._fetchWhenSave = true;
917917
}

lib/utils.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,7 @@
414414
if (objectId) {
415415
url += "/" + objectId;
416416
}
417-
if((route ==='users' || route === 'classes') && method === 'PUT' && dataObject._fetchWhenSave){
417+
if((route ==='users' || route === 'classes') && dataObject._fetchWhenSave){
418418
delete dataObject._fetchWhenSave;
419419
url += '?new=true';
420420
}

test/object.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -309,6 +309,17 @@ describe('Objects', function(){
309309
});
310310
});
311311

312+
it("should fetch when save when creating new object.", function(done){
313+
var p= new Person();
314+
p.set('pname', 'dennis');
315+
p.save().then(function(person) {
316+
expect(person.get('company')).to.be('leancloud');
317+
done();
318+
}).catch(function(err) {
319+
throw err;
320+
});
321+
});
322+
312323
/*
313324
314325
it("it should fetch relation post",function(done){

0 commit comments

Comments
 (0)