[openstreetmap/openstreetmap-website] Exclude `opening_hours` from semicolon splitting in tags helper (PR #6968)

Kevin Ratzel notifications at github.com
Mon Apr 6 16:38:15 UTC 2026


### Description
PR solves bug where `opening_hours` syntax splits by semicolon. A rare case that a semicolon is part of the spec for the tag value to delimit days. When sent to tag2link, it should be the full value.

### How has this been tested?
This is a simple `elseif` that forwards raw `opening_hours` values to tag2link instead of being captured by the `else` statement splitting by semicolon.

### LLM Disclaimer
This PR was made with suggested code by Codex by ChatGPT in an effort to find the correct file and line to use. Tests edited by hand.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/6968

-- Commit Summary --

  * Fix tag rendering for opening hours
  * Enhance browse_tags_helper_test with opening hours

-- File Changes --

    M app/helpers/browse_tags_helper.rb (2)
    M test/helpers/browse_tags_helper_test.rb (6)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/6968.patchhttps://github.com/openstreetmap/openstreetmap-website/pull/6968.diff
-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6968
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6968 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260406/46f0f5f0/attachment.htm>


More information about the rails-dev mailing list