ec-cube2

ec-cube2 copied to clipboard

API応答 ログインパスワードをマスクする

1

- 会員とメンバーで別々の対応が必要かも。(未確認)

seasoftjapan

LC_Page_Admin_* の extends にバラつきがある

2

基本は extends LC_Page_Admin_Ex だが、以下の例外がある。 ``` data\class\pages\admin\order\LC_Page_Admin_Order_Disp.php(33): class LC_Page_Admin_Order_Disp extends LC_Page_Admin_Order_Ex data\class\pages\admin\order\LC_Page_Admin_Order_Edit.php(33): class LC_Page_Admin_Order_Edit extends LC_Page_Admin_Order_Ex data\class\pages\admin\order\LC_Page_Admin_Order_Mail.php(33): class LC_Page_Admin_Order_Mail extends LC_Page_Admin_Order_Ex data\class\pages\admin\products\LC_Page_Admin_Products_ReviewEdit.php(33): class LC_Page_Admin_Products_ReviewEdit extends LC_Page_Admin_Products_Review ``` ケアレスミスだと思うが、意味があるならコメントを記述すべき。

seasoftjapan

商品規格登録画面でFatal Error

1

# 手順 - EC-CUBE 2.13.5 インストール直後の状態で、アイスクリームの商品規格編集画面を表示する。 - 任意の規格行のチェックボックスのチェックを外し、同じ行の税率を空白にする。 - 確認ボタン及び登録ボタンを押下する。 ``` Fatal error(E_USER_ERROR): DB処理でエラーが発生しました。 SQL: [UPDATE dtb_tax_rule SET calc_rule= $1, tax_rate= $2, tax_adjust= $3, apply_date= $4, member_id= $5, update_date= CURRENT_TIMESTAMP...

izayoi256

任意入力項目に対応できない。必須入力チエックは EXIST_CHECK で行える。 ```diff public function MIN_LENGTH_CHECK($value) { $disp_name = $value[0]; $keyname = $value[1]; $min_str_len = $value[2]; if (isset($this->arrErr[$keyname])) { return; } $this->createParam($value); // 文字数の取得 - if (mb_strlen($this->arrParam[$keyname]) < $min_str_len)...

seasoftjapan

目視によるソースコードレビュー。動作未確認。 ```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 処理は、単一選択肢の場合の省略か、不正入力時の代替値投入だろうか。 前者なら判定条件が不適切な気がする。後者なら入力値の変換とエラーチェックの過程で検出し、それでもすり抜けを生じうるのであれば、例外処理とすべき気がする。

seasoftjapan

Warning が発生した際に「不正なページ移動です」エラー画面になってしまう場合がある模様。 通常、「不正なページ移動です」エラー画面になった場合は、 `data/logs/error.log` が出力されるが、このケースは出力されないため、原因の特定が困難である。 #69 によるデグレード。

nanasess

Bump @babel/preset-env from 7.24.4 to 7.24.5

1

Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.24.4 to 7.24.5. Release notes Sourced from @​babel/preset-env's releases. v7.24.5 (2024-04-29) Thanks @​romgrk and @​sossost for your first PRs! :bug: Bug Fix babel-plugin-transform-classes, babel-traverse #16377 fix: TypeScript...

dependabot[bot]

Bump tar from 7.0.1 to 7.1.0

1

Bumps [tar](https://github.com/isaacs/node-tar) from 7.0.1 to 7.1.0. Changelog Sourced from tar's changelog. Changelog 7.1 Update minipass to v7.1.0 Update the type definitions of write() and end() methods on Unpack and Parser...

dependabot[bot]

Bump @babel/core from 7.24.4 to 7.24.5

1

Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.24.4 to 7.24.5. Release notes Sourced from @​babel/core's releases. v7.24.5 (2024-04-29) Thanks @​romgrk and @​sossost for your first PRs! :bug: Bug Fix babel-plugin-transform-classes, babel-traverse #16377 fix: TypeScript...

dependabot[bot]

Smarty modifier に array_key_exists を追加

1

admin/products/product_class_confirm.tpl で array_key_exists が使われてます。 _Originally posted by @bbkids in https://github.com/EC-CUBE/ec-cube2/issues/911#issuecomment-2078700829_

nanasess