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

Opened 15 years ago

Last modified 15 years ago

#2054 new enhancement

オープン制で、新規登録ボタンを自動的に表示してほしい

Reported by: tsukamoto Owned by: nobody
Priority: minor Milestone:
Component: 指定しない Version:
Keywords: 2.11要望 Cc:

Description

以下の件、仕様の再検討願えればと思います。

フリー登録制にするとo_login.tplで、
新規登録のリンクはちゃんと生成(表示)されますが
スキン画像の切替え(skin_login.jpg→skin_login_open.jpg)がされていない様です。
http://trac.openpne.jp/ticket/863

2.10.xで個別の画像ではなくプリセットでのスキン変更が可能になり、個々の画像ファイルを意識する必要がなくなりました。ログイン画面だけ、これを意識する必要があるというのはアンバランスに思えます。

2.10.4.1に対するパッチを添付します。

Attachments (1)

openpne_2.10.4.1_diff.txt (702 bytes) - added by tsukamoto 15 years ago.
Diff for 2.10.4.1 (trunk) - 非オープン制の画像URL生成部を修正

Download all attachments as: .zip

Change History (3)

Changed 15 years ago by tsukamoto

Attachment: openpne_2.10.4.1_diff.txt added

Diff for 2.10.4.1 (trunk) - 非オープン制の画像URL生成部を修正

comment:1 Changed 15 years ago by ebihara

Milestone: OpenPNE2.11.5

パッチの提供ありがとうございます。ごもっともな指摘だと思います。

内容を確認し、2.11.5に入れようと思います。

comment:2 Changed 15 years ago by ebihara

Milestone: OpenPNE2.11.5

パッチの内容を確認しましたが、

  • skin ディレクトリに skin_login_open がない場合にオープン制になっているケースが想定されていない
    • ログイン画面ではデフォルトのスキン画像が表示されてしまう
    • 管理画面のスキン変更ではオープン制用ではなくクローズ制用のログイン画像が表示されてしまう(従来通りの挙動)

という点で懸念があります。

このまま本線に取り込むことはちょっとできないかなぁという印象なので、Milestone を外し、次の機会にこのパッチを参考に実装をするなどしたいと思います。

Note: See TracTickets for help on using tickets.