diff --git a/upload/source/class/helper/helper_pm.php b/upload/source/class/helper/helper_pm.php
index b457359b6fab37c22697fb953c74b2ae41ff3f6e..a8150bf696597858d41edae343365b3d2e2279e4 100644
--- a/upload/source/class/helper/helper_pm.php
+++ b/upload/source/class/helper/helper_pm.php
@@ -19,14 +19,11 @@ class helper_pm {
 		if($fromid === '') {
 			$fromid = $_G['uid'];
 		}
-		$author = '';
 		if($fromid) {
 			if($fromid == $_G['uid']) {
 				$sendpmmaxnum = $_G['group']['allowsendpmmaxnum'];
-				$author = $_G['username'];
 			} else {
 				$user = getuserbyuid($fromid);
-				$author = $user['username'];
 				loadcache('usergroup_'.$user['groupid']);
 				$sendpmmaxnum = $_G['cache']['usergroup_'.$user['groupid']]['allowsendpmmaxnum'];
 			}
@@ -40,7 +37,7 @@ class helper_pm {
 		$return = uc_pm_send($fromid, $toid, addslashes($subject), addslashes($message), 1, $replypmid, $isusername, $type);
 		if($return > 0 && $fromid) {
 			foreach(explode(',', $fromid) as $v) {
-				useractionlog($fromid, 'pmid');
+				useractionlog($v, 'pmid');
 			}
 		}
 
diff --git a/upload/template/default/home/space_pm.htm b/upload/template/default/home/space_pm.htm
index bbadbbd04cc08b91bfbc7b00988e72bab32e9f26..7b8dc1bf398a047b4eadf8d746a34deee8cfbb7f 100644
--- a/upload/template/default/home/space_pm.htm
+++ b/upload/template/default/home/space_pm.htm
@@ -384,8 +384,8 @@
 								<!--{loop $grouppms $grouppm}-->
 									<dl id="gpmlist_$grouppm[id]" class="bbda cur1 cl{if !$gpmstatus[$grouppm[id]]} newpm{/if}">
 										<dd class="y mtm pm_o">
-											<a href="javascript:;" id="pm_o_$grouppm[id]" class="o" onmouseover="showMenu({'ctrlid':this.id, 'pos':'34'})">{lang menu}</a>
-											<div id="pm_o_$grouppm[id]_menu" class="p_pop" style="display: none;">
+											<a href="javascript:;" id="pm_g_$grouppm[id]" class="o" onmouseover="showMenu({'ctrlid':this.id, 'pos':'34'})">{lang menu}</a>
+											<div id="pm_g_$grouppm[id]_menu" class="p_pop" style="display: none;">
 												<ul>
 													<li><a href="javascript:;" onclick="ajaxget('home.php?mod=spacecp&ac=pm&op=delete&deletesubmit=1&deletepm_gpmid[]=$grouppm[id]', '', 'ajaxwaitid', '', 'none', '$(\'gpmlist_$grouppm[id]\').style.display=\'none\';');">{lang delete}</a></li>
 												</ul>