Skip to content

Fix wp checks +semver: major #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
guibranco merged 5 commits into from
Apr 23, 2026
Merged

Fix wp checks +semver: major #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
6fb9b61
[FEATURE] Fix WP Check warnings
guistracini-outsurance-ie Apr 23, 2026
d87b5d8
Fix files
guistracini-outsurance-ie Apr 23, 2026
a203a4e
Merge branch 'main' into feature/fix-wp-checks
gstraccini[bot] Apr 23, 2026
3ccebb3
Fix WP Checks warnings
guistracini-outsurance-ie Apr 23, 2026
73457e9
Merge branch 'feature/fix-wp-checks' of github.com:guibranco/ipquery-…
guistracini-outsurance-ie Apr 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
82 changes: 41 additions & 41 deletions admin/views/dashboard.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,45 +15,45 @@
<hr class="wp-header-end">

<?php
$total_visits = IpQuery_DB::get_total_visits();
$unique_ips = IpQuery_DB::get_unique_ips();
$risk_counts = IpQuery_DB::get_risk_counts();
$top_countries = IpQuery_DB::get_top_countries( 10 );
$top_cities = IpQuery_DB::get_top_cities( 10 );
$ipquery_total_visits = IpQuery_DB::get_total_visits();
$ipquery_unique_ips = IpQuery_DB::get_unique_ips();
$ipquery_risk_counts = IpQuery_DB::get_risk_counts();
$ipquery_top_countries = IpQuery_DB::get_top_countries( 10 );
$ipquery_top_cities = IpQuery_DB::get_top_cities( 10 );
?>

<!-- Stat cards -->
<div class="ipquery-cards">
<div class="ipquery-card ipquery-card--blue">
<div class="ipquery-card__icon dashicons dashicons-admin-users"></div>
<div class="ipquery-card__body">
<span class="ipquery-card__value"><?php echo esc_html( number_format_i18n( $total_visits ) ); ?></span>
<span class="ipquery-card__value"><?php echo esc_html( number_format_i18n( $ipquery_total_visits ) ); ?></span>
<span class="ipquery-card__label"><?php esc_html_e( 'Total Visits', 'ipquery' ); ?></span>
</div>
</div>
<div class="ipquery-card ipquery-card--green">
<div class="ipquery-card__icon dashicons dashicons-networking"></div>
<div class="ipquery-card__body">
<span class="ipquery-card__value"><?php echo esc_html( number_format_i18n( $unique_ips ) ); ?></span>
<span class="ipquery-card__value"><?php echo esc_html( number_format_i18n( $ipquery_unique_ips ) ); ?></span>
<span class="ipquery-card__label"><?php esc_html_e( 'Unique IPs', 'ipquery' ); ?></span>
</div>
</div>
<div class="ipquery-card ipquery-card--orange">
<div class="ipquery-card__icon dashicons dashicons-shield-alt"></div>
<div class="ipquery-card__body">
<span class="ipquery-card__value"><?php echo esc_html( number_format_i18n( $risk_counts['vpn'] + $risk_counts['proxy'] + $risk_counts['tor'] ) ); ?></span>
<span class="ipquery-card__value"><?php echo esc_html( number_format_i18n( $ipquery_risk_counts['vpn'] + $ipquery_risk_counts['proxy'] + $ipquery_risk_counts['tor'] ) ); ?></span>
<span class="ipquery-card__label"><?php esc_html_e( 'VPN / Proxy / Tor', 'ipquery' ); ?></span>
</div>
</div>
<div class="ipquery-card ipquery-card--red">
<div class="ipquery-card__icon dashicons dashicons-warning"></div>
<div class="ipquery-card__body">
<?php
$risky_pct = $unique_ips > 0
? round( ( ( $risk_counts['vpn'] + $risk_counts['proxy'] + $risk_counts['tor'] ) / $unique_ips ) * 100, 1 )
$ipquery_risky_pct = $ipquery_unique_ips > 0
? round( ( ( $ipquery_risk_counts['vpn'] + $ipquery_risk_counts['proxy'] + $ipquery_risk_counts['tor'] ) / $ipquery_unique_ips ) * 100, 1 )
: 0;
?>
<span class="ipquery-card__value"><?php echo esc_html( $risky_pct ); ?>%</span>
<span class="ipquery-card__value"><?php echo esc_html( $ipquery_risky_pct ); ?>%</span>
<span class="ipquery-card__label"><?php esc_html_e( 'Risk Rate', 'ipquery' ); ?></span>
</div>
</div>
Expand Down Expand Up @@ -100,22 +100,22 @@
</tr>
</thead>
<tbody>
<?php if ( empty( $top_countries ) ) : ?>
<?php if ( empty( $ipquery_top_countries ) ) : ?>
<tr><td colspan="3"><?php esc_html_e( 'No data yet.', 'ipquery' ); ?></td></tr>
<?php else : ?>
<?php foreach ( $top_countries as $row ) : ?>
<?php foreach ( $ipquery_top_countries as $ipquery_row ) : ?>
<tr>
<td>
<?php if ( $row['country_code'] ) : ?>
<img src="https://flagcdn.com/16x12/<?php echo esc_attr( strtolower( $row['country_code'] ) ); ?>.png"
<?php if ( $ipquery_row['country_code'] ) : ?>
<img src="https://flagcdn.com/16x12/<?php echo esc_attr( strtolower( $ipquery_row['country_code'] ) ); ?>.png"
width="16" height="12"
alt="<?php echo esc_attr( $row['country_code'] ); ?>"
alt="<?php echo esc_attr( $ipquery_row['country_code'] ); ?>"
style="vertical-align:middle;margin-right:4px;">
<?php endif; ?>
<?php echo esc_html( $row['country'] ?? $row['country_code'] ?? '—' ); ?>
<?php echo esc_html( $ipquery_row['country'] ?? $ipquery_row['country_code'] ?? '—' ); ?>
</td>
<td><?php echo esc_html( number_format_i18n( (int) $row['visits'] ) ); ?></td>
<td><?php echo $total_visits > 0 ? esc_html( round( ( $row['visits'] / $total_visits ) * 100, 1 ) ) . '%' : '—'; ?></td>
<td><?php echo esc_html( number_format_i18n( (int) $ipquery_row['visits'] ) ); ?></td>
<td><?php echo $ipquery_total_visits > 0 ? esc_html( round( ( $ipquery_row['visits'] / $ipquery_total_visits ) * 100, 1 ) ) . '%' : '—'; ?></td>
</tr>
<?php endforeach; ?>
<?php endif; ?>
Expand All @@ -139,23 +139,23 @@
</tr>
</thead>
<tbody>
<?php if ( empty( $top_cities ) ) : ?>
<?php if ( empty( $ipquery_top_cities ) ) : ?>
<tr><td colspan="4"><?php esc_html_e( 'No data yet.', 'ipquery' ); ?></td></tr>
<?php else : ?>
<?php foreach ( $top_cities as $row ) : ?>
<?php foreach ( $ipquery_top_cities as $ipquery_row ) : ?>
<tr>
<td><?php echo esc_html( $row['city'] ); ?></td>
<td><?php echo esc_html( $ipquery_row['city'] ); ?></td>
<td>
<?php if ( $row['country_code'] ) : ?>
<img src="https://flagcdn.com/16x12/<?php echo esc_attr( strtolower( $row['country_code'] ) ); ?>.png"
<?php if ( $ipquery_row['country_code'] ) : ?>
<img src="https://flagcdn.com/16x12/<?php echo esc_attr( strtolower( $ipquery_row['country_code'] ) ); ?>.png"
width="16" height="12"
alt="<?php echo esc_attr( $row['country_code'] ); ?>"
alt="<?php echo esc_attr( $ipquery_row['country_code'] ); ?>"
style="vertical-align:middle;margin-right:4px;">
<?php endif; ?>
<?php echo esc_html( $row['country'] ?? $row['country_code'] ?? '—' ); ?>
<?php echo esc_html( $ipquery_row['country'] ?? $ipquery_row['country_code'] ?? '—' ); ?>
</td>
<td><?php echo esc_html( number_format_i18n( (int) $row['visits'] ) ); ?></td>
<td><?php echo $total_visits > 0 ? esc_html( round( ( $row['visits'] / $total_visits ) * 100, 1 ) ) . '%' : '—'; ?></td>
<td><?php echo esc_html( number_format_i18n( (int) $ipquery_row['visits'] ) ); ?></td>
<td><?php echo $ipquery_total_visits > 0 ? esc_html( round( ( $ipquery_row['visits'] / $ipquery_total_visits ) * 100, 1 ) ) . '%' : '—'; ?></td>
</tr>
<?php endforeach; ?>
<?php endif; ?>
Expand All @@ -170,46 +170,46 @@
<h2><?php esc_html_e( 'Risk Details', 'ipquery' ); ?></h2>
<div class="ipquery-risk-grid">
<?php
$risk_items = array(
$ipquery_risk_items = array(
array(
'label' => __( 'VPN', 'ipquery' ),
'count' => $risk_counts['vpn'],
'count' => $ipquery_risk_counts['vpn'],
'icon' => 'lock',
'class' => 'orange',
),
array(
'label' => __( 'Proxy', 'ipquery' ),
'count' => $risk_counts['proxy'],
'count' => $ipquery_risk_counts['proxy'],
'icon' => 'update',
'class' => 'red',
),
array(
'label' => __( 'Tor', 'ipquery' ),
'count' => $risk_counts['tor'],
'count' => $ipquery_risk_counts['tor'],
'icon' => 'hidden',
'class' => 'red',
),
array(
'label' => __( 'Datacenter', 'ipquery' ),
'count' => $risk_counts['datacenter'],
'count' => $ipquery_risk_counts['datacenter'],
'icon' => 'cloud',
'class' => 'blue',
),
array(
'label' => __( 'Mobile', 'ipquery' ),
'count' => $risk_counts['mobile'],
'count' => $ipquery_risk_counts['mobile'],
'icon' => 'smartphone',
'class' => 'green',
),
);
foreach ( $risk_items as $item ) :
foreach ( $ipquery_risk_items as $ipquery_item ) :
?>
<div class="ipquery-risk-item ipquery-risk-item--<?php echo esc_attr( $item['class'] ); ?>">
<span class="dashicons dashicons-<?php echo esc_attr( $item['icon'] ); ?>"></span>
<strong><?php echo esc_html( number_format_i18n( $item['count'] ) ); ?></strong>
<small><?php echo esc_html( $item['label'] ); ?></small>
<?php if ( $unique_ips > 0 ) : ?>
<span class="ipquery-pct"><?php echo esc_html( round( ( $item['count'] / $unique_ips ) * 100, 1 ) ); ?>%</span>
<div class="ipquery-risk-item ipquery-risk-item--<?php echo esc_attr( $ipquery_item['class'] ); ?>">
<span class="dashicons dashicons-<?php echo esc_attr( $ipquery_item['icon'] ); ?>"></span>
<strong><?php echo esc_html( number_format_i18n( $ipquery_item['count'] ) ); ?></strong>
<small><?php echo esc_html( $ipquery_item['label'] ); ?></small>
<?php if ( $ipquery_unique_ips > 0 ) : ?>
<span class="ipquery-pct"><?php echo esc_html( round( ( $ipquery_item['count'] / $ipquery_unique_ips ) * 100, 1 ) ); ?>%</span>
<?php endif; ?>
</div>
<?php endforeach; ?>
Expand Down
4 changes: 2 additions & 2 deletions includes/class-ipquery-db.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -265,11 +265,11 @@ public static function get_visitors( array $args = array() ): array {

// Table name and validated orderby/order are safe to interpolate; values use placeholders.
$rows_sql = "SELECT * FROM {$table} {$where_sql} ORDER BY {$orderby} {$order} LIMIT %d OFFSET %d"; // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
$rows = $wpdb->get_results( $wpdb->prepare( $rows_sql, ...$query_values ), ARRAY_A ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared
$rows = $wpdb->get_results( $wpdb->prepare( $rows_sql, ...$query_values ), ARRAY_A ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared,PluginCheck.Security.DirectDB.UnescapedDBParameter

if ( $values ) {
$count_sql = "SELECT COUNT(*) FROM {$table} {$where_sql}"; // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
$total = (int) $wpdb->get_var( $wpdb->prepare( $count_sql, ...$values ) ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared
$total = (int) $wpdb->get_var( $wpdb->prepare( $count_sql, ...$values ) ); // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching,WordPress.DB.PreparedSQL.NotPrepared,PluginCheck.Security.DirectDB.UnescapedDBParameter
} else {
$total = (int) $wpdb->get_var( "SELECT COUNT(*) FROM {$table}" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared,WordPress.DB.DirectDatabaseQuery.DirectQuery,WordPress.DB.DirectDatabaseQuery.NoCaching
}
Expand Down
Loading