Merge pull request #220 from DigitalTimK/bugfix/meetinginfo

Transform to php74 (cont'd)

Author: GhaziTriki

src/Parameters/BaseParameters.php

@@ -25,17 +25,12 @@
  */
 abstract class BaseParameters
 {
-    /**
-     * @return string
-     */
-    abstract public function getHTTPQuery();
+    abstract public function getHTTPQuery(): string;
 
     /**
      * @param mixed $array
-     *
-     * @return string
      */
-    protected function buildHTTPQuery($array)
+    protected function buildHTTPQuery($array): string
     {
         return str_replace(['%20', '!', "'", '(', ')', '*'], ['+', '%21', '%27', '%28', '%29', '%2A'], http_build_query(array_filter($array), '', '&', \PHP_QUERY_RFC3986));
     }

src/Responses/ApiVersionResponse.php

@@ -25,26 +25,17 @@
  */
 class ApiVersionResponse extends BaseResponse
 {
-    /**
-     * @return string
-     */
-    public function getVersion()
+    public function getVersion(): string
     {
         return $this->rawXml->version->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getApiVersion()
+    public function getApiVersion(): string
     {
         return $this->rawXml->apiVersion->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getBbbVersion()
+    public function getBbbVersion(): string
     {
         return $this->rawXml->bbbVersion->__toString();
     }

src/Responses/BaseJsonResponse.php

@@ -46,10 +46,7 @@ public function getRawJson()
         return json_encode($this->data);
     }
 
-    /**
-     * @return null|mixed
-     */
-    public function getMessage()
+    public function getMessage(): ?string
     {
         if ($this->failed()) {
             return $this->data->response->message;
@@ -58,10 +55,7 @@ public function getMessage()
         return null;
     }
 
-    /**
-     * @return null|mixed
-     */
-    public function getMessageKey()
+    public function getMessageKey(): ?string
     {
         if ($this->failed()) {
             return $this->data->response->messageKey;
@@ -71,9 +65,11 @@ public function getMessageKey()
     }
 
     /**
-     * @return mixed
+     * Return will be either 'SUCCESS' or 'FAILED' (nothing else).
+     *
+     * @see: https://docs.bigbluebutton.org/development/api/#api-calls
      */
-    public function getReturnCode()
+    public function getReturnCode(): string
     {
         return $this->data->response->returncode;
     }

src/Responses/DeleteRecordingsResponse.php

@@ -25,11 +25,8 @@
  */
 class DeleteRecordingsResponse extends BaseResponse
 {
-    /**
-     * @return null|bool
-     */
-    public function isDeleted()
+    public function isDeleted(): bool
     {
-        return 'true' == $this->rawXml->deleted->__toString();
+        return 'true' === $this->rawXml->deleted->__toString();
     }
 }

src/Responses/GetMeetingInfoResponse.php

@@ -27,20 +27,8 @@
  */
 class GetMeetingInfoResponse extends BaseResponse
 {
-    /**
-     * @var Meeting
-     */
-    private $meeting;
-
-    /**
-     * @return Meeting
-     */
-    public function getMeeting()
+    public function getMeeting(): Meeting
     {
-        if (null === $this->meeting) {
-            $this->meeting = new Meeting($this->rawXml);
-        }
-
-        return $this->meeting;
+        return new Meeting($this->rawXml);
     }
 }

src/Responses/HooksCreateResponse.php

@@ -25,26 +25,17 @@
  */
 class HooksCreateResponse extends BaseResponse
 {
-    /**
-     * @return int
-     */
-    public function getHookId()
+    public function getHookId(): int
     {
         return (int) $this->rawXml->hookID->__toString();
     }
 
-    /**
-     * @return null|bool
-     */
-    public function isPermanentHook()
+    public function isPermanentHook(): bool
     {
         return 'true' === $this->rawXml->permanentHook->__toString();
     }
 
-    /**
-     * @return null|bool
-     */
-    public function hasRawData()
+    public function hasRawData(): bool
     {
         return 'true' === $this->rawXml->rawData->__toString();
     }

src/Responses/HooksDestroyResponse.php

@@ -25,10 +25,7 @@
  */
 class HooksDestroyResponse extends BaseResponse
 {
-    /**
-     * @return null|bool
-     */
-    public function removed()
+    public function removed(): bool
     {
         return 'true' === $this->rawXml->removed->__toString();
     }

src/Responses/IsMeetingRunningResponse.php

@@ -25,10 +25,7 @@
  */
 class IsMeetingRunningResponse extends BaseResponse
 {
-    /**
-     * @return null|bool
-     */
-    public function isRunning()
+    public function isRunning(): bool
     {
         return 'true' === $this->rawXml->running->__toString();
     }

src/Responses/JoinMeetingResponse.php

@@ -25,50 +25,32 @@
  */
 class JoinMeetingResponse extends BaseResponse
 {
-    /**
-     * @return string
-     */
-    public function getMeetingId()
+    public function getMeetingId(): string
     {
         return $this->rawXml->meeting_id->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getUserId()
+    public function getUserId(): string
     {
         return $this->rawXml->user_id->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getAuthToken()
+    public function getAuthToken(): string
     {
         return $this->rawXml->auth_token->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getSessionToken()
+    public function getSessionToken(): string
     {
         return $this->rawXml->session_token->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getGuestStatus()
+    public function getGuestStatus(): string
     {
         return $this->rawXml->guestStatus->__toString();
     }
 
-    /**
-     * @return string
-     */
-    public function getUrl()
+    public function getUrl(): string
     {
         return $this->rawXml->url->__toString();
     }

src/Responses/PublishRecordingsResponse.php

@@ -25,10 +25,7 @@
  */
 class PublishRecordingsResponse extends BaseResponse
 {
-    /**
-     * @return null|bool
-     */
-    public function isPublished()
+    public function isPublished(): bool
     {
         return 'true' === $this->rawXml->published->__toString();
     }