Inspection of "Merge branch '1.1'" - pamil/Sylius - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( adfc28...f3aa77 )

by Kamil

created 2017-12-28 22:48 UTC

Status

Doc Comments +9 added lines patch added patch discarded remove patch

@@ -43,6 +43,7 @@  discard block
 block discarded – undo
 
     /**
      * @param TaxonInterface|null $taxon
+     * @return void
      */
     public function setParent(?TaxonInterface $taxon): void;
 
@@ -70,11 +71,13 @@  discard block
 block discarded – undo
 
     /**
      * @param TaxonInterface $taxon
+     * @return void
      */
     public function addChild(TaxonInterface $taxon): void;
 
     /**
      * @param TaxonInterface $taxon
+     * @return void
      */
     public function removeChild(TaxonInterface $taxon): void;
 
@@ -85,6 +88,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string|null $name
+     * @return void
      */
     public function setName(?string $name): void;
 
@@ -95,6 +99,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string|null $description
+     * @return void
      */
     public function setDescription(?string $description): void;
 
@@ -105,6 +110,7 @@  discard block
 block discarded – undo
 
     /**
      * @param int|null $left
+     * @return void
      */
     public function setLeft(?int $left): void;
 
@@ -115,6 +121,7 @@  discard block
 block discarded – undo
 
     /**
      * @param int|null $right
+     * @return void
      */
     public function setRight(?int $right): void;
 
@@ -125,6 +132,7 @@  discard block
 block discarded – undo
 
     /**
      * @param int|null $level
+     * @return void
      */
     public function setLevel(?int $level): void;
 
@@ -135,6 +143,7 @@  discard block
 block discarded – undo
 
     /**
      * @param int|null $position
+     * @return void
      */
     public function setPosition(?int $position): void;
 

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/Administrator/CreatePageInterface.php 1 patch

Doc Comments +7 added lines patch added patch discarded remove patch

@@ -17,25 +17,32 @@
 block discarded – undo
 
 interface CreatePageInterface extends BaseCreatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
     /**
      * @param string $username
+     * @return void
      */
     public function specifyUsername($username);
 
     /**
      * @param string $email
+     * @return void
      */
     public function specifyEmail($email);
 
     /**
      * @param string $password
+     * @return void
      */
     public function specifyPassword($password);
 
     /**
      * @param string $localeCode
+     * @return void
      */
     public function specifyLocale($localeCode);
 }

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/Channel/CreatePageInterface.php 1 patch

Doc Comments +24 added lines patch added patch discarded remove patch

@@ -17,71 +17,95 @@
 block discarded – undo
 
 interface CreatePageInterface extends BaseCreatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $name
+     * @return void
      */
     public function nameIt($name);
 
     /**
      * @param string $code
+     * @return void
      */
     public function specifyCode($code);
 
     /**
      * @param string $description
+     * @return void
      */
     public function describeItAs($description);
 
     /**
      * @param string $hostname
+     * @return void
      */
     public function setHostname($hostname);
 
     /**
      * @param string $contactEmail
+     * @return void
      */
     public function setContactEmail($contactEmail);
 
     /**
      * @param string $color
+     * @return void
      */
     public function defineColor($color);
 
     /**
      * @param string $language
+     * @return void
      */
     public function chooseLocale($language);
 
     /**
      * @param string $currencyCode
+     * @return void
      */
     public function chooseCurrency($currencyCode);
 
     /**
      * @param string $taxZone
+     * @return void
      */
     public function chooseDefaultTaxZone($taxZone);
 
     /**
      * @param string $locale
+     * @return void
      */
     public function chooseDefaultLocale($locale);
 
     /**
      * @param string $currency
+     * @return void
      */
     public function chooseBaseCurrency($currency);
 
     /**
      * @param string $taxCalculationStrategy
+     * @return void
      */
     public function chooseTaxCalculationStrategy($taxCalculationStrategy);
 
+    /**
+     * @return void
+     */
     public function allowToSkipShippingStep();
 
+    /**
+     * @return void
+     */
     public function allowToSkipPaymentStep();
 }

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/Channel/UpdatePageInterface.php 1 patch

Doc Comments +12 added lines patch added patch discarded remove patch

@@ -18,17 +18,25 @@  discard block
 block discarded – undo
 
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $themeName
+     * @return void
      */
     public function setTheme($themeName);
 
     /**
      * @throws ElementNotFoundException
+     * @return void
      */
     public function unsetTheme();
 
@@ -39,6 +47,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string $language
+     * @return void
      */
     public function chooseLocale($language);
 
@@ -51,6 +60,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string $currencyCode
+     * @return void
      */
     public function chooseCurrency($currencyCode);
 
@@ -63,11 +73,13 @@  discard block
 block discarded – undo
 
     /**
      * @param string $taxZone
+     * @return void
      */
     public function chooseDefaultTaxZone($taxZone);
 
     /**
      * @param string $taxCalculationStrategy
+     * @return void
      */
     public function chooseTaxCalculationStrategy($taxCalculationStrategy);
 

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/Country/UpdatePageInterface.php 1 patch

Doc Comments +14 added lines patch added patch discarded remove patch

@@ -17,8 +17,14 @@  discard block
 block discarded – undo
 
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
@@ -44,28 +50,36 @@  discard block
 block discarded – undo
      * @param string $name
      * @param string $code
      * @param string|null $abbreviation
+     * @return void
      */
     public function addProvince($name, $code, $abbreviation = null);
 
     /**
      * @param string $provinceName
+     * @return void
      */
     public function removeProvince($provinceName);
 
+    /**
+     * @return void
+     */
     public function clickAddProvinceButton();
 
     /**
      * @param string $provinceName
+     * @return void
      */
     public function nameProvince($provinceName);
 
     /**
      * @param string $provinceName
+     * @return void
      */
     public function removeProvinceName($provinceName);
 
     /**
      * @param string $provinceCode
+     * @return void
      */
     public function specifyProvinceCode($provinceCode);
 }

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/Currency/UpdatePageInterface.php 1 patch

Doc Comments +7 added lines patch added patch discarded remove patch

@@ -17,8 +17,14 @@  discard block
 block discarded – undo
 
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
@@ -33,6 +39,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string $exchangeRate
+     * @return void
      */
     public function changeExchangeRate($exchangeRate);
 

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/Customer/UpdatePageInterface.php 1 patch

Doc Comments +13 added lines patch added patch discarded remove patch

@@ -17,8 +17,14 @@  discard block
 block discarded – undo
 
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
@@ -28,6 +34,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string $firstName
+     * @return void
      */
     public function changeFirstName($firstName);
 
@@ -38,6 +45,7 @@  discard block
 block discarded – undo
 
     /**
      * @param string $lastName
+     * @return void
      */
     public function changeLastName($lastName);
 
@@ -48,11 +56,13 @@  discard block
 block discarded – undo
 
     /**
      * @param string $email
+     * @return void
      */
     public function changeEmail($email);
 
     /**
      * @param string $password
+     * @return void
      */
     public function changePassword($password);
 
@@ -61,6 +71,9 @@  discard block
 block discarded – undo
      */
     public function getPassword();
 
+    /**
+     * @return void
+     */
     public function subscribeToTheNewsletter();
 
     /**

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/PaymentMethod/CreatePageInterface.php 1 patch

Doc Comments +16 added lines patch added patch discarded remove patch

@@ -17,60 +17,76 @@
 block discarded – undo
 
 interface CreatePageInterface extends BaseCreatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $name
      * @param string $languageCode
+     * @return void
      */
     public function nameIt($name, $languageCode);
 
     /**
      * @param string $code
+     * @return void
      */
     public function specifyCode($code);
 
     /**
      * @param string $channelName
+     * @return void
      */
     public function checkChannel($channelName);
 
     /**
      * @param string $description
      * @param string $languageCode
+     * @return void
      */
     public function describeIt($description, $languageCode);
 
     /**
      * @param string $instructions
      * @param string $languageCode
+     * @return void
      */
     public function setInstructions($instructions, $languageCode);
 
     /**
      * @param string $username
+     * @return void
      */
     public function setPaypalGatewayUsername($username);
 
     /**
      * @param string $password
+     * @return void
      */
     public function setPaypalGatewayPassword($password);
 
     /**
      * @param string $signature
+     * @return void
      */
     public function setPaypalGatewaySignature($signature);
 
     /**
      * @param string $secretKey
+     * @return void
      */
     public function setStripeSecretKey($secretKey);
 
     /**
      * @param string $publishableKey
+     * @return void
      */
     public function setStripePublishableKey($publishableKey);
 

Please login to merge, or discard this patch.

src/Sylius/Behat/Page/Admin/PaymentMethod/UpdatePageInterface.php 1 patch

Doc Comments +10 added lines patch added patch discarded remove patch

@@ -17,28 +17,38 @@
 block discarded – undo
 
 interface UpdatePageInterface extends BaseUpdatePageInterface
 {
+    /**
+     * @return void
+     */
     public function enable();
 
+    /**
+     * @return void
+     */
     public function disable();
 
     /**
      * @param string $name
      * @param string $languageCode
+     * @return void
      */
     public function nameIt($name, $languageCode);
 
     /**
      * @param string $username
+     * @return void
      */
     public function setPaypalGatewayUsername($username);
 
     /**
      * @param string $password
+     * @return void
      */
     public function setPaypalGatewayPassword($password);
 
     /**
      * @param string $signature
+     * @return void
      */
     public function setPaypalGatewaySignature($signature);
 

Please login to merge, or discard this patch.

		@@ -43,6 +43,7 @@ discard block
		block discarded – undo
43	43
44	44	/**
45	45	* @param TaxonInterface\|null $taxon
	46	+ * @return void
46	47	*/
47	48	public function setParent(?TaxonInterface $taxon): void;
48	49
		@@ -70,11 +71,13 @@ discard block
		block discarded – undo
70	71
71	72	/**
72	73	* @param TaxonInterface $taxon
	74	+ * @return void
73	75	*/
74	76	public function addChild(TaxonInterface $taxon): void;
75	77
76	78	/**
77	79	* @param TaxonInterface $taxon
	80	+ * @return void
78	81	*/
79	82	public function removeChild(TaxonInterface $taxon): void;
80	83
		@@ -85,6 +88,7 @@ discard block
		block discarded – undo
85	88
86	89	/**
87	90	* @param string\|null $name
	91	+ * @return void
88	92	*/
89	93	public function setName(?string $name): void;
90	94
		@@ -95,6 +99,7 @@ discard block
		block discarded – undo
95	99
96	100	/**
97	101	* @param string\|null $description
	102	+ * @return void
98	103	*/
99	104	public function setDescription(?string $description): void;
100	105
		@@ -105,6 +110,7 @@ discard block
		block discarded – undo
105	110
106	111	/**
107	112	* @param int\|null $left
	113	+ * @return void
108	114	*/
109	115	public function setLeft(?int $left): void;
110	116
		@@ -115,6 +121,7 @@ discard block
		block discarded – undo
115	121
116	122	/**
117	123	* @param int\|null $right
	124	+ * @return void
118	125	*/
119	126	public function setRight(?int $right): void;
120	127
		@@ -125,6 +132,7 @@ discard block
		block discarded – undo
125	132
126	133	/**
127	134	* @param int\|null $level
	135	+ * @return void
128	136	*/
129	137	public function setLevel(?int $level): void;
130	138
		@@ -135,6 +143,7 @@ discard block
		block discarded – undo
135	143
136	144	/**
137	145	* @param int\|null $position
	146	+ * @return void
138	147	*/
139	148	public function setPosition(?int $position): void;
140	149

		@@ -17,25 +17,32 @@
		block discarded – undo
17	17
18	18	interface CreatePageInterface extends BaseCreatePageInterface
19	19	{
	20	+ /**
	21	+ * @return void
	22	+ */
20	23	public function enable();
21	24
22	25	/**
23	26	* @param string $username
	27	+ * @return void
24	28	*/
25	29	public function specifyUsername($username);
26	30
27	31	/**
28	32	* @param string $email
	33	+ * @return void
29	34	*/
30	35	public function specifyEmail($email);
31	36
32	37	/**
33	38	* @param string $password
	39	+ * @return void
34	40	*/
35	41	public function specifyPassword($password);
36	42
37	43	/**
38	44	* @param string $localeCode
	45	+ * @return void
39	46	*/
40	47	public function specifyLocale($localeCode);
41	48	}

		@@ -17,71 +17,95 @@
		block discarded – undo
17	17
18	18	interface CreatePageInterface extends BaseCreatePageInterface
19	19	{
	20	+ /**
	21	+ * @return void
	22	+ */
20	23	public function enable();
21	24
	25	+ /**
	26	+ * @return void
	27	+ */
22	28	public function disable();
23	29
24	30	/**
25	31	* @param string $name
	32	+ * @return void
26	33	*/
27	34	public function nameIt($name);
28	35
29	36	/**
30	37	* @param string $code
	38	+ * @return void
31	39	*/
32	40	public function specifyCode($code);
33	41
34	42	/**
35	43	* @param string $description
	44	+ * @return void
36	45	*/
37	46	public function describeItAs($description);
38	47
39	48	/**
40	49	* @param string $hostname
	50	+ * @return void
41	51	*/
42	52	public function setHostname($hostname);
43	53
44	54	/**
45	55	* @param string $contactEmail
	56	+ * @return void
46	57	*/
47	58	public function setContactEmail($contactEmail);
48	59
49	60	/**
50	61	* @param string $color
	62	+ * @return void
51	63	*/
52	64	public function defineColor($color);
53	65
54	66	/**
55	67	* @param string $language
	68	+ * @return void
56	69	*/
57	70	public function chooseLocale($language);
58	71
59	72	/**
60	73	* @param string $currencyCode
	74	+ * @return void
61	75	*/
62	76	public function chooseCurrency($currencyCode);
63	77
64	78	/**
65	79	* @param string $taxZone
	80	+ * @return void
66	81	*/
67	82	public function chooseDefaultTaxZone($taxZone);
68	83
69	84	/**
70	85	* @param string $locale
	86	+ * @return void
71	87	*/
72	88	public function chooseDefaultLocale($locale);
73	89
74	90	/**
75	91	* @param string $currency
	92	+ * @return void
76	93	*/
77	94	public function chooseBaseCurrency($currency);
78	95
79	96	/**
80	97	* @param string $taxCalculationStrategy
	98	+ * @return void
81	99	*/
82	100	public function chooseTaxCalculationStrategy($taxCalculationStrategy);
83	101
	102	+ /**
	103	+ * @return void
	104	+ */
84	105	public function allowToSkipShippingStep();
85	106
	107	+ /**
	108	+ * @return void
	109	+ */
86	110	public function allowToSkipPaymentStep();
87	111	}

		@@ -18,17 +18,25 @@ discard block
		block discarded – undo
18	18
19	19	interface UpdatePageInterface extends BaseUpdatePageInterface
20	20	{
	21	+ /**
	22	+ * @return void
	23	+ */
21	24	public function enable();
22	25
	26	+ /**
	27	+ * @return void
	28	+ */
23	29	public function disable();
24	30
25	31	/**
26	32	* @param string $themeName
	33	+ * @return void
27	34	*/
28	35	public function setTheme($themeName);
29	36
30	37	/**
31	38	* @throws ElementNotFoundException
	39	+ * @return void
32	40	*/
33	41	public function unsetTheme();
34	42
		@@ -39,6 +47,7 @@ discard block
		block discarded – undo
39	47
40	48	/**
41	49	* @param string $language
	50	+ * @return void
42	51	*/
43	52	public function chooseLocale($language);
44	53
		@@ -51,6 +60,7 @@ discard block
		block discarded – undo
51	60
52	61	/**
53	62	* @param string $currencyCode
	63	+ * @return void
54	64	*/
55	65	public function chooseCurrency($currencyCode);
56	66
		@@ -63,11 +73,13 @@ discard block
		block discarded – undo
63	73
64	74	/**
65	75	* @param string $taxZone
	76	+ * @return void
66	77	*/
67	78	public function chooseDefaultTaxZone($taxZone);
68	79
69	80	/**
70	81	* @param string $taxCalculationStrategy
	82	+ * @return void
71	83	*/
72	84	public function chooseTaxCalculationStrategy($taxCalculationStrategy);
73	85

		@@ -17,8 +17,14 @@ discard block
		block discarded – undo
17	17
18	18	interface UpdatePageInterface extends BaseUpdatePageInterface
19	19	{
	20	+ /**
	21	+ * @return void
	22	+ */
20	23	public function enable();
21	24
	25	+ /**
	26	+ * @return void
	27	+ */
22	28	public function disable();
23	29
24	30	/**
		@@ -44,28 +50,36 @@ discard block
		block discarded – undo
44	50	* @param string $name
45	51	* @param string $code
46	52	* @param string\|null $abbreviation
	53	+ * @return void
47	54	*/
48	55	public function addProvince($name, $code, $abbreviation = null);
49	56
50	57	/**
51	58	* @param string $provinceName
	59	+ * @return void
52	60	*/
53	61	public function removeProvince($provinceName);
54	62
	63	+ /**
	64	+ * @return void
	65	+ */
55	66	public function clickAddProvinceButton();
56	67
57	68	/**
58	69	* @param string $provinceName
	70	+ * @return void
59	71	*/
60	72	public function nameProvince($provinceName);
61	73
62	74	/**
63	75	* @param string $provinceName
	76	+ * @return void
64	77	*/
65	78	public function removeProvinceName($provinceName);
66	79
67	80	/**
68	81	* @param string $provinceCode
	82	+ * @return void
69	83	*/
70	84	public function specifyProvinceCode($provinceCode);
71	85	}

		@@ -17,60 +17,76 @@
		block discarded – undo
17	17
18	18	interface CreatePageInterface extends BaseCreatePageInterface
19	19	{
	20	+ /**
	21	+ * @return void
	22	+ */
20	23	public function enable();
21	24
	25	+ /**
	26	+ * @return void
	27	+ */
22	28	public function disable();
23	29
24	30	/**
25	31	* @param string $name
26	32	* @param string $languageCode
	33	+ * @return void
27	34	*/
28	35	public function nameIt($name, $languageCode);
29	36
30	37	/**
31	38	* @param string $code
	39	+ * @return void
32	40	*/
33	41	public function specifyCode($code);
34	42
35	43	/**
36	44	* @param string $channelName
	45	+ * @return void
37	46	*/
38	47	public function checkChannel($channelName);
39	48
40	49	/**
41	50	* @param string $description
42	51	* @param string $languageCode
	52	+ * @return void
43	53	*/
44	54	public function describeIt($description, $languageCode);
45	55
46	56	/**
47	57	* @param string $instructions
48	58	* @param string $languageCode
	59	+ * @return void
49	60	*/
50	61	public function setInstructions($instructions, $languageCode);
51	62
52	63	/**
53	64	* @param string $username
	65	+ * @return void
54	66	*/
55	67	public function setPaypalGatewayUsername($username);
56	68
57	69	/**
58	70	* @param string $password
	71	+ * @return void
59	72	*/
60	73	public function setPaypalGatewayPassword($password);
61	74
62	75	/**
63	76	* @param string $signature
	77	+ * @return void
64	78	*/
65	79	public function setPaypalGatewaySignature($signature);
66	80
67	81	/**
68	82	* @param string $secretKey
	83	+ * @return void
69	84	*/
70	85	public function setStripeSecretKey($secretKey);
71	86
72	87	/**
73	88	* @param string $publishableKey
	89	+ * @return void
74	90	*/
75	91	public function setStripePublishableKey($publishableKey);
76	92

		@@ -17,28 +17,38 @@
		block discarded – undo
17	17
18	18	interface UpdatePageInterface extends BaseUpdatePageInterface
19	19	{
	20	+ /**
	21	+ * @return void
	22	+ */
20	23	public function enable();
21	24
	25	+ /**
	26	+ * @return void
	27	+ */
22	28	public function disable();
23	29
24	30	/**
25	31	* @param string $name
26	32	* @param string $languageCode
	33	+ * @return void
27	34	*/
28	35	public function nameIt($name, $languageCode);
29	36
30	37	/**
31	38	* @param string $username
	39	+ * @return void
32	40	*/
33	41	public function setPaypalGatewayUsername($username);
34	42
35	43	/**
36	44	* @param string $password
	45	+ * @return void
37	46	*/
38	47	public function setPaypalGatewayPassword($password);
39	48
40	49	/**
41	50	* @param string $signature
	51	+ * @return void
42	52	*/
43	53	public function setPaypalGatewaySignature($signature);
44	54

pamil / Sylius

Push — master ( adfc28...f3aa77 )

Status

Category

Doc Comments +9 added lines patch added patch discarded remove patch

Doc Comments +7 added lines patch added patch discarded remove patch

Doc Comments +24 added lines patch added patch discarded remove patch

Doc Comments +12 added lines patch added patch discarded remove patch

Doc Comments +14 added lines patch added patch discarded remove patch

Doc Comments +7 added lines patch added patch discarded remove patch

Doc Comments +13 added lines patch added patch discarded remove patch

Doc Comments +16 added lines patch added patch discarded remove patch

Doc Comments +10 added lines patch added patch discarded remove patch