[PR #792] [MERGED] Fixes handling of maps #976

Closed
opened 2026-02-25 20:37:17 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/benbusby/whoogle-search/pull/792
Author: @jacr13
Created: 6/15/2022
Status: Merged
Merged: 6/27/2022
Merged by: @benbusby

Base: mainHead: fix_maps_url


📝 Commits (2)

  • aed068e fixes map url, e.g. when no q parameter is given
  • 98fde77 move maps_args from results to filter where it is used

📊 Changes

1 file changed (+25 additions, -1 deletions)

View changed files

📝 app/filter.py (+25 -1)

📄 Description

When there is no q parameter in google maps, the directions url in search results points to an empty location.

This workaround fixes it by looking for other parameters that explain the location, if none is found it returns the true url.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/benbusby/whoogle-search/pull/792 **Author:** [@jacr13](https://github.com/jacr13) **Created:** 6/15/2022 **Status:** ✅ Merged **Merged:** 6/27/2022 **Merged by:** [@benbusby](https://github.com/benbusby) **Base:** `main` ← **Head:** `fix_maps_url` --- ### 📝 Commits (2) - [`aed068e`](https://github.com/benbusby/whoogle-search/commit/aed068eceffa7626bad096310ab60bd720a89aa4) fixes map url, e.g. when no q parameter is given - [`98fde77`](https://github.com/benbusby/whoogle-search/commit/98fde77f7c69a6e0a3ce8e4a9c7f69447d7753f9) move maps_args from results to filter where it is used ### 📊 Changes **1 file changed** (+25 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/filter.py` (+25 -1) </details> ### 📄 Description When there is no q parameter in google maps, the directions url in search results points to an empty location. This workaround fixes it by looking for other parameters that explain the location, if none is found it returns the true url. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-25 20:37:17 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/whoogle-search#976
No description provided.