Merge pull request #124 from Financial-Times/perf-n-bugs

Perf n bugs

lib/api/search.js

@@ -74,7 +74,7 @@ module.exports = function (term, opts) {
 	}
 
 	var requestBody = {
-		queryString: term,
+		queryString: term.replace(':', ':='),
 		queryContext: {
 			curations: ["ARTICLES", "BLOGS"]
 		},

lib/models/article.js

@@ -4,6 +4,7 @@ var _ = require('lodash');
 // body text processing
 var cheerio = require('cheerio');
 var dom = require('../utils/dom');
+var readingSpeed = 300; // wpm
 
 function Article (obj) {
 	if (obj && obj.item) {
@@ -23,13 +24,7 @@ Article.prototype.parse = function (obj) {
 	this.id = obj.item.id;
 	this.raw = obj;
 
-	var readingSpeed = 300; // wpm
-
 	if (!this.raw.item.body) return;
-
-	// FIXME - move to defineProperty getters
-	this.wordCount = this.body.split(' ').length;
-	this.readingTime = Math.round(this.wordCount / readingSpeed);
 };
 
 /**
@@ -61,6 +56,25 @@ Object.defineProperty(Article.prototype, 'outboundLinks', {
 	}
 });
 
+
+/**
+ * Returns the article body wordcount
+ */
+Object.defineProperty(Article.prototype, 'wordCount', {
+	get: function () {
+		return this.body.split(' ').length;
+	}
+});
+
+/**
+ * Returns an estimate of the reading time for the article
+ */
+Object.defineProperty(Article.prototype, 'readingTime', {
+	get: function () {
+		return Math.round(this.wordCount / readingSpeed);
+	}
+});
+
 /**
  * Returns the article headline
  */