プロジェクト

全般

プロフィール

Backport(バックポート) #3865

完了

PickHomeLayoutForm クラスで返り値が void となるメソッドを返り値に指定している

isao sano さんが約9年前に追加. ほぼ9年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
対象バージョン:
開始日:
2015-11-09
期日:
進捗率:

100%

予定工数:

説明

概要

PickHomeLayoutForm クラスで返り値が void となるメソッドを返り値に指定している

下記 73 行目では $snsConfig (SnsConfig クラス)の save() の返り値を用いているが,これは opDoctrineRecord クラスの save() となり,これは return を使ってないので void となる.

apps/pc_backend/modules/design/lib/PickHomeLayoutForm.class.php
 56 
 57   public function save()
 58   {
...
 64     $snsConfig = Doctrine::getTable('SnsConfig')->retrieveByName($this->layoutName);
 65     if (!$snsConfig)
 66     {
 67       $snsConfig = new SnsConfig();
 68       $snsConfig->setName($this->layoutName);
 69     }
 70     $value = $this->choices[$this->values['layout']];
 71     $snsConfig->setValue($value);
 72 
 73     return (bool)$snsConfig->save();
 74   }

lib/util/opDoctrineRecord.class.php
 56   public function save(Doctrine_Connection $conn = null)
 57   {
 58     if (is_null($conn))
 59     {
 60       $conn = opDoctrineQuery::chooseConnection(true);
 61     }
 62 
 63     parent::save($conn);
 64   }

関連するチケット 1 (0件未完了1件完了)

関連している OpenPNE 3 - Bug(バグ) #3492: PickHomeLayoutForm クラスで返り値が void となるメソッドを返り値に指定しているWon't fix(対応せず)Chiharu Nakajima2013-11-20

操作

isao sano さんが約9年前に更新

  • 関連している Bug(バグ) #3492: PickHomeLayoutForm クラスで返り値が void となるメソッドを返り値に指定している を追加

isao sano さんが約9年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

isao sano さんが約9年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

https://github.com/openpne/OpenPNE3/pull/275
にてプルリクエストを行いました。

レビューお願いします。

Rimpei Ogawa さんがほぼ9年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

Chiharu Nakajima さんがほぼ9年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更
  • 担当者isao sano から Chiharu Nakajima に変更

プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/290

Rimpei Ogawa さんがほぼ9年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

isao sano さんがほぼ9年前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

単体試験完了しました。
問題ありません。
マージお願いします。

Rimpei Ogawa さんがほぼ9年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

他の形式にエクスポート: Atom PDF