Description
【8/1】GIBJapan作業依頼
jolly/jr-web#2991
入力画面
リザルト画面
:成功
!スクリーンショット 2023-08-03 14
:失敗
!スクリーンショット 2023-08-03 14
jolly/jr-web#2991
:成功
!スクリーンショット 2023-08-03 14
:失敗
!スクリーンショット 2023-08-03 14
@wada
レビューお願いします。
@wada
分岐の条件を修正しました。
@suzuki $debug_typeの存在チェックのみだと、$object_noと同じ状態になってしまいます。
建物データの付与をした時のみ、建物データの付与チェックを実施するように修正をお願いします。
@wada
修正しました。
$debug_type = 7の分岐を新たに追加しています。
@wada
余分な記載のため、削除しました。
@suzuki
ありがとうございます。確認OKです。
@suzuki
建物データの付与条件が$object_noだけだと、今後$object_noを引数とする機能が追加された際に、意図せず建物データを付与してしまいます。
他の建築ツール処理に合わせて、$debug_typeで切り分ける形が良いと思います。
| base | 45b9950f3baf3835a3980e3800a7bf3a7e01e41f |
|---|---|
| head | 4db0360f5666fa26f46086621f0578d1d35da863 |
| base branch | jolly/jr-web:cpanel |
| head branch | jolly/jr-web:object_add0802 |
| status | file |
|---|---|
| modified | cpanel_html/template/m_build_event_debug.tpl |
| modified | cpanel_html/template/m_build_event_debug_result.tpl |
| modified | lib/cpanel/m_build_event_debug_result.inc |
@suzuki
建物データの付与条件が$object_noだけだと、今後$object_noを引数とする機能が追加された際に、意図せず建物データを付与してしまいます。
他の建築ツール処理に合わせて、$debug_typeで切り分ける形が良いと思います。