Merge pull request #27 from LibreCodeCoop/feat/multiselect-field-type

feat: add multiselect profile field type

Author: vitormattos

img/screenshots/admin-catalog-thumb.png

@@ -13,6 +13,7 @@ enum FieldType: string {
 	case TEXT = 'text';
 	case NUMBER = 'number';
 	case SELECT = 'select';
+	case MULTISELECT = 'multiselect';
 
 	/**
 	 * @return list<string>
@@ -22,6 +23,7 @@ public static function values(): array {
 			self::TEXT->value,
 			self::NUMBER->value,
 			self::SELECT->value,
+			self::MULTISELECT->value,
 		];
 	}
 

img/screenshots/admin-catalog.png

@@ -10,7 +10,7 @@
 namespace OCA\ProfileFields;
 
 /**
- * @psalm-type ProfileFieldsType = 'text'|'number'|'select'
+ * @psalm-type ProfileFieldsType = 'text'|'number'|'select'|'multiselect'
  * @psalm-type ProfileFieldsVisibility = 'private'|'users'|'public'
  * @psalm-type ProfileFieldsEditPolicy = 'admins'|'users'
  * @psalm-type ProfileFieldsExposurePolicy = 'hidden'|'private'|'users'|'public'

img/screenshots/personal-settings-thumb.png

@@ -29,7 +29,7 @@ class FieldDefinitionValidator {
 	 * @return array{
 	 *     field_key: non-empty-string,
 	 *     label: non-empty-string,
-	 *     type: 'text'|'number'|'select',
+	 *     type: 'text'|'number'|'select'|'multiselect',
 	 *     edit_policy: 'admins'|'users',
 	 *     exposure_policy: 'hidden'|'private'|'users'|'public',
 	 *     sort_order: int,
@@ -78,12 +78,13 @@ public function validate(array $definition): array {
 	 * @return list<string>|null
 	 */
 	private function validateOptions(string $type, mixed $options): ?array {
-		if ($type !== FieldType::SELECT->value) {
+		$isSelectLike = in_array($type, [FieldType::SELECT->value, FieldType::MULTISELECT->value], true);
+		if (!$isSelectLike) {
 			return null;
 		}
 
 		if (!is_array($options) || count($options) === 0) {
-			throw new InvalidArgumentException('select fields require at least one option');
+			throw new InvalidArgumentException($type . ' fields require at least one option');
 		}
 
 		$normalized = [];