🚿

Author: codemasher

phpcs.xml.dist

@@ -35,7 +35,7 @@
 	<rule ref="SlevomatCodingStandard.Attributes.RequireAttributeAfterDocComment"/>
 
 	<rule ref="SlevomatCodingStandard.Classes.ClassConstantVisibility"/>
-	<rule ref="SlevomatCodingStandard.Classes.DisallowConstructorPropertyPromotion"/>
+	<!--<rule ref="SlevomatCodingStandard.Classes.DisallowConstructorPropertyPromotion"/>-->
 	<rule ref="SlevomatCodingStandard.Classes.ForbiddenPublicProperty"/>
 	<rule ref="SlevomatCodingStandard.Classes.ModernClassNameReference"/>
 

src/Attributes/ThrowOnInvalidProperty.php

@@ -21,14 +21,8 @@
 #[Attribute(Attribute::TARGET_CLASS)]
 final class ThrowOnInvalidProperty{
 
-	private readonly bool $throwOnInvalid;
-
-	public function __construct(bool $throwOnInvalid){
-		$this->throwOnInvalid = $throwOnInvalid;
-	}
-
-	public function throwOnInvalid():bool{
-		return $this->throwOnInvalid;
-	}
+	public function __construct(
+		public readonly bool $throwOnInvalid,
+	){}
 
 }

src/SettingsContainerAbstract.php

@@ -153,8 +153,10 @@ final protected function throwOnInvalidProperty():bool{
 		if($attributes === []){
 			return false;
 		}
+		/** @var \chillerlan\Settings\Attributes\ThrowOnInvalidProperty $attr */
+		$attr = $attributes[0]->newInstance();
 
-		return $attributes[0]->newInstance()->throwOnInvalid();
+		return $attr->throwOnInvalid;
 	}
 
 	public function toArray():array{