alchemymvc 1.3.16 → 1.3.17

package/lib/app/datasource/mongo_datasource.js

@@ -182,39 +182,45 @@ Mongo.decorateMethod(Blast.Decorators.memoize({ignore_arguments: true}), functio
  * @since    0.2.0
  * @version  1.3.16
  *
+ * @param    {String}     name
  * @param    {Function}   callback
+ *
+ * @return   {Pledge<Collection>}
  */
 Mongo.setMethod(function collection(name, callback) {
 
-	var that = this;
+	const pledge = new Pledge();
 
-	if (this.collections[name]) {
-		setImmediate(function cachedCollection() {
-			callback(null, that.collections[name]);
-		});
+	pledge.done(callback);
 
-		return;
+	if (this.collections[name]) {
+		pledge.resolve(this.collections[name]);
+		return pledge;
 	}
 
+	const that = this;
+
 	this.connect().done(function gotConnection(err, db) {
 
 		if (err) {
-			return callback(err);
+			return pledge.reject(err);
 		}
 
 		Pledge.done(that.connection.collection(name), function createdCollection(err, collection) {
 
 			if (err) {
-				return callback(err);
+				return pledge.reject(err);
 			}
 
 			// Cache the collection
 			that.collections[name] = collection;
 
 			// Return it to the callback
-			callback(null, collection);
+			pledge.resolve(collection);
 		});
 	});
+
+	return pledge;
 });
 
 /**
@@ -560,43 +566,59 @@ Mongo.setMethod(function _remove(model, query, options, callback) {
  *
  * @author   Jelle De Loecker   <jelle@develry.be>
  * @since    0.2.0
- * @version  1.1.0
+ * @version  1.3.17
  */
 Mongo.setMethod(function _ensureIndex(model, index, callback) {
 
-	this.collection(model.table, function gotCollection(err, collection) {
-
-		var options,
-		    obj,
-		    key;
+	this.collection(model.table, async function gotCollection(err, collection) {
 
 		if (err != null) {
 			return callback(err);
 		}
 
-		options = {
-			name: index.options.name
+		let options = {
+			name   : index.options.name,
+			unique : index.options.unique ? true : false,
+			sparse : index.options.sparse ? true : false,
 		};
 
-		if (index.options.unique) {
-			options.unique = true;
-		}
-
-		if (index.options.sparse) {
-			options.sparse = true;
-		}
+		let index_specs;
 
 		// Hack in the text indexes
 		if (options.name == 'text') {
-			obj = {};
+			let key;
+
+			index_specs = {};
 
 			for (key in index.fields) {
-				obj[key] = 'text';
+				index_specs[key] = 'text';
 			}
-
-			collection.createIndex(obj, options, callback);
 		} else {
-			collection.createIndex(index.fields, options, callback);
+			index_specs = index.fields;
 		}
+
+		try {
+			await collection.createIndex(index_specs, options);
+		} catch (err) {
+
+			// Check for IndexOptionsConflict
+			if (err.code === 85) {
+
+				try {
+
+					// Index already exists, drop it
+					await collection.dropIndex(options.name);
+
+					// Try again
+					await collection.createIndex(index_specs, options);
+				} catch (second_err) {
+					return callback(second_err);
+				}
+			} else {
+				return callback(err);
+			}
+		}
+
+		callback();
 	});
 });

package/lib/app/element/time_ago.js

@@ -1,3 +1,5 @@
+const TIMEOUT_ID = Symbol('timeout_id');
+
 /**
  * The custom al-time-ago element
  *
@@ -27,8 +29,6 @@ TimeAgo.setAttribute(function date(str) {
 	return this._date;
 }, function setValue(value) {
 
-	var date;
-
 	if (value) {
 		this._date = Date.create(value);
 	} else {
@@ -73,57 +73,40 @@ TimeAgo.setProperty(function value() {
  * @version  0.1.0
  */
 TimeAgo.setMethod(function connected() {
-	this.refresh(true);
+	this.refresh();
 });
 
 /**
  * Refresh the content
  *
- * @author   Jelle De Loecker   <jelle@develry.be>
+ * @author   Jelle De Loecker   <jelle@elevenways.be>
  * @since    0.1.0
- * @version  0.1.7
- *
- * @param    {Boolean}   reset_timer
+ * @version  1.3.17
  */
-TimeAgo.setMethod(function refresh(reset_timer) {
+TimeAgo.setMethod(function refresh() {
 
-	let that = this,
-	    counter,
-	    timer,
-	    text = this.value,
-	    diff;
+	let text = this.value;
 
 	if (this.textContent != text) {
 		this.textContent = text;
 	}
 
-	if (!reset_timer || Blast.isNode) {
+	if (Blast.isNode || !this.isConnected || !this.date) {
 		return;
 	}
 
-	if (this._timer) {
-		clearInterval(this._timer);
-		this._timer = null;
+	let diff = Math.abs(Date.now() - this.date),
+	    timer;
+
+	if (diff < 1000 * 70) {
+		timer = 1000;
+	} else {
+		timer = 1000 * 29;
 	}
 
-	if (this.date) {
-		diff = Math.abs(Date.now() - this.date);
-		counter = 0;
-
-		if (diff < 1000 * 29) {
-			timer = 1000;
-		} else {
-			timer = 1000 * 29;
-		}
-
-		this._timer = setInterval(function doUpdate() {
-			counter++;
-
-			if (counter > 60) {
-				that.refresh(true);
-			} else {
-				that.refresh();
-			}
-		}, timer);
+	if (this[TIMEOUT_ID]) {
+		clearTimeout(this[TIMEOUT_ID]);
 	}
+
+	this[TIMEOUT_ID] = setTimeout(() => this.refresh(), timer);
 });

package/lib/app/helper/alchemy_helper.js

@@ -292,7 +292,7 @@ Alchemy.setMethod(function group(name, id, callback) {
  *
  * @author   Jelle De Loecker   <jelle@elevenways.be>
  * @since    0.5.0
- * @version  1.3.7
+ * @version  1.3.17
  *
  * @param    {String|Object}   options
  * @param    {Object}          data
@@ -325,7 +325,7 @@ Alchemy.setMethod(function segment(options, data) {
 			conduit = that.view.server_var('conduit');
 
 			if (!conduit) {
-				return next();
+				return next(null, '<!-- Failed to render segment ' + options.name + ': no conduit found -->');
 			}
 
 			options.params = data;