Use Webmozart for assertions

tvdijen · tvdijen · commit 9963fd5726f2 · 2019-05-28T11:32:23.000+02:00
diff --git a/lib/Auth/Process/Warning.php b/lib/Auth/Process/Warning.php
@@ -2,6 +2,8 @@
 
 namespace SimpleSAML\Module\preprodwarning\Auth\Process;
 
+use Webmozart\Assert\Assert;
+
 /**
  * Give a warning that the user is accessing a test system, not a production system.
  *
@@ -21,7 +23,7 @@ class Warning extends \SimpleSAML\Auth\ProcessingFilter
      */
     public function process(&$state)
     {
-        assert(is_array($state));
+        Assert::isArray($state);
 
         if (isset($state['isPassive']) && $state['isPassive'] === true) {
             // We have a passive request. Skip the warning

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,8 @@`
`2`	`2`
`3`	`3`	`namespace SimpleSAML\Module\preprodwarning\Auth\Process;`
`4`	`4`
	`5`	`+use Webmozart\Assert\Assert;`
	`6`	`+`
`5`	`7`	`/**`
`6`	`8`	`* Give a warning that the user is accessing a test system, not a production system.`
`7`	`9`	`*`
`@@ -21,7 +23,7 @@ class Warning extends \SimpleSAML\Auth\ProcessingFilter`
`21`	`23`	`*/`
`22`	`24`	`public function process(&$state)`
`23`	`25`	`{`
`24`		`- assert(is_array($state));`
	`26`	`+ Assert::isArray($state);`
`25`	`27`
`26`	`28`	`if (isset($state['isPassive']) && $state['isPassive'] === true) {`
`27`	`29`	`// We have a passive request. Skip the warning`