OpenPNE 3

#1113 の変更によってコンフリクトが発生する部分について¶

9f3f558e9c6075454cbcc095f110bf7eaf9c5e73 の変更によってgetMobileUID()に引数を付与することができるようになった．
また 89da5b0c6632cfac9c668df9da331aca3a037332 の変更によってgetMobileUID()を用いる部分で必要な箇所について適宜引数が与えれるように変更された．

3.4 では 89da5b0c6632cfac9c668df9da331aca3a037332 の対応部分においてコンフリクトが発生する．具体的には以下の部分である．
apps/mobile_frontend/modules/member/actions/actions.class.php
3.4

106         if ($request->hasParameter('update'))
107         {
108           if (!$request->getMobileUID())
109           {
110             $this->getUser()->setFlash('error', 'Your mobile UID was not registered.');
111             $this->redirect('member/configUID');
112           }
113 
114           $memberConfig = Doctrine::getTable('MemberConfig')->retrieveByNameAndMemberId('mobile_uid', $this->getUser()->getMemberId());
115           if (!$memberConfig)
116           {
117             $memberConfig = new MemberConfig();
118             $memberConfig->setMember($this->getUser()->getMember()); 
119             $memberConfig->setName('mobile_uid');
120           }
121           $memberConfig->setValue($request->getMobileUID());       
122           $memberConfig->save();
123           $this->getUser()->setFlash('notice', 'Your mobile UID was set successfully.');
124           $this->redirect('member/configUID');
125         }
126         elseif ($request->hasParameter('delete') && $this->isDeletableUid)
127         {
128           $mobileUid->delete();
129           $this->getUser()->setFlash('notice', 'Your mobile UID was deleted successfully.'); 
130           $this->redirect('member/configUID');
131         }

3.7

 82         if ($request->hasParameter('update'))
 83         {
 84           $cookieUid = sfContext::getInstance()->getResponse()->generateMobileUidCookie();
 85 
 86           if (!$request->getMobileUID(false) && !$cookieUid)
 87           {
 88             $this->getUser()->setFlash('error', 'Your mobile UID was not registered.');
 89             $this->redirect('member/configUID');
 90           }
 91 
 92           $member = $this->getUser()->getMember();
 93           $member->setConfig('mobile_uid', $request->getMobileUID(false));
 94           if ($cookieUid)
 95           {
 96             $member->setConfig('mobile_cookie_uid', $cookieUid);
 97           }
 98 
 99           $this->getUser()->setFlash('notice', 'Your mobile UID was set successfully.');
100           $this->redirect('member/configUID');
101         }
102         elseif ($request->hasParameter('delete') && $this->isDeletableUid)
103         {
104           $mobileUid->delete();
105           sfContext::getInstance()->getResponse()->deleteMobileUidCookie();
106           $this->getUser()->setFlash('notice', 'Your mobile UID was deleted successfully.');
107           $this->redirect('member/configUID');
108         }
109       }

#1113 は 3.4 では対応されていないため 3.4 の121行目と 3.7 の93行目について比較を行い， 3.4 の121行目のgetMobileUID()の引数にfalseが必要であるかどうかを検討したところ 内容的には 3.4 の内容とほぼ一致したことを確認したため必要であると判断した．

比較用コード断片¶

3.7 lib/model/doctrine/Member.class.php

 97   public function setConfig($configName, $value, $isDateTime = false)
 98   {
 99     Doctrine::getTable('MemberConfig')->setValue($this->getId(), $configName, $value, $isDateTime);
100   }

3.7 lib/model/doctrine/MemberConfigTable.class.php

 93   public function setValue($memberId, $name, $value, $isDateTime = false)
 94   {
 95     $config = $this->retrieveByNameAndMemberId($name, $memberId);
 96     if (!$config)
 97     {
 98       $config = new memberConfig();
 99       $config->setMemberId($memberId);
100       $config->setName($name);
101     }
102     if ($isDateTime)
103     {
104       $config->setValueDatetime($value);
105     }
106     $config->setValue($value);
107     $config->save();
108     $this->results[$memberId][$name] = $config;
109   }

メモ¶

#1810 の 534dd33ce9f2fbb46f747c74fd7bce558be95ca6 が原因でコンフリクトするが， #2241 で示される通り 3.4 では対応する必要がない．