nirjharlo / wp-plugin-framework

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);
 					}
 				}
 			}