Skip to content
Snippets Groups Projects

Convert uri to query

Merged Lera Shuklina requested to merge ITCP-287 into main

Problem: The move to query parameters was necessary due to an issue with incorrect handling of categories that contained special characters, including the slash (/) character. In the previous implementation, when such values ​​(e.g. Lap Swim/Exercise) were passed as part of the URL path, Drupal would interpret them incorrectly, resulting in incorrect or no schedule data.

Steps for review:

  1. Verify that the URL is formed using ? and query parameters (location, category, date, room, etc.).
  2. Ensure categories containing slashes (e.g. Lap Swim/Exercise) do not break the request.
  3. Confirm the request hits the correct route (/schedules/get-event-data).
  4. Verify location is passed and parsed as an array.
  5. Test filtering by category (including multiple selected).
  6. Check room parameter
  7. Validate excl and limit parameters for correct behavior.
  8. Test edge cases — empty or missing parameters (0, null, etc.).
  9. Test all input combinations:
  • date only,
  • category only,
  • room only,
  • all combined.
Edited by Andrii Podanenko

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading