EC-CUBE / EC-CUBE/ec-cube2

LC_Page_Shopping_Payment::action() mode=confirm の deliv_id 補完に疑問

Open
#119 0 comments 0 reactions 0 assignees View on GitHub
discussion question
Dominant language
PHP
Stars
92
Forks
97
Avg merge
4d 2h
Merged PRs (30d)
9

Description

目視によるソースコードレビュー。動作未確認。
```php
$deliv_id = $objFormParam->getValue('deliv_id');
if (strval($deliv_id) !== strval(intval($deliv_id))) {
$deliv_id = $this->arrDeliv[0]['deliv_id'];
$objFormParam->setValue('deliv_id', $deliv_id);
}
```
IF 処理は、単一選択肢の場合の省略か、不正入力時の代替値投入だろうか。

前者なら判定条件が不適切な気がする。後者なら入力値の変換とエラーチェックの過程で検出し、それでもすり抜けを生じうるのであれば、例外処理とすべき気がする。

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.