Merge pull request #52850 from nextcloud/backport/52813/stable30

[stable30] fix(files_sharing): Show note if note exists on share

apps/files_sharing/src/views/SharingDetailsTab.vue

@@ -836,8 +836,9 @@ export default {
 				this.advancedSectionAccordionExpanded = true
 			}
 
-			if (this.share.note) {
+			if (this.isValidShareAttribute(this.share.note)) {
 				this.writeNoteToRecipientIsChecked = true
+				this.advancedSectionAccordionExpanded = true
 			}
 
 		},

cypress/e2e/files_sharing/note-to-recipient.cy.ts

@@ -80,8 +80,6 @@ describe('files_sharing: Note to recipient', { testIsolation: true }, () => {
 		cy.get('[data-cy-sidebar]').within(() => {
 			// Open the share
 			cy.get('[data-cy-files-sharing-share-actions]').first().click()
-			// Open the custom settings
-			cy.get('[data-cy-files-sharing-share-permissions-bundle="custom"]').click()
 
 			cy.findByRole('checkbox', { name: /note to recipient/i })
 				.and('be.checked')
@@ -90,4 +88,5 @@ describe('files_sharing: Note to recipient', { testIsolation: true }, () => {
 				.and('have.value', 'Hello, this is the note.')
 		})
 	})
+
 })

dist/7248-7248.js.map.license

@@ -1 +0,0 @@
-7248-7248.js.license

dist/7493-7493.js.map.license

@@ -0,0 +1 @@
+7493-7493.js.license