Inspection of "Add response handler for authentication and update..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

jetpack

Completed

Push — whmcs-jetpack-partner-module ( 85a63c...12992f )

unknown

created 2018-09-03 11:26 UTC

Status

All Patches
120
New
1

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -181,7 +181,7 @@
 block discarded – undo
  *
  *
  * @param array $params WHMCS params
- * @return mixed A string with the access token or an error string beginning with 'JETPACK MODULE' indicating that
+ * @return null|string A string with the access token or an error string beginning with 'JETPACK MODULE' indicating that
  * an access token was not retrieved for provisioning or cancelling.
  * @throws Exception  An exception is thrown from make_api_request when there is a curl_error or an empty response
  */

Please login to merge, or discard this patch.

		@@ -181,7 +181,7 @@
		block discarded – undo
181	181	*
182	182	*
183	183	* @param array $params WHMCS params
184		- * @return mixed A string with the access token or an error string beginning with 'JETPACK MODULE' indicating that
	184	+ * @return null\|string A string with the access token or an error string beginning with 'JETPACK MODULE' indicating that
185	185	* an access token was not retrieved for provisioning or cancelling.
186	186	* @throws Exception An exception is thrown from make_api_request when there is a curl_error or an empty response
187	187	*/

Automattic / jetpack

Push — whmcs-jetpack-partner-module ( 85a63c...12992f )

Status

Category

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch