Skip to content

Hotfix: バス路線でGetStationsByLineIdが0件になる問題を種別なし駅一覧へのフォールバックで修正 - #1623

Merged
TinyKitten merged 1 commit into
masterfrom
hotfix/fix-bus-line-stations-empty
Aug 6, 2026
Merged

Hotfix: バス路線でGetStationsByLineIdが0件になる問題を種別なし駅一覧へのフォールバックで修正#1623
TinyKitten merged 1 commit into
masterfrom
hotfix/fix-bus-line-stations-empty

Conversation

@TinyKitten

Copy link
Copy Markdown
Member

概要

#1622(dev にマージ済み)の本番向け Hotfix です。バス路線で GetStationsByLineId が常に0件を返し、アプリでバス停一覧が全く表示されなくなっていた問題を修正します。master は #1621 で dev(#1614 を含む)を取り込んでおり、本番でも同じ退行が発生しているため、dev の修正コミット(008c507)を master へ cherry-pick しました。

#1614「列車種別なしの経路分岐を削除」で、種別を持たない駅向けのフォールバック(get_by_line_id_without_train_types)が削除され、get_by_line_id_with_train_type に一本化されました。しかし統合後のクエリの target_line_group CTE は t.kind IN (0,1) OR t.priority > 0 で種別グループを選択するため、BusRoutekind=7priority=0)しか種別を持たないバス路線ではグループが一切マッチせず、常に空の結果を返していました。

変更の種類

  • バグ修正
  • 新機能
  • データの修正・追加
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

テスト

  • cargo fmt --all -- --check が通ること
  • cargo clippy -- -D warnings が通ること(SQLX_OFFLINE=true
  • cargo testSQLX_OFFLINE=true)が通ること(427 + 41 件パス)

hotfix ブランチ上でも上記3コマンドを再実行して確認済みです。追加の統合テストは #1622 の時点でローカル PostgreSQL 16 に対して実行し、パスを確認しています。

関連Issue

Refs #1622

スクリーンショット(任意)

🤖 Generated with Claude Code

https://claude.ai/code/session_01LM9j6JJciEV1jmhLTt7Hod


Generated by Claude Code

@TinyKitten TinyKitten self-assigned this Aug 6, 2026
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d3b09b6d-33ad-4957-bb5c-11e805aad1ba

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@TinyKitten
TinyKitten merged commit 623f246 into master Aug 6, 2026
11 checks passed
@TinyKitten
TinyKitten deleted the hotfix/fix-bus-line-stations-empty branch August 6, 2026 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant