← All changes
|
core/dashboard/controllers/dashboard-controller.php
+4
-4
1.0.11
→
1.0.28
View file →
| @@ -130,9 +130,9 @@ | ||
| 130 | 130 | foreach ( self::REPORT_TYPES as $report_type ) { |
| 131 | 131 | $report_data = $request->get_param( $report_type ); |
| 132 | 132 | |
| 133 | 133 | if ( is_array( $report_data ) && isset( $report_data['start_date'], $report_data['end_date'] ) ) { |
| 134 | - $date_range = $report_data['start_date'] . ' to ' . $report_data['end_date']; | |
| 134 | + $date_range = sanitize_text_field( $report_data['start_date'] ) . ' to ' . sanitize_text_field( $report_data['end_date'] ); | |
| 135 | 135 | $data[ $report_type ] = $this->get_report_data( $report_type, $date_range ); |
| 136 | 136 | } |
| 137 | 137 | } |
| 138 | 138 | |
| @@ -215,9 +215,9 @@ | ||
| 215 | 215 | |
| 216 | 216 | // Calculate percentage change |
| 217 | 217 | $percentage_change = 0; |
| 218 | 218 | $trend = 'neutral'; |
| 219 | - if ( $previous_bookings != 0 ) { | |
| 219 | + if ( 0 != $previous_bookings ) { | |
| 220 | 220 | $percentage_change = ( ( $current_bookings - $previous_bookings ) / $previous_bookings ) * 100; |
| 221 | 221 | $trend = $this->get_trend( $percentage_change ); |
| 222 | 222 | } |
| 223 | 223 | |
| @@ -271,9 +271,9 @@ | ||
| 271 | 271 | $percentage_change = 0; |
| 272 | 272 | $trend = 'neutral'; |
| 273 | 273 | |
| 274 | 274 | // Ensure both values are valid numbers before calculation |
| 275 | - if ( is_numeric( $previous_revenue ) && $previous_revenue != 0 && is_numeric( $current_revenue ) ) { | |
| 275 | + if ( is_numeric( $previous_revenue ) && 0 != $previous_revenue && is_numeric( $current_revenue ) ) { | |
| 276 | 276 | $percentage_change = ( ( $current_revenue - $previous_revenue ) / $previous_revenue ) * 100; |
| 277 | 277 | |
| 278 | 278 | // Check if the result is a valid number |
| 279 | 279 | if ( !is_finite( $percentage_change ) ) { |
| @@ -356,9 +356,9 @@ | ||
| 356 | 356 | |
| 357 | 357 | // Calculate percentage change |
| 358 | 358 | $percentage_change = 0; |
| 359 | 359 | $trend = 'neutral'; |
| 360 | - if ( $previous_results != 0 ) { | |
| 360 | + if ( 0 != $previous_results ) { | |
| 361 | 361 | $percentage_change = ( ( $current_results - $previous_results ) / $previous_results ) * 100; |
| 362 | 362 | $trend = $this->get_trend( $percentage_change ); |
| 363 | 363 | } |
| 364 | 364 | |