Merge pull request #14 from Crowdhandler/fix/updatePerformnce

carl-wells-crowdhandler · web-flow · commit d44183a3c0e5 · 2022-12-16T12:18:42.000Z
Changed method to use property_exists for the responseID property
diff --git a/src/GateKeeper.php b/src/GateKeeper.php
@@ -288,7 +288,7 @@ private function setCookie($cookie)
      */
     public function recordPerformance($httpCode=200)
     {   
-        if(@$this->result->responseID) {
+        if(property_exists($this->result, 'responseID')) {
             $time = $this->timer->elapsed();
             $this->client->responses->put($this->result->responseID, array('code'=>$httpCode, 'time'=>$time));
             $this->debug('Page performance was recorded '.$time);

Original file line number	Diff line number	Diff line change
`@@ -288,7 +288,7 @@ private function setCookie($cookie)`
`288`	`288`	`*/`
`289`	`289`	`public function recordPerformance($httpCode=200)`
`290`	`290`	`{`
`291`		`- if(@$this->result->responseID) {`
	`291`	`+ if(property_exists($this->result, 'responseID')) {`
`292`	`292`	`$time = $this->timer->elapsed();`
`293`	`293`	`$this->client->responses->put($this->result->responseID, array('code'=>$httpCode, 'time'=>$time));`
`294`	`294`	`$this->debug('Page performance was recorded '.$time);`