Merge pull request #9262 from Sesquipedalian/3.0/post2

[3.0] Minor fixes for SMF\Actions\Post2

Author: Sesquipedalian

Sources/Actions/Post2.php

@@ -658,7 +658,7 @@ public function submit(): void
 		}
 
 		// Log an act of moderation - modifying.
-		if (!empty($moderationAction)) {
+		if (!empty($this->moderation_action)) {
 			Logging::logAction('modify', ['topic' => $this->existing_msg->id_topic, 'message' => $this->existing_msg->id, 'member' => $this->existing_msg->id_member, 'board' => $this->existing_msg->id_board]);
 		}
 
@@ -1031,13 +1031,13 @@ protected function prepareEdit(): void
 			User::$me->isAllowedTo('modify_replies');
 
 			// If you're modifying a reply, I say it better be logged...
-			$moderationAction = true;
+			$this->moderation_action = true;
 		} else {
 			User::$me->isAllowedTo('modify_any');
 
 			// Log it, assuming you're not modifying your own post.
 			if ($this->existing_msg->id_member != User::$me->id) {
-				$moderationAction = true;
+				$this->moderation_action = true;
 			}
 		}
 
@@ -1055,7 +1055,7 @@ protected function prepareEdit(): void
 		Utils::$context['poster_id'] = $this->existing_msg->id_member;
 
 		// Can they approve it?
-		$approve_checked = (!empty($REQUEST['approve']) ? true : false);
+		$approve_checked = (!empty($_REQUEST['approve']) ? true : false);
 		$this->becomes_approved = Config::$modSettings['postmod_active'] ? ($this->can_approve && !$this->existing_msg->approved ? $approve_checked : $this->existing_msg->approved > 0) : true;
 
 		if (!User::$me->allowedTo('moderate_forum') || !$this->authorIsGuest) {