操作
Backport(バックポート) #3866
完了PickHomeLayoutForm クラスで返り値が void となるメソッドを返り値に指定している
開始日:
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 }
isao sano さんが約9年前に更新
- 関連している Bug(バグ) #3492: PickHomeLayoutForm クラスで返り値が void となるメソッドを返り値に指定している を追加
isao sano さんが約9年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/276
にてプルリクエストを行いました。
レビューお願いします。
Rimpei Ogawa さんが約9年前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
Chiharu Nakajima さんが約9年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
- 担当者 を isao sano から Chiharu Nakajima に変更
プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/291
Rimpei Ogawa さんが約9年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Chiharu Nakajima さんが約9年前に更新
- ステータス を Pending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
- 進捗率 を 70 から 80 に変更
単体テストOK
Rimpei Ogawa さんが約9年前に更新
- ステータス を Pending Merge(マージ待ち) から Fixed(完了) に変更
- 進捗率 を 80 から 100 に変更
操作