PR

【管理画面】建物データの付与 #3001

suzuki opened this at 2023-08-03T05:09:24; closed

Comment

#29501 by suzuki at 2023-08-03T05:12:29; updated 2023-08-03T05:12:29

@wada
レビューお願いします。

Comment

#29630 by suzuki at 2023-08-07T18:49:28; updated 2023-08-07T18:49:28

@wada

分岐の条件を修正しました。

Comment

#29650 by wada at 2023-08-09T11:55:55; updated 2023-08-09T11:55:55

@suzuki
$debug_typeの存在チェックのみだと、$object_noと同じ状態になってしまいます。
建物データの付与をした時のみ、建物データの付与チェックを実施するように修正をお願いします。

suzuki change_title at 2023-08-10T11:45:10: 【管理画面】建物データの付与 [WIP]【管理画面】建物データの付与

Comment

#29777 by suzuki at 2023-08-21T07:36:32; updated 2023-08-21T07:36:32

@wada

修正しました。
$debug_type = 7の分岐を新たに追加しています。

suzuki change_title at 2023-08-21T07:37:07: [WIP]【管理画面】建物データの付与 【管理画面】建物データの付与

Comment

#29862 by suzuki at 2023-08-23T10:55:30; updated 2023-08-23T10:55:30

@wada

余分な記載のため、削除しました。

Comment

#29870 by wada at 2023-08-23T13:14:47; updated 2023-08-23T13:14:47

@suzuki
ありがとうございます。確認OKです。

wada add_label at 2023-08-23T13:14:50: 和田 LGTM
wada merge at 2023-08-23T15:51:11: 【管理画面】建物データの付与
wada close at 2023-08-23T15:51:11: Close

commit comment

#193 by wada at 2023-08-04T12:57:12; lib/cpanel/m_build_event_debug_result.inc / old:None new:71
9c4629efc610006453186019c95d63e6ae30f01c

@suzuki
建物データの付与条件が$object_noだけだと、今後$object_noを引数とする機能が追加された際に、意図せず建物データを付与してしまいます。

他の建築ツール処理に合わせて、$debug_typeで切り分ける形が良いと思います。

Commit range

base45b9950f3baf3835a3980e3800a7bf3a7e01e41f
head4db0360f5666fa26f46086621f0578d1d35da863
base branchjolly/jr-web:cpanel
head branchjolly/jr-web:object_add0802

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
modifiedcpanel_html/template/m_build_event_debug.tpl
modifiedcpanel_html/template/m_build_event_debug_result.tpl
modifiedlib/cpanel/m_build_event_debug_result.inc
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#193 by wada at 2023-08-04T12:57:12; lib/cpanel/m_build_event_debug_result.inc / old:None new:71
9c4629efc610006453186019c95d63e6ae30f01c

@suzuki
建物データの付与条件が$object_noだけだと、今後$object_noを引数とする機能が追加された際に、意図せず建物データを付与してしまいます。

他の建築ツール処理に合わせて、$debug_typeで切り分ける形が良いと思います。