Project

General

Profile

Enhancement(機能追加・改善) #2619

不要なガジェットの削除

Added by Shouta Kashiwagi almost 8 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2011-11-25
Due date:
% Done:

0%


Description

Overview (概要)

OpenPNE3のインストール(./symfony openpne:install 実行)後のデフォルトで登録されているガジェットのうち、不要なものを削除する。

Spec (仕様)

(1) 残すガジェットを以下に限定した。

ホーム画面 : インフォメーション、プロフィール写真、フレンドリスト、参加コミュニティリスト

プロフィール: プロフィール写真、フレンドリスト、参加コミュニティリスト、プロフィール

サイドバナー: 言語切り替え

携帯ホーム : インフォメーション

携帯プロフィール:フレンドリスト、参加コミュニティリスト

携帯ログイン : ログイン画面

(2) 以下のものについてはガジェットのスペースそのものを削除する.

・携帯版ヘッダーガジェット
・携帯版フッターガジェット
・デイリーニュース(PCメールアドレス向け)ガジェット
・デイリーニュース(携帯メールアドレス向け)ガジェット

(※2012/04/16 追記: 3.6.Xからのアップデート時にスペースそのものが削除されるのは不親切な為,一旦(2)実装は取り消します.)


Related issues

Related to OpenPNE 3 - Backport(バックポート) #2971: 不要なガジェットの削除 Fixed(完了) 2011-11-25

Associated revisions

Revision 6877bdd9 (diff)
Added by Shouta Kashiwagi almost 8 years ago

(refs #2619) remove default gadgets

History

#1 Updated by Shouta Kashiwagi almost 8 years ago

(3) 不要ガジェット削除に伴い、バンドルプラグインもopAuthMailAddressPlugin以外一旦すべてバンドルしないこととする。

#2 Updated by Youichi Kimura over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#3 Updated by Youichi Kimura over 7 years ago

  • Status changed from New(新規) to Accepted(着手)

#4 Updated by Youichi Kimura over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#5 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8RC1

#6 Updated by Yuma Sakata over 7 years ago

  • Description updated (diff)

#7 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

(2) 以下のものについてはガジェットのスペースそのものを削除する。

・携帯版ヘッダーガジェット
・携帯版フッターガジェット
・デイリーニュース(PCメールアドレス向け)ガジェット
・デイリーニュース(携帯メールアドレス向け)ガジェット

とありますが,実際には下記コード部などを例として,ガジェットのスペースは削除されていません.この変更ではデフォルトで追加されるガジェットが削除されているという状態になっています.しかし,実際にスペース自体を削除してしまうと 3.6 -> 3.8 のバージョンアップによって,今まで表示されていたものが表示されなくなり,代替方法もなくなるという問題が発生するため,デフォルトのガジェットを減らすという変更で問題ないと思います.

apps/mobile_frontend/modules/member/actions/actions.class.php

 20   public function executeHome($request)
 21   {
 22     $this->gadgetConfig = sfConfig::get('op_mobile_gadget_list');
 23 
 24     $gadgets = Doctrine::getTable('Gadget')->retrieveGadgetsByTypesName('mobile');
 25     $this->mobileTopGadgets = $gadgets['mobileTop'];
 26     $this->mobileContentsGadgets = $gadgets['mobileContents'];
 27     $this->mobileBottomGadgets = $gadgets['mobileBottom'];
 28 
 29     $filteredCategory = $this->filterConfigCategory();
 30     $this->categories = $filteredCategory['category'];
 31     $this->categoryCaptions = $filteredCategory['captions'];
 32 
 33     return parent::executeHome($request);
 34   }

また,

(3) 不要ガジェット削除に伴い、バンドルプラグインも opAuthMailAddressPlugin 以外一旦すべてバンドルしないこととする。

とありますが,実際には opMessagePlugin がバンドルされる予定となっています.

まとめ

チケットの本文の記述が適切なものに修正してください.

#8 Updated by Shouta Kashiwagi over 7 years ago

  • Description updated (diff)

#9 Updated by Shouta Kashiwagi over 7 years ago

  • Description updated (diff)

#10 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#11 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#12 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

コードから,下記ガジェットがインストール時のデフォルトから削除されるということを確認しました.

  • PC版トップページ検索窓
  • PC版トップページサイドバナー
  • PC版トップページバースデー
  • 携帯版トップページバースデー
  • PC版プロフィールページバースデー
  • 携帯版プロフィールページバースデー

スペース自体が削除される点については議論は別のチケットとして本チケットはレビューOKとしておきます.

#13 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Review(レビュー待ち)
  • % Done changed from 70 to 50

チケットを間違えました.

#14 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

本チケットに紐付いたコミットは master でのコミットのため stable-3.8.x のブランチには含まれていないため差し戻しとしておきます.

#15 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.8RC1 to OpenPNE 3.9.0-old

#16 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#2971 でレビューOKとなったのでテスト待ちにします.
また,当チケットでは,OpenPNE-3.9.0 (master)で扱うことにします.

#17 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8RC1 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF