The SDK will try to save object even if its child object not saved

[Nidal-Bakir]

New Issue Checklist

• [x] I am not disclosing a vulnerability.
• [x] I am not just asking a question.
• [x] I have searched through existing issues.
• [x] I can reproduce the issue with the latest version of Parse Server and the Parse Flutter SDK.

Issue Description

Steps to reproduce

This will not reproduce the error. I do not know how to force the parse server to fail to save an object so the error will appear. I catch this error while writing tests for the save() function.

1. create ParseObject planObject
2. create ParseObject dietPlansObject
3. set the planObject object as a child object for dietPlansObject object
4. call save() on the container object dietPlansObject

 final planObject = ParseObject('Plans')
          ..set('PlanName', 'plan 1'); // creating this object somehow should fail, then the error will appear.

 final dietPlansObject = ParseObject("Diet_Plans");

 dietPlansObject.set('Plan1', planObject);

 await dietPlansObject.save();

Actual Outcome

The SDK will try to Serialize the dietPlansObject to save it online but it can not because the child object is not saved. so an error will be thrown while trying to Serialize the Object.

Expected Outcome

The save() function fails to save the object and returns ParseError

Environment

Parse Flutter SDK

• SDK version: 4.0.0
• Operating system version: any

Server

• Parse Server version: any

Logs

package:parse_server_sdk/src/objects/parse_base.dart 269:76 ParseBase.toPointer package:parse_server_sdk/src/utils/parse_encoder.dart 49:20 parseEncode package:parse_server_sdk/src/objects/parse_base.dart 98:20 ParseBase.toJson. dart:collection _LinkedHashMapMixin.forEach package:parse_server_sdk/src/objects/parse_base.dart 96:12 ParseBase.toJson package:parse_server_sdk/src/objects/parse_object.dart 73:39 ParseObject.create package:parse_server_sdk/src/objects/parse_object.dart 116:26 ParseObject.save ===== asynchronous gap =========================== test/src/objects/parse_object/parse_object_save_test.dart 404:26 main..

Null check operator used on a null value

Process finished with exit code 1

[parse-github-assistant[bot]]

Thanks for opening this issue!

• 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

[Nidal-Bakir]

@mtrezza @mbfakourii

For all the issues that I have created and assigned myself to fix them. I have fixed them and I have written the test for all of them.

I will open new PRs to fix all of them once we are done with the transition to one package. #852

[mtrezza]

@Nidal-Bakir Reading this I understand now why you assigned yourself to issues. We don't use the assignment feature because it may deter others from picking up an issue that is assigned. You could keep a private list of issues you're planning to work on instead.

[Nidal-Bakir]

Got it, thanks