Merge branch 'master' of https://github.com/designcreateplay/NodeBB

public/templates/noscript/topic.tpl

@@ -1,6 +1,7 @@
 		<ul class="posts">
 			<!-- BEGIN main_posts -->
 			<li>
+				<a name="{main_posts.pid}"></a>
 				<div class="row">
 					<div class="col-lg-2 profile">
 						<img class="img-thumbnail" src="{main_posts.picture}" /><br />
@@ -14,6 +15,7 @@
 			<!-- END main_posts -->
 			<!-- BEGIN posts -->
 			<li>
+				<a name="{posts.pid}"></a>
 				<div class="row">
 					<div class="col-lg-2 profile">
 						<img class="img-thumbnail" src="{posts.picture}" /><br />

src/feed.js

@@ -5,10 +5,13 @@
 		topics = require('./topics.js'),
 		fs = require('fs'),
 		rss = require('node-rss'),
-		winston = require('winston');
+		winston = require('winston'),
+		path = require('path');
 
 	function saveFeed(location, feed) {
-		fs.writeFile(location, rss.getFeedXML(feed), function (err) {
+		var	savePath = path.join(__dirname, '../', location);
+
+		fs.writeFile(savePath, rss.getFeedXML(feed), function (err) {
 			if(err) {
 				winston.err(err);
 			}
@@ -21,18 +24,19 @@
 			feed_url,
 			description,
 			author,
-			xml_url, 
+			xml_url,
 			{
 				'urn' : urn
 			}
-		);	
+		);
 	}
 
 
 	Feed.updateTopic = function(tid, cid) {
+		winston.info('[RSS] Updating RSS feeds for topic ' + tid);
 		var cache_time_in_seconds = 60;
 
-		topics.getTopicWithPosts(tid, 0, function(err, topicData) {
+		topics.getTopicWithPosts(tid, 0, 0, -1, function(err, topicData) {
 			if (err) winston.error('Problem saving topic RSS feed', err.stack);
 
 			var location = '/topic/' + topicData.slug,
@@ -79,14 +83,14 @@
 
 			var urn = 'urn:' + cid;
 			var feed = createFeed(categoryData.category_name, '', location, xml_url, 'NodeBB', urn); // not exactly sure if author for a category should be site_title?
-			
+
 			var title;
 			var topics = categoryData.topics;
 
 			for (var i = 0, ii = topics.length; i < ii; i++) {
 				urn = 'urn:' + cid + ':' + topics[i].tid;
 				title = topics[i].title + '. Posted on ' + (new Date(parseInt(topics[i].timestamp, 10)).toUTCString());
-				
+
 				feed.addNewItem(
 					title,
 					location,

src/routes/api.js

@@ -110,7 +110,7 @@ var user = require('./../user.js'),
 
 		app.get('/api/topic/:id/:slug?', function(req, res, next) {
 			var uid = (req.user) ? req.user.uid : 0;
-			topics.getTopicWithPosts(req.params.id, uid, function(err, data) {
+			topics.getTopicWithPosts(req.params.id, uid, 0, 10, function(err, data) {
 				if(data.deleted === '1' && data.expose_tools === 0) {
 					return res.json(404, {});
 				}

src/topics.js

@@ -324,7 +324,7 @@ var	RDB = require('./redis.js')
 
 	}
 
-	Topics.getTopicWithPosts = function(tid, current_user, callback) {
+	Topics.getTopicWithPosts = function(tid, current_user, start, end, callback) {
 		threadTools.exists(tid, function(exists) {
 			if (!exists)
 				return callback(new Error('Topic tid \'' + tid + '\' not found'));
@@ -338,7 +338,7 @@ var	RDB = require('./redis.js')
 			}
 
 			function getTopicPosts(next) {
-				Topics.getTopicPosts(tid, 0, 10, current_user, function(topicPosts, privileges) {
+				Topics.getTopicPosts(tid, start, end, current_user, function(topicPosts, privileges) {
 					next(null, topicPosts);
 				});
 			}

src/webserver.js

@@ -225,7 +225,7 @@ var express = require('express'),
 
 			async.waterfall([
 				function(next) {
-					topics.getTopicWithPosts(tid, ((req.user) ? req.user.uid : 0), function(err, topicData) {
+					topics.getTopicWithPosts(tid, ((req.user) ? req.user.uid : 0), 0, -1, function(err, topicData) {
 						if(topicData) {
 							if(topicData.deleted === '1' && topicData.expose_tools === 0)
 								return next(new Error('Topic deleted'), null);