Update Request validation, so that it also knows about required schema attributes

Author: leenooks

app/Classes/LDAP/Schema/AttributeType.php

@@ -289,7 +289,6 @@ public function __get(string $key): mixed
 			case 'type': return $this->type;
 			case 'usage': return $this->usage;
 			case 'used_in_object_classes': return $this->used_in_object_classes;
-			case 'validation': return Arr::get(config('ldap.validation'),$this->name_lc);
 
 			default: return parent::__get($key);
 		}
@@ -601,4 +600,34 @@ public function setType($type) {
 
 		$this->type = $type;
 	}
+
+	/**
+	 * Return Request validation array
+	 *
+	 * This will merge configured validation with schema required attributes
+	 *
+	 * @param array $array
+	 * @return array|null
+	 */
+	public function validation(array $array): ?array
+	{
+		// For each item in array, we need to get the OC heirachy
+		$heirachy = collect($array)
+			->filter()
+			->map(fn($item)=>config('server')
+				->schema('objectclasses',$item)
+				->getSupClasses()
+				->push($item))
+			->flatten()
+			->unique();
+
+		$validation = collect(Arr::get(config('ldap.validation'),$this->name_lc,[]));
+		if (($heirachy->intersect($this->required_by_object_classes)->count() > 0)
+			&& (! collect($validation->get($this->name_lc))->contains('required'))) {
+			$validation->put($this->name_lc,array_merge(['required','min:1'],$validation->get($this->name_lc,[])))
+				->put($this->name_lc.'.*',array_merge(['required','min:1'],$validation->get($this->name_lc.'.*',[])));
+		}
+
+		return $validation->toArray();
+	}
 }

app/Http/Requests/EntryRequest.php

@@ -26,9 +26,9 @@ public function rules()
 		return config('server')
 			->schema('attributetypes')
 			->intersectByKeys($this->request)
-			->transform(function($item) { return $item->validation; })
+			->map(fn($item)=>$item->validation(request()->get('objectclass')))
 			->filter()
-			->flatMap(function($item) { return $item; })
+			->flatMap(fn($item)=>$item)
 			->toArray();
 	}
 }

config/ldap.php

@@ -1,5 +1,7 @@
 <?php
 
+use App\Rules\HasStructuralObjectClass;
+
 return [
 
 	/*
@@ -119,8 +121,10 @@
 	'validation' => [
 		'objectclass' => [
 			'objectclass'=>[
+				'required',
 				'array',
-				'min:1'
+				'min:1',
+				new HasStructuralObjectClass,
 			]
 		],
 		'gidnumber' => [
@@ -170,4 +174,4 @@
 			]
 		],
 	],
-];
+];