Compatible with PHP7.4

DoveChen · Aug 28, 2023 · 689a1b7 · 689a1b7
1 parent d127ddd
commit 689a1b7
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 9 deletions.
diff --git a/src/dataStructure/ExternalContactWay.php b/src/dataStructure/ExternalContactWay.php
@@ -64,7 +64,7 @@ public static function CheckExternalContactWayAddArgs ($externalContactWay)
 			if ($externalContactWay->type == ExternalContactWay::TYPE_ONLY) {
 				Utils::checkNotEmptyArray($externalContactWay->user, 'external contact user');
 
-				if (count($externalContactWay->user) > 1) {
+				if (is_array($externalContactWay->user) && count($externalContactWay->user) > 1) {
 					throw new \ParameterError('user only can be one');
 				}
 			}

diff --git a/src/dataStructure/LinkedcorpMessage.php b/src/dataStructure/LinkedcorpMessage.php
@@ -52,13 +52,13 @@ public static function pareFromArray ($arr)
 		 */
 		public static function CheckMessageSendArgs ($args)
 		{
-			if (count($args->touser) > 1000) {
+			if (is_array($args->touser) && count($args->touser) > 1000) {
 				throw new \QyApiError("touser should be no more than 1000");
 			}
-			if (count($args->toparty) > 100) {
+			if (is_array($args->toparty) && count($args->toparty) > 100) {
 				throw new \QyApiError("toparty should be no more than 100");
 			}
-			if (count($args->totag) > 100) {
+			if (is_array($args->totag) && count($args->totag) > 100) {
 				throw new \QyApiError("toparty should be no more than 100");
 			}
 

diff --git a/src/dataStructure/Message.php b/src/dataStructure/Message.php
@@ -58,13 +58,13 @@ public static function pareFromArray ($arr)
 		 */
 		public static function CheckMessageSendArgs ($args)
 		{
-			if (count($args->touser) > 1000) {
+			if (is_array($args->touser) && count($args->touser) > 1000) {
 				throw new \QyApiError("touser should be no more than 1000");
 			}
-			if (count($args->toparty) > 100) {
+			if (is_array($args->toparty) && count($args->toparty) > 100) {
 				throw new \QyApiError("toparty should be no more than 100");
 			}
-			if (count($args->totag) > 100) {
+			if (is_array($args->totag) && count($args->totag) > 100) {
 				throw new \QyApiError("toparty should be no more than 100");
 			}
 

diff --git a/src/dataStructure/TaskcardMesssageContent.php b/src/dataStructure/TaskcardMesssageContent.php
@@ -70,7 +70,7 @@ public static function CheckMessageSendArgs ($args)
 			Utils::checkNotEmptyStr($args->task_id, 'task_id');
 			Utils::checkNotEmptyArray($args->btn, 'btn');
 
-			if (count($args->btn) > 2) {
+			if (is_array($args->btn) && count($args->btn) > 2) {
 				throw new \QyApiError('btn only can be one or two');
 			}
 

diff --git a/src/dataStructure/User.php b/src/dataStructure/User.php
@@ -137,7 +137,7 @@ public static function CheckUserBatchDeleteArgs ($userIdList)
 			foreach ($userIdList as $userId) {
 				Utils::checkNotEmptyStr($userId, "userid");
 			}
-			if (count($userIdList) > 200) {
+			if (is_array($userIdList) && count($userIdList) > 200) {
 				throw new \QyApiError("no more than 200 userid once");
 			}
 		}