From 629d616de5275164349608827ad83e8013b4b7ed Mon Sep 17 00:00:00 2001
From: czaks <marcin@6irc.net>
Date: Sat, 20 Jul 2013 20:01:22 -0400
Subject: [PATCH] post search: internationalize and use config values

---
 search.php                 | 31 +++++++++++++++++++++----------
 templates/search_form.html |  8 ++++----
 2 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/search.php b/search.php
index d2dbdf24..30c05df1 100644
--- a/search.php
+++ b/search.php
@@ -1,11 +1,15 @@
 <?php
 	require 'inc/functions.php';
 	
-	$queries_per_minutes = Array(15, 2);
-	$queries_per_minutes_all = Array(50, 2);
-	$search_limit = 100;
+	if (!$config['search']['enable']) {
+		die(_("Post search is disabled"));
+	}
+
+	$queries_per_minutes = $config['search']['queries_per_minutes'];
+	$queries_per_minutes_all = $config['search']['queries_per_minutes_all'];
+	$search_limit = $config['search']['search_limit'];
 	
-	$boards = Array('new', 'r9k', 'v', 'edu', 'azn', 'h', 'meta');
+	$boards = $config['search']['boards'];
 	
 	$body = Element('search_form.html', Array('boards' => $boards, 'board' => isset($_POST['board']) ? $_POST['board'] : false, 'search' => isset($_POST['search']) ? str_replace('"', '&quot;', utf8tohtml($_POST['search'])) : false));
 	
@@ -18,13 +22,13 @@
 		$query->bindValue(':time', time() - ($queries_per_minutes[1] * 60));
 		$query->execute() or error(db_error($query));
 		if($query->fetchColumn() > $queries_per_minutes[0])
-			error('Wait a while before searching again, please.');
+			error(_('Wait a while before searching again, please.'));
 		
 		$query = prepare("SELECT COUNT(*) FROM `search_queries` WHERE `time` > :time");
 		$query->bindValue(':time', time() - ($queries_per_minutes_all[1] * 60));
 		$query->execute() or error(db_error($query));
 		if($query->fetchColumn() > $queries_per_minutes_all[0])
-			error('Wait a while before searching again, please.');
+			error(_('Wait a while before searching again, please.'));
 			
 		
 		$query = prepare("INSERT INTO `search_queries` VALUES (:ip, :time, :query)");
@@ -34,6 +38,11 @@
 		$query->execute() or error(db_error($query));
 		
 		_syslog(LOG_NOTICE, 'Searched /' . $_POST['board'] . '/ for "' . $phrase . '"');
+
+		// Cleanup search queries table
+		$query = prepare("DELETE FROM `search_queries` WHERE `time` <= :time");
+		$query->bindValue(':time', time() - ($queries_per_minutes_all[1] * 60));
+                $query->execute() or error(db_error($query));
 		
 		openBoard($_POST['board']);
 		
@@ -44,7 +53,7 @@
 			$name = $m[2];
 			$value = isset($m[4]) ? $m[4] : $m[3];
 			
-			if(!in_array($name, Array('id', 'thread', 'subject', 'name'))) {
+			if(!in_array($name, array('id', 'thread', 'subject', 'name'))) {
 				// unknown filter
 				return $m[0];
 			}
@@ -116,7 +125,7 @@
 		
 		if($query->rowCount() == $search_limit) {
 			_syslog(LOG_WARNING, 'Query too broad.');
-			$body .= '<p class="unimportant" style="text-align:center">(Query too broad.)</p>';
+			$body .= '<p class="unimportant" style="text-align:center">('._('Query too broad.').')</p>';
 			echo Element('page.html', Array(
 				'config'=>$config,
 				'title'=>'Search',
@@ -136,7 +145,9 @@
 		}
 		
 		if(!empty($temp))
-			$_body .= '<fieldset><legend>' . $query->rowCount() . ' result' . ($query->rowCount() != 1 ? 's' : '') . ' in <a href="/' .
+			$_body .= '<fieldset><legend>' . $query->rowCount() . 
+					sprintf(ngettext('%d result in', '%d results in', $query->rowCount()), 
+					$query->rowCount) . ' <a href="/' .
 					sprintf($config['board_path'], $board['uri']) . $config['file_index'] .
 			'">' .
 			sprintf($config['board_abbreviation'], $board['uri']) . ' - ' . $board['title'] .
@@ -146,7 +157,7 @@
 		if(!empty($_body))
 			$body .= $_body;
 		else
-			$body .= '<p style="text-align:center" class="unimportant">(No results.)</p>';
+			$body .= '<p style="text-align:center" class="unimportant">('._('No results.').')</p>';
 	}
 		
 	echo Element('page.html', Array(
diff --git a/templates/search_form.html b/templates/search_form.html
index 16b5d4af..a391ac53 100644
--- a/templates/search_form.html
+++ b/templates/search_form.html
@@ -1,11 +1,11 @@
 <div class="ban">
-	<h2>Search</h2>
+	<h2>{% trans %}Search{% endtrans %}</h2>
 	<form style="display:inline" action="" method="post">
 		<p>
-			<label style="display:inline" for="search">Phrase:</label> 
+			<label style="display:inline" for="search">{% trans %}Phrase:{% endtrans %}</label> 
 			<input id="search" name="search" type="text" size="55" value="{{ search }}">
 			<select name="board">
-				<option value="none">Select board&hellip;</option>
+				<option value="none">{% trans %}Select board{% endtrans %}&hellip;</option>
 				{% for b in boards %}
 					<option value="{{ b }}"{% if b == board %} selected{% endif %}>/{{ b }}/</option>
 				{% endfor %}
@@ -14,6 +14,6 @@
 		</p>
 	</form>
 	<p style="font-size:8pt;margin:5px">
-		Search is case-insensitive and based on keywords. To match exact phrases, use "quotes". Use an asterisk (*) for wildcard.</p><p style="font-size:8pt;margin:5px">You may apply the following filters to your searches: <strong>id</strong>, <strong>thread</strong>, <strong>subject</strong>, and <strong>name</strong>. To apply a filter, simply add to your query, for example, <em>name:Anonymous</em> or <em>subject:"Some Thread"</em>. Wildcards cannot be used in filters.
+		{% trans %}Search is case-insensitive and based on keywords. To match exact phrases, use "quotes". Use an asterisk (*) for wildcard.</p><p style="font-size:8pt;margin:5px">You may apply the following filters to your searches: <strong>id</strong>, <strong>thread</strong>, <strong>subject</strong>, and <strong>name</strong>. To apply a filter, simply add to your query, for example, <em>name:Anonymous</em> or <em>subject:"Some Thread"</em>. Wildcards cannot be used in filters.{% endtrans %}
 	</p>
 </div>