Inspection of "feat(donor-wall): allow to set donor as anonymous..." - WordImpress/Give - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#3657)

by Ravinder

created 2018-09-05 16:58 UTC

Status

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -11,9 +11,9 @@
 block discarded – undo
  * @since       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
-if ( is_active_sidebar( 'give-forms-sidebar' ) ) {
-	dynamic_sidebar( 'give-forms-sidebar' );
+if (is_active_sidebar('give-forms-sidebar')) {
+	dynamic_sidebar('give-forms-sidebar');
 }
\ No newline at end of file

Please login to merge, or discard this patch.

templates/single-give-form/content-single-give-form.php 1 patch

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -8,7 +8,7 @@  discard block
 block discarded – undo
  * @version       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
 
@@ -19,9 +19,9 @@  discard block
 block discarded – undo
  *
  * @since 1.0
  */
-do_action( 'give_before_single_form' );
+do_action('give_before_single_form');
 
-if ( post_password_required() ) {
+if (post_password_required()) {
 	echo get_the_password_form();
 	return;
 }
@@ -37,10 +37,10 @@  discard block
 block discarded – undo
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_before_single_form_summary' );
+		do_action('give_before_single_form_summary');
 		?>
 
-		<div class="<?php echo apply_filters( 'give_forms_single_summary_classes', 'summary entry-summary' ); ?>">
+		<div class="<?php echo apply_filters('give_forms_single_summary_classes', 'summary entry-summary'); ?>">
 
 			<?php
 			/**
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_single_form_summary' );
+			do_action('give_single_form_summary');
 			?>
 
 		</div>
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_after_single_form_summary' );
+		do_action('give_after_single_form_summary');
 		?>
 
 	</div><!-- #give-form-<?php the_ID(); ?> -->
@@ -77,5 +77,5 @@  discard block
 block discarded – undo
  *
  * @since 1.0
  */
-do_action( 'give_after_single_form' );
+do_action('give_after_single_form');
 ?>

Please login to merge, or discard this patch.

templates/single-give-form/title.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -11,7 +11,7 @@
 block discarded – undo
  * @since       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 } ?>
 <h1 itemprop="name" class="give-form-title entry-title"><?php the_title(); ?></h1>

Please login to merge, or discard this patch.

templates/payment-processing.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -5,11 +5,11 @@
 block discarded – undo
 ?>
 <div id="give-payment-processing">
 	<?php
-	Give()->notices->print_frontend_notice( sprintf(
+	Give()->notices->print_frontend_notice(sprintf(
 	/* translators: %s: success page URL */
-		__( 'Your donation is processing. This page will reload automatically in 8 seconds. If it does not, click <a href="%s">here</a>.', 'give' ),
+		__('Your donation is processing. This page will reload automatically in 8 seconds. If it does not, click <a href="%s">here</a>.', 'give'),
 		give_get_success_page_uri()
-	), true, 'success' );
+	), true, 'success');
 	?>
 	<span class="give-loading-animation"></span>
 	<script type="text/javascript">setTimeout(function () {

Please login to merge, or discard this patch.

templates/emails/header.php 2 patches

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -6,13 +6,13 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 // This is the footer used if no others are available
 ?>
 <!DOCTYPE html>
 <html>
 	<head>
 		<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
-		<title><?php echo get_bloginfo( 'name' ); ?></title>
+		<title><?php echo get_bloginfo('name'); ?></title>
 	</head>
 	<body>
\ No newline at end of file

Please login to merge, or discard this patch.

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,10 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 // This is the footer used if no others are available
 ?>
 <!DOCTYPE html>

Please login to merge, or discard this patch.

templates/emails/footer-default.php 2 patches

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -6,7 +6,7 @@  discard block
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 
 // For gmail compatibility, including CSS styles in head/body are stripped out therefore styles need to be inline. These variables contain rules which are added to the template inline.
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
                                                 <table border="0" cellpadding="10" cellspacing="0" width="100%">
                                                     <tr>
                                                         <td colspan="2" valign="middle" id="credit" style="<?php echo $credit; ?>">
-                                                           <?php echo wpautop( wp_kses_post( wptexturize( apply_filters( 'give_email_footer_text', '<a href="' . esc_url( home_url() ) . '">' . get_bloginfo( 'name' ) . '</a>' ) ) ) ); ?>
+                                                           <?php echo wpautop(wp_kses_post(wptexturize(apply_filters('give_email_footer_text', '<a href="'.esc_url(home_url()).'">'.get_bloginfo('name').'</a>')))); ?>
                                                         </td>
                                                     </tr>
                                                 </table>

Please login to merge, or discard this patch.

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,10 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 
 // For gmail compatibility, including CSS styles in head/body are stripped out therefore styles need to be inline. These variables contain rules which are added to the template inline.

Please login to merge, or discard this patch.

templates/emails/body-default.php 2 patches

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,7 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

Please login to merge, or discard this patch.

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,10 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

Please login to merge, or discard this patch.

templates/emails/footer.php 2 patches

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,7 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 // This is the footer used if no others are available
 

Please login to merge, or discard this patch.

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,10 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 // This is the footer used if no others are available
 

Please login to merge, or discard this patch.

templates/emails/body.php 2 patches

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,7 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

Please login to merge, or discard this patch.

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

@@ -6,7 +6,10 @@
 block discarded – undo
  * @version     1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly.
 
 // {email} is replaced by the content entered in Donations > Settings > Emails
 

Please login to merge, or discard this patch.

		@@ -11,9 +11,9 @@
		block discarded – undo
11	11	* @since 1.0
12	12	*/
13	13
14		-if ( ! defined( 'ABSPATH' ) ) {
	14	+if ( ! defined('ABSPATH')) {
15	15	exit; // Exit if accessed directly.
16	16	}
17		-if ( is_active_sidebar( 'give-forms-sidebar' ) ) {
18		- dynamic_sidebar( 'give-forms-sidebar' );
	17	+if (is_active_sidebar('give-forms-sidebar')) {
	18	+ dynamic_sidebar('give-forms-sidebar');
19	19	}
20	20	\ No newline at end of file

		@@ -8,7 +8,7 @@ discard block
		block discarded – undo
8	8	* @version 1.0
9	9	*/
10	10
11		-if ( ! defined( 'ABSPATH' ) ) {
	11	+if ( ! defined('ABSPATH')) {
12	12	exit; // Exit if accessed directly.
13	13	}
14	14
		@@ -19,9 +19,9 @@ discard block
		block discarded – undo
19	19	*
20	20	* @since 1.0
21	21	*/
22		-do_action( 'give_before_single_form' );
	22	+do_action('give_before_single_form');
23	23
24		-if ( post_password_required() ) {
	24	+if (post_password_required()) {
25	25	echo get_the_password_form();
26	26	return;
27	27	}
		@@ -37,10 +37,10 @@ discard block
		block discarded – undo
37	37	*
38	38	* @since 1.0
39	39	*/
40		- do_action( 'give_before_single_form_summary' );
	40	+ do_action('give_before_single_form_summary');
41	41	?>
42	42
43		- <div class="<?php echo apply_filters( 'give_forms_single_summary_classes', 'summary entry-summary' ); ?>">
	43	+ <div class="<?php echo apply_filters('give_forms_single_summary_classes', 'summary entry-summary'); ?>">
44	44
45	45	<?php
46	46	/**
		@@ -50,7 +50,7 @@ discard block
		block discarded – undo
50	50	*
51	51	* @since 1.0
52	52	*/
53		- do_action( 'give_single_form_summary' );
	53	+ do_action('give_single_form_summary');
54	54	?>
55	55
56	56	</div>
		@@ -64,7 +64,7 @@ discard block
		block discarded – undo
64	64	*
65	65	* @since 1.0
66	66	*/
67		- do_action( 'give_after_single_form_summary' );
	67	+ do_action('give_after_single_form_summary');
68	68	?>
69	69
70	70	</div><!-- #give-form-<?php the_ID(); ?> -->
		@@ -77,5 +77,5 @@ discard block
		block discarded – undo
77	77	*
78	78	* @since 1.0
79	79	*/
80		-do_action( 'give_after_single_form' );
	80	+do_action('give_after_single_form');
81	81	?>

		@@ -11,7 +11,7 @@
		block discarded – undo
11	11	* @since 1.0
12	12	*/
13	13
14		-if ( ! defined( 'ABSPATH' ) ) {
	14	+if ( ! defined('ABSPATH')) {
15	15	exit; // Exit if accessed directly.
16	16	} ?>
17	17	<h1 itemprop="name" class="give-form-title entry-title"><?php the_title(); ?></h1>

		@@ -5,11 +5,11 @@
		block discarded – undo
5	5	?>
6	6	<div id="give-payment-processing">
7	7	<?php
8		- Give()->notices->print_frontend_notice( sprintf(
	8	+ Give()->notices->print_frontend_notice(sprintf(
9	9	/* translators: %s: success page URL */
10		- __( 'Your donation is processing. This page will reload automatically in 8 seconds. If it does not, click <a href="%s">here</a>.', 'give' ),
	10	+ __('Your donation is processing. This page will reload automatically in 8 seconds. If it does not, click <a href="%s">here</a>.', 'give'),
11	11	give_get_success_page_uri()
12		- ), true, 'success' );
	12	+ ), true, 'success');
13	13	?>
14	14	<span class="give-loading-animation"></span>
15	15	<script type="text/javascript">setTimeout(function () {

		@@ -6,13 +6,13 @@
		block discarded – undo
6	6	* @version 1.0
7	7	*/
8	8
9		-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
	9	+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
10	10	// This is the footer used if no others are available
11	11	?>
12	12	<!DOCTYPE html>
13	13	<html>
14	14	<head>
15	15	<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
16		- <title><?php echo get_bloginfo( 'name' ); ?></title>
	16	+ <title><?php echo get_bloginfo('name'); ?></title>
17	17	</head>
18	18	<body>
19	19	\ No newline at end of file

		@@ -6,7 +6,10 @@
		block discarded – undo
6	6	* @version 1.0
7	7	*/
8	8
9		-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
	9	+if ( ! defined( 'ABSPATH' ) ) {
	10	+ exit;
	11	+}
	12	+// Exit if accessed directly.
10	13	// This is the footer used if no others are available
11	14	?>
12	15	<!DOCTYPE html>

		@@ -6,7 +6,7 @@ discard block
		block discarded – undo
6	6	* @version 1.0
7	7	*/
8	8
9		-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
	9	+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
10	10
11	11
12	12	// For gmail compatibility, including CSS styles in head/body are stripped out therefore styles need to be inline. These variables contain rules which are added to the template inline.
		@@ -44,7 +44,7 @@ discard block
		block discarded – undo
44	44	<table border="0" cellpadding="10" cellspacing="0" width="100%">
45	45	<tr>
46	46	<td colspan="2" valign="middle" id="credit" style="<?php echo $credit; ?>">
47		- <?php echo wpautop( wp_kses_post( wptexturize( apply_filters( 'give_email_footer_text', '<a href="' . esc_url( home_url() ) . '">' . get_bloginfo( 'name' ) . '</a>' ) ) ) ); ?>
	47	+ <?php echo wpautop(wp_kses_post(wptexturize(apply_filters('give_email_footer_text', '<a href="'.esc_url(home_url()).'">'.get_bloginfo('name').'</a>')))); ?>
48	48	</td>
49	49	</tr>
50	50	</table>

		@@ -6,7 +6,7 @@
		block discarded – undo
6	6	* @version 1.0
7	7	*/
8	8
9		-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.
	9	+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.
10	10
11	11	// {email} is replaced by the content entered in Donations > Settings > Emails
12	12

WordImpress / Give

Pull Request — master (#3657)

Status

Category

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +8 added lines, -8 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Braces +4 added lines, -1 removed lines patch added patch discarded remove patch