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

Opened 12 years ago

Closed 11 years ago

#1845 closed enhancement (fixed)

CHECK_IMG_AUTHをtrueにするとログイン前バナーが表示されない

Reported by: kiwa Owned by: shingo
Priority: major Milestone: OpenPNE2.13.4
Component: その他 Version: 2.10.x & 2.12.x & 2.14.x
Keywords: Cc:

Description (last modified by shingo)

■現象

config.phpでCHECK_IMG_AUTHをtrueにするとログイン前バナーが表示されない

■原因

CHECK_IMG_AUTHがtrueの場合、img.phpで処理される画像は(ファイル名がmodule_で始まるものの内の一部とスキンを除き)認証されていないと出力されないようになっている為、ログインしないで見ることのできるページ(o_で始まるページ)上では表示されない。

■修正内容

CHECK_IMG_AUTHがtrue時、管理画面のバナー設定で、ログイン前表示:× かつ ログイン後表示:○ となっているバナーのみ認証確認を行うように修正

■関連情報

  • 報告元
    要望というかバグなんでしょうか。。
    
    当方環境は2.10.3です。
    #弄ってるもんなので、ウチだけなのかな。。。
    
    
    config.phpの
    define('CHECK_IMG_AUTH', true);
    が、trueの場合、下記の2つの現象が確認できました。
    
    症状1:
    http://ホストドメイン/?m=admin&a=page_edit_c_banner
    で設置したバナーの「表示(ログイン前)」設定が
    ○にしても×にしても、ログイン画面(o_loginなど)で
    バナー画像までも表示できなくなってしまっていますが
    admin設定画面での機能側が優先されるべきだと思っています。
    
    症状2:
    同じく
    http://ホストドメイン/?m=admin&a=page_edit_c_banner
    の「画像」において。
    やはりバナー画像が表示できない。
    
    
    以上のことより
    CHECK_IMG_AUTHの影響がバナー画像にまで掛かっている様なので
    掛からないよう、していただきたいです。
    

Change History (23)

comment:1 Changed 12 years ago by kiwa

Priority: minormajor

comment:2 Changed 12 years ago by kiwa

Description: modified (diff)
Keywords: 再現待ち removed
Priority: majorminor
Summary: CHECK_IMG_AUTHをtrueにするとログイン前バナーや管理画面のバナー設定が表示されないCHECK_IMG_AUTHをtrueにするとログイン前バナーが表示されない

再現できたのでDescription変更します。

症状2: 同じく
http://ホストドメイン/?m=admin&a=page_edit_c_banner[[br]] の「画像」において。
やはりバナー画像が表示できない。

こちらは恐らく原因が同じと思われる#1851にまとめます

comment:3 Changed 12 years ago by kiwa

Version: 2.8.x & 2.10.x & 2.11.x

comment:4 Changed 12 years ago by kiwa

Component: pne-adminその他
Keywords: OpenPNE2.8.10 OpenPNE2.11.5 added
Milestone: OpenPNE2.10.5

comment:5 Changed 12 years ago by kiwa

Priority: minormajor

comment:6 Changed 12 years ago by ebihara

Keywords: OpenPNE2.8.10 OpenPNE2.11.5 removed
Milestone: OpenPNE2.10.5OpenPNE2.11.6

対応が困難なので、

  • 開発版での対処とします。開発版にて問題がなければ安定版にマージします
  • 今回のOpenPNE2.11.5 のリリース項目からは外します

comment:7 Changed 12 years ago by imamura623

Milestone: OpenPNE2.11.6OpenPNE2.11.7

2.11.6では見送ります。

comment:8 Changed 12 years ago by kiwa

Milestone: OpenPNE2.11.7OpenPNE2.12beta1

2.11.7では見送ります。

comment:9 Changed 12 years ago by ogawa

Milestone: OpenPNE2.12beta1
Priority: majorminor

comment:10 Changed 12 years ago by ogawa

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

comment:11 Changed 11 years ago by ebihara

Priority: minormajor

comment:12 Changed 11 years ago by ebihara

Milestone: OpenPNE2.13.5

認証周りなので enhancement チケットに変更し、開発版のみの対応とします。

comment:13 Changed 11 years ago by ebihara

Milestone: OpenPNE2.13.5OpenPNE2.13.4

comment:14 Changed 11 years ago by ebihara

Type: defectenhancement

comment:15 Changed 11 years ago by shingo

Owner: changed from nobody to shingo
Status: newassigned

作業します。

comment:16 Changed 11 years ago by shingo

Description: modified (diff)
Keywords: 確認中 added

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

comment:17 Changed 11 years ago by ebihara

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

$img_module という変数名は不自然な気がします。

あと、この修正であれば特に開発版で先行して対応ということでなく、最初から安定版にも対応する形でよさそうですかね。どうでしょう。

comment:18 Changed 11 years ago by ebihara

追加で、

66	    $c_banner =  db_get_row($sql, $params); 

の直後のスペースがひとつ余分な気がします。

comment:19 Changed 11 years ago by shingo

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

以下のリビジョンで comment:17 comment:18 にあげられた事項について修正しました。 ご確認ください。

comment:20 Changed 11 years ago by ebihara

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

$img_id_number という変数名ですと、モジュール用画像の場合に不自然なコードになってしまいます。

comment:21 Changed 11 years ago by shingo

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

以下のリビジョンで comment:20 にあげられた事項について修正しました。 ご確認ください。

comment:22 Changed 11 years ago by ebihara

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

comment:23 Changed 11 years ago by kiwa

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

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

Note: See TracTickets for help on using tickets.