fix: correct variable naming and syntax errors in webhook actions

Author: ritoban23

src/Core/Notification/Action/_ActionWebhook.php

@@ -173,9 +173,9 @@ protected function _DoExecute($oTrigger, $aContextArgs, &$oLog)
 			}
 
 			// Errors during preparation
-			if (!empty($this->m_aWebrequestErrors))
+			if (!empty($this->aRequestErrors))
 			{
-				return 'Errors: '.implode(', ', $this->m_aWebrequestErrors);
+				return 'Errors: '.implode(', ', $this->aRequestErrors);
 			}
 
 			if ($this->IsBeingTested() && empty($oRequest->GetURL()))
@@ -197,7 +197,7 @@ protected function _DoExecute($oTrigger, $aContextArgs, &$oLog)
 				case WebRequestSender::ENUM_SEND_STATE_PENDING:
 					return 'Pending';
 
-				case WebRequestSender::ENUM_SEND_STATE_ERROR;
+				case WebRequestSender::ENUM_SEND_STATE_ERROR:
 					return 'Errors: '.implode(', ', $this->aRequestErrors);
 			}
 		}

tests/php-unit-tests/_ActionWebhookTest.php

@@ -13,8 +13,6 @@
 use Combodo\iTop\Service\CallbackService;
 use Combodo\iTop\Test\UnitTest\ItopTestCase;
 use DBObject;
-use Exception;
-use ReflectionException;
 
 class _ActionWebhookTest extends ItopTestCase
 {