Skip to content

Commit 4da9ddd

Browse files
committed
👌 IMPROVE: Moved the db-table code to it's own file
1 parent 2f9f7ea commit 4da9ddd

2 files changed

Lines changed: 30 additions & 23 deletions

File tree

admin/db-table.php

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
<?php
2+
3+
// If this file is called directly, abort.
4+
if ( ! defined( 'WPINC' ) ) {
5+
die;
6+
}
7+
8+
/**
9+
* Summary of benchpress_create_snapshots_table
10+
*
11+
* @since 1.0.0
12+
* @return void
13+
*/
14+
function benchpress_create_snapshots_table() {
15+
global $wpdb;
16+
$table_name = $wpdb->prefix . 'benchpress_snapshots';
17+
$charset_collate = $wpdb->get_charset_collate();
18+
19+
$sql = "CREATE TABLE $table_name (
20+
id mediumint(9) NOT NULL AUTO_INCREMENT,
21+
snapshot_data longtext NOT NULL,
22+
created_at datetime DEFAULT CURRENT_TIMESTAMP NOT NULL,
23+
PRIMARY KEY (id)
24+
) $charset_collate;";
25+
26+
require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
27+
dbDelta( $sql );
28+
}
29+
register_activation_hook( __FILE__, 'benchpress_create_snapshots_table' );

benchpress.php

Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -92,30 +92,8 @@ function benchpress_load_textdomain() {
9292
}
9393
add_action( 'plugins_loaded', 'benchpress_load_textdomain' );
9494

95-
/**
96-
* Summary of benchpress_create_snapshots_table
97-
*
98-
* @since 1.0.0
99-
* @return void
100-
*/
101-
function benchpress_create_snapshots_table() {
102-
global $wpdb;
103-
$table_name = $wpdb->prefix . 'benchpress_snapshots';
104-
$charset_collate = $wpdb->get_charset_collate();
105-
106-
$sql = "CREATE TABLE $table_name (
107-
id mediumint(9) NOT NULL AUTO_INCREMENT,
108-
snapshot_data longtext NOT NULL,
109-
created_at datetime DEFAULT CURRENT_TIMESTAMP NOT NULL,
110-
PRIMARY KEY (id)
111-
) $charset_collate;";
112-
113-
require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
114-
dbDelta( $sql );
115-
}
116-
register_activation_hook( __FILE__, 'benchpress_create_snapshots_table' );
117-
11895
// Include necessary files.
96+
require_once BENCHPRESS_PLUGIN_DIR . 'admin/db-table.php';
11997
require_once BENCHPRESS_PLUGIN_DIR . 'classes/BenchPress_Table.php';
12098
require_once BENCHPRESS_PLUGIN_DIR . 'classes/BenchPress_Snapshots_Table.php';
12199
require_once BENCHPRESS_PLUGIN_DIR . 'includes/helper-functions.php';

0 commit comments

Comments
 (0)