Passed
Pull Request — dev/2.3.0 (#196)
by Sudar
46:49 queued 31:53
created

TableManager::fetch_log_items()   F

Complexity

Conditions 24
Paths 144

Size

Total Lines 106
Code Lines 82

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 600

Importance

Changes 0
Metric Value
cc 24
eloc 82
nc 144
nop 3
dl 0
loc 106
ccs 0
cts 50
cp 0
crap 600
rs 3.8
c 0
b 0
f 0

How to fix   Long Method    Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php namespace EmailLog\Core\DB;
2
3
/**
4
 * Handle installation and db table creation.
5
 */
6
use EmailLog\Core\Loadie;
7
use EmailLog\Util;
8
9
defined( 'ABSPATH' ) || exit; // Exit if accessed directly.
10
11
/**
12
 * Helper class to create table.
13
 *
14
 * @since 2.0.0
15
 */
16
class TableManager implements Loadie {
17
18
	/* Database table name */
19
	const LOG_TABLE_NAME = 'email_log';
20
21
	/* Database option name */
22
	const DB_OPTION_NAME = 'email-log-db';
23
24
	/* Database version */
25
	const DB_VERSION = '0.2';
26
27
	/**
28
	 * Setup hooks.
29
	 */
30
	public function load() {
31
		add_action( 'wpmu_new_blog', array( $this, 'create_table_for_new_blog' ) );
32
33
		add_filter( 'wpmu_drop_tables', array( $this, 'delete_table_from_deleted_blog' ) );
34
35
		// Do any DB upgrades.
36
		$this->update_table_if_needed();
37
	}
38
39
	/**
40
	 * On plugin activation, create table if needed.
41
	 *
42
	 * @param bool $network_wide True if the plugin was network activated.
43
	 */
44
	public function on_activate( $network_wide ) {
45
		if ( is_multisite() && $network_wide ) {
46
			// Note: if there are more than 10,000 blogs or
47
			// if `wp_is_large_network` filter is set, then this may fail.
48
			$sites = get_sites();
49
50
			foreach ( $sites as $site ) {
51
				switch_to_blog( $site['blog_id'] );
52
				$this->create_table_if_needed();
53
				restore_current_blog();
54
			}
55
		} else {
56
			$this->create_table_if_needed();
57
		}
58
	}
59
60
	/**
61
	 * Create email log table when a new blog is created.
62
	 *
63
	 * @param int $blog_id Blog Id.
64
	 */
65
	public function create_table_for_new_blog( $blog_id ) {
66
		if ( is_plugin_active_for_network( 'email-log/email-log.php' ) ) {
67
			switch_to_blog( $blog_id );
68
			$this->create_table_if_needed();
69
			restore_current_blog();
70
		}
71
	}
72
73
	/**
74
	 * Add email log table to the list of tables deleted when a blog is deleted.
75
	 *
76
	 * @param array $tables List of tables to be deleted.
77
	 *
78
	 * @return string[] $tables Modified list of tables to be deleted.
79
	 */
80 1
	public function delete_table_from_deleted_blog( $tables ) {
81 1
		$tables[] = $this->get_log_table_name();
82
83 1
		return $tables;
84
	}
85
86
	/**
87
	 * Get email log table name.
88
	 *
89
	 * @return string Email Log Table name.
90
	 */
91 2
	public function get_log_table_name() {
92 2
		global $wpdb;
93
94 2
		return $wpdb->prefix . self::LOG_TABLE_NAME;
95
	}
96
97
	/**
98
	 * Insert log data into DB.
99
	 *
100
	 * @param array $data Data to be inserted.
101
	 */
102
	public function insert_log( $data ) {
103
		global $wpdb;
104
105
		$table_name = $this->get_log_table_name();
106
		$wpdb->insert( $table_name, $data );
107
	}
108
109
	/**
110
	 * Delete log entries by ids.
111
	 *
112
	 * @param string $ids Comma separated list of log ids.
113
	 *
114
	 * @return false|int Number of log entries that got deleted. False on failure.
115
	 */
116
	public function delete_logs( $ids ) {
117
		global $wpdb;
118
119
		$table_name = $this->get_log_table_name();
120
121
		// Can't use wpdb->prepare for the below query. If used it results in this bug // https://github.com/sudar/email-log/issues/13.
122
		$ids = esc_sql( $ids );
123
124
		return $wpdb->query( "DELETE FROM {$table_name} where id IN ( {$ids} )" ); //@codingStandardsIgnoreLine
125
	}
126
127
	/**
128
	 * Delete all log entries.
129
	 *
130
	 * @return false|int Number of log entries that got deleted. False on failure.
131
	 */
132
	public function delete_all_logs() {
133
		global $wpdb;
134
135
		$table_name = $this->get_log_table_name();
136
137
		return $wpdb->query( "DELETE FROM {$table_name}" ); //@codingStandardsIgnoreLine
138
	}
139
140
	/**
141
	 * Deletes Email Logs older than the specified interval.
142
	 *
143
	 * @param int $interval_in_days No. of days beyond which logs are to be deleted.
144
	 *
145
	 * @return int $deleted_rows_count  Count of rows deleted.
146
	 */
147
	public function delete_logs_older_than( $interval_in_days ) {
148
		global $wpdb;
149
		$table_name = $this->get_log_table_name();
150
151
		$query              = $wpdb->prepare( "DELETE FROM {$table_name} WHERE sent_date < DATE_SUB( CURDATE(), INTERVAL %d DAY )", $interval_in_days );
152
		$deleted_rows_count = $wpdb->query( $query );
153
154
		return $deleted_rows_count;
155
	}
156
157
	/**
158
	 * Fetch log item by ID.
159
	 *
160
	 * @param array $ids             Optional. Array of IDs of the log items to be retrieved.
161
	 * @param array $additional_args {
162
	 *                               Optional. Array of additional args.
163
	 *
164
	 * @type string $date_column_format MySQL date column format. Refer
165
	 *
166
	 * @link  https://dev.mysql.com/doc/refman/5.5/en/date-and-time-functions.html#function_date-format
167
	 * }
168
	 *
169
	 * @return array Log item(s).
170
	 */
171
	public function fetch_log_items_by_id( $ids = array(), $additional_args = array() ) {
172
		global $wpdb;
173
		$table_name = $this->get_log_table_name();
174
175
		$query = "SELECT * FROM {$table_name}";
176
177
		// When `date_column_format` exists, should replace the `$query` var.
178
		$date_column_format_key = 'date_column_format';
179
		if ( array_key_exists( $date_column_format_key, $additional_args ) && ! empty( $additional_args[ $date_column_format_key ] ) ) {
180
			$query = "SELECT DATE_FORMAT(sent_date, \"{$additional_args[ $date_column_format_key ]}\") as sent_date_custom, el.* FROM {$table_name} as el";
181
		}
182
183
		if ( ! empty( $ids ) ) {
184
			$ids = array_map( 'absint', $ids );
185
186
			// Can't use wpdb->prepare for the below query. If used it results in this bug https://github.com/sudar/email-log/issues/13.
187
			$ids_list = esc_sql( implode( ',', $ids ) );
188
189
			$query .= " where id IN ( {$ids_list} )";
190
		}
191
192
		return $wpdb->get_results( $query, 'ARRAY_A' ); //@codingStandardsIgnoreLine
193
	}
194
195
	/**
196
	 * Fetch log items.
197
	 *
198
	 * @param array $request         Request object.
199
	 * @param int   $per_page        Entries per page.
200
	 * @param int   $current_page_no Current page no.
201
	 *
202
	 * @return array Log entries and total items count.
203
	 */
204
	public function fetch_log_items( $request, $per_page, $current_page_no ) {
205
		global $wpdb;
206
		$table_name = $this->get_log_table_name();
207
208
		$query       = 'SELECT * FROM ' . $table_name;
209
		$count_query = 'SELECT count(*) FROM ' . $table_name;
210
		$query_cond  = '';
211
212
		if ( isset( $request['s'] ) && is_string( $request['s'] ) && $request['s'] !== '' ) {
213
			$search_term = trim( esc_sql( $request['s'] ) );
0 ignored issues
show
Bug introduced by
It seems like esc_sql($request['s']) can also be of type array; however, parameter $str of trim() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

213
			$search_term = trim( /** @scrutinizer ignore-type */ esc_sql( $request['s'] ) );
Loading history...
214
215
			if ( Util\is_advanced_search_term( $search_term ) ) {
0 ignored issues
show
Bug introduced by
The function is_advanced_search_term was not found. Maybe you did not declare it correctly or list all dependencies? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

215
			if ( /** @scrutinizer ignore-call */ Util\is_advanced_search_term( $search_term ) ) {
Loading history...
216
				$predicates = Util\get_advanced_search_term_predicates( $search_term );
0 ignored issues
show
Bug introduced by
The function get_advanced_search_term_predicates was not found. Maybe you did not declare it correctly or list all dependencies? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

216
				$predicates = /** @scrutinizer ignore-call */ Util\get_advanced_search_term_predicates( $search_term );
Loading history...
217
218
				foreach ( $predicates as $column => $email ) {
219
					switch ( $column ) {
220
						case 'to':
221
							$query_cond .= empty( $query_cond ) ? ' WHERE ' : ' AND ';
222
							$query_cond .= "to_email LIKE '%$email%'";
223
							break;
224
						case 'email':
225
							$query_cond .= empty( $query_cond ) ? ' WHERE ' : ' AND ';
226
							$query_cond .= ' ( '; /* Begin 1st */
227
							$query_cond .= " ( to_email LIKE '%$email%' OR subject LIKE '%$email%' ) "; /* Begin 2nd & End 2nd */
228
							$query_cond .= ' OR ';
229
							$query_cond .= ' ( '; /* Begin 3rd */
230
							$query_cond .= "headers <> ''";
231
							$query_cond .= ' AND ';
232
							$query_cond .= ' ( '; /* Begin 4th */
233
							$query_cond .= "headers REGEXP '[F|f]rom:.*$email' OR ";
234
							$query_cond .= "headers REGEXP '[CC|Cc|cc]:.*$email' OR ";
235
							$query_cond .= "headers REGEXP '[BCC|Bcc|bcc]:.*$email' OR ";
236
							$query_cond .= "headers REGEXP '[R|r]eply-[T|t]o:.*$email'";
237
							$query_cond .= ' ) '; /* End 4th */
238
							$query_cond .= ' ) '; /* End 3rd */
239
							$query_cond .= ' ) '; /* End 1st */
240
							break;
241
						case 'cc':
242
							$query_cond .= empty( $query_cond ) ? ' WHERE ' : ' AND ';
243
							$query_cond .= ' ( '; /* Begin 1st */
244
							$query_cond .= "headers <> ''";
245
							$query_cond .= ' AND ';
246
							$query_cond .= ' ( '; /* Begin 2nd */
247
							$query_cond .= "headers REGEXP '[CC|Cc|cc]:.*$email' ";
248
							$query_cond .= ' ) '; /* End 2nd */
249
							$query_cond .= ' ) '; /* End 1st */
250
							break;
251
						case 'bcc':
252
							$query_cond .= empty( $query_cond ) ? ' WHERE ' : ' AND ';
253
							$query_cond .= ' ( '; /* Begin 1st */
254
							$query_cond .= "headers <> ''";
255
							$query_cond .= ' AND ';
256
							$query_cond .= ' ( '; /* Begin 2nd */
257
							$query_cond .= "headers REGEXP '[BCC|Bcc|bcc]:.*$email' ";
258
							$query_cond .= ' ) '; /* End 2nd */
259
							$query_cond .= ' ) '; /* End 1st */
260
							break;
261
						case 'reply-to':
262
							$query_cond .= empty( $query_cond ) ? ' WHERE ' : ' AND ';
263
							$query_cond .= ' ( '; /* Begin 1st */
264
							$query_cond .= "headers <> ''";
265
							$query_cond .= ' AND ';
266
							$query_cond .= ' ( '; /* Begin 2nd */
267
							$query_cond .= "headers REGEXP '[R|r]eply-to:.*$email' ";
268
							$query_cond .= ' ) '; /* End 2nd */
269
							$query_cond .= ' ) '; /* End 1st */
270
							break;
271
					}
272
				}
273
			} else {
274
				$query_cond .= " WHERE ( to_email LIKE '%$search_term%' OR subject LIKE '%$search_term%' ) ";
275
			}
276
		}
277
278
		if ( isset( $request['d'] ) && $request['d'] !== '' ) {
279
			$search_date = trim( esc_sql( $request['d'] ) );
280
			if ( '' === $query_cond ) {
281
				$query_cond .= " WHERE sent_date BETWEEN '$search_date 00:00:00' AND '$search_date 23:59:59' ";
282
			} else {
283
				$query_cond .= " AND sent_date BETWEEN '$search_date 00:00:00' AND '$search_date 23:59:59' ";
284
			}
285
		}
286
287
		// Ordering parameters.
288
		$orderby = ! empty( $request['orderby'] ) ? esc_sql( $request['orderby'] ) : 'sent_date';
289
		$order   = ! empty( $request['order'] ) ? esc_sql( $request['order'] ) : 'DESC';
290
291
		if ( ! empty( $orderby ) & ! empty( $order ) ) {
0 ignored issues
show
Bug introduced by
Are you sure you want to use the bitwise & or did you mean &&?
Loading history...
292
			$query_cond .= ' ORDER BY ' . $orderby . ' ' . $order;
0 ignored issues
show
Bug introduced by
Are you sure $order of type array|string can be used in concatenation? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

292
			$query_cond .= ' ORDER BY ' . $orderby . ' ' . /** @scrutinizer ignore-type */ $order;
Loading history...
Bug introduced by
Are you sure $orderby of type array|string can be used in concatenation? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

292
			$query_cond .= ' ORDER BY ' . /** @scrutinizer ignore-type */ $orderby . ' ' . $order;
Loading history...
293
		}
294
295
		// Find total number of items.
296
		$count_query = $count_query . $query_cond;
297
		$total_items = $wpdb->get_var( $count_query );
298
299
		// Adjust the query to take pagination into account.
300
		if ( ! empty( $current_page_no ) && ! empty( $per_page ) ) {
301
			$offset = ( $current_page_no - 1 ) * $per_page;
302
			$query_cond .= ' LIMIT ' . (int) $offset . ',' . (int) $per_page;
303
		}
304
305
		// Fetch the items.
306
		$query = $query . $query_cond;
307
		$items = $wpdb->get_results( $query );
308
309
		return array( $items, $total_items );
310
	}
311
312
	/**
313
	 * Create email log table.
314
	 *
315
	 * @access private
316
	 *
317
	 * @global object $wpdb
318
	 */
319
	private function create_table_if_needed() {
320
		global $wpdb;
321
322
		$table_name = $this->get_log_table_name();
323
324
		if ( $wpdb->get_var( "show tables like '{$table_name}'" ) != $table_name ) {
325
326
			$sql = $this->get_create_table_query();
327
328
			require_once ABSPATH . 'wp-admin/includes/upgrade.php';
329
			dbDelta( $sql );
330
331
			add_option( self::DB_OPTION_NAME, self::DB_VERSION );
332
		}
333
	}
334
335
	/**
336
	 * Get the total number of email logs.
337
	 *
338
	 * @return int Total email log count
339
	 */
340
	public function get_logs_count() {
341
		global $wpdb;
342
343
		$query = 'SELECT count(*) FROM ' . $this->get_log_table_name();
344
345
		return $wpdb->get_var( $query );
346
	}
347
348
	/**
349
	 * Fetches the log item by the item data.
350
	 *
351
	 * Use this method to get the log item when the error instance only returns the log item data.
352
	 *
353
	 * @param array $data Array of Email information. {
354
	 *
355
	 * @type array|string to
356
	 * @type string       subject
357
	 * @type string       message
358
	 * @type array|string headers
359
	 * @type array|string attachments
360
	 *                    }
361
	 *
362
	 * @return int
363
	 */
364
	public function fetch_log_item_by_item_data( $data ) {
365
		if ( empty( $data ) || ! is_array( $data ) ) {
366
			return 0;
367
		}
368
369
		global $wpdb;
370
		$table_name = $this->get_log_table_name();
371
372
		$query      = "SELECT ID FROM {$table_name}";
373
		$query_cond = '';
374
		$where      = array();
375
376
		// Execute the following `if` conditions only when $data is array.
377
		if ( array_key_exists( 'to', $data ) ) {
378
			// Since the value is stored as CSV in DB, convert the values from error data to CSV to compare.
379
			$to_email = Util\join_array_elements_with_delimiter( $data['to'] );
0 ignored issues
show
Bug introduced by
The function join_array_elements_with_delimiter was not found. Maybe you did not declare it correctly or list all dependencies? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

379
			$to_email = /** @scrutinizer ignore-call */ Util\join_array_elements_with_delimiter( $data['to'] );
Loading history...
380
381
			$to_email = trim( esc_sql( $to_email ) );
0 ignored issues
show
Bug introduced by
It seems like esc_sql($to_email) can also be of type array; however, parameter $str of trim() does only seem to accept string, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

381
			$to_email = trim( /** @scrutinizer ignore-type */ esc_sql( $to_email ) );
Loading history...
382
			$where[]  = "to_email = '$to_email'";
383
		}
384
385
		if ( array_key_exists( 'subject', $data ) ) {
386
			$subject = trim( esc_sql( $data['subject'] ) );
387
			$where[] = "subject = '$subject'";
388
		}
389
390
		if ( array_key_exists( 'attachments', $data ) ) {
391
			if ( is_array( $data['attachments'] ) ) {
392
				$attachments = count( $data['attachments'] ) > 0 ? 'true' : 'false';
393
			} else {
394
				$attachments = empty( $data['attachments'] ) ? 'false' : 'true';
395
			}
396
			$attachments = trim( esc_sql( $attachments ) );
397
			$where[]     = "attachments = '$attachments'";
398
		}
399
400
		foreach ( $where as $index => $value ) {
401
			$query_cond .= 0 === $index ? ' WHERE ' : ' AND ';
402
			$query_cond .= $value;
403
		}
404
405
		// Get only the latest logged item when multiple rows match.
406
		$query_cond .= ' ORDER BY id DESC LIMIT 1';
407
408
		$query = $query . $query_cond;
409
410
		return absint( $wpdb->get_var( $query ) );
411
	}
412
413
	/**
414
	 * Sets email sent status as failed for the given log item.
415
	 *
416
	 * @since 2.3.0
417
	 *
418
	 * @param int $log_item_id ID of the log item whose email sent status should be set to failed.
419
	 */
420
	public function set_log_item_fail_status_by_id( $log_item_id ) {
421
		global $wpdb;
422
		$table_name = $this->get_log_table_name();
423
424
		$wpdb->update(
425
			$table_name,
426
			array( 'result' => '0' ),
427
			array( 'ID'     => $log_item_id ),
428
			array( '%d' ),
429
			array( '%d' )
430
		);
431
	}
432
433
	/**
434
	 * Updates the DB schema.
435
	 *
436
	 * Adds new columns to the Database as of v0.2.
437
	 *
438
	 * @since 2.3.0
439
	 */
440
	private function update_table_if_needed() {
441
		$existing_db_version = get_option( self::DB_OPTION_NAME, false );
442
		$updated_db_version  = self::DB_VERSION;
443
444
		// Bail out when the DB version is `0.1` or equals to self::DB_VERSION
445
		if ( ! $existing_db_version || $existing_db_version !== '0.1' || $existing_db_version === $updated_db_version ) {
446
			return;
447
		}
448
449
		$sql = $this->get_create_table_query();
450
451
		require_once ABSPATH . 'wp-admin/includes/upgrade.php';
452
		dbDelta( $sql );
453
454
		update_option( self::DB_OPTION_NAME, self::DB_VERSION );
455
	}
456
457
	/**
458
	 * Gets the Create Table query.
459
	 *
460
	 * @since 2.3.0
461
	 *
462
	 * @return string
463
	 */
464
	private function get_create_table_query() {
465
		global $wpdb;
466
		$table_name      = $this->get_log_table_name();
467
		$charset_collate = $wpdb->get_charset_collate();
468
469
		$sql = 'CREATE TABLE ' . $table_name . ' (
470
				id mediumint(9) NOT NULL AUTO_INCREMENT,
471
				to_email VARCHAR(500) NOT NULL,
472
				subject VARCHAR(500) NOT NULL,
473
				message TEXT NOT NULL,
474
				headers TEXT NOT NULL,
475
				attachments TEXT NOT NULL,
476
				sent_date timestamp NOT NULL,
477
				attachment_name VARCHAR(1000),
478
				ip_address VARCHAR(15),
479
				result TINYINT(1),
480
				PRIMARY KEY  (id)
481
			) ' . $charset_collate . ';';
482
483
		return $sql;
484
	}
485
}
486