| @@ -259,8 +259,11 @@ | ||
| 259 | 259 | 'trip_id' => (int) $request->get_param('trip_id'), |
| 260 | 260 | 'search' => $request->get_param('search') ?: '', |
| 261 | 261 | 'date_from' => $request->get_param('date_from') ?: '', |
| 262 | 262 | 'date_to' => $request->get_param('date_to') ?: '', |
| 263 | + // Column sorting from the table headers — whitelisted in the repository. | |
| 264 | + 'orderby' => $request->get_param('orderby') ?: '', | |
| 265 | + 'order' => $request->get_param('order') ?: '', | |
| 263 | 266 | ]; |
| 264 | 267 | |
| 265 | 268 | // Delegate to service |
| 266 | 269 | $result = $this->bookingService->getBookings($filters); |
| @@ -820,9 +823,15 @@ | ||
| 820 | 823 | 'trip_title' => $trip ? ($trip->title ?? $booking['trip_title'] ?? __('Trip Booking', 'yatra')) : ($booking['trip_title'] ?? __('Trip Booking', 'yatra')), |
| 821 | 824 | // Trip duration comes from duration_days/duration_nights (there is no |
| 822 | 825 | // `duration` column — accessing it caused a blank value + PHP notice). |
| 823 | 826 | 'trip_duration' => $trip |
| 824 | - ? yatra_format_duration((int) ($trip->duration_days ?? 0), isset($trip->duration_nights) ? (int) $trip->duration_nights : null) | |
| 827 | + ? yatra_format_duration( | |
| 828 | + (int) ($trip->duration_days ?? 0), | |
| 829 | + isset($trip->duration_nights) ? (int) $trip->duration_nights : null, | |
| 830 | + // Hour-based day tours: "8 hours" instead of "1 day". Absent | |
| 831 | + // or NULL on every day-based trip, which keeps its wording. | |
| 832 | + (int) ($trip->duration_hours ?? 0) | |
| 833 | + ) | |
| 825 | 834 | : '', |
| 826 | 835 | 'trip_difficulty' => $trip ? ($trip->difficulty_name ?? '') : '', |
| 827 | 836 | 'departure_location' => $trip ? ($trip->departure_location ?? '') : '', |
| 828 | 837 | 'destination' => $trip ? ($trip->destination ?? '') : '', |
