Give_DB_Log_Meta::create_table() - Code Metrics - Inspection of "Release/2.0" - WordImpress/Give - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#2668)

by Devin

created 2018-01-17 19:02 UTC

Give_DB_Log_Meta::create_table() A

↳ Parent: Give_DB_Log_Meta

Complexity

Conditions	1
Paths	1

Size

Total Lines	19
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	8
nc	1
nop	0
dl	0
loc	19
rs	9.4285
c	0
b	0
f	0

<?php
/**
 * Logs Meta DB class
 *
 * @package     Give
 * @subpackage  Classes/DB Log Meta
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       2.0
 */

// Exit if accessed directly.
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}

/**
 * Class Give_DB_Log_Meta
 *
 * This class is for interacting with the log meta database table.
 *
 * @since 2.0
 */
class Give_DB_Log_Meta extends Give_DB_Meta {
	/**
	 * Meta supports.
	 *
	 * @since  2.0
	 * @access protected
	 * @var array
	 */
	protected $supports = array();

	/**
	 * Meta type
	 *
	 * @since  2.0
	 * @access protected
	 * @var bool
	 */
	protected $meta_type = 'log';

	/**
	 * Give_DB_Log_Meta constructor.
	 *
	 * @access  public
	 * @since   2.0
	 */
	public function __construct() {
		/* @var WPDB $wpdb */
		global $wpdb;

		$wpdb->logmeta     = $this->table_name = $wpdb->prefix . 'give_logmeta';
		$this->primary_key = 'meta_id';
		$this->version     = '1.0';

		$this->register_table();

		parent::__construct();
	}

	/**
	 * Get table columns and data types.
	 *
	 * @access  public
	 * @since   2.0
	 *
	 * @return  array  Columns and formats.
	 */
	public function get_columns() {
		return array(
			'meta_id'    => '%d',
			'log_id'     => '%d',
			'meta_key'   => '%s',

			'meta_value' => '%s',

		);
	}

	/**
	 * Delete all log meta
	 *
	 * @since  2.0
	 * @access public
	 *
	 * @param int $log_id
	 *
	 * @return bool
	 */
	public function delete_row( $log_id = 0 ) {

		/* @var WPDB $wpdb */
		global $wpdb;

		// Row ID must be positive integer
		$log_id = absint( $log_id );

		if ( empty( $log_id ) ) {
			return false;
		}

		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE log_id = %d", $log_id ) ) ) {

			return false;
		}

		return true;
	}

	/**
	 * Create the table
	 *
	 * @access public
	 * @since  2.0
	 *
	 * @return void
	 */
	public function create_table() {
		global $wpdb;
		$charset_collate = $wpdb->get_charset_collate();

		$sql = "CREATE TABLE {$wpdb->logmeta} (
			meta_id bigint(20) NOT NULL AUTO_INCREMENT,
			log_id bigint(20) NOT NULL,
			meta_key varchar(255) DEFAULT NULL,
			meta_value longtext,
			PRIMARY KEY  (meta_id),
			KEY log_id (log_id),
			KEY meta_key (meta_key)
			) {$charset_collate};";

		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
		dbDelta( $sql );

		update_option( $this->table_name . '_db_version', $this->version );
	}

	/**
	 * Check if current id is valid
	 *
	 * @since  2.0
	 * @access protected
	 *
	 * @param $ID
	 *
	 * @return bool
	 */
	protected function is_valid_post_type( $ID ) {
		return $ID && true;
	}
}


1		<?php
2		/**
3		* Logs Meta DB class
4		*
5		* @package Give
6		* @subpackage Classes/DB Log Meta
7		* @copyright Copyright (c) 2016, WordImpress
8		* @license https://opensource.org/licenses/gpl-license GNU Public License
9		* @since 2.0
10		*/
11
12		// Exit if accessed directly.
13		if ( ! defined( 'ABSPATH' ) ) {
14		exit;
15		}
16
17		/**
18		* Class Give_DB_Log_Meta
19		*
20		* This class is for interacting with the log meta database table.
21		*
22		* @since 2.0
23		*/
24		class Give_DB_Log_Meta extends Give_DB_Meta {
25		/**
26		* Meta supports.
27		*
28		* @since 2.0
29		* @access protected
30		* @var array
31		*/
32		protected $supports = array();
33
34		/**
35		* Meta type
36		*
37		* @since 2.0
38		* @access protected
39		* @var bool
40		*/
41		protected $meta_type = 'log';
42
43		/**
44		* Give_DB_Log_Meta constructor.
45		*
46		* @access public
47		* @since 2.0
48		*/
49		public function __construct() {
50		/* @var WPDB $wpdb */
51		global $wpdb;
52
53		$wpdb->logmeta = $this->table_name = $wpdb->prefix . 'give_logmeta';
54		$this->primary_key = 'meta_id';
55		$this->version = '1.0';
56
57		$this->register_table();
58
59		parent::__construct();
60		}
61
62		/**
63		* Get table columns and data types.
64		*
65		* @access public
66		* @since 2.0
67		*
68		* @return array Columns and formats.
69		*/
70		public function get_columns() {
71		return array(
72		'meta_id' => '%d',
73		'log_id' => '%d',
74		'meta_key' => '%s',
		0 ignored issues – show introduced 2018-01-17 19:08 UTC by Report Bug Copy Issue Report Detected usage of meta_key, possible slow query. Loading history...
75		'meta_value' => '%s',
		0 ignored issues – show introduced 2018-01-17 19:08 UTC by Report Bug Copy Issue Report Detected usage of meta_value, possible slow query. Loading history...
76		);
77		}
78
79		/**
80		* Delete all log meta
81		*
82		* @since 2.0
83		* @access public
84		*
85		* @param int $log_id
86		*
87		* @return bool
88		*/
89	View Code Duplication	public function delete_row( $log_id = 0 ) {
		0 ignored issues – show Duplication introduced 2018-01-17 19:08 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
90		/* @var WPDB $wpdb */
91		global $wpdb;
92
93		// Row ID must be positive integer
94		$log_id = absint( $log_id );
95
96		if ( empty( $log_id ) ) {
97		return false;
98		}
99
100		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE log_id = %d", $log_id ) ) ) {
		0 ignored issues – show introduced 2018-01-17 19:08 UTC by Report Bug Copy Issue Report Usage of a direct database call is discouraged. Loading history... introduced 2018-01-17 19:08 UTC by Report Bug Copy Issue Report Usage of a direct database call without caching is prohibited. Use wp_cache_get / wp_cache_set. Loading history...
101		return false;
102		}
103
104		return true;
105		}
106
107		/**
108		* Create the table
109		*
110		* @access public
111		* @since 2.0
112		*
113		* @return void
114		*/
115		public function create_table() {
116		global $wpdb;
117		$charset_collate = $wpdb->get_charset_collate();
118
119		$sql = "CREATE TABLE {$wpdb->logmeta} (
120		meta_id bigint(20) NOT NULL AUTO_INCREMENT,
121		log_id bigint(20) NOT NULL,
122		meta_key varchar(255) DEFAULT NULL,
123		meta_value longtext,
124		PRIMARY KEY (meta_id),
125		KEY log_id (log_id),
126		KEY meta_key (meta_key)
127		) {$charset_collate};";
128
129		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
130		dbDelta( $sql );
131
132		update_option( $this->table_name . '_db_version', $this->version );
133		}
134
135		/**
136		* Check if current id is valid
137		*
138		* @since 2.0
139		* @access protected
140		*
141		* @param $ID
142		*
143		* @return bool
144		*/
145		protected function is_valid_post_type( $ID ) {
146		return $ID && true;
147		}
148		}
149

WordImpress / Give

Pull Request — master (#2668)

Give_DB_Log_Meta::create_table() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like