nirjharlo / wp-plugin-framework

plugin/Src/Widget.php

Spacing +14 added lines, -14 removed lines

@@ -3,7 +3,7 @@ 
 
 use WP_WIDGET as WP_WIDGET;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * Widget class
@@ -11,7 +11,7 @@ 
  * @author     Nirjhar Lo
  * @package    wp-plugin-framework
  */
-if ( ! class_exists( 'Widget' ) ) {
+if ( ! class_exists('Widget')) {
 
 	final class Widget extends WP_WIDGET {
 
@@ -25,9 +25,9 @@ 
 
 			$widget_ops = array(
 							'classname' => 'plugin_widget',
-							'description' => __( 'Plugin widget', 'textdomain' ),
+							'description' => __('Plugin widget', 'textdomain'),
 							);
-			parent::__construct( 'my_widget_id', __( 'Plugin widget', 'textdomain' ), $widget_ops );
+			parent::__construct('my_widget_id', __('Plugin widget', 'textdomain'), $widget_ops);
 		}
 
 
@@ -39,13 +39,13 @@ 
 		 *
 		 * @return Html
 		 */
-		public function widget( $args, $instance ) {
+		public function widget($args, $instance) {
 
 			echo $args['before_widget'];
-			if ( ! empty( $instance['title'] ) ) {
-				echo $args['before_title'] . apply_filters( 'widget_title', $instance['title'] ) . $args['after_title'];
+			if ( ! empty($instance['title'])) {
+				echo $args['before_title'] . apply_filters('widget_title', $instance['title']) . $args['after_title'];
 			}
-			echo esc_html__( 'Hello, World!', 'textdomain' );
+			echo esc_html__('Hello, World!', 'textdomain');
 			echo $args['after_widget'];
 		}
 
@@ -57,12 +57,12 @@ 
 		 *
 		 * @return Html
 		 */
-		public function form( $instance ) {
+		public function form($instance) {
 
-			$title = ! empty( $instance['title'] ) ? $instance['title'] : esc_html__( 'Internal Link Master', 'textdomain' ); ?>
+			$title = ! empty($instance['title']) ? $instance['title'] : esc_html__('Internal Link Master', 'textdomain'); ?>
 			<p>
-				<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php esc_attr_e( 'Title:', 'textdomain' ); ?></label>
-				<input class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>" type="text" value="<?php echo esc_attr( $title ); ?>">
+				<label for="<?php echo esc_attr($this->get_field_id('title')); ?>"><?php esc_attr_e('Title:', 'textdomain'); ?></label>
+				<input class="widefat" id="<?php echo esc_attr($this->get_field_id('title')); ?>" name="<?php echo esc_attr($this->get_field_name('title')); ?>" type="text" value="<?php echo esc_attr($title); ?>">
 			</p>
 		<?php
 		}
@@ -76,10 +76,10 @@ 
 		 *
 		 * @return Array
 		 */
-		public function update( $new_instance, $old_instance ) {
+		public function update($new_instance, $old_instance) {
 
 			$instance = array();
-			$instance['title'] = ( ! empty( $new_instance['title'] ) ? sanitize_text_field( $new_instance['title'] ) : '' );
+			$instance['title'] = ( ! empty($new_instance['title']) ? sanitize_text_field($new_instance['title']) : '');
 			return $instance;
 		}
 	}

Braces +3 added lines, -1 removed lines

@@ -3,7 +3,9 @@
 
 use WP_WIDGET as WP_WIDGET;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Widget class

plugin/Src/Db.php

Indentation +3 added lines, -3 removed lines

@@ -44,13 +44,13 @@
 			global $wpdb;
 			$wpdb->hide_errors();
 			$collate = "";
-		    if ( $wpdb->has_cap( 'collation' ) ) {
+			if ( $wpdb->has_cap( 'collation' ) ) {
 				if( ! empty($wpdb->charset ) )
 					$collate .= "DEFAULT CHARACTER SET $wpdb->charset";
 				if( ! empty($wpdb->collate ) )
 					$collate .= " COLLATE $wpdb->collate";
-		    }
-    		require_once( $this->up_path );
+			}
+			require_once( $this->up_path );
 			return $collate;
 		}
 

Spacing +13 added lines, -13 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 namespace NirjharLo\WP_Plugin_Framework\Src;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * DB installation class
@@ -9,7 +9,7 @@ 
  * @author     Nirjhar Lo
  * @package    wp-plugin-framework
  */
-if ( ! class_exists( 'Db' ) ) {
+if ( ! class_exists('Db')) {
 
 	class Db {
 
@@ -46,10 +46,10 @@ 
 			global $wpdb;
 			$wpdb->hide_errors();
 			$this->table_name = $wpdb->prefix . $this->table;
-			update_option( '_plugin_db_exist', 0 );
-			if ( $wpdb->get_var( "SHOW TABLES LIKE '$this->table_name'" ) != $this->table_name ) {
-				$execute_sql = $this->execute( $this->table_name, $this->collate(), $this->sql );
-				dbDelta( $execute_sql );
+			update_option('_plugin_db_exist', 0);
+			if ($wpdb->get_var("SHOW TABLES LIKE '$this->table_name'") != $this->table_name) {
+				$execute_sql = $this->execute($this->table_name, $this->collate(), $this->sql);
+				dbDelta($execute_sql);
 			}
 		}
 
@@ -64,13 +64,13 @@ 
 			global $wpdb;
 			$wpdb->hide_errors();
 			$collate = "";
-		    if ( $wpdb->has_cap( 'collation' ) ) {
-				if( ! empty($wpdb->charset ) )
+		    if ($wpdb->has_cap('collation')) {
+				if ( ! empty($wpdb->charset))
 					$collate .= "DEFAULT CHARACTER SET $wpdb->charset";
-				if( ! empty($wpdb->collate ) )
+				if ( ! empty($wpdb->collate))
 					$collate .= " COLLATE $wpdb->collate";
 		    }
-    		require_once( $this->up_path );
+    		require_once($this->up_path);
 			return $collate;
 		}
 
@@ -84,7 +84,7 @@ 
 		 *
 		 * @return String
 		 */
-		public function execute( $table_name, $collate, $sql ) {
+		public function execute($table_name, $collate, $sql) {
 
 			return "CREATE TABLE $table_name ( $sql ) $collate;";
 		}
@@ -100,9 +100,9 @@ 
 			global $wpdb;
 
 			$this->table_name = $wpdb->prefix . $this->table;
-			if ( $wpdb->get_var( "SHOW TABLES LIKE '$this->table_name'" ) == $this->table_name ) {
+			if ($wpdb->get_var("SHOW TABLES LIKE '$this->table_name'") == $this->table_name) {
 
-				update_option( '_plugin_db_exist', 1 );
+				update_option('_plugin_db_exist', 1);
 			}
 		}
 	}

Braces +9 added lines, -5 removed lines

@@ -1,7 +1,9 @@ 
 <?php
 namespace NirjharLo\WP_Plugin_Framework\Src;
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * DB installation class
@@ -65,10 +67,12 @@ 
 			$wpdb->hide_errors();
 			$collate = "";
 		    if ( $wpdb->has_cap( 'collation' ) ) {
-				if( ! empty($wpdb->charset ) )
-					$collate .= "DEFAULT CHARACTER SET $wpdb->charset";
-				if( ! empty($wpdb->collate ) )
-					$collate .= " COLLATE $wpdb->collate";
+				if( ! empty($wpdb->charset ) ) {
+									$collate .= "DEFAULT CHARACTER SET $wpdb->charset";
+				}
+				if( ! empty($wpdb->collate ) ) {
+									$collate .= " COLLATE $wpdb->collate";
+				}
 		    }
     		require_once( $this->up_path );
 			return $collate;

plugin/Lib/Table.php

Indentation +6 added lines, -6 removed lines

@@ -14,8 +14,8 @@ 
 if ( ! class_exists( 'Table' ) ) {
 
 	if ( ! class_exists( 'WP_List_Table' ) ) {
-    	require_once( ABSPATH . 'wp-admin/includes/screen.php' );
-    	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+		require_once( ABSPATH . 'wp-admin/includes/screen.php' );
+		require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
 	}
 
 	final class Table extends WP_List_Table {
@@ -230,10 +230,10 @@ 
 			$current_page = $this->get_pagenum();
 			$total_items  = self::record_count();
 			$this->_column_headers = [
-    			$this->get_columns(),
-    			[], // hidden columns
-    			$this->get_sortable_columns(),
-    			$this->get_primary_column_name(),
+				$this->get_columns(),
+				[], // hidden columns
+				$this->get_sortable_columns(),
+				$this->get_primary_column_name(),
 			];
 			$this->set_pagination_args( array(
 				'total_items' => $total_items,

Spacing +54 added lines, -54 removed lines

@@ -11,11 +11,11 @@ 
  * @author     Nirjhar Lo
  * @package    wp-plugin-framework
  */
-if ( ! class_exists( 'Table' ) ) {
+if ( ! class_exists('Table')) {
 
-	if ( ! class_exists( 'WP_List_Table' ) ) {
-    	require_once( ABSPATH . 'wp-admin/includes/screen.php' );
-    	require_once( ABSPATH . 'wp-admin/includes/class-wp-list-table.php' );
+	if ( ! class_exists('WP_List_Table')) {
+    	require_once(ABSPATH . 'wp-admin/includes/screen.php');
+    	require_once(ABSPATH . 'wp-admin/includes/class-wp-list-table.php');
 	}
 
 	final class Table extends WP_List_Table {
@@ -28,11 +28,11 @@ 
 		 */
 		public function __construct() {
 
-			parent::__construct( [
-				'singular' => __( 'Name', 'textdomain' ),
-				'plural'   => __( 'Names', 'textdomain' ),
+			parent::__construct([
+				'singular' => __('Name', 'textdomain'),
+				'plural'   => __('Names', 'textdomain'),
 				'ajax'     => false,
-			] );
+			]);
 		}
 
 
@@ -41,27 +41,27 @@ 
 		 *
 		 * @return Array
 		 */
-		public static function get_Table( $per_page = 5, $page_number = 1 ) {
+		public static function get_Table($per_page = 5, $page_number = 1) {
 
 			global $wpdb;
 
 			//Take pivotal from URL
-			$link = ( isset( $_GET['link'] ) ? $_GET['link'] : 'link' );
+			$link = (isset($_GET['link']) ? $_GET['link'] : 'link');
 
 			//Build the db query base
 			$sql = "SELECT * FROM {$wpdb->prefix}wordpress_table";
 			$sql .= " QUERIES with $link'";
 
 			//Set filters in the query using $_REQUEST
-			if ( ! empty( $_REQUEST['orderby'] ) ) {
-				$sql .= ' ORDER BY ' . esc_sql( $_REQUEST['orderby'] );
-				$sql .= ! empty( $_REQUEST['order'] ) ? ' ' . esc_sql( $_REQUEST['order'] ) : ' ASC';
+			if ( ! empty($_REQUEST['orderby'])) {
+				$sql .= ' ORDER BY ' . esc_sql($_REQUEST['orderby']);
+				$sql .= ! empty($_REQUEST['order']) ? ' ' . esc_sql($_REQUEST['order']) : ' ASC';
 			}
 			$sql .= " LIMIT $per_page";
-			$sql .= ' OFFSET ' . ( $page_number - 1 ) * $per_page;
+			$sql .= ' OFFSET ' . ($page_number - 1) * $per_page;
 
 			//get the data from database
-			$result = $wpdb->get_results( $sql, 'ARRAY_A' );
+			$result = $wpdb->get_results($sql, 'ARRAY_A');
 
 			return $result;
 		}
@@ -72,11 +72,11 @@ 
 		 *
 		 * @return Void
 		 */
-		public static function delete_url( $id ) {
+		public static function delete_url($id) {
 
 			global $wpdb;
 
-			$wpdb->delete( "{$wpdb->prefix}wordpress_table", array( 'ID' => $id ), array( '%s' ) );
+			$wpdb->delete("{$wpdb->prefix}wordpress_table", array('ID' => $id), array('%s'));
 		}
 
 
@@ -87,7 +87,7 @@ 
 		 */
 		public function no_items() {
 
-			_e( 'No Items Added yet.', 'textDomain' );
+			_e('No Items Added yet.', 'textDomain');
 		}
 
 
@@ -101,13 +101,13 @@ 
 			global $wpdb;
 
 			//Take pivotal from URL
-			$link = ( isset( $_GET['link'] ) ? $_GET['link'] : 'link' );
+			$link = (isset($_GET['link']) ? $_GET['link'] : 'link');
 
 			//Build the db query base
 			$sql = "SELECT COUNT(*) FROM {$wpdb->prefix}wordpress_table";
 			$sql .= " QUERIES with $link'";
 
-			return $wpdb->get_var( $sql );
+			return $wpdb->get_var($sql);
 		}
 
 
@@ -116,17 +116,17 @@ 
 		 *
 		 * @return Html
 		 */
-		public function column_name( $item ) {
+		public function column_name($item) {
 
-			$delete_nonce = wp_create_nonce( 'delete_url' );
-			$title = sprintf( '<strong>%s</strong>', $item['item_name'] );
+			$delete_nonce = wp_create_nonce('delete_url');
+			$title = sprintf('<strong>%s</strong>', $item['item_name']);
 
 			//Change the page instruction where you want to show it
 			$actions = array(
-					'delete' => sprintf( '<a href="?page=%s&action=%s&instruction=%s&_wpnonce=%s">%s</a>', esc_attr( $_REQUEST['page'] ), 'delete', absint( $item['ID'] ), $delete_nonce, __( 'Delete', 'textdomain' ) )
+					'delete' => sprintf('<a href="?page=%s&action=%s&instruction=%s&_wpnonce=%s">%s</a>', esc_attr($_REQUEST['page']), 'delete', absint($item['ID']), $delete_nonce, __('Delete', 'textdomain'))
 					);
 
-			return $title . $this->row_actions( $actions );
+			return $title . $this->row_actions($actions);
 		}
 
 
@@ -135,22 +135,22 @@ 
 		 *
 		 * @return Html
 		 */
-		public function column_default( $item, $column_name ) {
+		public function column_default($item, $column_name) {
 
-			switch ( $column_name ) {
+			switch ($column_name) {
 
 				case 'name':
 					//This is the first column
-					return $this->column_name( $item );
+					return $this->column_name($item);
 				case 'caseOne':
 				case 'caseTwo':
 				case 'caseThree':
-					return $item[ $column_name ];
+					return $item[$column_name];
 
 				default:
 
 					//Show the whole array for troubleshooting purposes
-					return print_r( $item, true );
+					return print_r($item, true);
 			}
 		}
 
@@ -160,9 +160,9 @@ 
 		 *
 		 * @return Html
 		 */
-		public function column_cb( $item ) {
+		public function column_cb($item) {
 
-			return sprintf( '<input type="checkbox" name="bulk-select[]" value="%s" />', $item['ID'] );
+			return sprintf('<input type="checkbox" name="bulk-select[]" value="%s" />', $item['ID']);
 		}
 
 
@@ -175,10 +175,10 @@ 
 
 			$columns = array(
 							'cb'		=> '<input type="checkbox" />',
-							'name'	=> __( 'Name', 'textdomain' ),
-							'caseOne'	=> __( 'Case One', 'textdomain' ),
-							'caseTwo'	=> __( 'Case Two', 'textdomain' ),
-							'caseThree'	=> __( 'Case Three', 'textdomain' ),
+							'name'	=> __('Name', 'textdomain'),
+							'caseOne'	=> __('Case One', 'textdomain'),
+							'caseTwo'	=> __('Case Two', 'textdomain'),
+							'caseThree'	=> __('Case Three', 'textdomain'),
 						);
 
 			return $columns;
@@ -193,9 +193,9 @@ 
 		public function get_sortable_columns() {
 
 			$sortable_columns = array(
-				'name' => array( 'name', true ),
-				'caseOne' => array( 'caseOne', false ),
-				'caseTwo' => array( 'caseTwo', false ),
+				'name' => array('name', true),
+				'caseOne' => array('caseOne', false),
+				'caseTwo' => array('caseTwo', false),
 			);
 
 			return $sortable_columns;
@@ -209,7 +209,7 @@ 
 		 */
 		public function get_bulk_actions() {
 
-			$actions = array( 'bulk-delete' => 'Delete'	);
+			$actions = array('bulk-delete' => 'Delete');
 
 			return $actions;
 		}
@@ -226,7 +226,7 @@ 
 
 			/** Process bulk action */
 			$this->process_bulk_action();
-			$per_page     = $this->get_items_per_page( 'option_name_per_page', 5 );
+			$per_page     = $this->get_items_per_page('option_name_per_page', 5);
 			$current_page = $this->get_pagenum();
 			$total_items  = self::record_count();
 			$this->_column_headers = [
@@ -235,12 +235,12 @@ 
     			$this->get_sortable_columns(),
     			$this->get_primary_column_name(),
 			];
-			$this->set_pagination_args( array(
+			$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $per_page,
-			) );
+			));
 
-			$this->items = self::get_Table( $per_page, $current_page );
+			$this->items = self::get_Table($per_page, $current_page);
 		}
 
 
@@ -252,24 +252,24 @@ 
 		public function process_bulk_action() {
 
 			//Detect when a bulk action is being triggered...
-			if ( 'delete' === $this->current_action() ) {
+			if ('delete' === $this->current_action()) {
 
 				//In our file that handles the request, verify the nonce.
-				$nonce = esc_attr( $_REQUEST['_wpnonce'] );
+				$nonce = esc_attr($_REQUEST['_wpnonce']);
 
-				if ( ! wp_verify_nonce( $nonce, 'delete_url' ) ) {
-					die( 'Go get a live script kiddies' );
+				if ( ! wp_verify_nonce($nonce, 'delete_url')) {
+					die('Go get a live script kiddies');
 				} else {
-					self::delete_url( absint( $_GET['instruction'] ) ); //Remember the instruction param from column_name method
+					self::delete_url(absint($_GET['instruction'])); //Remember the instruction param from column_name method
 				}
 			}
 
 			//If the delete bulk action is triggered
-			if ( isset( $_POST['action'] ) ) {
-				if ( ( isset( $_POST['action'] ) && $_POST['action'] == 'bulk-delete' ) ) {
-					$delete_ids = esc_sql( $_POST['bulk-select'] );
-					foreach ( $delete_ids as $id ) {
-						self::delete_url( $id );
+			if (isset($_POST['action'])) {
+				if ((isset($_POST['action']) && $_POST['action'] == 'bulk-delete')) {
+					$delete_ids = esc_sql($_POST['bulk-select']);
+					foreach ($delete_ids as $id) {
+						self::delete_url($id);
 					}
 				}
 			}