maripo / JOSM_easypresets

22 stars 7 forks source link

「保存」ボタンでProsetsリストが更新されるようにしました #43

Closed yuuhayashi closed 4 years ago

yuuhayashi commented 4 years ago

Modified so that the edited result in the PresetEditorDialog is reflected to the ManagePresetsDialog and the GroupPresetMenu without restarting JOSM.

PresetEditorDialogでの編集結果がJOSMを再起動しなくてもManagePresetsDialogとGroupPresetMenuに反映されるように修正しました。

yuuhayashi commented 4 years ago

This fork is based on v1.15. The base side was updated to v1.16 two hours before posting the pull request, but fortunately v1.15 and v1.16 seem to be the same, so I think that it is possible to merge as it is. (Is #42-v1.16 resolved by a rebuild?) Since I don't understand modern programming techniques, I only test with primitive methods. I have checked all the functions of Easy Preset that I know. (Linux mint 19, OpenJDK-11, JOSM 16043)

このフォークは v1.15 をベースにしています。pull requestを投稿する2時間前に ベース側が v1.16 にアップデートされましたが、幸いなことに v1.15とv1.16は同一のようですのでこのままmerge可能だと思います。(#42-v1.16はリビルドで解決ですか?) 最近のプログラミング手法を理解していないので原始的な方法でのテストしか行っていません。私が知っているEasyPresetの機能のみ一通り動作確認しています。(Linux mint 19, OpenJDK-11, JOSM 16043)

maripo commented 4 years ago

ありがとうございます。1.16は1行変更しただけですので問題ないかと思います。 確認して取り込ませていただこうと思います。 (最近メンテナンスしていないので色々思い出さないといけないことがいっぱいで、少し作業に時間がかかるかもしれません。)

yuuhayashi commented 4 years ago

下記URLからベータ版をダウンロードできるようにしました。 http://surveyor.mydns.jp/archiva/repository/haya4/org/maripo/josm/easypresets/1.16b.1/easypresets-1.16b.1.jar

突然のPULLリクエストに答えていただきありがとうございます。 わずかな改善ですが広範囲にソースを書き換えています。動作確認にお手数をかけてしまい申し訳ございません。 Talk-jaに投稿してベータ版の動作テストの協力を呼びかけてもよろしいですか?

maripo commented 4 years ago

ありがとうございます。手元でのビルドも試してみました。プリセット更新後にちゃんと反映されるのを確認しました。実際動かしてざっと読ませていただいた限り、おそらく大丈夫そうなのですが、けっこう大きめの変更なのでリリース前にもういっぺんcoreのコードも合わせて読むことにしようと思います。

Talk-jaに投稿してベータ版の動作テストの協力を呼びかけてもよろしいですか? ぜひぜひ。よろしくおねがいします。

そうえば「保存したらリストが更新されるようにしたいけれど実際やろうとするとかなり大掛かりになりそうだからやめとこう」と思ってあきらめていたのを思い出しました。