Skip to content

タグ転記でも情報源を変更セットに残す - #47

Merged
nyampire merged 1 commit into
mainfrom
fix/plateau-source-on-tag-transfer
Aug 11, 2026
Merged

タグ転記でも情報源を変更セットに残す#47
nyampire merged 1 commit into
mainfrom
fix/plateau-source-on-tag-transfer

Conversation

@nyampire

Copy link
Copy Markdown
Owner

ご報告ありがとうございます。以下のとおり修正しました!(以下Claudeさん)

Fixes #45

何が起きていたか

タグ転記モードで既存の建物にタグだけ追加して保存すると、変更セットに情報源が付きませんでした。
建物を追加する経路では MLIT_PLATEAU / RapiD_Plateau_JP / source_ref が揃います。

原因

変更セットの source は、各編集の annotation に付いた dataUsed から組み立てられます(EditSystem_gatherSources)。

経路 annotation 結果
建物を追加 UiRapidInspectordataUsed: dataset?.dataUsed を付けます 情報源が揃います
タグ転記 type / plateauID / entityID のみで、dataUsed がありません 情報源が付きません

commit.jsdataUsed の値を /plateau/i で判定しているので、値が無いと Plateau を使ったと判断されませんでした。

直しかた

転記の annotation に dataUsed を足しました。
データセットの引き当ては受け入れ経路と同じ形にしています(rapid.datasets.get(datasetID)、catalog に無ければデータセット id そのもの)。

実地確認

ローカルの dev server に宇城市の Plateau データ(8,994 件、本番 API から取得)を読み込ませ、実際に転記を実行しました。

annotation の dataUsed : ["osmf.jp", "Plateau Buildings"]
sourcesUsed().data     : ["osmf.jp", "Plateau Buildings"]
source                 : MLIT_PLATEAU;RapiD_Plateau_JP
source_ref             : https://wiki.openstreetmap.org/wiki/MLIT_PLATEAU/imports_outline

転記先の OSM 建物だけは合成です。dev server では OSM の認証が無く実データを引けないためで、それ以外は実際の Plateau データと実際のコードを通しています。

保存パネルそのものは、OSM にログインしないと開けないので確認していません。

テスト

2 件追加しました。実装前に、どちらも expected undefined to deeply equal [...] で落ちることを確認しています。

  • annotation にデータセットの dataUsed が乗ること
  • catalog に無いときはデータセット id にフォールバックすること

browser 753 completed / 5 skipped、unit 1209 passed。

建物を追加する経路は annotation に dataUsed を付けるが、タグ転記の経路は
付けていなかった。変更セットの source は EditSystem がこの値から組み立てる
ため、転記だけで保存すると MLIT_PLATEAU も RapiD_Plateau_JP も source_ref も
付かない。報告の「タグ転記モードで既存の建物を選択してタグだけ追加すると
情報源が揃わない」はこれ。

データセットの引き当ては UiRapidInspector の受け入れ経路と同じ形にした。
catalog に無ければデータセット id そのものを使う。
@nyampire
nyampire merged commit df91315 into main Aug 11, 2026
3 checks passed
@nyampire
nyampire deleted the fix/plateau-source-on-tag-transfer branch August 11, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

情報源にPLATEAUが入ったり、入らなかったりする。

1 participant