MAPIException::getDisplayMessage() - Code Metrics - Inspection of "lang: typo succesful -> successful" - grommunio/grommunio-web - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 647c72...cd42b5 )

unknown

created 2022-05-28 23:49 UTC

MAPIException::getDisplayMessage() D

↳ Parent: MAPIException

Complexity

Conditions	21
Paths	21

Size

Total Lines	37
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	21
eloc	31
nc	21
nop	0
dl	0
loc	37
rs	4.1666
c	1
b	0
f	0

How to fix Complexity

<?php
	/**
	 * MAPIException
	 * if enabled using mapi_enable_exceptions then php-ext can throw exceptions when
	 * any error occurs in mapi calls. this exception will only be thrown when severity bit is set in
	 * error code that means it will be thrown only for mapi errors not for mapi warnings.
	 */
	class MAPIException extends BaseException
	{
		/**
		 * Function will return display message of exception if its set by the callee.
		 * if it is not set then we are generating some default display messages based
		 * on mapi error code.
		 * @return string returns error-message that should be sent to client to display.
		 */
		public function getDisplayMessage()
		{
			if(!empty($this->displayMessage))
				return $this->displayMessage;

			switch($this->getCode()) 
			{
				/* see also class.webappauthentication.php:getErrorMessage for more instances */

				case MAPI_E_NO_ACCESS:
					return dgettext("zarafa","You have insufficient privileges to open this object.");
				case ecUnknownUser:
				case MAPI_E_LOGON_FAILED:
				case MAPI_E_UNCONFIGURED:
					return dgettext("zarafa","Logon Failed. Please check your name/password.");
				case MAPI_E_NETWORK_ERROR:
					return dgettext("zarafa","Can not connect to Gromox.");
				case MAPI_E_UNKNOWN_ENTRYID:
					return dgettext("zarafa","Can not open object with provided id.");
				case MAPI_E_NO_RECIPIENTS:
					return dgettext("zarafa","There are no recipients in the message.");
				case MAPI_E_NOT_FOUND:
					return dgettext("zarafa","Can not find object.");
				case MAPI_E_NOT_ENOUGH_MEMORY:
					return dgettext("zarafa","Operation failed: Server does not have enough memory.");
				case MAPI_E_INTERFACE_NOT_SUPPORTED:
				case MAPI_E_INVALID_PARAMETER:
				case MAPI_E_INVALID_ENTRYID:
				case MAPI_E_INVALID_OBJECT:
				case MAPI_E_TOO_COMPLEX:
				case MAPI_E_CORRUPT_DATA:
				case MAPI_E_END_OF_SESSION:
				case MAPI_E_AMBIGUOUS_RECIP:
				case MAPI_E_COLLISION:
				case MAPI_E_UNCONFIGURED:
				default :
					return sprintf(dgettext("zarafa","Unknown MAPI Error: %s"), get_mapi_error_name($this->getCode()));
			}
		}
	}

	// Tell the PHP extension which exception class to instantiate
	if (function_exists('mapi_enable_exceptions')) {
		mapi_enable_exceptions("mapiexception");
	}
?>



1			<?php
2			/**
3			* MAPIException
4			* if enabled using mapi_enable_exceptions then php-ext can throw exceptions when
5			* any error occurs in mapi calls. this exception will only be thrown when severity bit is set in
6			* error code that means it will be thrown only for mapi errors not for mapi warnings.
7			*/
8			class MAPIException extends BaseException
9			{
10			/**
11			* Function will return display message of exception if its set by the callee.
12			* if it is not set then we are generating some default display messages based
13			* on mapi error code.
14			* @return string returns error-message that should be sent to client to display.
15			*/
16			public function getDisplayMessage()
17			{
18			if(!empty($this->displayMessage))
19			return $this->displayMessage;
20
21			switch($this->getCode())
22			{
23			/* see also class.webappauthentication.php:getErrorMessage for more instances */
24
25			case MAPI_E_NO_ACCESS:
26			return dgettext("zarafa","You have insufficient privileges to open this object.");
27			case ecUnknownUser:
28			case MAPI_E_LOGON_FAILED:
29			case MAPI_E_UNCONFIGURED:
30			return dgettext("zarafa","Logon Failed. Please check your name/password.");
31			case MAPI_E_NETWORK_ERROR:
32			return dgettext("zarafa","Can not connect to Gromox.");
33			case MAPI_E_UNKNOWN_ENTRYID:
34			return dgettext("zarafa","Can not open object with provided id.");
35			case MAPI_E_NO_RECIPIENTS:
36			return dgettext("zarafa","There are no recipients in the message.");
37			case MAPI_E_NOT_FOUND:
38			return dgettext("zarafa","Can not find object.");
39			case MAPI_E_NOT_ENOUGH_MEMORY:
40			return dgettext("zarafa","Operation failed: Server does not have enough memory.");
41			case MAPI_E_INTERFACE_NOT_SUPPORTED:
42			case MAPI_E_INVALID_PARAMETER:
43			case MAPI_E_INVALID_ENTRYID:
44			case MAPI_E_INVALID_OBJECT:
45			case MAPI_E_TOO_COMPLEX:
46			case MAPI_E_CORRUPT_DATA:
47			case MAPI_E_END_OF_SESSION:
48			case MAPI_E_AMBIGUOUS_RECIP:
49			case MAPI_E_COLLISION:
50			case MAPI_E_UNCONFIGURED:
51			default :
52			return sprintf(dgettext("zarafa","Unknown MAPI Error: %s"), get_mapi_error_name($this->getCode()));
53			}
54			}
55			}
56
57			// Tell the PHP extension which exception class to instantiate
58			if (function_exists('mapi_enable_exceptions')) {
59			mapi_enable_exceptions("mapiexception");
60			}
61			?>
			0 ignored issues – show Best Practice introduced 2022-05-28 23:59 UTC by Report Bug Copy Issue Report It is not recommended to use PHP's closing tag `?>` in files other than templates. Using a closing tag in PHP files that only contain PHP code is not recommended as you might accidentally add whitespace after the closing tag which would then be output by PHP. This can cause severe problems, for example headers cannot be sent anymore. A simple precaution is to leave off the closing tag as it is not required, and it also has no negative effects whatsoever. Loading history...
62

grommunio / grommunio-web

Push — master ( 647c72...cd42b5 )

MAPIException::getDisplayMessage() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like