Ticket #752 (closed バグ指摘: 修正済)

Opened 10 years ago

Last modified 10 years ago

商品登録CSV 最大カラム数($colmax)の比較の誤り

Reported by: Seasoft Owned by: AMUAMU
Priority: Milestone: EC-CUBE2.11.0
Component: 管理画面 Version: コミュニティ (eccube-comu)
Keywords: Cc:
修正済み:

Description

LC_Page_Admin_Products_UploadCSV#process

イコールでチェックしているが、上限としてチェックすべき。

また、項目数カウントの変数名が $max というのも不適切だと思う。

Change History

comment:1 Changed 10 years ago by AMUAMU

  • Owner changed from somebody to AMUAMU
  • Status changed from new to assigned

comment:2 Changed 10 years ago by AMUAMU

  • Milestone changed from EC-CUBE2.5.0beta to EC-CUBE2.5.0

#19950 にて修正。 上限ではなく制限とし引き続きイコールチェックとし、カラム列数の省略は認めないものとしました(アップロードする文字列中にカンマ記号が入り、さらに文字列のダブルコーテーション囲みを不用意に削除してしまった場合を考慮すると列ずれが生じて危険な為)。 列の変更は #870 により、CSV設定にて変更可能。

comment:3 Changed 10 years ago by AMUAMU

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.