From 86e4d0a1eaa17302878e2772caa46dd26ceb9174 Mon Sep 17 00:00:00 2001
From: Shinichi Urabe <urabe@tejimaya.com>
Date: Sun, 20 Mar 2011 22:16:01 +0900
Subject: [PATCH] fixed bugs.

---
 .../opIdCall/actions/actions.class.php        | 23 +++++++++++--------
 1 file changed, 13 insertions(+), 10 deletions(-)

diff --git a/apps/mobile_mail_frontend/modules/opIdCall/actions/actions.class.php b/apps/mobile_mail_frontend/modules/opIdCall/actions/actions.class.php
index babe8fc..d7cb269 100644
--- a/apps/mobile_mail_frontend/modules/opIdCall/actions/actions.class.php
+++ b/apps/mobile_mail_frontend/modules/opIdCall/actions/actions.class.php
@@ -12,7 +12,7 @@ class opIdCallActions extends sfActions
   public function preExecute()
   {
     $this->member = $this->getRoute()->getMember();
-    if ($this->member)
+    if (!$this->member)
     {
       exit;
     }
@@ -100,7 +100,7 @@ public function executeDiaryComment(sfWebRequest $request)
     {
       return sfView::NONE;
     }
-    if (!$this->isDiaryPostable($diaryComment->Diary))
+    if (!$this->isDiaryPostable($diary = $diaryComment->Diary))
     {
       return sfView::NONE;
     }
@@ -252,15 +252,18 @@ private function postCommunityTopicComment($communityTopic)
     return $communityTopicComment->save();
   }
 
-  private function postActivityData($replyActivityData)
+  private function postActivityData($activityData)
   {
-    $activityData = new ActivityData();
-    $activityData->setInReplyTo($replyActivityData);
-    $activityData->setMember($this->member);
-    $activityData->setBody($this->generateBody());
-    $activityData->setIsPc($replyActivityData->is_pc);
-    $activityData->setIsMobile($replyActivityData->is_mobile);
-    $activityData->setPublicFlag($this->member->getConfig(MemberConfigIdCallForm::ID_CALL_ACTIVITY_PUBLIC_FLAG, ActivityDataTable::PUBLIC_FLAG_SNS));
+    $options = array(
+      'public_flag' => $this->member->getConfig(
+        MemberConfigIdCallForm::ID_CALL_ACTIVITY_PUBLIC_FLAG,
+        ActivityDataTable::PUBLIC_FLAG_SNS
+      ),
+      'is_pc' => $activityData->is_pc,
+      'is_mobile' => $activityData->is_mobile,
+    );
+
+    return Doctrine_Core::getTable('ActivityData')->updateActivity($this->member->id, $this->generateBody(), $options);
   }
 
   private function generateBody($toName)