Merge pull request #909 from nextcloud/fix/add-child-for-schedule-cal…

…endar-transparency fix: add child to schedule-calendar-transp property
nextcloud · Jul 17, 2024 · dc2471b · dc2471b
2 parents 30618ee + 17b7142
commit dc2471b
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 1 deletion.
diff --git a/src/propset/calendarPropSet.js b/src/propset/calendarPropSet.js
@@ -88,7 +88,9 @@ export default function calendarPropSet(props) {
 		case '{urn:ietf:params:xml:ns:caldav}schedule-calendar-transp':
 			xmlified.push({
 				name: [NS.IETF_CALDAV, 'schedule-calendar-transp'],
-				value,
+				children: [{
+					name: [NS.IETF_CALDAV, value],
+				}],
 			})
 			break
 		default:

diff --git a/test/unit/propset/calendarPropSetTest.js b/test/unit/propset/calendarPropSetTest.js
@@ -118,4 +118,36 @@ describe('Calendar prop-set', () => {
 			}
 		]);
 	});
+
+	it('should serialize {urn:ietf:params:xml:ns:caldav}schedule-calendar-transp correctly - transparent', () => {
+		expect(calendarPropSet({
+			'{Foo:}bar': 123,
+			'{urn:ietf:params:xml:ns:caldav}schedule-calendar-transp': 'transparent'
+		})).toEqual([
+			{
+				name: ['urn:ietf:params:xml:ns:caldav', 'schedule-calendar-transp'],
+				children: [
+					{
+						name: ['urn:ietf:params:xml:ns:caldav', 'transparent'],
+					},
+				],
+			}
+		]);
+	});
+
+	it('should serialize {urn:ietf:params:xml:ns:caldav}schedule-calendar-transp correctly - opaque', () => {
+		expect(calendarPropSet({
+			'{Foo:}bar': 123,
+			'{urn:ietf:params:xml:ns:caldav}schedule-calendar-transp': 'opaque'
+		})).toEqual([
+			{
+				name: ['urn:ietf:params:xml:ns:caldav', 'schedule-calendar-transp'],
+				children: [
+					{
+						name: ['urn:ietf:params:xml:ns:caldav', 'opaque'],
+					},
+				],
+			}
+		]);
+	});
 });