Emuera不具合発覚により顕在化したユーザ関数の引数における型の不整合

Bug #1420233 reported by revkoishi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Project Reverse
Fix Released
High
revkoishi
1.0
Fix Released
High
revkoishi

Bug Description

互換性オプション「ユーザー関数の引数に自動的にTOSTRを補完する」が逆の効用になっていた
ため、文字列型の引数の型チェックが不完全だったことにより見過ごされていたバグの顕在化。
補完されるのが正しい仕様と誤認して作りこんでいる箇所があるかもしれないので、
それぞれ呼び出し側と関数側のどちらを修正すべきか、および修正に伴う影響範囲なども調べて、
修正方法を検討する必要があると思われる。

エラー発生関数と発生箇所/使用箇所
----------------------------------------
@ACTM_NUM 28箇所/28箇所
@HTMLBUTTON 1箇所/45箇所
@ADD_FTAG 4箇所/4箇所
@COM_VITALITY 4箇所/20箇所
@TOUCH_SET 46箇所/52箇所
@SEX_SET  1箇所/12箇所
@CHARA_LIST_W 2箇所/4箇所
@ALLCHKDATA_DATE_LATEST 1箇所/3箇所
@GET_KOJO_MENU_CORRECTION 1箇所/1箇所
@GET_KOJO_CORRECTION 2箇所/2箇所 ※@ACTM_NUMと同一行で発生
----------------------------------------
計90箇所/171箇所

Tags: function
revkoishi (revkoishi)
description: updated
revkoishi (revkoishi)
description: updated
revkoishi (revkoishi)
description: updated
Revision history for this message
revkoishi (revkoishi) wrote :

@ACTM_NUM→すべての使用箇所で数値が与えられていたため、引数を数値型に変更。
文字列型を引数に取る関数@ACTMNAME_NUMを追加
@HTMLBUTTON→@SHOW_SHOPにて使われている箇所を修正。その他@SHOW_SHOPを微調整
@ADD_FTAG→文字列で引き受けるの前提の引数であるため、呼び出し側@UPCHECK_REVを修正
@COM_VITALITY→文字列で引き受けるの前提の引数であるため、呼び出し側@COMABLE、@SET_COM_VITALITYを修正
@TOUCH_SET→基本的に文字列で指定するのが望ましいので、呼び出し側@TOUCH_SUCCESSIONを修正。
その他@TOUCH_SUCCESSION、@TOUCH_SETのバグ修正と微調整等も行った。@TOUCH_PARTを追加
@SEX_SET→文字列で引き受けるの前提の引数であるため、呼び出し側@TOUCH_SUCCESSIONを修正。
その他@SEX_SETも微調整
@CHARA_LIST_W→文字列で引き受けるの前提の引数であるため、呼び出し側@SELECT_CHARA_LIST_OPTを修正
@ALLCHKDATA_DATE_LATEST→引数を途中から数値→文字列に変えているらしく、呼び出し側@QUICKLOADを修正。
一部使用意図が不明な箇所があったが多分kap=サン案件なので安心です、ゆえに現状のままとする(@DATAMENU)
@GET_KOJO_MENU_CORRECTION→関数仕様を尊重し、呼び出し側@ACT_MENU_KOJO_CORRECTION_Tを修正
@GET_KOJO_CORRECTION→関数仕様を尊重し、呼び出し側@ACT_ACTM_KOJO_CORRECTION_Tを修正

Revision history for this message
revkoishi (revkoishi) wrote :

CALLFORM系で呼び出している中にエラーが発生する箇所があったので修正

L7switch (layer7-inc)
no longer affects: eratohoreverse/1.1
Changed in eratohoreverse:
milestone: 1.110 → 1.021
L7switch (layer7-inc)
Changed in eratohoreverse:
milestone: 1.021 → none
L7switch (layer7-inc)
Changed in eratohoreverse:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.