Merge pull request nextcloud#5196 from nextcloud/backport/5170/stable3.5

[stable3.5] Fix issue with Attendee not being string crashing tooltip

src/utils/attendee.js

@@ -27,6 +27,10 @@
  * @return {string} URI without a mailto prefix
  */
 export function removeMailtoPrefix(uri) {
+	if (typeof uri !== 'string') {
+		return ''
+	}
+
 	if (uri.startsWith('mailto:')) {
 		return uri.substr(7)
 	}
@@ -41,6 +45,10 @@ export function removeMailtoPrefix(uri) {
  * @return {string} URI with a mailto prefix
  */
 export function addMailtoPrefix(uri) {
+	if (typeof uri !== 'string') {
+		return 'mailto:'
+	}
+
 	if (uri.startsWith('mailto:')) {
 		return uri
 	}

tests/javascript/unit/utils/attendee.test.js

@@ -33,12 +33,22 @@ describe('utils/attendee test suite', () => {
 		expect(removeMailtoPrefix(`mailto:${uri}`)).toEqual(uri)
 	})
 
+	it('should return blank strings when uris are not of type string', () => {
+		expect(removeMailtoPrefix(null)).toEqual('')
+		expect(removeMailtoPrefix(undefined)).toEqual('')
+	})
+
 	it('should add mailto prefixes to uris', () => {
 		const uri = '[email protected]'
 		const uriWithPrefix = `mailto:${uri}`
 		expect(addMailtoPrefix(uri)).toEqual(uriWithPrefix)
 		expect(addMailtoPrefix(uriWithPrefix)).toEqual(uriWithPrefix)
 	})
+
+	it('should add mailto prefixes to uris when they are not of type string', () => {
+		expect(addMailtoPrefix(null)).toEqual("mailto:")
+		expect(addMailtoPrefix(undefined)).toEqual("mailto:")
+	})
 
 	it('should extract a display name of an organizer', () => {
 		const commonName = 'My Name'