Inspection of "Calypso Nudges: Add admin notices nudging the user..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

jetpack

Completed

Pull Request — master (#8227)

by Kirk

created 2017-11-22 00:55 UTC

Status

All Patches
114
New
1

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

@@ -121,6 +121,9 @@  discard block
 block discarded – undo
 		return $nudge_url;
 	}
 
+	/**
+	 * @param string $desc
+	 */
 	protected function display_editor_nudge( $nudge_url, $desc ) {
 		switch ( $desc ) {
 			case 'post':
@@ -166,6 +169,9 @@  discard block
 block discarded – undo
 		<?php
 	}
 
+	/**
+	 * @param string $desc
+	 */
 	protected function display_management_nudge( $nudge_url, $desc ) {
 		switch ( $desc ) {
 			case 'post':

Please login to merge, or discard this patch.

		@@ -121,6 +121,9 @@ discard block
		block discarded – undo
121	121	return $nudge_url;
122	122	}
123	123
	124	+ /**
	125	+ * @param string $desc
	126	+ */
124	127	protected function display_editor_nudge( $nudge_url, $desc ) {
125	128	switch ( $desc ) {
126	129	case 'post':
		@@ -166,6 +169,9 @@ discard block
		block discarded – undo
166	169	<?php
167	170	}
168	171
	172	+ /**
	173	+ * @param string $desc
	174	+ */
169	175	protected function display_management_nudge( $nudge_url, $desc ) {
170	176	switch ( $desc ) {
171	177	case 'post':

Automattic / jetpack

Pull Request — master (#8227)

Status

Category

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