From bba731d5b4085d7f5b929be41b2114b4dcc7595e Mon Sep 17 00:00:00 2001
From: marcus <marcus@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Thu, 15 Jan 2009 11:01:20 +0000
Subject: Editing post no longer takes ownership

git-svn-id: https://code.elgg.org/elgg/trunk@2570 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 mod/groups/actions/forums/edittopic.php | 78 +++++++++++++++++----------------
 1 file changed, 41 insertions(+), 37 deletions(-)

(limited to 'mod/groups/actions')

diff --git a/mod/groups/actions/forums/edittopic.php b/mod/groups/actions/forums/edittopic.php
index bff8dbe53..3f46292a8 100644
--- a/mod/groups/actions/forums/edittopic.php
+++ b/mod/groups/actions/forums/edittopic.php
@@ -26,54 +26,58 @@
 		$topic_guid = get_input('topic');
 		$access = get_input('access_id');
 		$group_guid = get_input('group_guid');
-		$user = $_SESSION['user']->getGUID(); // you need to be logged in to comment on a group forum
+		//$user = $_SESSION['user']->getGUID(); // you need to be logged in to comment on a group forum
 		$status = get_input('status'); // sticky, resolved, closed
 		
 	// Convert string of tags into a preformatted array
 		 $tagarray = string_to_tag_array($tags);
 		
 	// Make sure we actually have permission to edit
-		$topic = get_entity($topic_guid);
-		
-		if ($topic->getSubtype() == "groupforumtopic") {
-	
-		// Convert string of tags into a preformatted array
-			$tagarray = string_to_tag_array($tags);
+		$topic = get_entity($topic_guid);
+		if ($topic)
+		{
+		
+			$user = $topic->getOwner(); 
 			
-		// Make sure the title isn't blank
-			if (empty($title) || empty($message)) {
-				register_error(elgg_echo("groupstopic:blank"));
+			if ($topic->getSubtype() == "groupforumtopic") {
 		
-		// Otherwise, save the forum
-			} else {
-				
-		        $topic->access_id = $access;
-				
-		// Set its title
-				$topic->title = $title;
+			// Convert string of tags into a preformatted array
+				$tagarray = string_to_tag_array($tags);
 				
-		// if no tags are present, clear existing ones
-				if (is_array($tagarray)) {
-					$topic->tags = $tagarray;
-				} else $topic->clearMetadata('tags');
-				
-		// edit metadata
-		       $topic->status = $status; // the current status i.e sticky, closed, resolved
-		       
-        // now let's edit the message annotation
-			   update_annotation($message_id, "group_topic_post", $message, "",$user, $access);
-		       
-		// save the changes
-            if (!$topic->save()) {
-			//		register_error(elgg_echo("forumtopic:error"));
-			}
+			// Make sure the title isn't blank
+				if (empty($title) || empty($message)) {
+					register_error(elgg_echo("groupstopic:blank"));
 			
-           // Success message
-				system_message(elgg_echo("groups:forumtopic:edited"));
+			// Otherwise, save the forum
+				} else {
+					
+			        $topic->access_id = $access;
+					
+			// Set its title
+					$topic->title = $title;
+					
+			// if no tags are present, clear existing ones
+					if (is_array($tagarray)) {
+						$topic->tags = $tagarray;
+					} else $topic->clearMetadata('tags');
+					
+			// edit metadata
+			       $topic->status = $status; // the current status i.e sticky, closed, resolved
+			       
+	        // now let's edit the message annotation
+				   update_annotation($message_id, "group_topic_post", $message, "",$user, $access);
+			       
+			// save the changes
+	            if (!$topic->save()) {
+				//		register_error(elgg_echo("forumtopic:error"));
+				}
 				
-		    }
-        }
-		
+	           // Success message
+					system_message(elgg_echo("groups:forumtopic:edited"));
+					
+			    }
+	        }
+		}
 		// Forward to the group forum page
 	        global $CONFIG;
 	        $url = $CONFIG->wwwroot . "pg/groups/forum/{$group_guid}/";
-- 
cgit v1.2.3