Inspection of "basic commit" - TheRakeshPurohit/rtCamp - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( cf3219...0b31bd )

by Rakesh

created 2019-06-11 12:08 UTC

Status

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -89,11 +89,11 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'scheme':
-            $this->_scheme = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'scheme':
+                $this->_scheme = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Media/Extension/MediaCopyright.php 1 patch

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -87,11 +87,11 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'url':
-            $this->_url = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'url':
+                $this->_url = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Media/Extension/MediaText.php 1 patch

Switch Indentation +14 added lines, -14 removed lines patch added patch discarded remove patch

@@ -120,20 +120,20 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'type':
-            $this->_type = $attribute->nodeValue;
-            break;
-        case 'lang':
-            $this->_lang = $attribute->nodeValue;
-            break;
-        case 'start':
-            $this->_start = $attribute->nodeValue;
-            break;
-        case 'end':
-            $this->_end = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'type':
+                $this->_type = $attribute->nodeValue;
+                break;
+            case 'lang':
+                $this->_lang = $attribute->nodeValue;
+                break;
+            case 'start':
+                $this->_start = $attribute->nodeValue;
+                break;
+            case 'end':
+                $this->_end = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Media/Entry.php 1 patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -97,14 +97,14 @@
 block discarded – undo
     {
         $absoluteNodeName = $child->namespaceURI . ':' . $child->localName;
         switch ($absoluteNodeName) {
-        case $this->lookupNamespace('media') . ':' . 'group':
-            $mediaGroup = new Zend_Gdata_Media_Extension_MediaGroup();
-            $mediaGroup->transferFromDOM($child);
-            $this->_mediaGroup = $mediaGroup;
-            break;
-        default:
-            parent::takeChildFromDOM($child);
-            break;
+            case $this->lookupNamespace('media') . ':' . 'group':
+                $mediaGroup = new Zend_Gdata_Media_Extension_MediaGroup();
+                $mediaGroup->transferFromDOM($child);
+                $this->_mediaGroup = $mediaGroup;
+                break;
+            default:
+                parent::takeChildFromDOM($child);
+                break;
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Gapps/Extension/Name.php 1 patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -110,14 +110,14 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'familyName':
-            $this->_familyName = $attribute->nodeValue;
-            break;
-        case 'givenName':
-            $this->_givenName = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'familyName':
+                $this->_familyName = $attribute->nodeValue;
+                break;
+            case 'givenName':
+                $this->_givenName = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Gapps/Extension/EmailList.php 1 patch

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -97,11 +97,11 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'name':
-            $this->_name = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'name':
+                $this->_name = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Gapps/Extension/Quota.php 1 patch

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -97,11 +97,11 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'limit':
-            $this->_limit = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'limit':
+                $this->_limit = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Gapps/Extension/Login.php 1 patch

Switch Indentation +59 added lines, -59 removed lines patch added patch discarded remove patch

@@ -192,65 +192,65 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'userName':
-            $this->_username = $attribute->nodeValue;
-            break;
-        case 'password':
-            $this->_password = $attribute->nodeValue;
-            break;
-        case 'hashFunctionName':
-            $this->_hashFunctionName = $attribute->nodeValue;
-            break;
-        case 'admin':
-            if ($attribute->nodeValue == "true") {
-                $this->_admin = true;
-            }
-            else if ($attribute->nodeValue == "false") {
-                $this->_admin = false;
-            }
-            else {
-                require_once('Zend/Gdata/App/InvalidArgumentException.php');
-                throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#admin.");
-            }
-            break;
-        case 'agreedToTerms':
-            if ($attribute->nodeValue == "true") {
-                $this->_agreedToTerms = true;
-            }
-            else if ($attribute->nodeValue == "false") {
-                $this->_agreedToTerms = false;
-            }
-            else {
-                require_once('Zend/Gdata/App/InvalidArgumentException.php');
-                throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#agreedToTerms.");
-            }
-            break;
-        case 'suspended':
-            if ($attribute->nodeValue == "true") {
-                $this->_suspended = true;
-            }
-            else if ($attribute->nodeValue == "false") {
-                $this->_suspended = false;
-            }
-            else {
-                require_once('Zend/Gdata/App/InvalidArgumentException.php');
-                throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#suspended.");
-            }
-            break;
-        case 'changePasswordAtNextLogin':
-            if ($attribute->nodeValue == "true") {
-                $this->_changePasswordAtNextLogin = true;
-            }
-            else if ($attribute->nodeValue == "false") {
-                $this->_changePasswordAtNextLogin = false;
-            }
-            else {
-                require_once('Zend/Gdata/App/InvalidArgumentException.php');
-                throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#changePasswordAtNextLogin.");
-            }
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'userName':
+                $this->_username = $attribute->nodeValue;
+                break;
+            case 'password':
+                $this->_password = $attribute->nodeValue;
+                break;
+            case 'hashFunctionName':
+                $this->_hashFunctionName = $attribute->nodeValue;
+                break;
+            case 'admin':
+                if ($attribute->nodeValue == "true") {
+                    $this->_admin = true;
+                }
+                else if ($attribute->nodeValue == "false") {
+                    $this->_admin = false;
+                }
+                else {
+                    require_once('Zend/Gdata/App/InvalidArgumentException.php');
+                    throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#admin.");
+                }
+                break;
+            case 'agreedToTerms':
+                if ($attribute->nodeValue == "true") {
+                    $this->_agreedToTerms = true;
+                }
+                else if ($attribute->nodeValue == "false") {
+                    $this->_agreedToTerms = false;
+                }
+                else {
+                    require_once('Zend/Gdata/App/InvalidArgumentException.php');
+                    throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#agreedToTerms.");
+                }
+                break;
+            case 'suspended':
+                if ($attribute->nodeValue == "true") {
+                    $this->_suspended = true;
+                }
+                else if ($attribute->nodeValue == "false") {
+                    $this->_suspended = false;
+                }
+                else {
+                    require_once('Zend/Gdata/App/InvalidArgumentException.php');
+                    throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#suspended.");
+                }
+                break;
+            case 'changePasswordAtNextLogin':
+                if ($attribute->nodeValue == "true") {
+                    $this->_changePasswordAtNextLogin = true;
+                }
+                else if ($attribute->nodeValue == "false") {
+                    $this->_changePasswordAtNextLogin = false;
+                }
+                else {
+                    require_once('Zend/Gdata/App/InvalidArgumentException.php');
+                    throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#changePasswordAtNextLogin.");
+                }
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

lib/Zend/Gdata/Gapps/Error.php 1 patch

Switch Indentation +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -206,17 +206,17 @@
 block discarded – undo
     protected function takeAttributeFromDOM($attribute)
     {
         switch ($attribute->localName) {
-        case 'errorCode':
-            $this->_errorCode = $attribute->nodeValue;
-            break;
-        case 'reason':
-            $this->_reason = $attribute->nodeValue;
-            break;
-        case 'invalidInput':
-            $this->_invalidInput = $attribute->nodeValue;
-            break;
-        default:
-            parent::takeAttributeFromDOM($attribute);
+            case 'errorCode':
+                $this->_errorCode = $attribute->nodeValue;
+                break;
+            case 'reason':
+                $this->_reason = $attribute->nodeValue;
+                break;
+            case 'invalidInput':
+                $this->_invalidInput = $attribute->nodeValue;
+                break;
+            default:
+                parent::takeAttributeFromDOM($attribute);
         }
     }
 

Please login to merge, or discard this patch.

		@@ -120,20 +120,20 @@
		block discarded – undo
120	120	protected function takeAttributeFromDOM($attribute)
121	121	{
122	122	switch ($attribute->localName) {
123		- case 'type':
124		- $this->_type = $attribute->nodeValue;
125		- break;
126		- case 'lang':
127		- $this->_lang = $attribute->nodeValue;
128		- break;
129		- case 'start':
130		- $this->_start = $attribute->nodeValue;
131		- break;
132		- case 'end':
133		- $this->_end = $attribute->nodeValue;
134		- break;
135		- default:
136		- parent::takeAttributeFromDOM($attribute);
	123	+ case 'type':
	124	+ $this->_type = $attribute->nodeValue;
	125	+ break;
	126	+ case 'lang':
	127	+ $this->_lang = $attribute->nodeValue;
	128	+ break;
	129	+ case 'start':
	130	+ $this->_start = $attribute->nodeValue;
	131	+ break;
	132	+ case 'end':
	133	+ $this->_end = $attribute->nodeValue;
	134	+ break;
	135	+ default:
	136	+ parent::takeAttributeFromDOM($attribute);
137	137	}
138	138	}
139	139

		@@ -97,11 +97,11 @@
		block discarded – undo
97	97	protected function takeAttributeFromDOM($attribute)
98	98	{
99	99	switch ($attribute->localName) {
100		- case 'name':
101		- $this->_name = $attribute->nodeValue;
102		- break;
103		- default:
104		- parent::takeAttributeFromDOM($attribute);
	100	+ case 'name':
	101	+ $this->_name = $attribute->nodeValue;
	102	+ break;
	103	+ default:
	104	+ parent::takeAttributeFromDOM($attribute);
105	105	}
106	106	}
107	107

		@@ -97,11 +97,11 @@
		block discarded – undo
97	97	protected function takeAttributeFromDOM($attribute)
98	98	{
99	99	switch ($attribute->localName) {
100		- case 'limit':
101		- $this->_limit = $attribute->nodeValue;
102		- break;
103		- default:
104		- parent::takeAttributeFromDOM($attribute);
	100	+ case 'limit':
	101	+ $this->_limit = $attribute->nodeValue;
	102	+ break;
	103	+ default:
	104	+ parent::takeAttributeFromDOM($attribute);
105	105	}
106	106	}
107	107

		@@ -89,11 +89,11 @@
		block discarded – undo
89	89	protected function takeAttributeFromDOM($attribute)
90	90	{
91	91	switch ($attribute->localName) {
92		- case 'scheme':
93		- $this->_scheme = $attribute->nodeValue;
94		- break;
95		- default:
96		- parent::takeAttributeFromDOM($attribute);
	92	+ case 'scheme':
	93	+ $this->_scheme = $attribute->nodeValue;
	94	+ break;
	95	+ default:
	96	+ parent::takeAttributeFromDOM($attribute);
97	97	}
98	98	}
99	99

		@@ -87,11 +87,11 @@
		block discarded – undo
87	87	protected function takeAttributeFromDOM($attribute)
88	88	{
89	89	switch ($attribute->localName) {
90		- case 'url':
91		- $this->_url = $attribute->nodeValue;
92		- break;
93		- default:
94		- parent::takeAttributeFromDOM($attribute);
	90	+ case 'url':
	91	+ $this->_url = $attribute->nodeValue;
	92	+ break;
	93	+ default:
	94	+ parent::takeAttributeFromDOM($attribute);
95	95	}
96	96	}
97	97

		@@ -97,14 +97,14 @@
		block discarded – undo
97	97	{
98	98	$absoluteNodeName = $child->namespaceURI . ':' . $child->localName;
99	99	switch ($absoluteNodeName) {
100		- case $this->lookupNamespace('media') . ':' . 'group':
101		- $mediaGroup = new Zend_Gdata_Media_Extension_MediaGroup();
102		- $mediaGroup->transferFromDOM($child);
103		- $this->_mediaGroup = $mediaGroup;
104		- break;
105		- default:
106		- parent::takeChildFromDOM($child);
107		- break;
	100	+ case $this->lookupNamespace('media') . ':' . 'group':
	101	+ $mediaGroup = new Zend_Gdata_Media_Extension_MediaGroup();
	102	+ $mediaGroup->transferFromDOM($child);
	103	+ $this->_mediaGroup = $mediaGroup;
	104	+ break;
	105	+ default:
	106	+ parent::takeChildFromDOM($child);
	107	+ break;
108	108	}
109	109	}
110	110

		@@ -110,14 +110,14 @@
		block discarded – undo
110	110	protected function takeAttributeFromDOM($attribute)
111	111	{
112	112	switch ($attribute->localName) {
113		- case 'familyName':
114		- $this->_familyName = $attribute->nodeValue;
115		- break;
116		- case 'givenName':
117		- $this->_givenName = $attribute->nodeValue;
118		- break;
119		- default:
120		- parent::takeAttributeFromDOM($attribute);
	113	+ case 'familyName':
	114	+ $this->_familyName = $attribute->nodeValue;
	115	+ break;
	116	+ case 'givenName':
	117	+ $this->_givenName = $attribute->nodeValue;
	118	+ break;
	119	+ default:
	120	+ parent::takeAttributeFromDOM($attribute);
121	121	}
122	122	}
123	123

		@@ -192,65 +192,65 @@
		block discarded – undo
192	192	protected function takeAttributeFromDOM($attribute)
193	193	{
194	194	switch ($attribute->localName) {
195		- case 'userName':
196		- $this->_username = $attribute->nodeValue;
197		- break;
198		- case 'password':
199		- $this->_password = $attribute->nodeValue;
200		- break;
201		- case 'hashFunctionName':
202		- $this->_hashFunctionName = $attribute->nodeValue;
203		- break;
204		- case 'admin':
205		- if ($attribute->nodeValue == "true") {
206		- $this->_admin = true;
207		- }
208		- else if ($attribute->nodeValue == "false") {
209		- $this->_admin = false;
210		- }
211		- else {
212		- require_once('Zend/Gdata/App/InvalidArgumentException.php');
213		- throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#admin.");
214		- }
215		- break;
216		- case 'agreedToTerms':
217		- if ($attribute->nodeValue == "true") {
218		- $this->_agreedToTerms = true;
219		- }
220		- else if ($attribute->nodeValue == "false") {
221		- $this->_agreedToTerms = false;
222		- }
223		- else {
224		- require_once('Zend/Gdata/App/InvalidArgumentException.php');
225		- throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#agreedToTerms.");
226		- }
227		- break;
228		- case 'suspended':
229		- if ($attribute->nodeValue == "true") {
230		- $this->_suspended = true;
231		- }
232		- else if ($attribute->nodeValue == "false") {
233		- $this->_suspended = false;
234		- }
235		- else {
236		- require_once('Zend/Gdata/App/InvalidArgumentException.php');
237		- throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#suspended.");
238		- }
239		- break;
240		- case 'changePasswordAtNextLogin':
241		- if ($attribute->nodeValue == "true") {
242		- $this->_changePasswordAtNextLogin = true;
243		- }
244		- else if ($attribute->nodeValue == "false") {
245		- $this->_changePasswordAtNextLogin = false;
246		- }
247		- else {
248		- require_once('Zend/Gdata/App/InvalidArgumentException.php');
249		- throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#changePasswordAtNextLogin.");
250		- }
251		- break;
252		- default:
253		- parent::takeAttributeFromDOM($attribute);
	195	+ case 'userName':
	196	+ $this->_username = $attribute->nodeValue;
	197	+ break;
	198	+ case 'password':
	199	+ $this->_password = $attribute->nodeValue;
	200	+ break;
	201	+ case 'hashFunctionName':
	202	+ $this->_hashFunctionName = $attribute->nodeValue;
	203	+ break;
	204	+ case 'admin':
	205	+ if ($attribute->nodeValue == "true") {
	206	+ $this->_admin = true;
	207	+ }
	208	+ else if ($attribute->nodeValue == "false") {
	209	+ $this->_admin = false;
	210	+ }
	211	+ else {
	212	+ require_once('Zend/Gdata/App/InvalidArgumentException.php');
	213	+ throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#admin.");
	214	+ }
	215	+ break;
	216	+ case 'agreedToTerms':
	217	+ if ($attribute->nodeValue == "true") {
	218	+ $this->_agreedToTerms = true;
	219	+ }
	220	+ else if ($attribute->nodeValue == "false") {
	221	+ $this->_agreedToTerms = false;
	222	+ }
	223	+ else {
	224	+ require_once('Zend/Gdata/App/InvalidArgumentException.php');
	225	+ throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#agreedToTerms.");
	226	+ }
	227	+ break;
	228	+ case 'suspended':
	229	+ if ($attribute->nodeValue == "true") {
	230	+ $this->_suspended = true;
	231	+ }
	232	+ else if ($attribute->nodeValue == "false") {
	233	+ $this->_suspended = false;
	234	+ }
	235	+ else {
	236	+ require_once('Zend/Gdata/App/InvalidArgumentException.php');
	237	+ throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#suspended.");
	238	+ }
	239	+ break;
	240	+ case 'changePasswordAtNextLogin':
	241	+ if ($attribute->nodeValue == "true") {
	242	+ $this->_changePasswordAtNextLogin = true;
	243	+ }
	244	+ else if ($attribute->nodeValue == "false") {
	245	+ $this->_changePasswordAtNextLogin = false;
	246	+ }
	247	+ else {
	248	+ require_once('Zend/Gdata/App/InvalidArgumentException.php');
	249	+ throw new Zend_Gdata_App_InvalidArgumentException("Expected 'true' or 'false' for apps:login#changePasswordAtNextLogin.");
	250	+ }
	251	+ break;
	252	+ default:
	253	+ parent::takeAttributeFromDOM($attribute);
254	254	}
255	255	}
256	256

		@@ -206,17 +206,17 @@
		block discarded – undo
206	206	protected function takeAttributeFromDOM($attribute)
207	207	{
208	208	switch ($attribute->localName) {
209		- case 'errorCode':
210		- $this->_errorCode = $attribute->nodeValue;
211		- break;
212		- case 'reason':
213		- $this->_reason = $attribute->nodeValue;
214		- break;
215		- case 'invalidInput':
216		- $this->_invalidInput = $attribute->nodeValue;
217		- break;
218		- default:
219		- parent::takeAttributeFromDOM($attribute);
	209	+ case 'errorCode':
	210	+ $this->_errorCode = $attribute->nodeValue;
	211	+ break;
	212	+ case 'reason':
	213	+ $this->_reason = $attribute->nodeValue;
	214	+ break;
	215	+ case 'invalidInput':
	216	+ $this->_invalidInput = $attribute->nodeValue;
	217	+ break;
	218	+ default:
	219	+ parent::takeAttributeFromDOM($attribute);
220	220	}
221	221	}
222	222

TheRakeshPurohit / rtCamp

Push — master ( cf3219...0b31bd )

Status

Category

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

Switch Indentation +14 added lines, -14 removed lines patch added patch discarded remove patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Switch Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

Switch Indentation +5 added lines, -5 removed lines patch added patch discarded remove patch

Switch Indentation +59 added lines, -59 removed lines patch added patch discarded remove patch

Switch Indentation +11 added lines, -11 removed lines patch added patch discarded remove patch