Merge branch 'master' of git://github.com/Yousha/Tinyboard into yousha

README.md

@@ -1,6 +1,8 @@
-Tinyboard -- A lightweight PHP imageboard.
+Tinyboard - A lightweight PHP imageboard.
 ==========================================
 
+About
+------------
 Tinyboard is a light-weight, fast, highly configurable and user-friendly
 imageboard software package released under a non-restrictive open-source
 license. It is written in PHP and has few dependencies.
@@ -19,8 +21,11 @@ it need one.
 
 Contributing
 ------------
-Use GitHub to submit a pull request. If you need help developing a patch, join
+* Use GitHub to submit a pull request.
-our IRC channel.
+* Send feedbacks
+* Fix bugs or add features
+* Write/Edit the Wiki
+If you need help developing a patch, join our IRC channel.
 
 Installation
 -------------

inc/anti-bot.php

@@ -247,14 +247,11 @@ function checkSpam(array $extra_salt = array()) {
 	$query = prepare('SELECT `passed` FROM `antispam` WHERE `hash` = :hash');
 	$query->bindValue(':hash', $hash);
 	$query->execute() or error(db_error($query));
-	if (($passed = $query->fetchColumn(0)) === false) {
+	if ((($passed = $query->fetchColumn(0)) === false) || ($passed > $config['spam']['hidden_inputs_max_pass'])) {
 		// there was no database entry for this hash. most likely expired.
 		return true;
 	}
 
-	if ($passed > $config['spam']['hidden_inputs_max_pass'])
-		return true;
-	
 	return $hash;
 }
 
@@ -263,5 +260,3 @@ function incrementSpamHash($hash) {
 	$query->bindValue(':hash', $hash);
 	$query->execute() or error(db_error($query));
 }
-
-

inc/database.php

@@ -73,6 +73,7 @@ function prepare($query) {
 	
 	if ($config['debug'])
 		return new PreparedQueryDebug($query);
+
 	return $pdo->prepare($query);
 }
 
@@ -93,19 +94,19 @@ function query($query) {
 			'time' => '~' . $time
 		);
 		return $query;
-	} else {
-		return $pdo->query($query);
 	}
+
+	return $pdo->query($query);
 }
 
 function db_error($PDOStatement=null) {
 	global $pdo;
+
 	if (isset($PDOStatement)) {
 		$err = $PDOStatement->errorInfo();
 		return $err[2];
-	} else {
+	}
+
 	$err = $pdo->errorInfo();
 	return $err[2];
 }
-}
-

inc/functions.php

@@ -241,6 +241,7 @@ function create_antibot($board, $thread = null) {
 function rebuildThemes($action) {
 	// List themes
 	$query = query("SELECT `theme` FROM `theme_settings` WHERE `name` IS NULL AND `value` IS NULL") or error(db_error());
+
 	while ($theme = $query->fetch()) {
 		rebuildTheme($theme['theme'], $action);
 	}
@@ -327,6 +328,7 @@ function setupBoard($array) {
 
 function openBoard($uri) {
 	global $config;
+
 	if ($config['cache']['enabled'] && ($board = cache::get('board_' . $uri))) {
 		setupBoard($board);
 		return true;
@@ -341,7 +343,9 @@ function openBoard($uri) {
 			cache::set('board_' . $uri, $board);
 		setupBoard($board);
 		return true;
-	} else return false;
+	}
+
+	return false;
 }
 
 function boardTitle($uri) {
@@ -356,7 +360,9 @@ function boardTitle($uri) {
 	
 	if ($title = $query->fetch()) {
 		return $title['title'];
-	} else return false;
+	}
+
+	return false;
 }
 
 function purge($uri) {
@@ -546,9 +552,9 @@ function until($timestamp) {
 		return ($num = round($difference/(60*60*24))) . ' day' . ($num != 1 ? 's' : '');
 	} elseif ($difference < 60*60*24*365) {
 		return ($num = round($difference/(60*60*24*7))) . ' week' . ($num != 1 ? 's' : '');
-	} else {
-		return ($num = round($difference/(60*60*24*365))) . ' year' . ($num != 1 ? 's' : '');
 	}
+
+	return ($num = round($difference/(60*60*24*365))) . ' year' . ($num != 1 ? 's' : '');
 }
 
 function ago($timestamp) {
@@ -563,9 +569,9 @@ function ago($timestamp) {
 		return ($num = round($difference/(60*60*24))) . ' day' . ($num != 1 ? 's' : '');
 	} elseif ($difference < 60*60*24*365) {
 		return ($num = round($difference/(60*60*24*7))) . ' week' . ($num != 1 ? 's' : '');
-	} else {
-		return ($num = round($difference/(60*60*24*365))) . ' year' . ($num != 1 ? 's' : '');
 	}
+
+	return ($num = round($difference/(60*60*24*365))) . ' year' . ($num != 1 ? 's' : '');
 }
 
 function displayBan($ban) {
@@ -683,7 +689,9 @@ function threadExists($id) {
 	
 	if ($query->rowCount()) {
 		return true;
-	} else return false;
+	}
+
+	return false;
 }
 
 function post(array $post) {
@@ -841,10 +849,7 @@ function rebuildPost($id) {
 	$query->bindValue(':id', $id, PDO::PARAM_INT);
 	$query->execute() or error(db_error($query));
 	
-	if (!$post = $query->fetch())
+	if ((!$post = $query->fetch()) || !$post['body_nomarkup'])
-		return false;
-	
-	if (!$post['body_nomarkup'])
 		return false;
 	
 	markup($body = &$post['body_nomarkup']);
@@ -1101,10 +1106,7 @@ function makerobot($body) {
 }
 
 function checkRobot($body) {
-	if (empty($body))
+	if (empty($body) || event('check-robot', $body))
-		return true;
-	
-	if (event('check-robot', $body))
 		return true;
 	
 	$body = makerobot($body);
@@ -1114,15 +1116,14 @@ function checkRobot($body) {
 	
 	if ($query->fetch()) {
 		return true;
-	} else {
+	}
-		// Insert new hash
 
+	// Insert new hash
 	$query = prepare("INSERT INTO `robot` VALUES (:hash)");
 	$query->bindValue(':hash', $body);
 	$query->execute() or error(db_error($query));
 	return false;
 }
-}
 
 function numPosts($id) {
 	global $board;
@@ -1197,7 +1198,6 @@ function checkMute() {
 	}
 }
 
-
 function buildIndex() {
 	global $board, $config;
 	
@@ -1539,7 +1539,7 @@ function buildThread($id, $return=false, $mod=false) {
 
 	if ($return)
 		return $body;
-	else
+
 	file_write($board['dir'] . $config['dir']['res'] . sprintf($config['file_page'], $id), $body);
 }
 
@@ -1634,8 +1634,6 @@ function fraction($numerator, $denominator, $sep) {
 	return "{$numerator}{$sep}{$denominator}";
 }
 
-
-
 function getPostByHash($hash) {
 	global $board;
 	$query = prepare(sprintf("SELECT `id`,`thread` FROM `posts_%s` WHERE `filehash` = :hash", $board['uri']));
@@ -1706,4 +1704,3 @@ function DNS($host) {
 	
 	return $ip_addr;
 }
-

install.sql

@@ -44,7 +44,7 @@ CREATE TABLE IF NOT EXISTS `antispam` (
 --
 
 CREATE TABLE IF NOT EXISTS `bans` (
-  `id` int(11) NOT NULL AUTO_INCREMENT,
+  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
   `ip` varchar(45) NOT NULL,
   `mod` int(11) NOT NULL COMMENT 'which mod made the ban',
   `set` int(11) NOT NULL,
@@ -97,7 +97,7 @@ CREATE TABLE IF NOT EXISTS `cites` (
 --
 
 CREATE TABLE IF NOT EXISTS `ip_notes` (
-  `id` int(11) NOT NULL AUTO_INCREMENT,
+  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
   `ip` varchar(45) NOT NULL,
   `mod` int(11) DEFAULT NULL,
   `time` int(11) NOT NULL,
@@ -128,7 +128,7 @@ CREATE TABLE IF NOT EXISTS `modlogs` (
 --
 
 CREATE TABLE IF NOT EXISTS `mods` (
-  `id` smallint(6) NOT NULL AUTO_INCREMENT,
+  `id` smallint(6) unsigned NOT NULL AUTO_INCREMENT,
   `username` varchar(30) NOT NULL,
   `password` char(40) NOT NULL COMMENT 'SHA1',
   `type` smallint(1) NOT NULL COMMENT '0: janitor, 1: mod, 2: admin',
@@ -163,7 +163,7 @@ CREATE TABLE IF NOT EXISTS `mutes` (
 --
 
 CREATE TABLE IF NOT EXISTS `news` (
-  `id` int(11) NOT NULL AUTO_INCREMENT,
+  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
   `name` text NOT NULL,
   `time` int(11) NOT NULL,
   `subject` text NOT NULL,
@@ -179,7 +179,7 @@ CREATE TABLE IF NOT EXISTS `news` (
 --
 
 CREATE TABLE IF NOT EXISTS `noticeboard` (
-  `id` int(11) NOT NULL AUTO_INCREMENT,
+  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
   `mod` int(11) NOT NULL,
   `time` int(11) NOT NULL,
   `subject` text NOT NULL,
@@ -194,7 +194,7 @@ CREATE TABLE IF NOT EXISTS `noticeboard` (
 --
 
 CREATE TABLE IF NOT EXISTS `pms` (
-  `id` int(11) NOT NULL AUTO_INCREMENT,
+  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
   `sender` int(11) NOT NULL,
   `to` int(11) NOT NULL,
   `message` text NOT NULL,
@@ -210,7 +210,7 @@ CREATE TABLE IF NOT EXISTS `pms` (
 --
 
 CREATE TABLE IF NOT EXISTS `reports` (
-  `id` int(11) NOT NULL AUTO_INCREMENT,
+  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
   `time` int(11) NOT NULL,
   `ip` varchar(45) NOT NULL,
   `board` varchar(120) DEFAULT NULL,