Use update_dirty_bit_to in pre_save to allow null user fields during deserialization

[ozer550]

Summary

• Pass save kwargs to pre_save and use update_dirty_bit_to for user field null enforcement.
• Update tests to incorporate new changes.

References

• closes Align course model pre_save logic with awareness of deserialization #14130

Reviewer guidance

• Check all instances of classes that inherit AbstractFacilityDataMode have the pre_save method updated.
• Defensive logic for user ID fields to be null is appropriate as mentioned in the issue Align course model pre_save logic with awareness of deserialization #14130 .

[github-actions]

Build Artifacts

Asset type	Download link
PEX file	kolibri-0.19.3a0.dev0_git.11.gc965847b.pex
Windows Installer (EXE)	kolibri-0.19.3a0.dev0+git.11.gc965847b-windows-setup-unsigned.exe
Debian Package	kolibri_0.19.3a0.dev0+git.11.gc965847b-0ubuntu1_all.deb
Mac Installer (DMG)	kolibri-0.19.3a0.dev0+git.11.gc965847b.dmg
Android Package (APK)	kolibri-0.19.3a0.dev0+git.11.gc965847b-0.1.7-debug.apk
Raspberry Pi Image	kolibri-pi-image-0.19.3a0.dev0+git.11.gc965847b.zip
TAR file	kolibri-0.19.3a0.dev0+git.11.gc965847b.tar.gz
WHL file	kolibri-0.19.3a0.dev0+git.11.gc965847b-py2.py3-none-any.whl

Smoke test screenshot

Download screenshot

[bjester]

Overall, changes look good! I think it would still be good to wait until after this one is merged (then rebase), since then we can run the morango integ tests and have extra assurance on the syncing aspects.

[rtibbles]

This should be retargeted to release-v0.19.x - but I'm also going to block it for a little longer to allow the integration tests to be merged, and also to wait until 0.19.2 has been released!

[bjester]

@rtibbles ah great catch! 🤦

Resolved