refactor(MapperOptions): Simplify option names.

Author: priyadi

CHANGELOG.md

@@ -11,6 +11,7 @@
 * feat(`ObjectToObjectTransformer`): Option to disable target value reading.
 * refactor(`MainTransformer`): Make manual GC interval a static variable.
 * refactor(`ObjectToObjectMetadataFactory`): Remove `Context`.
+* refactor(`MapperOptions`): Simplify option names.
 
 ## 0.7.3
 

src/Context/MapperOptions.php

@@ -19,12 +19,12 @@
 final readonly class MapperOptions
 {
     /**
-     * @param boolean $enableLazyLoading Enable or disable lazy loading.
-     * @param boolean $enableTargetValueReading If disabled, values on the target side will not be read, and assumed to be null.
+     * @param boolean $lazyLoading Enable or disable lazy loading.
+     * @param boolean $readTargetValue If disabled, values on the target side will not be read, and assumed to be null.
      */
     public function __construct(
-        public bool $enableLazyLoading = true,
-        public bool $enableTargetValueReading = true,
+        public bool $lazyLoading = true,
+        public bool $readTargetValue = true,
     ) {
     }
 }

src/Transformer/ObjectToObjectTransformer.php

@@ -112,7 +112,7 @@ public function transform(
 
         if (
             $objectToObjectMetadata->isTargetReadOnly()
-            || !$context(MapperOptions::class)?->enableTargetValueReading
+            || !$context(MapperOptions::class)?->readTargetValue
         ) {
             $target = null;
         }
@@ -121,7 +121,7 @@ public function transform(
 
         if (null === $target) {
             $canUseTargetProxy = $objectToObjectMetadata->canUseTargetProxy()
-                && $context(MapperOptions::class)?->enableLazyLoading;
+                && $context(MapperOptions::class)?->lazyLoading;
 
             if ($canUseTargetProxy) {
                 $target = $this->instantiateTargetProxy(
@@ -522,7 +522,7 @@ private function transformValue(
 
         if (
             is_object($target)
-            && $context(MapperOptions::class)?->enableTargetValueReading
+            && $context(MapperOptions::class)?->readTargetValue
         ) {
             // if this is for a property mapping, not a constructor argument
 

tests/IntegrationTest/LazyObjectTest.php

@@ -118,7 +118,7 @@ public function testEagerAndLazyPropertyInConstruct(): void
 
     public function testEnablingLazyObject(): void
     {
-        $options = new MapperOptions(enableLazyLoading: true);
+        $options = new MapperOptions(lazyLoading: true);
         $context = Context::create($options);
 
         $source = new ObjectWithScalarProperties();
@@ -128,7 +128,7 @@ public function testEnablingLazyObject(): void
 
     public function testDisablingLazyObject(): void
     {
-        $options = new MapperOptions(enableLazyLoading: false);
+        $options = new MapperOptions(lazyLoading: false);
         $context = Context::create($options);
 
         $source = new ObjectWithScalarProperties();