Fix test

Author: asika32764

packages/database/src/Hydrator/SimpleHydrator.php

@@ -21,9 +21,12 @@ class SimpleHydrator implements HydratorInterface, FieldHydratorInterface
      */
     public function extract(object $object): array
     {
-        if ($object instanceof DumpableInterface) {
-            return new AccessibleHydrator()->extract($object);
-        }
+        // Since dump() use get_object_dump_values(), protected fields will not return
+        // We must use total dump by ReflectAccessor::getPropertiesValues($object);
+
+        // if ($object instanceof DumpableInterface) {
+        //     return new AccessibleHydrator()->extract($object);
+        // }
 
         if ($object instanceof Traversable) {
             return iterator_to_array($object);

packages/dom/src/HTML5Factory.php

@@ -161,15 +161,20 @@ public static function createFromString(
      *
      * @param  string  $text
      *
-     * @return  HTMLElement
+     * @return  HTMLElement|Node
      */
-    public static function parse(string $text): HTMLElement
+    public static function parse(string $text): HTMLElement|Node
     {
         $doc = static::document();
         $root = $doc->createElement('root');
         $root->innerHTML = $text;
 
-        return $root->firstElementChild;
+        return $root->firstChild;
+    }
+
+    public static function parseAsElement(string $text): HTMLElement
+    {
+        return static::parse($text);
     }
 
     /**

packages/form/test/FormTest.php

@@ -411,7 +411,7 @@ public function testRenderField()
     {
         $form = $this->getByDefine('windwalker');
 
-        $html = '<div id="input-windwalker-id-wrapper" class="control-input" data-field-wrapper><label id="input-windwalker-id-label" data-field-label for="input-windwalker-id">ID</label><div><input id="input-windwalker-id" name="windwalker[id]" data-field-input type="text" value></div></div>';
+        $html = '<div id="input-windwalker-id-wrapper" class="control-input" data-field-wrapper><label id="input-windwalker-id-label" data-field-label for="input-windwalker-id">ID</label><div><input id="input-windwalker-id" name="windwalker[id]" data-field-input type="text" value=""></div></div>';
 
         self::assertEquals($html, $form->renderField('id'));
 

packages/orm/src/EntityMapper.php

@@ -1740,7 +1740,7 @@ public function toEntity(array|object $data): object
         }
 
         if (is_object($data)) {
-            $data = TypeCast::toArray($data);
+            $data = ReflectAccessor::getPropertiesValues($data);
         }
 
         // Only ORM has Hydrator, we must call ORM to do this.

packages/orm/src/NestedSetMapper.php

@@ -332,6 +332,15 @@ protected function preprocessSave(BeforeSaveEvent $event, bool $new = false): vo
         $data = $event->data;
         /** @var NestedEntityInterface $entity */
         $entity = $this->toEntity($event->source);
+
+        if (is_array($event->source)) {
+            $position = $event->source['position'];
+        } else {
+            $position = $event->source->getPosition();
+        }
+
+        $this->hydrate(compact('position'), $entity);
+
         $position = $entity->getPosition();
         $className = $this->getMetadata()->getClassName();
 
@@ -353,11 +362,18 @@ protected function preprocessSave(BeforeSaveEvent $event, bool $new = false): vo
 
             // We are inserting a node relative to the last root node.
             if (!$position->getReferenceId()) {
+                $parentId = $entity->getParentId() ?: $this->getEmptyParentId();
                 $reference = $this->select($k, 'parent_id', 'level', 'lft', 'rgt')
-                    ->where('id', $entity->getParentId() ?: $this->getEmptyParentId())
+                    ->where('id', $parentId)
                     ->order('lft', 'DESC')
                     ->limit(1)
                     ->get($className);
+
+                if (!$reference) {
+                    throw new NestedHandleException(
+                        sprintf('Reference of parent %s not found.', $parentId)
+                    );
+                }
             } else {
                 // We have a real node set as a location reference.
                 // Get the reference node by primary key.

packages/orm/test/Entity/StubLocation.php

@@ -17,6 +17,7 @@
 use Windwalker\ORM\Metadata\EntityMetadata;
 use Windwalker\ORM\Relation\Action;
 use Windwalker\ORM\Relation\RelationCollection;
+use Windwalker\Utilities\Attributes\Expose;
 
 /**
  * The Location class.
@@ -44,8 +45,10 @@ class StubLocation implements EntityInterface
         OnUpdate(Action::CASCADE),
         OnDelete(Action::CASCADE)
     ]
+    #[Expose]
     protected ?StubLocationData $data = null;
 
+    #[Expose]
     protected RelationCollection|null $sakuras = null;
 
     protected RelationCollection|null $roses = null;

packages/orm/test/Entity/StubLocationData.php

@@ -11,6 +11,7 @@
 use Windwalker\ORM\Attributes\Table;
 use Windwalker\ORM\EntityInterface;
 use Windwalker\ORM\EntityTrait;
+use Windwalker\Utilities\Attributes\Expose;
 
 /**
  * The LocationData class.
@@ -28,6 +29,7 @@ class StubLocationData implements EntityInterface
     protected string $locationNo = '';
 
     #[Column('data')]
+    #[Expose]
     protected string|stdClass $data = '';
 
     /**

packages/utilities/src/TypeCast.php

@@ -69,6 +69,24 @@ public static function extractEnum(\UnitEnum $data): string|int
         return $data->name;
     }
 
+    public static function dumpAll(object $object): array
+    {
+        static $propsCache = [];
+
+        $ref = new \ReflectionObject($object);
+        $props = $propsCache[$object::class] ??= $ref->getProperties(
+            \ReflectionProperty::IS_PUBLIC | \ReflectionProperty::IS_PROTECTED | \ReflectionProperty::IS_PRIVATE
+        );
+
+        $values = [];
+
+        foreach ($props as $prop) {
+            $values[$prop->getName()] = $prop->getValue($object);
+        }
+
+        return $values;
+    }
+
     /**
      * Utility function to convert all types to an array.
      *