OpenPNE 3

変更対象ファイル群．

apps/pc_frontend/templates/smtLayoutMember.php
2:<?php $sf_response->removeStylesheet('/opSkinBasicPlugin/css/main.css') ?>

apps/pc_frontend/templates/smtLayoutHome.php
2:<?php $sf_response->removeStylesheet('/opSkinBasicPlugin/css/main.css') ?>

apps/pc_frontend/templates/smtLayoutSns.php
2:<?php $sf_response->removeStylesheet('/opSkinBasicPlugin/css/main.css') ?>

apps/pc_frontend/templates/smtLayoutGroup.php
2:<?php $sf_response->removeStylesheet('/opSkinBasicPlugin/css/main.css') ?>

修正方針¶

response に addStylesheet() している部分は view.yml のキャッシュ部分であるため直接追加部分のソースの変更を行うことはできない．
よって修正方針としては以下の３つが考えられる．

1. ビュー部分で不必要な css や js をすべて $response->removeStylesheet() や $response->removeJavascript() などで使用しないようにする
2. include_stylesheet() や include_javasript() のスマートフォンを対象とした処理を追加して不必要な css や js を読み込まないようにする
3. view.yml のキャッシュ生成時に不必要な css や js を読み込まないようにする．

1 の方法だと，各ビューで同じような処理を追加する必要があり，そうでなくても不必要なファイルを読み込んでから削除するという処理が発生する無駄があるので取りうるべきではない．
2 の方法も無難ではあるが，Symfony の作法とは別に OpenPNE での作法となるため，カスタマイズなどでスマートフォン対応をする際に実装ミスの原因となる可能性がある．また，PC版と共通のファイルを読み込みたい場合などであっても，PC版とスマートフォン版でそれぞれ読み込む処理を追加しなければならない．これらはすべてソースコードで設定を行わなければならないため，PC版のように view.yml から設定を行うことができない．
3 の方法が最も適切であると思うが，PC版と同じアプリケーションで実装されているためキャッシュがPC版と混在してしまう．これを避けるため，2のようにスマートフォンで必要な css や js を得る手段が必要であると考えられる．

2 または 3 の方法を選択する場合にはいずれにせよスマートフォン用のファイルを得る手段が必要なため，今回は実装が簡単な 2 の方法を選択することにする． 3 の方法は今後の改善要求として 別チケットで作成しておくこととする．（ #3043 ）

その他¶

本チケットではPC版のスキンプラグインが読み込まれてしまうという問題であるが， 管理画面で設定したカスタマイズCSSもPC版用として設定されたものが適用されてしまう．この問題については別チケットで対応する．（ #3040 ）

「使用されていないプロパティ displayMember/displayCommunity が残っていますが、これについては別チケットで扱う」としていましたが，対象チケットがなく，影響範囲もないということで本チケットで同時に修正を行います．

更新履歴 9c524c3855389400e2b397766a954e60ab16afa6 で適用されました。