Only clear share password model when actually saved

Instead of always clearing the password in the share model, now only do
it when it was actually saved.

Fixes an issue where saving another field would clear the password too
early and prevent it to be saved when the dropdown closed.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>

apps/files_sharing/src/mixins/SharesMixin.js

@@ -248,11 +248,14 @@ export default {
 					try {
 						await this.updateShare(this.share.id, properties)
 
+						if (propertyNames.indexOf('password') >= 0) {
+							// reset password state after sync
+							this.$delete(this.share, 'newPassword')
+						}
+
 						// clear any previous errors
 						this.$delete(this.errors, propertyNames[0])
 
-						// reset password state after sync
-						this.$delete(this.share, 'newPassword')
 					} catch ({ message }) {
 						if (message && message !== '') {
 							this.onSyncError(propertyNames[0], message)