Try to reproduce serialize issues

dereuromark · dereuromark · commit d9b63681a148 · 2025-11-02T14:18:07.000+01:00
diff --git a/src/ToolbarService.php b/src/ToolbarService.php
@@ -291,10 +291,31 @@ public function saveData(ServerRequest $request, ResponseInterface $response): R
         foreach ($this->registry->loaded() as $name) {
             $panel = $this->registry->{$name};
             try {
-                $content = serialize($panel->data());
+                $data = $panel->data();
+
+                // Set error handler to catch warnings/errors during serialization
+                set_error_handler(function ($errno, $errstr) use ($name): void {
+                    throw new Exception("Serialization error in panel '{$name}': {$errstr}");
+                });
+
+                $content = serialize($data);
+
+                restore_error_handler();
             } catch (Exception $e) {
+                restore_error_handler();
+
+                $errorMessage = sprintf(
+                    'Failed to serialize data for panel "%s": %s',
+                    $name,
+                    $e->getMessage(),
+                );
+
+                Log::warning($errorMessage);
+                Log::debug('Panel data type: ' . gettype($data ?? null));
+
                 $content = serialize([
-                    'error' => $e->getMessage(),
+                    'error' => $errorMessage,
+                    'panel' => $name,
                 ]);
             }
             $row->panels[] = $requests->Panels->newEntity([
diff --git a/tests/TestCase/ToolbarServiceTest.php b/tests/TestCase/ToolbarServiceTest.php
@@ -506,4 +506,53 @@ public function testIsEnabledForceEnableCallable()
         ]);
         $this->assertTrue($bar->isEnabled(), 'debug is off, panel is forced on');
     }
+
+    /**
+     * Test that saveData handles serialization errors gracefully
+     *
+     * @return void
+     */
+    public function testSaveDataSerializationError()
+    {
+        $request = new Request([
+            'url' => '/articles',
+            'environment' => ['REQUEST_METHOD' => 'GET'],
+        ]);
+        $response = new Response([
+            'statusCode' => 200,
+            'type' => 'text/html',
+            'body' => '<html><title>test</title><body><p>some text</p></body>',
+        ]);
+
+        $bar = new ToolbarService($this->events, []);
+        $bar->loadPanels();
+
+        // Create a panel with unserializable data
+        $panel = $bar->registry()->load('DebugKit.TestApp\Panel\SimplePanel');
+        // Mock the data() method to return something problematic
+        $panel->_data = ['closure' => fn () => 'test'];
+
+        $row = $bar->saveData($request, $response);
+        $this->assertNotEmpty($row, 'Should save data even with serialization errors');
+
+        $requests = $this->getTableLocator()->get('DebugKit.Requests');
+        $result = $requests->find()
+            ->orderBy(['Requests.requested_at' => 'DESC'])
+            ->contain('Panels')
+            ->first();
+
+        // Find the SimplePanel in the results
+        $simplePanel = null;
+        foreach ($result->panels as $p) {
+            if ($p->panel === 'SimplePanel') {
+                $simplePanel = $p;
+                break;
+            }
+        }
+
+        $this->assertNotNull($simplePanel, 'SimplePanel should be present');
+        $content = unserialize($simplePanel->content);
+        $this->assertArrayHasKey('error', $content, 'Should have error key');
+        $this->assertStringContainsString('SimplePanel', $content['error'], 'Error should mention panel name');
+    }
 }
