fix: testing fixes
Some checks failed
Build Test / test (pull_request) Successful in 26s
JS Unit Tests / test (pull_request) Failing after 27s
PHP Unit Tests / test (pull_request) Successful in 54s

Signed-off-by: Sebastian Krupinski <krupinski01@gmail.com>
This commit is contained in:
2026-05-16 00:10:02 -04:00
parent e8612c4e10
commit 7afee695c6
2 changed files with 16 additions and 17 deletions

View File

@@ -608,13 +608,11 @@ class Manager {
throw new InvalidArgumentException("Service '{$service->identifier()}' is not capable of creating collections"); throw new InvalidArgumentException("Service '{$service->identifier()}' is not capable of creating collections");
} }
// convert properties if necessary // convert properties if necessary
if (is_array($properties)) { if ($properties instanceof CollectionPropertiesMutableInterface === false) {
$collection = $service->collectionFresh()->getProperties()->jsonDeserialize($properties); $properties = $service->collectionFresh()->getProperties()->jsonDeserialize($properties);
} else {
$collection = $properties;
} }
// Create collection // Create collection
return $service->collectionCreate($target, $collection, $options); return $service->collectionCreate($target, $properties, $options);
} }
/** /**
@@ -642,13 +640,11 @@ class Manager {
throw new InvalidArgumentException("Service '{$service->identifier()}' is not capable of updating collections"); throw new InvalidArgumentException("Service '{$service->identifier()}' is not capable of updating collections");
} }
// convert properties if necessary // convert properties if necessary
if (is_array($properties)) { if ($properties instanceof CollectionPropertiesMutableInterface === false) {
$mutation = $service->collectionFresh()->getProperties()->jsonDeserialize($properties); $properties = $service->collectionFresh()->getProperties()->jsonDeserialize($properties);
} else {
$mutation = $properties;
} }
// Update collection // Update collection
return $service->collectionUpdate($target, $mutation); return $service->collectionUpdate($target, $properties);
} }
/** /**

View File

@@ -296,9 +296,10 @@ export const useCollectionsStore = defineStore('mailCollectionsStore', () => {
properties: properties.toJson() properties: properties.toJson()
}) })
// Merge created collection into state if (response instanceof CollectionObject) {
_collections.value[response.identifier] = response _collections.value[response.identifier] = response
indexCollection(response) indexCollection(response)
}
console.debug('[Mail Manager][Store] - Successfully created collection:', response.identifier) console.debug('[Mail Manager][Store] - Successfully created collection:', response.identifier)
return response return response
@@ -326,12 +327,14 @@ export const useCollectionsStore = defineStore('mailCollectionsStore', () => {
properties: properties.toJson() properties: properties.toJson()
}) })
if (_collections.value[response.identifier]) { if (_collections.value[target]) {
deindexCollection(_collections.value[response.identifier]) deindexCollection(_collections.value[target])
} }
if (response instanceof CollectionObject) {
_collections.value[response.identifier] = response _collections.value[response.identifier] = response
indexCollection(response) indexCollection(response)
}
console.debug('[Mail Manager][Store] - Successfully updated collection:', response.identifier) console.debug('[Mail Manager][Store] - Successfully updated collection:', response.identifier)
return response return response