tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

BUG Fix issue with DebugView failing on class name of existing class

Browse Source 
Fixes #7827
This commit is contained in:
Damian Mooyman 2018-02-05 10:45:49 +13:00
parent b81ac41b5d
commit 288aaf083c
No known key found for this signature in database
GPG Key ID: 78B823A10DE27D1A
2 changed files with 19 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Dev/DebugView.php
View File

@ -421,7 +421,7 @@ class DebugView
public function debugVariableText($val) public function debugVariableText($val)
{ {
// Check debug // Check debug
if (ClassInfo::hasMethod($val, 'debug')) { if (is_object($val) && ClassInfo::hasMethod($val, 'debug')) {
return $val->debug(); return $val->debug();
} }

22
tests/php/Dev/DebugViewTest.php
View File

@ -31,7 +31,7 @@ class DebugViewTest extends SapphireTest
<<<EOS <<<EOS
<div style="background-color: white; text-align: left;"> <div style="background-color: white; text-align: left;">
<hr> <hr>
<h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\Dev\Tests\DebugViewTest::setUp())</span> <h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\\Dev\\Tests\\DebugViewTest::setUp())</span>
</h3> </h3>
<pre style="font-family: Courier new, serif">string</pre> <pre style="font-family: Courier new, serif">string</pre>
</div> </div>
@ -44,7 +44,7 @@ EOS
<<<EOS <<<EOS
<div style="background-color: white; text-align: left;"> <div style="background-color: white; text-align: left;">
<hr> <hr>
<h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\Dev\Tests\DebugViewTest::setUp())</span> <h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\\Dev\\Tests\\DebugViewTest::setUp())</span>
</h3> </h3>
<ul> <ul>
<li>key = <pre style="font-family: Courier new, serif">value</pre> <li>key = <pre style="font-family: Courier new, serif">value</pre>
@ -62,12 +62,26 @@ EOS
<<<EOS <<<EOS
<div style="background-color: white; text-align: left;"> <div style="background-color: white; text-align: left;">
<hr> <hr>
<h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\Dev\Tests\DebugViewTest::setUp())</span> <h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\\Dev\\Tests\\DebugViewTest::setUp())</span>
</h3> </h3>
SilverStripe\Dev\Tests\DebugViewTest\ObjectWithDebug::debug() custom content</div> SilverStripe\\Dev\\Tests\\DebugViewTest\\ObjectWithDebug::debug() custom content</div>
EOS EOS
, ,
$view->debugVariable(new ObjectWithDebug(), $this->caller) $view->debugVariable(new ObjectWithDebug(), $this->caller)
); );
$this->assertEquals(
<<<EOS
<div style="background-color: white; text-align: left;">
<hr>
<h3>Debug <span style="font-size: 65%">(DebugViewTest.php:17 - SilverStripe\\Dev\\Tests\\DebugViewTest::setUp())</span>
</h3>
<pre style="font-family: Courier new, serif">SilverStripe\\Dev\\Tests\\DebugViewTest\\ObjectWithDebug</pre>
</div>
EOS
,
$view->debugVariable(ObjectWithDebug::class, $this->caller)
);
} }
} }