ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#2163 closed defect (fixed)

「CSVダウンロード」にて、「右側のID入力欄」に「左側のID入力欄」より少ない数字を入力し「ダウンロード」を押しても、CSVファイルをダウンロードできてしまう

Reported by: kiwa Owned by: shingo
Priority: minor Milestone: OpenPNE2.12.4
Component: pne-admin Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.10 OpenPNE2.13.2 Cc:

Description (last modified by ebihara)

■現象

「CSVダウンロード」にて、「右側のID入力欄」に「左側のID入力欄」より少ない数字を入力し「ダウンロード」を押しても、CSVファイルをダウンロードできてしまう。

■原因

ファイルダウンロード処理をおこなう前の、入力値チェックが不足していた。

■修正内容

「右側のID入力欄」に「左側のID入力欄」より少ない数字を入力し、「ダウンロード」を押すとエラーになるように修正。

■関連情報

「管理画面」>「メンバー管理」の「CSVダウンロード」画面にて、「メンバーIDを指定してダウンロード」の「左側のID入力欄」(入力欄1)に「右側の入力欄」(入力欄2)より大きな数値を入力し、「ダウンロード」ボタンを押下した際、エラーが表示されず、CSVファイルのダウンロードが可能となっております。

【検証手順の例】

1)「メンバーIDを指定してダウンロード」の【ID入力欄1】に50、【ID入力欄2】に30と入力し、「ダウンロード」ボタンを押下します。

例> 50 ~ 30

2)エラー表示等がされず、CSVファイルが正常にダウンロードされることを確認

※上記現象によりダウンロードしたCSVファイルを確認したところ、ユーザーデータは記載されておりません

Change History (20)

comment:1 Changed 11 years ago by ebihara

Keywords: OpenPNE2.8.11 OpenPNE2.12beta2 added
Milestone: OpenPNE2.10.6

comment:2 Changed 11 years ago by ebihara

Keywords: OpenPNE2.12beta3 added; OpenPNE2.8.11 OpenPNE2.12beta2 removed
Milestone: OpenPNE2.10.6OpenPNE2.10.7

comment:3 Changed 11 years ago by ogawa

Version: 2.8.x & 2.10.x & 2.11.x2.10.x & 2.12.x & 2.13.x

comment:4 Changed 11 years ago by kiwa

Keywords: OpenPNE2.12RC1 added; OpenPNE2.12beta3 removed

comment:5 Changed 11 years ago by ebihara

Keywords: OpenPNE2.12RC2 added; OpenPNE2.12RC1 removed

comment:6 Changed 11 years ago by ogawa

Keywords: OpenPNE2.10.7 added; OpenPNE2.12RC2 removed
Milestone: OpenPNE2.10.7OpenPNE2.12.1

安定版リリースに回します。

comment:7 Changed 11 years ago by ogawa

Keywords: OpenPNE2.13.2 added

comment:8 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.8 added; OpenPNE2.10.7 removed
Milestone: OpenPNE2.12.1OpenPNE2.12.2

comment:9 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.9 added; OpenPNE2.10.8 removed
Milestone: OpenPNE2.12.2OpenPNE2.12.3

comment:10 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.10 added; OpenPNE2.10.9 removed
Milestone: OpenPNE2.12.3OpenPNE2.12.4

comment:11 Changed 11 years ago by kiwa

Keywords: OpenPNE2.13.3 added; OpenPNE2.13.2 removed

comment:12 Changed 11 years ago by shingo

Keywords: 確認中 added
Owner: changed from nobody to shingo
Status: newassigned

修正します。

comment:13 Changed 11 years ago by shingo

Description: modified (diff)

comment:14 Changed 11 years ago by shingo

Description: modified (diff)

以下のリビジョンで修正しました。ご確認ください。

comment:15 Changed 11 years ago by kiwa

Keywords: OpenPNE2.13.2 added; OpenPNE2.13.3 removed

comment:16 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

今回の修正ではなく元のコードが良くなかったと思いますが、エラーチェックに引っかからなかった場合、webapp/modules/admin/do/csv_member.php の以下の部分で Notice レベルのエラーが発生してしまいます(未定義の値を参照しようとしているため)。

 32         if ($errors) {

comment:17 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで修正しました。ご確認ください。

comment:18 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:19 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: assignedclosed

comment:20 Changed 11 years ago by ebihara

Description: modified (diff)
Summary: 「CSVダウンロード」にて、「右側のID入力欄」に「左側のID入力欄」より少ない数字を入力し、「ダウンロード」を押すと、CSVファイルをダウンロードする事が出来る「CSVダウンロード」にて、「右側のID入力欄」に「左側のID入力欄」より少ない数字を入力し「ダウンロード」を押しても、CSVファイルをダウンロードできてしまう
Note: See TracTickets for help on using tickets.