insideout10 / wordlift-plugin

src/admin/partials/wordlift-admin-deactivation-feedback-popup.php

Indentation +68 added lines, -68 removed lines

@@ -1,62 +1,62 @@ 
 <?php
 $too_complicate_reason_label = sprintf(
-	'<a target="_blank" href="%s">%s</a>',
-	/* translators: the link https://wordlift.io/contact-us/ should be changed to language version of the page */
-	esc_attr__( 'https://wordlift.io/contact-us/', 'wordlift' ),
-	esc_html__( 'Contact Us', 'wordlift' )
+    '<a target="_blank" href="%s">%s</a>',
+    /* translators: the link https://wordlift.io/contact-us/ should be changed to language version of the page */
+    esc_attr__( 'https://wordlift.io/contact-us/', 'wordlift' ),
+    esc_html__( 'Contact Us', 'wordlift' )
 );
 
 $reasons = array(
-	array(
-		'id'      => 'TOO_COMPLICATED',
-		'text'    => esc_html__( 'It was too complicated and unclear to me', 'wordlift' ),
-		'message' => array(
-			'text' => sprintf(
-				esc_html__( 'Need help? We are ready to answer your questions. %s', 'wordlift' ),
-				$too_complicate_reason_label
-			),
-		),
-	),
-	array(
-		'id'      => 'NOT_ENOUGH_FEATURES',
-		'text'    => esc_html__( 'It misses some important feature to me', 'wordlift' ),
-		'message' => array(
-			'field' => 'text',
-			'text'  => esc_html__( 'Tell us what this feature is.', 'wordlift' ),
-		),
-	),
-	array(
-		'id'      => 'COSTS_TOO_MUCH',
-		'text'    => esc_html__( 'It costs too much', 'wordlift' ),
-		'message' => array(
-			'field' => 'text',
-			'text'  => esc_html__( 'How much you would like to pay?', 'wordlift' ),
-		),
-	),
-	array(
-		'id'      => 'FOUND_ANOTHER_TOOL',
-		'text'    => esc_html__( 'I found another tool that I like better', 'wordlift' ),
-		'message' => array(
-			'field' => 'text',
-			'text'  => esc_html__( 'Please tell us some more details.', 'wordlift' ),
-		),
-	),
-	array(
-		'id'   => 'I_DONT_USE_IT',
-		'text' => esc_html__( 'I\'m not using it right now', 'wordlift' ),
-	),
-	array(
-		'id'   => 'SOMETHING_DIDNT_WORK',
-		'text' => esc_html__( 'Something didn\'t work right', 'wordlift' ),
-	),
-	array(
-		'id'      => 'ANOTHER_REASON',
-		'text'    => esc_html__( 'Another reason', 'wordlift' ),
-		'message' => array(
-			'field' => 'textarea',
-			'text'  => esc_html__( 'Please tell us the reason so we can improve it.', 'wordlift' ),
-		),
-	),
+    array(
+        'id'      => 'TOO_COMPLICATED',
+        'text'    => esc_html__( 'It was too complicated and unclear to me', 'wordlift' ),
+        'message' => array(
+            'text' => sprintf(
+                esc_html__( 'Need help? We are ready to answer your questions. %s', 'wordlift' ),
+                $too_complicate_reason_label
+            ),
+        ),
+    ),
+    array(
+        'id'      => 'NOT_ENOUGH_FEATURES',
+        'text'    => esc_html__( 'It misses some important feature to me', 'wordlift' ),
+        'message' => array(
+            'field' => 'text',
+            'text'  => esc_html__( 'Tell us what this feature is.', 'wordlift' ),
+        ),
+    ),
+    array(
+        'id'      => 'COSTS_TOO_MUCH',
+        'text'    => esc_html__( 'It costs too much', 'wordlift' ),
+        'message' => array(
+            'field' => 'text',
+            'text'  => esc_html__( 'How much you would like to pay?', 'wordlift' ),
+        ),
+    ),
+    array(
+        'id'      => 'FOUND_ANOTHER_TOOL',
+        'text'    => esc_html__( 'I found another tool that I like better', 'wordlift' ),
+        'message' => array(
+            'field' => 'text',
+            'text'  => esc_html__( 'Please tell us some more details.', 'wordlift' ),
+        ),
+    ),
+    array(
+        'id'   => 'I_DONT_USE_IT',
+        'text' => esc_html__( 'I\'m not using it right now', 'wordlift' ),
+    ),
+    array(
+        'id'   => 'SOMETHING_DIDNT_WORK',
+        'text' => esc_html__( 'Something didn\'t work right', 'wordlift' ),
+    ),
+    array(
+        'id'      => 'ANOTHER_REASON',
+        'text'    => esc_html__( 'Another reason', 'wordlift' ),
+        'message' => array(
+            'field' => 'textarea',
+            'text'  => esc_html__( 'Please tell us the reason so we can improve it.', 'wordlift' ),
+        ),
+    ),
 );
 ?>
 <div class="wl-modal-deactivation-feedback" style="display: none">
@@ -92,15 +92,15 @@ 
 							<?php if ( ! empty( $reason['message'] ) ) : ?>
                                 <div class="additional-info <?php echo ( ! empty( $reason['message']['field'] ) ) ? 'has-field' : ''; ?>">
 									<?php
-									if ( ! empty( $reason['message']['field'] ) ) {
-										if ( $reason['message']['field'] === 'text' ) {
-											echo '<input type="text" name="wl-details" class="wl-details"/>';
-										} else {
-											echo '<textarea name="wl-details" class="wl-details"></textarea>';
-										}
-									}
-									echo wp_kses( wpautop( $reason['message']['text'] ), array( 'p' => array() ) )
-									?>
+                                    if ( ! empty( $reason['message']['field'] ) ) {
+                                        if ( $reason['message']['field'] === 'text' ) {
+                                            echo '<input type="text" name="wl-details" class="wl-details"/>';
+                                        } else {
+                                            echo '<textarea name="wl-details" class="wl-details"></textarea>';
+                                        }
+                                    }
+                                    echo wp_kses( wpautop( $reason['message']['text'] ), array( 'p' => array() ) )
+                                    ?>
                                 </div>
 							<?php endif ?>
                         </li>
@@ -111,11 +111,11 @@ 
             <div class="notes">
                 <p>
 					<?php
-					echo wp_kses( sprintf(
-						__( 'Important notice: Uninstalling the plugin will delete your vocabulary.<br>Maybe you would like to <a href="%s" target="_blank">download your data</a> first.', 'wordlift' ),
-						add_query_arg( array( 'page' => 'wl_download_your_data' ), admin_url( 'admin.php' ) )
-					), array( 'br' => array(), 'a' => array( 'href' => array(), 'target' => array() ) ) );
-					?>
+                    echo wp_kses( sprintf(
+                        __( 'Important notice: Uninstalling the plugin will delete your vocabulary.<br>Maybe you would like to <a href="%s" target="_blank">download your data</a> first.', 'wordlift' ),
+                        add_query_arg( array( 'page' => 'wl_download_your_data' ), admin_url( 'admin.php' ) )
+                    ), array( 'br' => array(), 'a' => array( 'href' => array(), 'target' => array() ) ) );
+                    ?>
                 </p>
             </div>
 

Spacing +33 added lines, -33 removed lines

@@ -2,59 +2,59 @@ 
 $too_complicate_reason_label = sprintf(
 	'<a target="_blank" href="%s">%s</a>',
 	/* translators: the link https://wordlift.io/contact-us/ should be changed to language version of the page */
-	esc_attr__( 'https://wordlift.io/contact-us/', 'wordlift' ),
-	esc_html__( 'Contact Us', 'wordlift' )
+	esc_attr__('https://wordlift.io/contact-us/', 'wordlift'),
+	esc_html__('Contact Us', 'wordlift')
 );
 
 $reasons = array(
 	array(
 		'id'      => 'TOO_COMPLICATED',
-		'text'    => esc_html__( 'It was too complicated and unclear to me', 'wordlift' ),
+		'text'    => esc_html__('It was too complicated and unclear to me', 'wordlift'),
 		'message' => array(
 			'text' => sprintf(
-				esc_html__( 'Need help? We are ready to answer your questions. %s', 'wordlift' ),
+				esc_html__('Need help? We are ready to answer your questions. %s', 'wordlift'),
 				$too_complicate_reason_label
 			),
 		),
 	),
 	array(
 		'id'      => 'NOT_ENOUGH_FEATURES',
-		'text'    => esc_html__( 'It misses some important feature to me', 'wordlift' ),
+		'text'    => esc_html__('It misses some important feature to me', 'wordlift'),
 		'message' => array(
 			'field' => 'text',
-			'text'  => esc_html__( 'Tell us what this feature is.', 'wordlift' ),
+			'text'  => esc_html__('Tell us what this feature is.', 'wordlift'),
 		),
 	),
 	array(
 		'id'      => 'COSTS_TOO_MUCH',
-		'text'    => esc_html__( 'It costs too much', 'wordlift' ),
+		'text'    => esc_html__('It costs too much', 'wordlift'),
 		'message' => array(
 			'field' => 'text',
-			'text'  => esc_html__( 'How much you would like to pay?', 'wordlift' ),
+			'text'  => esc_html__('How much you would like to pay?', 'wordlift'),
 		),
 	),
 	array(
 		'id'      => 'FOUND_ANOTHER_TOOL',
-		'text'    => esc_html__( 'I found another tool that I like better', 'wordlift' ),
+		'text'    => esc_html__('I found another tool that I like better', 'wordlift'),
 		'message' => array(
 			'field' => 'text',
-			'text'  => esc_html__( 'Please tell us some more details.', 'wordlift' ),
+			'text'  => esc_html__('Please tell us some more details.', 'wordlift'),
 		),
 	),
 	array(
 		'id'   => 'I_DONT_USE_IT',
-		'text' => esc_html__( 'I\'m not using it right now', 'wordlift' ),
+		'text' => esc_html__('I\'m not using it right now', 'wordlift'),
 	),
 	array(
 		'id'   => 'SOMETHING_DIDNT_WORK',
-		'text' => esc_html__( 'Something didn\'t work right', 'wordlift' ),
+		'text' => esc_html__('Something didn\'t work right', 'wordlift'),
 	),
 	array(
 		'id'      => 'ANOTHER_REASON',
-		'text'    => esc_html__( 'Another reason', 'wordlift' ),
+		'text'    => esc_html__('Another reason', 'wordlift'),
 		'message' => array(
 			'field' => 'textarea',
-			'text'  => esc_html__( 'Please tell us the reason so we can improve it.', 'wordlift' ),
+			'text'  => esc_html__('Please tell us the reason so we can improve it.', 'wordlift'),
 		),
 	),
 );
@@ -63,43 +63,43 @@ 
     <div class="wl-modal">
         <div class="wl-modal-body">
             <h2>
-				<?php esc_html_e( 'We\'re sorry to see you go!', 'wordlift' ); ?>
+				<?php esc_html_e('We\'re sorry to see you go!', 'wordlift'); ?>
             </h2>
 
             <div class="wl-modal-panel active">
                 <h4>
-					<?php esc_html_e( 'If you have a moment, please let us know why you are deactivating', 'wordlift' ); ?>
+					<?php esc_html_e('If you have a moment, please let us know why you are deactivating', 'wordlift'); ?>
                     :
                 </h4>
 
                 <ul>
-					<?php foreach ( $reasons as $reason ) : ?>
-                        <li class="wl-reason-item <?php echo ( $reason['id'] == 'I_DONT_USE_IT' ) ? 'selected' : ''; ?>">
+					<?php foreach ($reasons as $reason) : ?>
+                        <li class="wl-reason-item <?php echo ($reason['id'] == 'I_DONT_USE_IT') ? 'selected' : ''; ?>">
                             <label>
                                 <input
                                         type="radio"
                                         name="wl-code"
                                         class="wl-code"
-									<?php checked( 'I_DONT_USE_IT', $reason['id'], true ); ?>
-                                        value="<?php echo esc_attr( $reason['id'] ); ?>"
+									<?php checked('I_DONT_USE_IT', $reason['id'], true); ?>
+                                        value="<?php echo esc_attr($reason['id']); ?>"
                                 />
 
                                 <span class="description">
-									<?php echo esc_html( $reason['text'] ); ?>
+									<?php echo esc_html($reason['text']); ?>
 								</span>
                             </label>
 
-							<?php if ( ! empty( $reason['message'] ) ) : ?>
-                                <div class="additional-info <?php echo ( ! empty( $reason['message']['field'] ) ) ? 'has-field' : ''; ?>">
+							<?php if ( ! empty($reason['message'])) : ?>
+                                <div class="additional-info <?php echo ( ! empty($reason['message']['field'])) ? 'has-field' : ''; ?>">
 									<?php
-									if ( ! empty( $reason['message']['field'] ) ) {
-										if ( $reason['message']['field'] === 'text' ) {
+									if ( ! empty($reason['message']['field'])) {
+										if ($reason['message']['field'] === 'text') {
 											echo '<input type="text" name="wl-details" class="wl-details"/>';
 										} else {
 											echo '<textarea name="wl-details" class="wl-details"></textarea>';
 										}
 									}
-									echo wp_kses( wpautop( $reason['message']['text'] ), array( 'p' => array() ) )
+									echo wp_kses(wpautop($reason['message']['text']), array('p' => array()))
 									?>
                                 </div>
 							<?php endif ?>
@@ -111,10 +111,10 @@ 
             <div class="notes">
                 <p>
 					<?php
-					echo wp_kses( sprintf(
-						__( 'Important notice: Uninstalling the plugin will delete your vocabulary.<br>Maybe you would like to <a href="%s" target="_blank">download your data</a> first.', 'wordlift' ),
-						add_query_arg( array( 'page' => 'wl_download_your_data' ), admin_url( 'admin.php' ) )
-					), array( 'br' => array(), 'a' => array( 'href' => array(), 'target' => array() ) ) );
+					echo wp_kses(sprintf(
+						__('Important notice: Uninstalling the plugin will delete your vocabulary.<br>Maybe you would like to <a href="%s" target="_blank">download your data</a> first.', 'wordlift'),
+						add_query_arg(array('page' => 'wl_download_your_data'), admin_url('admin.php'))
+					), array('br' => array(), 'a' => array('href' => array(), 'target' => array())));
 					?>
                 </p>
             </div>
@@ -124,11 +124,11 @@ 
 
         <div class="wl-modal-footer">
             <a href="#" class="button button-secondary wl-modal-button-close">
-				<?php esc_html_e( 'Cancel', 'wordlift' ); ?>
+				<?php esc_html_e('Cancel', 'wordlift'); ?>
             </a>
 
             <a href="#" class="button button-primary wl-modal-button-deactivate">
-				<?php esc_html_e( 'Deactivate', 'wordlift' ); ?>
+				<?php esc_html_e('Deactivate', 'wordlift'); ?>
             </a>
             <div class="clear"></div>
         </div>
@@ -137,7 +137,7 @@ 
                 type="hidden"
                 name="wl_deactivation_feedback_nonce"
                 class="wl_deactivation_feedback_nonce"
-                value="<?php echo esc_attr( wp_create_nonce( 'wl_deactivation_feedback_nonce' ) ); ?>"
+                value="<?php echo esc_attr(wp_create_nonce('wl_deactivation_feedback_nonce')); ?>"
         >
     </div>
 </div>

src/admin/class-wordlift-admin-user-profile-page.php

Indentation +131 added lines, -131 removed lines

@@ -18,77 +18,77 @@ 
  */
 class Wordlift_Admin_User_Profile_Page {
 
-	/**
-	 * The {@link Wordlift_Admin_Person_Element} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Admin_Author_Element $plugin The person entity
-	 *                selection element rendering the possible persons.
-	 */
-	private $author_element;
-
-	/**
-	 * The {@link Wordlift_User_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
-	 */
-	private $user_service;
-
-	/**
-	 * Create the {@link Wordlift_Admin_User_Profile_Page} instance.
-	 *
-	 * @param \Wordlift_Admin_Author_Element $author_element The person entity selection
-	 *                                                       element rendering the possible persons.
-	 * @param \Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
-	 *
-	 * @since 3.14.0
-	 *
-	 */
-	function __construct( $author_element, $user_service ) {
-
-		$this->author_element = $author_element;
-		$this->user_service   = $user_service;
-
-		/*
+    /**
+     * The {@link Wordlift_Admin_Person_Element} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Admin_Author_Element $plugin The person entity
+     *                selection element rendering the possible persons.
+     */
+    private $author_element;
+
+    /**
+     * The {@link Wordlift_User_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
+     */
+    private $user_service;
+
+    /**
+     * Create the {@link Wordlift_Admin_User_Profile_Page} instance.
+     *
+     * @param \Wordlift_Admin_Author_Element $author_element The person entity selection
+     *                                                       element rendering the possible persons.
+     * @param \Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
+     *
+     * @since 3.14.0
+     *
+     */
+    function __construct( $author_element, $user_service ) {
+
+        $this->author_element = $author_element;
+        $this->user_service   = $user_service;
+
+        /*
 		 * When an admin (or similar permissions) edits his own profile a
 		 * different action than the usual is being triggered.
 		 * It is too early in the wordpress boot to do user capabilities filtering
 		 * here and it is deferred to the handler.
 		 */
-		add_action( 'show_user_profile', array( $this, 'edit_user_profile' ) );
-		add_action( 'edit_user_profile', array( $this, 'edit_user_profile' ) );
-		add_action( 'edit_user_profile_update', array(
-			$this,
-			'edit_user_profile_update',
-		) );
-		add_action( 'personal_options_update', array(
-			$this,
-			'edit_user_profile_update',
-		) );
-
-	}
-
-	/**
-	 * Add a WordLift section in the user profile which lets
-	 * the admin to associate a wordpress user with a person entity.
-	 *
-	 * @param WP_User $user The current WP_User object of the user being edited.
-	 *
-	 * @since 3.14.0
-	 *
-	 */
-	public function edit_user_profile( $user ) {
-
-		// In case it is a user editing his own profile, make sure he has admin
-		// like capabilities.
-		if ( ! current_user_can( 'edit_users' ) ) {
-			return;
-		}
-
-		?>
+        add_action( 'show_user_profile', array( $this, 'edit_user_profile' ) );
+        add_action( 'edit_user_profile', array( $this, 'edit_user_profile' ) );
+        add_action( 'edit_user_profile_update', array(
+            $this,
+            'edit_user_profile_update',
+        ) );
+        add_action( 'personal_options_update', array(
+            $this,
+            'edit_user_profile_update',
+        ) );
+
+    }
+
+    /**
+     * Add a WordLift section in the user profile which lets
+     * the admin to associate a wordpress user with a person entity.
+     *
+     * @param WP_User $user The current WP_User object of the user being edited.
+     *
+     * @since 3.14.0
+     *
+     */
+    public function edit_user_profile( $user ) {
+
+        // In case it is a user editing his own profile, make sure he has admin
+        // like capabilities.
+        if ( ! current_user_can( 'edit_users' ) ) {
+            return;
+        }
+
+        ?>
         <h2><?php esc_html_e( 'WordLift', 'wordlift' ); ?></h2>
 
         <table class="form-table">
@@ -99,12 +99,12 @@ 
                 </th>
                 <td>
 					<?php
-					$this->author_element->render( array(
-						'id'             => 'wl_person',
-						'name'           => 'wl_person',
-						'current_entity' => $this->user_service->get_entity( $user->ID ),
-					) );
-					?>
+                    $this->author_element->render( array(
+                        'id'             => 'wl_person',
+                        'name'           => 'wl_person',
+                        'current_entity' => $this->user_service->get_entity( $user->ID ),
+                    ) );
+                    ?>
                     <p class="description"><?php esc_html_e( 'The entity, person or organization, from the vocabulary to associate with this author.', 'wordlift' ); ?></p>
                 </td>
             </tr>
@@ -122,66 +122,66 @@ 
                 </td>
 				<?php } ?>
 				<?php
-				/**
-				 * Action name: wordlift_user_settings_page
-				 * An action to render the wordlift user settings.
-				 * @since 3.30.0
-				 *
-				 */
-				do_action( 'wordlift_user_settings_page' );
-				?>
+                /**
+                 * Action name: wordlift_user_settings_page
+                 * An action to render the wordlift user settings.
+                 * @since 3.30.0
+                 *
+                 */
+                do_action( 'wordlift_user_settings_page' );
+                ?>
         </table>
 		<?php
-	}
-
-	/**
-	 * Handle storing the person entity associated with the user.
-	 *
-	 * @param int $user_id The user id of the user being saved.
-	 *
-	 * @since 3.14.0
-	 *
-	 */
-	public function edit_user_profile_update( $user_id ) {
-
-		// In case it is a user editing his own profile, make sure he has admin
-		// like capabilities.
-		if ( ! current_user_can( 'edit_users' ) ) {
-			return;
-		}
-
-		// Link an entity to the user.
-		$this->link_entity( $user_id, $_POST );
-
-		// Deny and enable the edit entity capability
-		if ( isset( $_POST['wl_can_create_entities'] ) ) {
-			// User has capability so remove the deny indication if present.
-			$this->user_service->allow_editor_entity_create( $user_id );
-		} else {
-			$this->user_service->deny_editor_entity_create( $user_id );
-		}
-
-	}
-
-	/**
-	 * Link an entity (specified in the `$_POST` array) to the {@link WP_User}
-	 * with the specified `id`.
-	 *
-	 * @param int $user_id The {@link WP_User} `id`.
-	 * @param array $post The `$_POST` array.
-	 *
-	 * @since 3.14.0
-	 *
-	 */
-	private function link_entity( $user_id, $post ) {
-
-		// Bail out if the `wl_person` parameter isn't set.
-		if ( ! isset( $post['wl_person'] ) || ! is_numeric( $post['wl_person'] ) ) {
-			return;
-		}
-
-		$this->user_service->set_entity( $user_id, intval( $post['wl_person'] ) );
-
-	}
+    }
+
+    /**
+     * Handle storing the person entity associated with the user.
+     *
+     * @param int $user_id The user id of the user being saved.
+     *
+     * @since 3.14.0
+     *
+     */
+    public function edit_user_profile_update( $user_id ) {
+
+        // In case it is a user editing his own profile, make sure he has admin
+        // like capabilities.
+        if ( ! current_user_can( 'edit_users' ) ) {
+            return;
+        }
+
+        // Link an entity to the user.
+        $this->link_entity( $user_id, $_POST );
+
+        // Deny and enable the edit entity capability
+        if ( isset( $_POST['wl_can_create_entities'] ) ) {
+            // User has capability so remove the deny indication if present.
+            $this->user_service->allow_editor_entity_create( $user_id );
+        } else {
+            $this->user_service->deny_editor_entity_create( $user_id );
+        }
+
+    }
+
+    /**
+     * Link an entity (specified in the `$_POST` array) to the {@link WP_User}
+     * with the specified `id`.
+     *
+     * @param int $user_id The {@link WP_User} `id`.
+     * @param array $post The `$_POST` array.
+     *
+     * @since 3.14.0
+     *
+     */
+    private function link_entity( $user_id, $post ) {
+
+        // Bail out if the `wl_person` parameter isn't set.
+        if ( ! isset( $post['wl_person'] ) || ! is_numeric( $post['wl_person'] ) ) {
+            return;
+        }
+
+        $this->user_service->set_entity( $user_id, intval( $post['wl_person'] ) );
+
+    }
 
 }

Spacing +29 added lines, -29 removed lines

@@ -47,7 +47,7 @@ 
 	 * @since 3.14.0
 	 *
 	 */
-	function __construct( $author_element, $user_service ) {
+	function __construct($author_element, $user_service) {
 
 		$this->author_element = $author_element;
 		$this->user_service   = $user_service;
@@ -58,16 +58,16 @@ 
 		 * It is too early in the wordpress boot to do user capabilities filtering
 		 * here and it is deferred to the handler.
 		 */
-		add_action( 'show_user_profile', array( $this, 'edit_user_profile' ) );
-		add_action( 'edit_user_profile', array( $this, 'edit_user_profile' ) );
-		add_action( 'edit_user_profile_update', array(
+		add_action('show_user_profile', array($this, 'edit_user_profile'));
+		add_action('edit_user_profile', array($this, 'edit_user_profile'));
+		add_action('edit_user_profile_update', array(
 			$this,
 			'edit_user_profile_update',
-		) );
-		add_action( 'personal_options_update', array(
+		));
+		add_action('personal_options_update', array(
 			$this,
 			'edit_user_profile_update',
-		) );
+		));
 
 	}
 
@@ -80,45 +80,45 @@ 
 	 * @since 3.14.0
 	 *
 	 */
-	public function edit_user_profile( $user ) {
+	public function edit_user_profile($user) {
 
 		// In case it is a user editing his own profile, make sure he has admin
 		// like capabilities.
-		if ( ! current_user_can( 'edit_users' ) ) {
+		if ( ! current_user_can('edit_users')) {
 			return;
 		}
 
 		?>
-        <h2><?php esc_html_e( 'WordLift', 'wordlift' ); ?></h2>
+        <h2><?php esc_html_e('WordLift', 'wordlift'); ?></h2>
 
         <table class="form-table">
-		    <?php if ( apply_filters( 'wl_feature__enable__user-author', true ) ) { ?>
+		    <?php if (apply_filters('wl_feature__enable__user-author', true)) { ?>
             <tr class="user-description-wrap">
                 <th><label
-                            for="wl_person"><?php esc_html_e( 'Author from the vocabulary', 'wordlift' ); ?></label>
+                            for="wl_person"><?php esc_html_e('Author from the vocabulary', 'wordlift'); ?></label>
                 </th>
                 <td>
 					<?php
-					$this->author_element->render( array(
+					$this->author_element->render(array(
 						'id'             => 'wl_person',
 						'name'           => 'wl_person',
-						'current_entity' => $this->user_service->get_entity( $user->ID ),
-					) );
+						'current_entity' => $this->user_service->get_entity($user->ID),
+					));
 					?>
-                    <p class="description"><?php esc_html_e( 'The entity, person or organization, from the vocabulary to associate with this author.', 'wordlift' ); ?></p>
+                    <p class="description"><?php esc_html_e('The entity, person or organization, from the vocabulary to associate with this author.', 'wordlift'); ?></p>
                 </td>
             </tr>
 		    <?php } ?>
-			<?php if ( $this->user_service->is_editor( $user->ID ) ) { ?>
+			<?php if ($this->user_service->is_editor($user->ID)) { ?>
             <tr>
                 <th>
                     <label
-                            for="wl_can_create_entities"><?php esc_html_e( 'Can create new entities', 'wordlift' ) ?></label>
+                            for="wl_can_create_entities"><?php esc_html_e('Can create new entities', 'wordlift') ?></label>
                 </th>
                 <td>
                     <input id="wl_can_create_entities"
                            name="wl_can_create_entities"
-                           type="checkbox" <?php checked( $this->user_service->editor_can_create_entities( $user->ID ) ) ?>
+                           type="checkbox" <?php checked($this->user_service->editor_can_create_entities($user->ID)) ?>
                 </td>
 				<?php } ?>
 				<?php
@@ -128,7 +128,7 @@ 
 				 * @since 3.30.0
 				 *
 				 */
-				do_action( 'wordlift_user_settings_page' );
+				do_action('wordlift_user_settings_page');
 				?>
         </table>
 		<?php
@@ -142,23 +142,23 @@ 
 	 * @since 3.14.0
 	 *
 	 */
-	public function edit_user_profile_update( $user_id ) {
+	public function edit_user_profile_update($user_id) {
 
 		// In case it is a user editing his own profile, make sure he has admin
 		// like capabilities.
-		if ( ! current_user_can( 'edit_users' ) ) {
+		if ( ! current_user_can('edit_users')) {
 			return;
 		}
 
 		// Link an entity to the user.
-		$this->link_entity( $user_id, $_POST );
+		$this->link_entity($user_id, $_POST);
 
 		// Deny and enable the edit entity capability
-		if ( isset( $_POST['wl_can_create_entities'] ) ) {
+		if (isset($_POST['wl_can_create_entities'])) {
 			// User has capability so remove the deny indication if present.
-			$this->user_service->allow_editor_entity_create( $user_id );
+			$this->user_service->allow_editor_entity_create($user_id);
 		} else {
-			$this->user_service->deny_editor_entity_create( $user_id );
+			$this->user_service->deny_editor_entity_create($user_id);
 		}
 
 	}
@@ -173,14 +173,14 @@ 
 	 * @since 3.14.0
 	 *
 	 */
-	private function link_entity( $user_id, $post ) {
+	private function link_entity($user_id, $post) {
 
 		// Bail out if the `wl_person` parameter isn't set.
-		if ( ! isset( $post['wl_person'] ) || ! is_numeric( $post['wl_person'] ) ) {
+		if ( ! isset($post['wl_person']) || ! is_numeric($post['wl_person'])) {
 			return;
 		}
 
-		$this->user_service->set_entity( $user_id, intval( $post['wl_person'] ) );
+		$this->user_service->set_entity($user_id, intval($post['wl_person']));
 
 	}
 

src/admin/class-wordlift-entity-type-settings.php

Indentation +204 added lines, -204 removed lines

@@ -13,7 +13,7 @@ 
  */
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -30,219 +30,219 @@ 
  */
 class Wordlift_Admin_Entity_Type_Settings {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * Create a {@link Wordlift_Admin_Entity_Type_Settings} instance.
-	 *
-	 * @since 3.14.0
-	 */
-	public function __construct() {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Admin_Entity_Type_Settings' );
-
-	}
-
-	/**
-	 * Handle menu registration.
-	 *
-	 * The registration is required, although we do not want to actually to add
-	 * an item to the menu, in order to "whitelist" the access to the settings page in
-	 * the admin.
-	 *
-	 * @since 3.11.0
-	 */
-	public function admin_menu() {
-
-		/*
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * Create a {@link Wordlift_Admin_Entity_Type_Settings} instance.
+     *
+     * @since 3.14.0
+     */
+    public function __construct() {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Admin_Entity_Type_Settings' );
+
+    }
+
+    /**
+     * Handle menu registration.
+     *
+     * The registration is required, although we do not want to actually to add
+     * an item to the menu, in order to "whitelist" the access to the settings page in
+     * the admin.
+     *
+     * @since 3.11.0
+     */
+    public function admin_menu() {
+
+        /*
 		 * Before anything else check if an settings form was submitted.
 		 * This has to be done before any output happens in order to be able to
 		 * display proper "die" error messages and redirect.
 		 */
-		if ( isset( $_GET['page'] ) && ( 'wl_entity_type_settings' === $_GET['page'] ) ) {
-
-			// Validate inputs. Do not return on invalid parameters or capabilities.
-			$this->validate_proper_term();
-
-			// If proper form submission, handle it and redirect back to the settings page.
-			if ( isset( $_POST['action'] ) && ( 'wl_edit_entity_type_term' === $_POST['action'] ) ) {
-				$this->handle_form_submission();
-			}
-			/**
-			 * Filter: wl_feature__enable__notices.
-			 *
-			 * @param bool whether the notices needs to be enabled or not.
-			 *
-			 * @return bool
-			 * @since 3.27.6
-			 */
-			if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
-				// Register admin notices handler.
-				add_action( 'admin_notices', array( $this, 'admin_notice' ) );
-			}
-
-		}
-
-		/*
+        if ( isset( $_GET['page'] ) && ( 'wl_entity_type_settings' === $_GET['page'] ) ) {
+
+            // Validate inputs. Do not return on invalid parameters or capabilities.
+            $this->validate_proper_term();
+
+            // If proper form submission, handle it and redirect back to the settings page.
+            if ( isset( $_POST['action'] ) && ( 'wl_edit_entity_type_term' === $_POST['action'] ) ) {
+                $this->handle_form_submission();
+            }
+            /**
+             * Filter: wl_feature__enable__notices.
+             *
+             * @param bool whether the notices needs to be enabled or not.
+             *
+             * @return bool
+             * @since 3.27.6
+             */
+            if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+                // Register admin notices handler.
+                add_action( 'admin_notices', array( $this, 'admin_notice' ) );
+            }
+
+        }
+
+        /*
 		 * Use a null parent slug to prevent the menu from actually appearing
 		 * in the admin menu.
 		 */
-		// @todo: use the new {@link Wordlift_Admin_Page}.
-		add_submenu_page(
-			null,
-			__( 'Edit Entity term', 'wordlift' ),
-			__( 'Edit Entity term', 'wordlift' ),
-			'manage_options',
-			'wl_entity_type_settings',
-			array( $this, 'render' )
-		);
-	}
-
-	/**
-	 * Output admin notices if needed, based on the message url parameter.
-	 * A value of 1 indicates that a successful save was done.
-	 *
-	 * @since 3.11.0
-	 */
-	function admin_notice() {
-		if ( isset( $_GET['message'] ) && ( '1' === $_GET['message'] ) ) {
-			?>
+        // @todo: use the new {@link Wordlift_Admin_Page}.
+        add_submenu_page(
+            null,
+            __( 'Edit Entity term', 'wordlift' ),
+            __( 'Edit Entity term', 'wordlift' ),
+            'manage_options',
+            'wl_entity_type_settings',
+            array( $this, 'render' )
+        );
+    }
+
+    /**
+     * Output admin notices if needed, based on the message url parameter.
+     * A value of 1 indicates that a successful save was done.
+     *
+     * @since 3.11.0
+     */
+    function admin_notice() {
+        if ( isset( $_GET['message'] ) && ( '1' === $_GET['message'] ) ) {
+            ?>
             <div class="notice notice-success is-dismissible">
                 <p><?php esc_html_e( 'Settings saved', 'wordlift' ) ?></p>
             </div>
 			<?php
-		}
-	}
-
-	/**
-	 * Validate the existence of the entity type indicated by the tag_ID url
-	 * parameter before doing any processing. Done before any output to mimic
-	 * the way WordPress handles same situation with "normal" term editing screens.
-	 *
-	 * @since 3.11.0
-	 */
-	function validate_proper_term() {
-
-		// Validate capabilities.
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_die(
-				'<h1>' . esc_html__( 'Cheatin&#8217; uh?' ) . '</h1>' .
-				'<p>' . esc_html__( 'Sorry, you are not allowed to edit this item.' ) . '</p>',
-				403
-			);
-		}
-
-		// Get the term id and the actual term.
-		$term_id = isset( $_REQUEST['tag_ID'] ) ? (int) $_REQUEST['tag_ID'] : 0;
-
-		if ( ! term_exists( $term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME ) ) {
-			wp_die( esc_html__( 'You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift' ) );
-		}
-
-	}
-
-	/**
-	 * Handle the form submission of the settings form. On successful
-	 * handling redirect tp the setting edit page.
-	 *
-	 * @since 3.11.0
-	 */
-	function handle_form_submission() {
-
-		$term_id = isset( $_POST['tag_ID'] ) ? (int) $_POST['tag_ID'] : 0;
-
-		// Check the nonce.
-		check_admin_referer( 'update-entity_type_term_' . $term_id );
-
-		$term = get_term( $term_id, 'wl_entity_type' );
-
-		$title       = isset( $_POST['title'] ) ? (string) $_POST['title'] : '';
-		$description = isset( $_POST['description'] ) ? (string) $_POST['description'] : '';
-		$this->set_setting(
-			$term_id,
-			trim( wp_unslash( $title ) ),
-			wp_unslash( $description )
-		);
-
-		// Redirect back to the term settings page and indicate a save was done.
-		$url = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1" );
-
-		wp_redirect( $url );
-		exit;
-
-	}
-
-	/**
-	 * Render the settings page for the term.
-	 *
-	 * Access and parameter validity is assumed to be done earlier.
-	 *
-	 * @since 3.11.0
-	 */
-	function render() {
-
-		// Set variables used by the partial
-		$term_id  = isset( $_REQUEST['tag_ID'] ) ? absint( $_REQUEST['tag_ID'] ) : 0;
-		$settings = $this->get_setting( $term_id );
-
-		include plugin_dir_path( dirname( __FILE__ ) ) . 'admin/partials/wordlift-admin-entity-type-settings.php';
-
-	}
-
-	/**
-	 * Store the entity type term settings in the DB
-	 *
-	 * @param integer $term_id The ID of the entity type term
-	 * @param string $title The override for the terms title.
-	 * @param string $description The override for the terms description.
-	 *
-	 * @since 3.11.0
-	 *
-	 */
-	function set_setting( $term_id, $title, $description ) {
-
-		$settings             = get_option( 'wl_entity_type_settings', array() );
-		$settings[ $term_id ] = array(
-			'title'       => $title,
-			'description' => $description,
-		);
-		update_option( 'wl_entity_type_settings', $settings );
-
-	}
-
-	/**
-	 * Retrieve the entity type term settings from the DB
-	 *
-	 * @param integer $term_id The ID of the entity type term
-	 *
-	 * @return    null|array {
-	 *                null is returned when there are no settings otherwise
-	 *                an array is returned with following fields
-	 *
-	 * @type    string    title    The overriding title for the term
-	 * @type    string    description    The overriding description for the term
-	 *            }
-	 * @since 3.11.0
-	 *
-	 */
-	function get_setting( $term_id ) {
-
-		$settings = get_option( 'wl_entity_type_settings', array() );
-
-		if ( isset( $settings[ $term_id ] ) ) {
-			return $settings[ $term_id ];
-		}
-
-		$this->log->warn( "No settings found for term id $term_id." );
-
-		return null;
-	}
+        }
+    }
+
+    /**
+     * Validate the existence of the entity type indicated by the tag_ID url
+     * parameter before doing any processing. Done before any output to mimic
+     * the way WordPress handles same situation with "normal" term editing screens.
+     *
+     * @since 3.11.0
+     */
+    function validate_proper_term() {
+
+        // Validate capabilities.
+        if ( ! current_user_can( 'manage_options' ) ) {
+            wp_die(
+                '<h1>' . esc_html__( 'Cheatin&#8217; uh?' ) . '</h1>' .
+                '<p>' . esc_html__( 'Sorry, you are not allowed to edit this item.' ) . '</p>',
+                403
+            );
+        }
+
+        // Get the term id and the actual term.
+        $term_id = isset( $_REQUEST['tag_ID'] ) ? (int) $_REQUEST['tag_ID'] : 0;
+
+        if ( ! term_exists( $term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME ) ) {
+            wp_die( esc_html__( 'You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift' ) );
+        }
+
+    }
+
+    /**
+     * Handle the form submission of the settings form. On successful
+     * handling redirect tp the setting edit page.
+     *
+     * @since 3.11.0
+     */
+    function handle_form_submission() {
+
+        $term_id = isset( $_POST['tag_ID'] ) ? (int) $_POST['tag_ID'] : 0;
+
+        // Check the nonce.
+        check_admin_referer( 'update-entity_type_term_' . $term_id );
+
+        $term = get_term( $term_id, 'wl_entity_type' );
+
+        $title       = isset( $_POST['title'] ) ? (string) $_POST['title'] : '';
+        $description = isset( $_POST['description'] ) ? (string) $_POST['description'] : '';
+        $this->set_setting(
+            $term_id,
+            trim( wp_unslash( $title ) ),
+            wp_unslash( $description )
+        );
+
+        // Redirect back to the term settings page and indicate a save was done.
+        $url = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1" );
+
+        wp_redirect( $url );
+        exit;
+
+    }
+
+    /**
+     * Render the settings page for the term.
+     *
+     * Access and parameter validity is assumed to be done earlier.
+     *
+     * @since 3.11.0
+     */
+    function render() {
+
+        // Set variables used by the partial
+        $term_id  = isset( $_REQUEST['tag_ID'] ) ? absint( $_REQUEST['tag_ID'] ) : 0;
+        $settings = $this->get_setting( $term_id );
+
+        include plugin_dir_path( dirname( __FILE__ ) ) . 'admin/partials/wordlift-admin-entity-type-settings.php';
+
+    }
+
+    /**
+     * Store the entity type term settings in the DB
+     *
+     * @param integer $term_id The ID of the entity type term
+     * @param string $title The override for the terms title.
+     * @param string $description The override for the terms description.
+     *
+     * @since 3.11.0
+     *
+     */
+    function set_setting( $term_id, $title, $description ) {
+
+        $settings             = get_option( 'wl_entity_type_settings', array() );
+        $settings[ $term_id ] = array(
+            'title'       => $title,
+            'description' => $description,
+        );
+        update_option( 'wl_entity_type_settings', $settings );
+
+    }
+
+    /**
+     * Retrieve the entity type term settings from the DB
+     *
+     * @param integer $term_id The ID of the entity type term
+     *
+     * @return    null|array {
+     *                null is returned when there are no settings otherwise
+     *                an array is returned with following fields
+     *
+     * @type    string    title    The overriding title for the term
+     * @type    string    description    The overriding description for the term
+     *            }
+     * @since 3.11.0
+     *
+     */
+    function get_setting( $term_id ) {
+
+        $settings = get_option( 'wl_entity_type_settings', array() );
+
+        if ( isset( $settings[ $term_id ] ) ) {
+            return $settings[ $term_id ];
+        }
+
+        $this->log->warn( "No settings found for term id $term_id." );
+
+        return null;
+    }
 
 }

Spacing +38 added lines, -38 removed lines

@@ -12,7 +12,7 @@ 
  * @since      3.11.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,7 +46,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Admin_Entity_Type_Settings' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Admin_Entity_Type_Settings');
 
 	}
 
@@ -66,13 +66,13 @@ 
 		 * This has to be done before any output happens in order to be able to
 		 * display proper "die" error messages and redirect.
 		 */
-		if ( isset( $_GET['page'] ) && ( 'wl_entity_type_settings' === $_GET['page'] ) ) {
+		if (isset($_GET['page']) && ('wl_entity_type_settings' === $_GET['page'])) {
 
 			// Validate inputs. Do not return on invalid parameters or capabilities.
 			$this->validate_proper_term();
 
 			// If proper form submission, handle it and redirect back to the settings page.
-			if ( isset( $_POST['action'] ) && ( 'wl_edit_entity_type_term' === $_POST['action'] ) ) {
+			if (isset($_POST['action']) && ('wl_edit_entity_type_term' === $_POST['action'])) {
 				$this->handle_form_submission();
 			}
 			/**
@@ -83,9 +83,9 @@ 
 			 * @return bool
 			 * @since 3.27.6
 			 */
-			if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+			if (apply_filters('wl_feature__enable__notices', true)) {
 				// Register admin notices handler.
-				add_action( 'admin_notices', array( $this, 'admin_notice' ) );
+				add_action('admin_notices', array($this, 'admin_notice'));
 			}
 
 		}
@@ -97,11 +97,11 @@ 
 		// @todo: use the new {@link Wordlift_Admin_Page}.
 		add_submenu_page(
 			null,
-			__( 'Edit Entity term', 'wordlift' ),
-			__( 'Edit Entity term', 'wordlift' ),
+			__('Edit Entity term', 'wordlift'),
+			__('Edit Entity term', 'wordlift'),
 			'manage_options',
 			'wl_entity_type_settings',
-			array( $this, 'render' )
+			array($this, 'render')
 		);
 	}
 
@@ -112,10 +112,10 @@ 
 	 * @since 3.11.0
 	 */
 	function admin_notice() {
-		if ( isset( $_GET['message'] ) && ( '1' === $_GET['message'] ) ) {
+		if (isset($_GET['message']) && ('1' === $_GET['message'])) {
 			?>
             <div class="notice notice-success is-dismissible">
-                <p><?php esc_html_e( 'Settings saved', 'wordlift' ) ?></p>
+                <p><?php esc_html_e('Settings saved', 'wordlift') ?></p>
             </div>
 			<?php
 		}
@@ -131,19 +131,19 @@ 
 	function validate_proper_term() {
 
 		// Validate capabilities.
-		if ( ! current_user_can( 'manage_options' ) ) {
+		if ( ! current_user_can('manage_options')) {
 			wp_die(
-				'<h1>' . esc_html__( 'Cheatin&#8217; uh?' ) . '</h1>' .
-				'<p>' . esc_html__( 'Sorry, you are not allowed to edit this item.' ) . '</p>',
+				'<h1>'.esc_html__('Cheatin&#8217; uh?').'</h1>'.
+				'<p>'.esc_html__('Sorry, you are not allowed to edit this item.').'</p>',
 				403
 			);
 		}
 
 		// Get the term id and the actual term.
-		$term_id = isset( $_REQUEST['tag_ID'] ) ? (int) $_REQUEST['tag_ID'] : 0;
+		$term_id = isset($_REQUEST['tag_ID']) ? (int) $_REQUEST['tag_ID'] : 0;
 
-		if ( ! term_exists( $term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME ) ) {
-			wp_die( esc_html__( 'You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift' ) );
+		if ( ! term_exists($term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME)) {
+			wp_die(esc_html__('You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift'));
 		}
 
 	}
@@ -156,25 +156,25 @@ 
 	 */
 	function handle_form_submission() {
 
-		$term_id = isset( $_POST['tag_ID'] ) ? (int) $_POST['tag_ID'] : 0;
+		$term_id = isset($_POST['tag_ID']) ? (int) $_POST['tag_ID'] : 0;
 
 		// Check the nonce.
-		check_admin_referer( 'update-entity_type_term_' . $term_id );
+		check_admin_referer('update-entity_type_term_'.$term_id);
 
-		$term = get_term( $term_id, 'wl_entity_type' );
+		$term = get_term($term_id, 'wl_entity_type');
 
-		$title       = isset( $_POST['title'] ) ? (string) $_POST['title'] : '';
-		$description = isset( $_POST['description'] ) ? (string) $_POST['description'] : '';
+		$title       = isset($_POST['title']) ? (string) $_POST['title'] : '';
+		$description = isset($_POST['description']) ? (string) $_POST['description'] : '';
 		$this->set_setting(
 			$term_id,
-			trim( wp_unslash( $title ) ),
-			wp_unslash( $description )
+			trim(wp_unslash($title)),
+			wp_unslash($description)
 		);
 
 		// Redirect back to the term settings page and indicate a save was done.
-		$url = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1" );
+		$url = admin_url("admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1");
 
-		wp_redirect( $url );
+		wp_redirect($url);
 		exit;
 
 	}
@@ -189,10 +189,10 @@ 
 	function render() {
 
 		// Set variables used by the partial
-		$term_id  = isset( $_REQUEST['tag_ID'] ) ? absint( $_REQUEST['tag_ID'] ) : 0;
-		$settings = $this->get_setting( $term_id );
+		$term_id  = isset($_REQUEST['tag_ID']) ? absint($_REQUEST['tag_ID']) : 0;
+		$settings = $this->get_setting($term_id);
 
-		include plugin_dir_path( dirname( __FILE__ ) ) . 'admin/partials/wordlift-admin-entity-type-settings.php';
+		include plugin_dir_path(dirname(__FILE__)).'admin/partials/wordlift-admin-entity-type-settings.php';
 
 	}
 
@@ -206,14 +206,14 @@ 
 	 * @since 3.11.0
 	 *
 	 */
-	function set_setting( $term_id, $title, $description ) {
+	function set_setting($term_id, $title, $description) {
 
-		$settings             = get_option( 'wl_entity_type_settings', array() );
-		$settings[ $term_id ] = array(
+		$settings             = get_option('wl_entity_type_settings', array());
+		$settings[$term_id] = array(
 			'title'       => $title,
 			'description' => $description,
 		);
-		update_option( 'wl_entity_type_settings', $settings );
+		update_option('wl_entity_type_settings', $settings);
 
 	}
 
@@ -232,15 +232,15 @@ 
 	 * @since 3.11.0
 	 *
 	 */
-	function get_setting( $term_id ) {
+	function get_setting($term_id) {
 
-		$settings = get_option( 'wl_entity_type_settings', array() );
+		$settings = get_option('wl_entity_type_settings', array());
 
-		if ( isset( $settings[ $term_id ] ) ) {
-			return $settings[ $term_id ];
+		if (isset($settings[$term_id])) {
+			return $settings[$term_id];
 		}
 
-		$this->log->warn( "No settings found for term id $term_id." );
+		$this->log->warn("No settings found for term id $term_id.");
 
 		return null;
 	}

src/admin/elements/class-wordlift-admin-select-element.php

Indentation +147 added lines, -147 removed lines

@@ -18,28 +18,28 @@ 
  */
 abstract class Wordlift_Admin_Select_Element implements Wordlift_Admin_Element {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function render( $args ) {
-		// Some select fields may need custom script/styles to work.
-		$this->enqueue_resources();
-
-		// Parse the arguments and merge with default values.
-		$params = wp_parse_args(
-			$args,
-			array(
-				'id'          => uniqid( 'wl-input-' ),
-				'name'        => uniqid( 'wl-input-' ),
-				'value'       => '',
-				'class'       => '',
-				'notice'      => '',
-				'description' => false,
-				'data'        => array(),
-				'options'     => array(),
-			)
-		);
-		?>
+    /**
+     * @inheritdoc
+     */
+    public function render( $args ) {
+        // Some select fields may need custom script/styles to work.
+        $this->enqueue_resources();
+
+        // Parse the arguments and merge with default values.
+        $params = wp_parse_args(
+            $args,
+            array(
+                'id'          => uniqid( 'wl-input-' ),
+                'name'        => uniqid( 'wl-input-' ),
+                'value'       => '',
+                'class'       => '',
+                'notice'      => '',
+                'description' => false,
+                'data'        => array(),
+                'options'     => array(),
+            )
+        );
+        ?>
 		<select
 				id="<?php echo esc_attr( $params['id'] ); ?>"
 				name="<?php echo esc_attr( $params['name'] ); ?>"
@@ -50,130 +50,130 @@ 
 		</select>
 
 		<?php
-		// Print the notice message if there is such.
-		$this->print_notice( $params['notice'] );
-
-		// Print the field description.
-		echo wp_kses( $this->get_description( $params['description'] ), array( 'p' => array() ) );
-
-		return $this;
-	}
-
-	/**
-	 * Returns html escaped description string.
-	 * Note: Only `a` tags are allowed with only `href` attributes.
-	 *
-	 * @param string|bool $description The field description or false if not set.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @return string|void The description or null if not set.
-	 */
-	public function get_description( $description ) {
-		// Bail if the description is not set.
-		if ( empty( $description ) ) {
-			return;
-		}
-
-		// Remove all characters except links.
-		$filtered_descrption = wp_kses(
-			$description,
-			array(
-				'a' => array(
-					'href' => array(),
-				),
-			)
-		);
-
-		return wpautop( $filtered_descrption );
-	}
-
-	/**
-	 * Prints the field notice that will be shown on error.
-	 *
-	 * @param string $notice The notice to add.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @return void
-	 */
-	public function print_notice( $notice ) {
-		// Bail if the notice is empty.
-		if ( empty( $notice ) ) {
-			return;
-		}
-
-		// Print the notice message.
-		printf(
-			'<small class="wl-select-notices">%s</small>',
-			esc_html( $notice )
-		);
-	}
-
-	/**
-	 * Adds data attributes to select element.
-	 *
-	 * We need to use method here, because different select elements
-	 * may have different data attributes.
-	 *
-	 * @param array $pre_attributes Array of all data attributes.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @return string $output The data attributes or empty string
-	 */
-	private function get_data_attributes( $pre_attributes ) {
-		// The output.
-		$output = '';
-
-		/**
-		 * Filter: 'wl_admin_select_element_data_attributes' - Allow third
-		 * parties to modify the field data attrbutes.
-		 *
-		 * @since  3.18.0
-		 *
-		 * @param  array $pre_attributes Array of the field data attributes.
-		 */
-		$attributes = apply_filters( 'wl_admin_select_element_data_attributes', $pre_attributes );
-
-		// Bail is there are no data attributes.
-		if ( empty( $attributes ) ) {
-			return $output;
-		}
-
-		// Loop throught all data attributes and build the output string.
-		foreach ( $attributes as $name => $value ) {
-			$output .= sprintf(
-				'data-%s="%s" ',
-				$name,
-				esc_attr( $value )
-			);
-		}
-
-		// Finally return the output escaped.
-		return $output;
-	}
-
-	/**
-	 * Prints specific resources(scripts/styles) if the select requires them.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @return void
-	 */
-	protected function enqueue_resources() {
-
-	}
-
-	/**
-	 * Abstract method that renders the select options.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @param array $params Select params.
-	 *
-	 * @return Prints the select options.
-	 */
-	abstract function render_options( $params );
+        // Print the notice message if there is such.
+        $this->print_notice( $params['notice'] );
+
+        // Print the field description.
+        echo wp_kses( $this->get_description( $params['description'] ), array( 'p' => array() ) );
+
+        return $this;
+    }
+
+    /**
+     * Returns html escaped description string.
+     * Note: Only `a` tags are allowed with only `href` attributes.
+     *
+     * @param string|bool $description The field description or false if not set.
+     *
+     * @since 3.18.0
+     *
+     * @return string|void The description or null if not set.
+     */
+    public function get_description( $description ) {
+        // Bail if the description is not set.
+        if ( empty( $description ) ) {
+            return;
+        }
+
+        // Remove all characters except links.
+        $filtered_descrption = wp_kses(
+            $description,
+            array(
+                'a' => array(
+                    'href' => array(),
+                ),
+            )
+        );
+
+        return wpautop( $filtered_descrption );
+    }
+
+    /**
+     * Prints the field notice that will be shown on error.
+     *
+     * @param string $notice The notice to add.
+     *
+     * @since 3.18.0
+     *
+     * @return void
+     */
+    public function print_notice( $notice ) {
+        // Bail if the notice is empty.
+        if ( empty( $notice ) ) {
+            return;
+        }
+
+        // Print the notice message.
+        printf(
+            '<small class="wl-select-notices">%s</small>',
+            esc_html( $notice )
+        );
+    }
+
+    /**
+     * Adds data attributes to select element.
+     *
+     * We need to use method here, because different select elements
+     * may have different data attributes.
+     *
+     * @param array $pre_attributes Array of all data attributes.
+     *
+     * @since 3.18.0
+     *
+     * @return string $output The data attributes or empty string
+     */
+    private function get_data_attributes( $pre_attributes ) {
+        // The output.
+        $output = '';
+
+        /**
+         * Filter: 'wl_admin_select_element_data_attributes' - Allow third
+         * parties to modify the field data attrbutes.
+         *
+         * @since  3.18.0
+         *
+         * @param  array $pre_attributes Array of the field data attributes.
+         */
+        $attributes = apply_filters( 'wl_admin_select_element_data_attributes', $pre_attributes );
+
+        // Bail is there are no data attributes.
+        if ( empty( $attributes ) ) {
+            return $output;
+        }
+
+        // Loop throught all data attributes and build the output string.
+        foreach ( $attributes as $name => $value ) {
+            $output .= sprintf(
+                'data-%s="%s" ',
+                $name,
+                esc_attr( $value )
+            );
+        }
+
+        // Finally return the output escaped.
+        return $output;
+    }
+
+    /**
+     * Prints specific resources(scripts/styles) if the select requires them.
+     *
+     * @since 3.18.0
+     *
+     * @return void
+     */
+    protected function enqueue_resources() {
+
+    }
+
+    /**
+     * Abstract method that renders the select options.
+     *
+     * @since 3.18.0
+     *
+     * @param array $params Select params.
+     *
+     * @return Prints the select options.
+     */
+    abstract function render_options( $params );
 
 }

Spacing +22 added lines, -22 removed lines

@@ -21,7 +21,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function render( $args ) {
+	public function render($args) {
 		// Some select fields may need custom script/styles to work.
 		$this->enqueue_resources();
 
@@ -29,8 +29,8 @@ 
 		$params = wp_parse_args(
 			$args,
 			array(
-				'id'          => uniqid( 'wl-input-' ),
-				'name'        => uniqid( 'wl-input-' ),
+				'id'          => uniqid('wl-input-'),
+				'name'        => uniqid('wl-input-'),
 				'value'       => '',
 				'class'       => '',
 				'notice'      => '',
@@ -41,20 +41,20 @@ 
 		);
 		?>
 		<select
-				id="<?php echo esc_attr( $params['id'] ); ?>"
-				name="<?php echo esc_attr( $params['name'] ); ?>"
-				class="<?php echo esc_attr( $params['class'] ); ?>"
-			<?php echo $this->get_data_attributes( $params['data'] ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
+				id="<?php echo esc_attr($params['id']); ?>"
+				name="<?php echo esc_attr($params['name']); ?>"
+				class="<?php echo esc_attr($params['class']); ?>"
+			<?php echo $this->get_data_attributes($params['data']); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
 		>
-			<?php $this->render_options( $params ); ?>
+			<?php $this->render_options($params); ?>
 		</select>
 
 		<?php
 		// Print the notice message if there is such.
-		$this->print_notice( $params['notice'] );
+		$this->print_notice($params['notice']);
 
 		// Print the field description.
-		echo wp_kses( $this->get_description( $params['description'] ), array( 'p' => array() ) );
+		echo wp_kses($this->get_description($params['description']), array('p' => array()));
 
 		return $this;
 	}
@@ -69,9 +69,9 @@ 
 	 *
 	 * @return string|void The description or null if not set.
 	 */
-	public function get_description( $description ) {
+	public function get_description($description) {
 		// Bail if the description is not set.
-		if ( empty( $description ) ) {
+		if (empty($description)) {
 			return;
 		}
 
@@ -85,7 +85,7 @@ 
 			)
 		);
 
-		return wpautop( $filtered_descrption );
+		return wpautop($filtered_descrption);
 	}
 
 	/**
@@ -97,16 +97,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function print_notice( $notice ) {
+	public function print_notice($notice) {
 		// Bail if the notice is empty.
-		if ( empty( $notice ) ) {
+		if (empty($notice)) {
 			return;
 		}
 
 		// Print the notice message.
 		printf(
 			'<small class="wl-select-notices">%s</small>',
-			esc_html( $notice )
+			esc_html($notice)
 		);
 	}
 
@@ -122,7 +122,7 @@ 
 	 *
 	 * @return string $output The data attributes or empty string
 	 */
-	private function get_data_attributes( $pre_attributes ) {
+	private function get_data_attributes($pre_attributes) {
 		// The output.
 		$output = '';
 
@@ -134,19 +134,19 @@ 
 		 *
 		 * @param  array $pre_attributes Array of the field data attributes.
 		 */
-		$attributes = apply_filters( 'wl_admin_select_element_data_attributes', $pre_attributes );
+		$attributes = apply_filters('wl_admin_select_element_data_attributes', $pre_attributes);
 
 		// Bail is there are no data attributes.
-		if ( empty( $attributes ) ) {
+		if (empty($attributes)) {
 			return $output;
 		}
 
 		// Loop throught all data attributes and build the output string.
-		foreach ( $attributes as $name => $value ) {
+		foreach ($attributes as $name => $value) {
 			$output .= sprintf(
 				'data-%s="%s" ',
 				$name,
-				esc_attr( $value )
+				esc_attr($value)
 			);
 		}
 
@@ -174,6 +174,6 @@ 
 	 *
 	 * @return Prints the select options.
 	 */
-	abstract function render_options( $params );
+	abstract function render_options($params);
 
 }

src/admin/elements/class-wordlift-admin-input-radio-element.php

Indentation +39 added lines, -39 removed lines

@@ -18,48 +18,48 @@ 
  */
 class Wordlift_Admin_Radio_Input_Element implements Wordlift_Admin_Element {
 
-	/**
-	 * Output the HTML for an input box type settings_page.
-	 *
-	 * @param array $args {
-	 *                           An array of arguments.
-	 *
-	 * @type string $name The name attribute of the input element. Mandatory.
-	 * @type string $id The id attribute of the input element. Optional,
-	 *                           randomly generated one is used if not supplied.
-	 * @type string $value The value of the input element. Optional, defaults
-	 *                           to empty string.
-	 * @type string $css_class The class attribute for the input element. If empty
-	 *                           string no class attribute will be added. Optional,
-	 *                           defaults to empty string.
-	 * @type string $description The description text to be displayed below the element.
-	 *                           Can include some HTML element. If empty string no
-	 *                           description will be displayed. Optional, defaults to
-	 *                           empty string.
-	 * }
-	 * @return $this|Wordlift_Admin_Element
-	 * @since      3.13.0
-	 *
-	 */
-	public function render( $args ) {
-		/*
+    /**
+     * Output the HTML for an input box type settings_page.
+     *
+     * @param array $args {
+     *                           An array of arguments.
+     *
+     * @type string $name The name attribute of the input element. Mandatory.
+     * @type string $id The id attribute of the input element. Optional,
+     *                           randomly generated one is used if not supplied.
+     * @type string $value The value of the input element. Optional, defaults
+     *                           to empty string.
+     * @type string $css_class The class attribute for the input element. If empty
+     *                           string no class attribute will be added. Optional,
+     *                           defaults to empty string.
+     * @type string $description The description text to be displayed below the element.
+     *                           Can include some HTML element. If empty string no
+     *                           description will be displayed. Optional, defaults to
+     *                           empty string.
+     * }
+     * @return $this|Wordlift_Admin_Element
+     * @since      3.13.0
+     *
+     */
+    public function render( $args ) {
+        /*
 		 * Parse the arguments and merge with default values.
 		 * Name intentionally do not have a default as it has to be in SyncEvent
 		 * with form handling code
 		 */
-		$params = wp_parse_args( $args, array(
-			'id'          => uniqid( 'wl-input-' ),
-			'value'       => '',
-			'css_class'   => '',
-			'description' => '',
-		) );
+        $params = wp_parse_args( $args, array(
+            'id'          => uniqid( 'wl-input-' ),
+            'value'       => '',
+            'css_class'   => '',
+            'description' => '',
+        ) );
 
-		// Set the readonly and class attributes and the description.
-		$value       = $params['value'];
-		$css_class   = ! empty( $params['css_class'] ) ? ' class="' . esc_attr( $params['css_class'] ) . '"' : '';
-		$description = ! empty( $params['description'] ) ? '<p>' . wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ) . '</p>' : '';
+        // Set the readonly and class attributes and the description.
+        $value       = $params['value'];
+        $css_class   = ! empty( $params['css_class'] ) ? ' class="' . esc_attr( $params['css_class'] ) . '"' : '';
+        $description = ! empty( $params['description'] ) ? '<p>' . wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ) . '</p>' : '';
 
-		?>
+        ?>
 
         <input type="radio" id="<?php echo esc_attr( $params['id'] ); ?>"
                name="<?php echo esc_attr( $params['name'] ); ?>"
@@ -73,9 +73,9 @@ 
         /> No
 
 		<?php
-		echo  wp_kses( $description, array( 'a' => array( 'href' => array() ) ) );
+        echo  wp_kses( $description, array( 'a' => array( 'href' => array() ) ) );
 
-		return $this;
-	}
+        return $this;
+    }
 
 }

Spacing +13 added lines, -13 removed lines

@@ -41,39 +41,39 @@
 	 * @since      3.13.0
 	 *
 	 */
-	public function render( $args ) {
+	public function render($args) {
 		/*
 		 * Parse the arguments and merge with default values.
 		 * Name intentionally do not have a default as it has to be in SyncEvent
 		 * with form handling code
 		 */
-		$params = wp_parse_args( $args, array(
-			'id'          => uniqid( 'wl-input-' ),
+		$params = wp_parse_args($args, array(
+			'id'          => uniqid('wl-input-'),
 			'value'       => '',
 			'css_class'   => '',
 			'description' => '',
-		) );
+		));
 
 		// Set the readonly and class attributes and the description.
 		$value       = $params['value'];
-		$css_class   = ! empty( $params['css_class'] ) ? ' class="' . esc_attr( $params['css_class'] ) . '"' : '';
-		$description = ! empty( $params['description'] ) ? '<p>' . wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ) . '</p>' : '';
+		$css_class   = ! empty($params['css_class']) ? ' class="'.esc_attr($params['css_class']).'"' : '';
+		$description = ! empty($params['description']) ? '<p>'.wp_kses($params['description'], array('a' => array('href' => array()))).'</p>' : '';
 
 		?>
 
-        <input type="radio" id="<?php echo esc_attr( $params['id'] ); ?>"
-               name="<?php echo esc_attr( $params['name'] ); ?>"
+        <input type="radio" id="<?php echo esc_attr($params['id']); ?>"
+               name="<?php echo esc_attr($params['name']); ?>"
                value="yes" <?php echo $css_class; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
-			<?php checked( $value, 'yes' ); ?>
+			<?php checked($value, 'yes'); ?>
         /> Yes
-        <input type="radio" id="<?php echo esc_attr( $params['id'] ); ?>"
-               name="<?php echo esc_attr( $params['name'] ); ?>"
+        <input type="radio" id="<?php echo esc_attr($params['id']); ?>"
+               name="<?php echo esc_attr($params['name']); ?>"
                value="no" <?php echo $css_class; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped?>
-			<?php checked( $value, 'no' ); ?>
+			<?php checked($value, 'no'); ?>
         /> No
 
 		<?php
-		echo  wp_kses( $description, array( 'a' => array( 'href' => array() ) ) );
+		echo  wp_kses($description, array('a' => array('href' => array())));
 
 		return $this;
 	}

src/admin/wordlift_admin_menu.php

Indentation +31 added lines, -31 removed lines

@@ -15,24 +15,24 @@ 
  */
 function wl_admin_menu() {
 
-	$menu_slug  = 'wl_admin_menu';
-	$capability = 'manage_options';
+    $menu_slug  = 'wl_admin_menu';
+    $capability = 'manage_options';
 
-	// see http://codex.wordpress.org/Function_Reference/add_menu_page
-	add_menu_page(
-		__( 'WordLift', 'wordlift' ), // page title
-		__( 'WordLift', 'wordlift' ), // menu title
-		$capability,                 // capabilities
-		$menu_slug,                  // menu slug
-		//'wl_admin_menu_callback',  // TODO: function callback to draw the coming dashboard
-		'wl_configuration_admin_menu_callback',
-		WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-logo-icon.svg' );  // icon URL 20x20 px
+    // see http://codex.wordpress.org/Function_Reference/add_menu_page
+    add_menu_page(
+        __( 'WordLift', 'wordlift' ), // page title
+        __( 'WordLift', 'wordlift' ), // menu title
+        $capability,                 // capabilities
+        $menu_slug,                  // menu slug
+        //'wl_admin_menu_callback',  // TODO: function callback to draw the coming dashboard
+        'wl_configuration_admin_menu_callback',
+        WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-logo-icon.svg' );  // icon URL 20x20 px
 
-	// Call hooked functions.
-	do_action( 'wl_admin_menu', $menu_slug, $capability );
+    // Call hooked functions.
+    do_action( 'wl_admin_menu', $menu_slug, $capability );
 
-	// Remove duplicate 'WordLift' subpage created by WordPress.
-	remove_submenu_page( $menu_slug, $menu_slug );
+    // Remove duplicate 'WordLift' subpage created by WordPress.
+    remove_submenu_page( $menu_slug, $menu_slug );
 
 }
 
@@ -46,20 +46,20 @@ 
  * @since 3.15.0
  */
 function wl_remove_entity_type_menu() {
-	/*
+    /*
 	 * Remove from the menu links to the entity type admin page when
 	 * under non entity hierarchy.
 	 */
-	foreach ( Wordlift_Entity_Service::valid_entity_post_types() as $post_type ) {
-		// In the context of admin menues post has no explicit indication of post type in the urls.
-		if ( 'post' !== $post_type ) {
-			remove_submenu_page( 'edit.php', 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
-		}
+    foreach ( Wordlift_Entity_Service::valid_entity_post_types() as $post_type ) {
+        // In the context of admin menues post has no explicit indication of post type in the urls.
+        if ( 'post' !== $post_type ) {
+            remove_submenu_page( 'edit.php', 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+        }
 
-		if ( Wordlift_Entity_Service::TYPE_NAME !== $post_type ) {
-			remove_submenu_page( 'edit.php?post_type=' . $post_type, 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME . '&post_type=' . $post_type );
-		}
-	}
+        if ( Wordlift_Entity_Service::TYPE_NAME !== $post_type ) {
+            remove_submenu_page( 'edit.php?post_type=' . $post_type, 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME . '&post_type=' . $post_type );
+        }
+    }
 }
 
 add_action( 'admin_menu', 'wl_remove_entity_type_menu', 100 );
@@ -71,12 +71,12 @@ 
  */
 function wl_admin_menu_callback() {
 
-	if ( ! current_user_can( 'manage_options' ) ) {
-		wp_die( esc_html__( 'You do not have sufficient permissions to access this page.' ) );
-	}
+    if ( ! current_user_can( 'manage_options' ) ) {
+        wp_die( esc_html__( 'You do not have sufficient permissions to access this page.' ) );
+    }
 
-	echo '<div class="wrap">';
-	echo '<p>Here is where the form would go if I actually had options.</p>';
-	echo '</div>';
+    echo '<div class="wrap">';
+    echo '<p>Here is where the form would go if I actually had options.</p>';
+    echo '</div>';
 
 }

Spacing +16 added lines, -16 removed lines

@@ -20,23 +20,23 @@ 
 
 	// see http://codex.wordpress.org/Function_Reference/add_menu_page
 	add_menu_page(
-		__( 'WordLift', 'wordlift' ), // page title
-		__( 'WordLift', 'wordlift' ), // menu title
-		$capability,                 // capabilities
-		$menu_slug,                  // menu slug
+		__('WordLift', 'wordlift'), // page title
+		__('WordLift', 'wordlift'), // menu title
+		$capability, // capabilities
+		$menu_slug, // menu slug
 		//'wl_admin_menu_callback',  // TODO: function callback to draw the coming dashboard
 		'wl_configuration_admin_menu_callback',
-		WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-logo-icon.svg' );  // icon URL 20x20 px
+		WP_CONTENT_URL.'/plugins/wordlift/images/svg/wl-logo-icon.svg' ); // icon URL 20x20 px
 
 	// Call hooked functions.
-	do_action( 'wl_admin_menu', $menu_slug, $capability );
+	do_action('wl_admin_menu', $menu_slug, $capability);
 
 	// Remove duplicate 'WordLift' subpage created by WordPress.
-	remove_submenu_page( $menu_slug, $menu_slug );
+	remove_submenu_page($menu_slug, $menu_slug);
 
 }
 
-add_action( 'admin_menu', 'wl_admin_menu' );
+add_action('admin_menu', 'wl_admin_menu');
 
 /**
  * This function is called by the *admin_menu* hook to remove for the admin menu
@@ -50,19 +50,19 @@ 
 	 * Remove from the menu links to the entity type admin page when
 	 * under non entity hierarchy.
 	 */
-	foreach ( Wordlift_Entity_Service::valid_entity_post_types() as $post_type ) {
+	foreach (Wordlift_Entity_Service::valid_entity_post_types() as $post_type) {
 		// In the context of admin menues post has no explicit indication of post type in the urls.
-		if ( 'post' !== $post_type ) {
-			remove_submenu_page( 'edit.php', 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+		if ('post' !== $post_type) {
+			remove_submenu_page('edit.php', 'edit-tags.php?taxonomy='.Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME);
 		}
 
-		if ( Wordlift_Entity_Service::TYPE_NAME !== $post_type ) {
-			remove_submenu_page( 'edit.php?post_type=' . $post_type, 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME . '&post_type=' . $post_type );
+		if (Wordlift_Entity_Service::TYPE_NAME !== $post_type) {
+			remove_submenu_page('edit.php?post_type='.$post_type, 'edit-tags.php?taxonomy='.Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME.'&post_type='.$post_type);
 		}
 	}
 }
 
-add_action( 'admin_menu', 'wl_remove_entity_type_menu', 100 );
+add_action('admin_menu', 'wl_remove_entity_type_menu', 100);
 
 /**
  * This function is called as a callback by the *wl_admin_menu* to display the actual page.
@@ -71,8 +71,8 @@ 
  */
 function wl_admin_menu_callback() {
 
-	if ( ! current_user_can( 'manage_options' ) ) {
-		wp_die( esc_html__( 'You do not have sufficient permissions to access this page.' ) );
+	if ( ! current_user_can('manage_options')) {
+		wp_die(esc_html__('You do not have sufficient permissions to access this page.'));
 	}
 
 	echo '<div class="wrap">';

src/admin/class-wordlift-admin-dashboard-v2.php

Spacing +29 added lines, -29 removed lines

@@ -21,14 +21,14 @@ 
 	 * @since 3.20.0
 	 *
 	 */
-	public function __construct( $entity_service ) {
+	public function __construct($entity_service) {
 
-		add_action( 'wp_dashboard_setup', array( $this, 'dashboard_setup' ) );
+		add_action('wp_dashboard_setup', array($this, 'dashboard_setup'));
 
 		// Define where to access the tip.
-		defined( 'WL_TODAYS_TIP_JSON_URL' ) || define( 'WL_TODAYS_TIP_JSON_URL', 'https://wordlift.io/blog' );
-		defined( 'WL_TODAYS_TIP_JSON_URL_IT' ) || define( 'WL_TODAYS_TIP_JSON_URL_IT', '/it/wp-json/wp/v2/posts?context=embed&per_page=1&categories=27' );
-		defined( 'WL_TODAYS_TIP_JSON_URL_EN' ) || define( 'WL_TODAYS_TIP_JSON_URL_EN', '/en/wp-json/wp/v2/posts?context=embed&per_page=1&categories=38' );
+		defined('WL_TODAYS_TIP_JSON_URL') || define('WL_TODAYS_TIP_JSON_URL', 'https://wordlift.io/blog');
+		defined('WL_TODAYS_TIP_JSON_URL_IT') || define('WL_TODAYS_TIP_JSON_URL_IT', '/it/wp-json/wp/v2/posts?context=embed&per_page=1&categories=27');
+		defined('WL_TODAYS_TIP_JSON_URL_EN') || define('WL_TODAYS_TIP_JSON_URL_EN', '/en/wp-json/wp/v2/posts?context=embed&per_page=1&categories=38');
 		$this->entity_service = $entity_service;
 
 	}
@@ -44,11 +44,11 @@ 
 		 * Feature flag to enable / disable dashboard
 		 * @since 3.30.0
 		 */
-		if ( apply_filters( 'wl_feature__enable__wordlift-dashboard', true ) ) {
+		if (apply_filters('wl_feature__enable__wordlift-dashboard', true)) {
 			wp_add_dashboard_widget(
 				'wl-dashboard-v2',
-				__( 'WordLift Dashboard', 'wordlift' ),
-				array( $this, 'dashboard_setup_callback' )
+				__('WordLift Dashboard', 'wordlift'),
+				array($this, 'dashboard_setup_callback')
 			);
 		}
 
@@ -61,7 +61,7 @@ 
 	 */
 	public function dashboard_setup_callback() {
 
-		require_once plugin_dir_path( dirname( __FILE__ ) ) . 'admin/partials/wordlift-admin-dashboard-v2.php';
+		require_once plugin_dir_path(dirname(__FILE__)).'admin/partials/wordlift-admin-dashboard-v2.php';
 
 	}
 
@@ -79,7 +79,7 @@ 
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/1214
 		 * Top entities are generated by cron now.
 		 */
-		return get_option( Top_Entities::OPTION_KEY, array() );
+		return get_option(Top_Entities::OPTION_KEY, array());
 	}
 
 	/**
@@ -92,20 +92,20 @@ 
 		$data = @self::get_todays_tip_data();
 
 		// Unable to get data from the local cache, nor from the remote URL.
-		if ( false === $data ) {
+		if (false === $data) {
 			return;
 		}
 		?>
 
         <div id="wl-todays-tip" class="wl-dashboard__block wl-dashboard__block--todays-tip">
             <header>
-                <h3><?php esc_html_e( "Today's Tip", 'wordlift' ); ?></h3>
+                <h3><?php esc_html_e("Today's Tip", 'wordlift'); ?></h3>
             </header>
             <article>
-                <p><strong><?php echo esc_html( wp_strip_all_tags( $data['title'] ) ); ?></strong>
-					<?php echo esc_html( wp_strip_all_tags( $data['excerpt'] ) ); ?>
+                <p><strong><?php echo esc_html(wp_strip_all_tags($data['title'])); ?></strong>
+					<?php echo esc_html(wp_strip_all_tags($data['excerpt'])); ?>
                     <a target="_blank"
-                       href="<?php echo esc_attr( $data['link'] ); ?>"><?php echo esc_html( __( 'Read more', 'wordlift' ) ); ?></a>
+                       href="<?php echo esc_attr($data['link']); ?>"><?php echo esc_html(__('Read more', 'wordlift')); ?></a>
                 </p>
         </div>
 		<?php
@@ -122,39 +122,39 @@ 
 	private static function get_todays_tip_data() {
 
 		// Return the transient.
-		if ( false !== get_transient( self::TODAYS_TIP ) ) {
-			return get_transient( self::TODAYS_TIP );
+		if (false !== get_transient(self::TODAYS_TIP)) {
+			return get_transient(self::TODAYS_TIP);
 		}
 
 		// If the transient isn't available, query the remote web site.
 		$url = WL_TODAYS_TIP_JSON_URL
-		       . ( 'it' === get_bloginfo( 'language' ) ? WL_TODAYS_TIP_JSON_URL_IT : WL_TODAYS_TIP_JSON_URL_EN );
+		       . ('it' === get_bloginfo('language') ? WL_TODAYS_TIP_JSON_URL_IT : WL_TODAYS_TIP_JSON_URL_EN);
 
-		$response = wp_remote_get( $url );
+		$response = wp_remote_get($url);
 
-		if ( is_wp_error( $response )
-		     || ! isset( $response['response']['code'] )
-		     || 2 !== (int) $response['response']['code'] / 100 ) {
+		if (is_wp_error($response)
+		     || ! isset($response['response']['code'])
+		     || 2 !== (int) $response['response']['code'] / 100) {
 			return false;
 		}
 
-		$json = json_decode( $response['body'], true );
+		$json = json_decode($response['body'], true);
 
-		if ( empty( $json )
-		     || ! isset( $json[0]['title']['rendered'] )
-		     || ! isset( $json[0]['excerpt']['rendered'] )
-		     || ! isset( $json[0]['link'] ) ) {
+		if (empty($json)
+		     || ! isset($json[0]['title']['rendered'])
+		     || ! isset($json[0]['excerpt']['rendered'])
+		     || ! isset($json[0]['link'])) {
 			return false;
 		}
 
 		$value = array(
 			'title'   => $json[0]['title']['rendered'],
-			'excerpt' => '<!-- cached -->' . $json[0]['excerpt']['rendered'],
+			'excerpt' => '<!-- cached -->'.$json[0]['excerpt']['rendered'],
 			'link'    => $json[0]['link'],
 		);
 
 		// Store the results for one day.
-		set_transient( self::TODAYS_TIP, $value, 86400 );
+		set_transient(self::TODAYS_TIP, $value, 86400);
 
 		return $value;
 	}

src/admin/class-wordlift-download-your-data-page.php

Indentation +125 added lines, -125 removed lines

@@ -18,133 +18,133 @@
  * @since 3.6.0
  */
 class Wordlift_Admin_Download_Your_Data_Page {
-	/**
-	 * Used to check if the requested file is supported.
-	 *
-	 * @since  3.16.0
-	 * @access private
-	 * @var $allowed_formats array Allowed formats.
-	 */
-	private $allowed_formats = array(
-		'json',
-		'rdf',
-		'ttl',
-		'n3',
-	);
-
-
-	/**
-	 * The list of headers allowed by the endpoint.
-	 * @since 3.28.2
-	 * @var string[]
-	 */
-	private $allowed_headers = array(
-		'json' => 'application/ld+json',
-		'rdf'  => 'application/rdf+xml',
-		'n3'   => 'text/n3',
-		'ttl'  => 'text/turtle'
-	);
-
-	/**
-	 * Hook to 'admin_menu' to add the 'Download Your Data' page.
-	 *
-	 * @since 3.6.0
-	 */
-	public function admin_menu() {
-
-		// Add a callback to our 'page' function.
-		add_submenu_page(
-			'wl_admin_menu',
-			_x( 'Download Your Data', 'Page title', 'wordlift' ),
-			_x( 'Download Your Data', 'Menu title', 'wordlift' ),
-			'manage_options',
-			'wl_download_your_data',
-			array( $this, 'page' )
-		);
-
-	}
-
-	/**
-	 * The admin menu callback to render the page.
-	 *
-	 * @since 3.6.0
-	 */
-	public function page() {
-
-		// Include the partial.
-		include( plugin_dir_path( __FILE__ ) . 'partials/wordlift-admin-download-your-data.php' );
-
-	}
-
-	/**
-	 * Ajax call to redirect to a download URL.
-	 *
-	 * @since 3.6.0
-	 */
-	public function download_your_data() {
-
-		$default_api_service = Default_Api_Service::get_instance();
-
-		// Avoid PHP notices when buffer is empty.
-		if ( ob_get_contents() ) {
-			ob_end_clean();
-		}
-
-		// Get WL's key.
-		$key = Wordlift_Configuration_Service::get_instance()->get_key();
-
-		// Use json suffix by default.
-		$suffix = 'json';
-
-		// Check if there is suffix.
-		if ( isset( $_GET['out'] ) ) {
-			$suffix = sanitize_text_field( wp_unslash( $_GET['out'] ) );
-		}
-
-		// Create filename.
-		$filename = 'dataset.' . $suffix;
-
-		if ( ! in_array( $suffix, $this->allowed_formats, true ) ) {
-			// The file type is not from allowed types.
-			wp_die( esc_html__( 'The format is not supported.', 'wordlift' ) );
-		}
-
-
-		$accept_header_format = $this->allowed_headers[ $suffix ];
-
-		$headers = array(
-			'Accept' => $accept_header_format
-		);
-
-		$response = $default_api_service->get( '/dataset/export', $headers );
-
-		$response = $response->get_response();
-
-		if (
-			is_wp_error( $response ) ||
-			200 !== (int) $response['response']['code']
-		) {
-			// Something is not working properly, so display error message.
-			wp_die( esc_html__( 'There was an error trying to connect to the server. Please try again later.', 'wordlift' ) );
-		}
-
-
-		// Get response body.
-		$body     = wp_remote_retrieve_body( $response );
-		$type     = wp_remote_retrieve_header( $response, 'content-type' );
-		$filename = 'dataset-' . date( 'Y-m-d-H-i-s' ) . '.' . $suffix;
-
-		// Add proper file headers.
-		header( "Content-Disposition: attachment; filename=$filename" );
-		header( "Content-Type: $type" );
-
-		/*
+    /**
+     * Used to check if the requested file is supported.
+     *
+     * @since  3.16.0
+     * @access private
+     * @var $allowed_formats array Allowed formats.
+     */
+    private $allowed_formats = array(
+        'json',
+        'rdf',
+        'ttl',
+        'n3',
+    );
+
+
+    /**
+     * The list of headers allowed by the endpoint.
+     * @since 3.28.2
+     * @var string[]
+     */
+    private $allowed_headers = array(
+        'json' => 'application/ld+json',
+        'rdf'  => 'application/rdf+xml',
+        'n3'   => 'text/n3',
+        'ttl'  => 'text/turtle'
+    );
+
+    /**
+     * Hook to 'admin_menu' to add the 'Download Your Data' page.
+     *
+     * @since 3.6.0
+     */
+    public function admin_menu() {
+
+        // Add a callback to our 'page' function.
+        add_submenu_page(
+            'wl_admin_menu',
+            _x( 'Download Your Data', 'Page title', 'wordlift' ),
+            _x( 'Download Your Data', 'Menu title', 'wordlift' ),
+            'manage_options',
+            'wl_download_your_data',
+            array( $this, 'page' )
+        );
+
+    }
+
+    /**
+     * The admin menu callback to render the page.
+     *
+     * @since 3.6.0
+     */
+    public function page() {
+
+        // Include the partial.
+        include( plugin_dir_path( __FILE__ ) . 'partials/wordlift-admin-download-your-data.php' );
+
+    }
+
+    /**
+     * Ajax call to redirect to a download URL.
+     *
+     * @since 3.6.0
+     */
+    public function download_your_data() {
+
+        $default_api_service = Default_Api_Service::get_instance();
+
+        // Avoid PHP notices when buffer is empty.
+        if ( ob_get_contents() ) {
+            ob_end_clean();
+        }
+
+        // Get WL's key.
+        $key = Wordlift_Configuration_Service::get_instance()->get_key();
+
+        // Use json suffix by default.
+        $suffix = 'json';
+
+        // Check if there is suffix.
+        if ( isset( $_GET['out'] ) ) {
+            $suffix = sanitize_text_field( wp_unslash( $_GET['out'] ) );
+        }
+
+        // Create filename.
+        $filename = 'dataset.' . $suffix;
+
+        if ( ! in_array( $suffix, $this->allowed_formats, true ) ) {
+            // The file type is not from allowed types.
+            wp_die( esc_html__( 'The format is not supported.', 'wordlift' ) );
+        }
+
+
+        $accept_header_format = $this->allowed_headers[ $suffix ];
+
+        $headers = array(
+            'Accept' => $accept_header_format
+        );
+
+        $response = $default_api_service->get( '/dataset/export', $headers );
+
+        $response = $response->get_response();
+
+        if (
+            is_wp_error( $response ) ||
+            200 !== (int) $response['response']['code']
+        ) {
+            // Something is not working properly, so display error message.
+            wp_die( esc_html__( 'There was an error trying to connect to the server. Please try again later.', 'wordlift' ) );
+        }
+
+
+        // Get response body.
+        $body     = wp_remote_retrieve_body( $response );
+        $type     = wp_remote_retrieve_header( $response, 'content-type' );
+        $filename = 'dataset-' . date( 'Y-m-d-H-i-s' ) . '.' . $suffix;
+
+        // Add proper file headers.
+        header( "Content-Disposition: attachment; filename=$filename" );
+        header( "Content-Type: $type" );
+
+        /*
 		 * Echo the response body. As this is not HTML we can not escape it
 		 * and neither sanitize it, therefor turning off the linter notice.
 		 */
-		echo $body; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+        echo $body; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 
-		// Exit in both cases.
-		exit;
-	}
+        // Exit in both cases.
+        exit;
+    }
 }

src/admin/class-wordlift-notice-service.php

Indentation +189 added lines, -189 removed lines

@@ -7,194 +7,194 @@
  */
 class Wordlift_Notice_Service {
 
-	/**
-	 * The template used to display notices. The <em>notice dismissible</em> style classes make this notice dismissible
-	 * on the WordPress UI (via a small X button on the right side of the notice).
-	 *
-	 * @since 3.2.0
-	 */
-	const TEMPLATE = '<div class="wl-notice notice is-dismissible %s"><p>%s</p></div>';
-
-	/**
-	 * The standard WordPress <em>update</em> style class.
-	 *
-	 * @since 3.2.0
-	 */
-	const UPDATE = 'update';
-
-	/**
-	 * The standard WordPress <em>update-nag</em> style class.
-	 *
-	 * @since 3.2.0
-	 */
-	const UPDATE_NAG = 'update-nag';
-
-	/**
-	 * The standard WordPress <em>error</em> style class.
-	 *
-	 * @since 3.2.0
-	 */
-	const ERROR = 'error';
-
-	/**
-	 * A custom WordLift css style class used for WordLift suggestions.
-	 *
-	 * @since 3.3.0
-	 */
-	const SUGGESTION = 'wl-suggestion';
-
-	/**
-	 * The array of notices.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var array $notices The array of notices.
-	 */
-	private $notices = array();
-
-	/**
-	 * A singleton instance of the Notice service.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Notice_Service $instance A singleton instance of the Notice service.
-	 */
-	private static $instance;
-
-	/**
-	 * Create an instance of the Notice service.
-	 *
-	 * @since 3.2.0
-	 */
-	public function __construct() {
-		/**
-		 * Filter: wl_feature__enable__notices.
-		 *
-		 * @param bool whether the notices needs to be enabled or not.
-		 *
-		 * @return bool
-		 * @since 3.27.6
-		 */
-		if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
-			// Hook to be called when to display notices.
-			add_action( 'admin_notices', array( $this, 'admin_notices' ) );
-		}
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the singleton instance of the Notice service.
-	 *
-	 * @return \Wordlift_Notice_Service The singleton instance of the Notice service.
-	 * @since 3.2.0
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Add a notice.
-	 *
-	 * @param string $class The css class.
-	 * @param string $message The message.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function add( $class, $message ) {
-
-		$this->notices[] = sprintf( self::TEMPLATE, $class, $this->transform( $message ) );
-
-	}
-
-	/**
-	 * Add an update notice (message with a white background and a green left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function add_update( $message ) {
-
-		$this->add( self::UPDATE, $message );
-
-	}
-
-	/**
-	 * Add an update nag notice (message with a white background and a yellow left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function add_update_nag( $message ) {
-
-		$this->add( self::UPDATE_NAG, $message );
-
-	}
-
-	/**
-	 * Add an error notice (message with a white background and a red left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function add_error( $message ) {
-
-		$this->add( self::ERROR, $message );
-
-	}
-
-	/**
-	 * Add a suggestion notice (message with a white background and a WordLift brand colored left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.3.0
-	 *
-	 */
-	public function add_suggestion( $message ) {
-
-		$this->add( self::SUGGESTION, $message );
-
-	}
-
-	/**
-	 * Print out the notices when the admin_notices action is called.
-	 *
-	 * @since 3.2.0
-	 */
-	public function admin_notices() {
-
-		foreach ( $this->notices as $notice ) {
-			echo( $notice ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
-		}
-
-	}
-
-	/**
-	 * Transform message depending on message type. Return a string
-	 *
-	 * @param string $message The message.
-	 *
-	 * @since 3.3.0
-	 *
-	 */
-	private function transform( $message ) {
-
-		switch ( gettype( $message ) ) {
-			case 'array':
-				return implode('<br />', $message );
-			default:
-				return $message;
-		}
-
-	}
+    /**
+     * The template used to display notices. The <em>notice dismissible</em> style classes make this notice dismissible
+     * on the WordPress UI (via a small X button on the right side of the notice).
+     *
+     * @since 3.2.0
+     */
+    const TEMPLATE = '<div class="wl-notice notice is-dismissible %s"><p>%s</p></div>';
+
+    /**
+     * The standard WordPress <em>update</em> style class.
+     *
+     * @since 3.2.0
+     */
+    const UPDATE = 'update';
+
+    /**
+     * The standard WordPress <em>update-nag</em> style class.
+     *
+     * @since 3.2.0
+     */
+    const UPDATE_NAG = 'update-nag';
+
+    /**
+     * The standard WordPress <em>error</em> style class.
+     *
+     * @since 3.2.0
+     */
+    const ERROR = 'error';
+
+    /**
+     * A custom WordLift css style class used for WordLift suggestions.
+     *
+     * @since 3.3.0
+     */
+    const SUGGESTION = 'wl-suggestion';
+
+    /**
+     * The array of notices.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var array $notices The array of notices.
+     */
+    private $notices = array();
+
+    /**
+     * A singleton instance of the Notice service.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Notice_Service $instance A singleton instance of the Notice service.
+     */
+    private static $instance;
+
+    /**
+     * Create an instance of the Notice service.
+     *
+     * @since 3.2.0
+     */
+    public function __construct() {
+        /**
+         * Filter: wl_feature__enable__notices.
+         *
+         * @param bool whether the notices needs to be enabled or not.
+         *
+         * @return bool
+         * @since 3.27.6
+         */
+        if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+            // Hook to be called when to display notices.
+            add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+        }
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the singleton instance of the Notice service.
+     *
+     * @return \Wordlift_Notice_Service The singleton instance of the Notice service.
+     * @since 3.2.0
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Add a notice.
+     *
+     * @param string $class The css class.
+     * @param string $message The message.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function add( $class, $message ) {
+
+        $this->notices[] = sprintf( self::TEMPLATE, $class, $this->transform( $message ) );
+
+    }
+
+    /**
+     * Add an update notice (message with a white background and a green left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function add_update( $message ) {
+
+        $this->add( self::UPDATE, $message );
+
+    }
+
+    /**
+     * Add an update nag notice (message with a white background and a yellow left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function add_update_nag( $message ) {
+
+        $this->add( self::UPDATE_NAG, $message );
+
+    }
+
+    /**
+     * Add an error notice (message with a white background and a red left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function add_error( $message ) {
+
+        $this->add( self::ERROR, $message );
+
+    }
+
+    /**
+     * Add a suggestion notice (message with a white background and a WordLift brand colored left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.3.0
+     *
+     */
+    public function add_suggestion( $message ) {
+
+        $this->add( self::SUGGESTION, $message );
+
+    }
+
+    /**
+     * Print out the notices when the admin_notices action is called.
+     *
+     * @since 3.2.0
+     */
+    public function admin_notices() {
+
+        foreach ( $this->notices as $notice ) {
+            echo( $notice ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+        }
+
+    }
+
+    /**
+     * Transform message depending on message type. Return a string
+     *
+     * @param string $message The message.
+     *
+     * @since 3.3.0
+     *
+     */
+    private function transform( $message ) {
+
+        switch ( gettype( $message ) ) {
+            case 'array':
+                return implode('<br />', $message );
+            default:
+                return $message;
+        }
+
+    }
 
 }

Spacing +17 added lines, -17 removed lines

@@ -75,9 +75,9 @@ 
 		 * @return bool
 		 * @since 3.27.6
 		 */
-		if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+		if (apply_filters('wl_feature__enable__notices', true)) {
 			// Hook to be called when to display notices.
-			add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+			add_action('admin_notices', array($this, 'admin_notices'));
 		}
 		self::$instance = $this;
 
@@ -103,9 +103,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function add( $class, $message ) {
+	public function add($class, $message) {
 
-		$this->notices[] = sprintf( self::TEMPLATE, $class, $this->transform( $message ) );
+		$this->notices[] = sprintf(self::TEMPLATE, $class, $this->transform($message));
 
 	}
 
@@ -117,9 +117,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function add_update( $message ) {
+	public function add_update($message) {
 
-		$this->add( self::UPDATE, $message );
+		$this->add(self::UPDATE, $message);
 
 	}
 
@@ -131,9 +131,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function add_update_nag( $message ) {
+	public function add_update_nag($message) {
 
-		$this->add( self::UPDATE_NAG, $message );
+		$this->add(self::UPDATE_NAG, $message);
 
 	}
 
@@ -145,9 +145,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function add_error( $message ) {
+	public function add_error($message) {
 
-		$this->add( self::ERROR, $message );
+		$this->add(self::ERROR, $message);
 
 	}
 
@@ -159,9 +159,9 @@ 
 	 * @since 3.3.0
 	 *
 	 */
-	public function add_suggestion( $message ) {
+	public function add_suggestion($message) {
 
-		$this->add( self::SUGGESTION, $message );
+		$this->add(self::SUGGESTION, $message);
 
 	}
 
@@ -172,8 +172,8 @@ 
 	 */
 	public function admin_notices() {
 
-		foreach ( $this->notices as $notice ) {
-			echo( $notice ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+		foreach ($this->notices as $notice) {
+			echo($notice); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 		}
 
 	}
@@ -186,11 +186,11 @@ 
 	 * @since 3.3.0
 	 *
 	 */
-	private function transform( $message ) {
+	private function transform($message) {
 
-		switch ( gettype( $message ) ) {
+		switch (gettype($message)) {
 			case 'array':
-				return implode('<br />', $message );
+				return implode('<br />', $message);
 			default:
 				return $message;
 		}