Fixed some PHPDocs and outputs

opencart · May 2, 2024 · 7423172 · 7423172
1 parent 2267160
commit 7423172
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/upload/catalog/model/extension/payment/alipay.php b/upload/catalog/model/extension/payment/alipay.php
@@ -195,10 +195,6 @@ public function pageExecute($request, string $httpmethod = 'POST') {
 	 * @return bool
 	 */
 	private function checkEmpty(string $value): bool {
-		if (!isset($value)) {
-			return true;
-		}
-
 		if ($value === null) {
 			return true;
 		}

diff --git a/upload/catalog/model/extension/payment/globalpay_remote.php b/upload/catalog/model/extension/payment/globalpay_remote.php
@@ -110,9 +110,9 @@ public function checkEnrollment(string $account, float $amount, string $currency
 	 * @param string $card_name
 	 * @param string $pares
 	 *
-	 * @return SimpleXMElement
+	 * @return mixed
 	 */
-	public function enrollmentSignature(string $account, float $amount, string $currency, string $order_ref, int $card_number, int $card_expire, string $card_type, string $card_name, string $pares): SimpleXMElement {
+	public function enrollmentSignature(string $account, float $amount, string $currency, string $order_ref, int $card_number, int $card_expire, string $card_type, string $card_name, string $pares) {
 		// Orders
 		$this->load->model('checkout/order');
 

diff --git a/upload/catalog/model/extension/payment/squareup.php b/upload/catalog/model/extension/payment/squareup.php
@@ -451,7 +451,7 @@ public function updateSubscriptionTrial(int $subscription_id): bool {
 	 *
 	 * @param int $subscription_id
 	 *
-	 * @return void
+	 * @return int
 	 */
 	public function suspendSubscriptionProfile(int $subscription_id): int {
 		$this->db->query("UPDATE `" . DB_PREFIX . "squareup_subscription` SET `status` = '" . self::RECURRING_SUSPENDED . "' WHERE `subscription_id` = '" . (int)$subscription_id . "'");