Included comments in notifications

Author: rbonick

controller/rating_controller.php

@@ -103,7 +103,7 @@ public function __construct(\phpbb\auth\auth $auth, \phpbb\config\config $config
 	/**
 	* Display the post rating page
 	*
-	* @param string $mode		Mode taken from the URL 
+	* @param string $mode		Mode taken from the URL
 	* @param int $post_id		Post ID taken from the URL
 	* @return Symfony\Component\HttpFoundation\Response A Symfony Response object
 	* @access public
@@ -434,7 +434,7 @@ public function post($mode, $post_id)
 		// Save vote
 		if ($submit)
 		{
-			// Prevent cheater to break the forum permissions to give negative points or give more points than they can 
+			// Prevent cheater to break the forum permissions to give negative points or give more points than they can
 			if (!$this->auth->acl_get('f_rs_rate_negative', $row['forum_id']) && $points < 0 || $points < 0 && $this->config['rs_min_rep_negative'] && ($this->user->data['user_reputation'] < $this->config['rs_min_rep_negative']) || $this->config['rs_enable_power'] && (($points > $max_voting_allowed) || ($points < -$max_voting_allowed)))
 			{
 				$submit = false;
@@ -483,6 +483,7 @@ public function post($mode, $post_id)
 				'user_id_from'		=> $this->user->data['user_id'],
 				'post_id'			=> $post_id,
 				'post_subject'		=> $row['post_subject'],
+				'comment' => $comment,
 			);
 
 			$notification_type = ($points > 0) ? 'pico.reputation.notification.type.rate_post_positive' : 'pico.reputation.notification.type.rate_post_negative';
@@ -825,7 +826,7 @@ public function user($uid)
 
 		if ($submit)
 		{
-			//Prevent cheater to break the forum permissions to give negative points or give more points than they can 
+			//Prevent cheater to break the forum permissions to give negative points or give more points than they can
 			if (!$this->auth->acl_get('u_rs_rate_negative') && $points < 0 || $points < 0 && $this->config['rs_min_rep_negative'] && ($this->user->data['user_reputation'] < $this->config['rs_min_rep_negative']) || $this->config['rs_enable_power'] && (($points > $max_voting_allowed) || ($points < -$max_voting_allowed)))
 			{
 				$submit = false;
@@ -898,4 +899,4 @@ public function user($uid)
 
 		return $this->helper->render('rateuser.html', $this->user->lang('RS_USER_RATING'));
 	}
-}
+}

controller/reputation_common.php

@@ -0,0 +1,59 @@
+<?php
+/**
+*
+* Reputation System
+*
+* @copyright (c) 2014 Lukasz Kaczynski
+* @license GNU General Public License, version 2 (GPL-2.0)
+*
+*/
+
+/**
+* DO NOT CHANGE
+*/
+if (!defined('IN_PHPBB'))
+{
+	exit;
+}
+
+if (empty($lang) || !is_array($lang))
+{
+	$lang = array();
+}
+
+// DEVELOPERS PLEASE NOTE
+//
+// All language files should use UTF-8 as their encoding and the files must not contain a BOM.
+//
+// Placeholders can now contain order information, e.g. instead of
+// 'Page %s of %s' you can (and should) write 'Page %1$s of %2$s', this allows
+// translators to re-order the output of data while ensuring it remains correct
+//
+// You do not need this where single placeholders are used, e.g. 'Message %d' is fine
+// equally where a string contains only two placeholders which are used to wrap text
+// in a url you again do not need to specify an order e.g., 'Click %sHERE%s' is fine
+//
+// Some characters you may want to copy&paste:
+// ’ » “ ” …
+//
+
+$lang = array_merge($lang, array(
+	'REPUTATION'		=> 'Reputation',
+
+	'RS_DISABLED'		=> 'Sorry, but the board administrator has disabled this feature.',
+
+	'RS_COMMENT'		=> 'Comment',
+	'RS_POINTS'			=> 'Points',
+
+	'RS_POST_REPUTATION'		=> 'Post reputation',
+	'RS_POST_RATED'			=> 'You have rated this post',
+	'RS_RATE_POST_POSITIVE'	=> 'Rate post positive',
+	'RS_RATE_POST_NEGATIVE'	=> 'Rate post negative',
+	'RS_RATE_USER'			=> 'Rate user',
+	'RS_VIEW_DETAILS'		=> 'View details',
+
+	'NOTIFICATION_TYPE_REPUTATION'		=> 'Someone gives you reputation point',
+	'NOTIFICATION_RATE_POST_POSITIVE'	=> '<strong>Rated positively</strong> by %s for post %s',
+	'NOTIFICATION_RATE_POST_NEGATIVE'	=> '<strong>Rated negatively</strong> by %s for post %s',
+	'NOTIFICATION_RATE_USER'			=> '<strong>Rated</strong> by %s',
+));

language/en/reputation_common.php

@@ -53,7 +53,7 @@
 	'RS_VIEW_DETAILS'		=> 'View details',
 
 	'NOTIFICATION_TYPE_REPUTATION'		=> 'Someone gives you reputation point',
-	'NOTIFICATION_RATE_POST_POSITIVE'	=> '<strong>Rated positively</strong> by %s for post',
-	'NOTIFICATION_RATE_POST_NEGATIVE'	=> '<strong>Rated negatively</strong> by %s for post',
+	'NOTIFICATION_RATE_POST_POSITIVE'	=> '<strong>Rated positively</strong> by %s for post %s',
+	'NOTIFICATION_RATE_POST_NEGATIVE'	=> '<strong>Rated negatively</strong> by %s for post %s',
 	'NOTIFICATION_RATE_USER'			=> '<strong>Rated</strong> by %s',
 ));

notification/type/rate_post_positive.php

@@ -228,6 +228,7 @@ public function get_title()
 		return $this->user->lang(
 			$this->language_key,
 			phpbb_generate_string_list($usernames, $this->user),
+			censor_text($this->get_data('post_subject')),
 			$trimmed_voting_users_cnt
 		);
 	}
@@ -239,9 +240,12 @@ public function get_title()
 	*/
 	public function get_reference()
 	{
+		if (empty($this->get_data('comment'))) {
+			return null;
+		}
 		return $this->user->lang(
 			'NOTIFICATION_REFERENCE',
-			censor_text($this->get_data('post_subject'))
+			censor_text($this->get_data('comment'))
 		);
 	}
 
@@ -305,6 +309,7 @@ public function create_insert_array($data, $pre_create_data = array())
 		$this->set_data('user_id_from', $data['user_id_from']);
 		$this->set_data('post_id', $data['post_id']);
 		$this->set_data('post_subject', $data['post_subject']);
+		$this->set_data('comment', $data['comment']);
 
 		parent::create_insert_array($data, $pre_create_data);
 	}