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

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#3058 closed defect (fixed)

管理画面からのファイルダウンロードがページ名ランダム生成に対応していない

Reported by: imamura623 Owned by: mori
Priority: minor Milestone: OpenPNE2.12.6
Component: 指定しない Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.12 OpenPNE2.13.4 Cc:

Description (last modified by kudo)

■現象

ページ名ランダム生成時に、管理画面からアップロードファイルをダウンロードしようとするとエラーになる。

■原因

該当テンプレートファイルの修正漏れ。t_url関数利用時にうまくハッシュ化されていなかった。

■修正内容

上記ファイルの Smaty プラグイン関数の使用ミスを修正。 これにより、ページ名をランダム化している場合もファイルをダウンロードできるようになった。

■関連情報

元のDescription

http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=19239より転記

OpenPNE-2.10系


・現象
「ページ名ランダム生成」中に、画像書き込み管理からアップロードファイルリストからファイルがダウンロードできない

・原因
file_downloadがまずページ名ランダム生成されていない

・修正???
よく私自身読み込めていませんので、以下だけでうまくいくかもわかりませんが、
webapp/modules/admin/templates/list_c_file.tpl
l.63のリンクはhashからはaction名を持ってきていない模様

取り急ぎ報告まで

Change History (19)

comment:1 Changed 11 years ago by kiwa

Keywords: 再現待ち removed

再現できるかやってみました。

概要

管理画面のページ名ランダム生成時、アップロードファイルリスト(admin_page_list_c_file)からファイルがダウンロードできないかどうか試してみる。

結果

管理画面のページ名ランダム生成時はファイルをダウンロードできない(再現した)

version

  • 2.10.10
  • 2.12.4
  • 2.13.3

設定

config.php

  • OPENPNE_USE_FILEUPLOAD: true

その他

  • 既にファイルアップロード機能でファイルが1つ以上アップロードされている

手順

  1. ページ名ランダム生成(admin_page_update_hash_table)にて、ページ名ランダム生成を実行する
  2. アップロードファイルリスト(admin_page_list_c_file)に遷移する
  3. アップロード済みのファイルのファイル名を押下する → ファイルをダウンロードできるかどうか

結果詳細

以下のようなエラー画面に遷移され、ファイルをダウンロードすることができない。

エラーが発生しました。

    * アクションの指定が正しくありません

このときのURLは以下のようになっている。

http://hogehoge/?m=admin&a=do_file_download&filename=ファイル名&sessid=hogehoge

comment:2 Changed 11 years ago by kiwa

Summary: 管理画面 - ファイルダウンロード管理画面からのファイルダウンロードがページ名ランダム生成に対応していない

comment:3 Changed 11 years ago by ebihara

Keywords: OpenPNE2.10.12 OpenPNE2.13.4 added
Milestone: OpenPNE2.12.6

comment:4 Changed 11 years ago by mori

Owner: changed from nobody to mori
Status: newassigned

担当します。

comment:5 Changed 11 years ago by mori

Keywords: 確認中 added

以下のリビジョンを修正しました。

comment:6 Changed 11 years ago by mori

Description: modified (diff)

comment:7 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • comment:5 のリビジョンで修正されている do_file_download へのURLですが、実際にはランダム生成されていないように見えます
  • webapp/modules/admin/templates/delete_topic.tpl についても同様に修正する必要があるのではないでしょうか
  • webapp/modules/admin/templates/topic_list.tpl についても同様に修正する必要があるのではないでしょうか

comment:8 Changed 11 years ago by mori

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

以下のリビジョンを再修正しました。

comment:9 Changed 11 years ago by ebihara

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

comment:8 の修正では、 admin_page_delete_topic においてファイルをダウンロードすることができなくなってしまいます。

comment:10 Changed 11 years ago by ebihara

Keywords: 差し戻し added; テスト待ち removed

すいませんキーワード間違えました。

comment:11 Changed 11 years ago by mori

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

以下のリビジョンを修正しました。

comment:12 Changed 11 years ago by ebihara

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

comment:13 Changed 11 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

Descriptionについての指摘です。下記の修正をお願いします。

  1. 元のDescription(「■現象」の上にある内容)は、#2910のように「関連情報」に入れてください
  2. ■原因について、具体的に何が原因だったのかを分かるように記載してほしいです
  3. ■修正内容について、ユーザ側から見てどのような対応になったのかも記載してほしいです
    • 例: #2910 「絞込みをすると1ページ目に移動するように修正。」

※ 今回#2910を例に挙げてますが、条件を満たしたものを適当に選んだだけなので「これが最良例」というわけではありません

comment:14 Changed 11 years ago by mori

Description: modified (diff)
Keywords: テスト待ち added; 差し戻し removed

comment:15 Changed 11 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

重ねてですが、以下の修正をお願いします。

■原因について、具体的に何が原因だったのかを分かるように記載してほしいです 

■修正内容について、ユーザ側から見てどのような対応になったのかも記載してほしいです
    * 例: #2910 「絞込みをすると1ページ目に移動するように修正。」 
  • 原因について、もしファイルダウンロードのリンクがページ名ランダム生成に対応する記述になっていなかったのなら、そのような事がわかるようにしてください
  • 修正内容について、もしファイルダウンロードのリンクがページ名ランダム生成に対応したのなら、その事がわかるようにしてください

comment:16 Changed 11 years ago by mori

Description: modified (diff)
Keywords: 確認中 added; 差し戻し removed

comment:17 Changed 11 years ago by mori

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

comment:18 Changed 11 years ago by kiwa

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

確認しました。問題ありません。

comment:19 Changed 11 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.