updated how tasks are added to queue

lib/epubjs/queue.js

@@ -7,17 +7,39 @@ EPUBJS.Queue = function(_context){
 
 // Add an item to the queue
 EPUBJS.Queue.prototype.enqueue = function(task, args, context) {
+  var deferred, promise;
+  var queued;
+  
   // Handle single args without context
   if(args && !args.length) {
     args = [args];
   }
+  
+  if(typeof task === "function"){
+    
+    deferred = new RSVP.defer();
+    promise = deferred.promise;
+    
+    queued = {
+      "task" : task,
+      "args"     : args,
+      "context"  : context,
+      "deferred" : deferred,
+      "promise" : promise
+    };
+    
+  } else {
+    // Task is a promise
+    queued = {
+      "promise" : task
+    };
+    
+  }
+  
+  this._q.push(queued);
 
-  this._q.push({
+  
-    "task" : task,
+  return queued.promise;
-    "args"     : args,
-    "context"  : context
-  });
-  return this._q;
 };
 
 // Run one item
@@ -28,12 +50,15 @@ EPUBJS.Queue.prototype.dequeue = function(){
     inwait = this._q.shift();
     task = inwait.task;
 
-    if(typeof task === "function"){
+    if(task){
       // Task is a function that returns a promise
-      return task.apply(inwait.context || this.context, inwait.args);
+      return task.apply(inwait.context || this.context, inwait.args).then(function(){
+        inwait.deferred.resolve.apply(inwait.context || this.context, arguments);
+      }.bind(this));
+      
     } else {
       // Task is a promise
-      return task;
+      return inwait.promise;
     }
 
   } else {
@@ -71,13 +96,11 @@ EPUBJS.Queue.prototype.length = function(){
 
 // Create a new tast from a callback
 EPUBJS.Task = function(task, args, context){
-  var toApply = args || [];
-  var scope = context || this.context;
 
   return function(){
-
+    var toApply = arguments || [];
+    
     return new RSVP.Promise(function(resolve, reject) {
-
       var callback = function(value){
         resolve(value);
       };
@@ -85,9 +108,9 @@ EPUBJS.Task = function(task, args, context){
       toApply.push(callback);
 
       // Apply all arguments to the functions
-      task.apply(scope, toApply);
+      task.apply(this, toApply);
     
-    });
+  }.bind(this));
 
   };
 

lib/epubjs/rendition.js

@@ -143,41 +143,29 @@ EPUBJS.Rendition.prototype.bounds = function() {
 };
 
 EPUBJS.Rendition.prototype.display = function(what){
-	var displaying = new RSVP.defer();
+	
-	var displayed = displaying.promise;
+	return this.q.enqueue(this.load, what);
 
-	// Check for fragments
-	if(typeof what === 'string') {
-		what = what.split("#")[0];
-	}
-
-	this.q.enqueue(this.move, what);
-
-
-	return displayed;
 };
 
-EPUBJS.Rendition.prototype.move = function(what){
+EPUBJS.Rendition.prototype.load = function(what){
 	var displaying = new RSVP.defer();
 	var displayed = displaying.promise;
 
 	var section = this.book.spine.get(what);
 	var view;
-
+		
 	this.displaying = true;
 
 	if(section){
 		view = new EPUBJS.View(section);
-		// Clear views
+		
-		// this.clear();
+		// This will clear all previous views
 		this.fill(view);
-		// rendered = this.render(section);
+		
+		// Move to correct place within the section, if needed
+		// this.moveTo(what)
 
-		// if(this.settings.infinite) {
-		// 	rendered.then(function(){
-		// 		return this.fill.call(this);
-		// 	}.bind(this));
-		// }
 		this.check();
 		
 		view.displayed.then(function(){
@@ -193,6 +181,11 @@ EPUBJS.Rendition.prototype.move = function(what){
 	return displayed;
 };
 
+// Takes a cfi, fragment or page?
+EPUBJS.Rendition.prototype.moveTo = function(what){
+	
+};
+
 EPUBJS.Rendition.prototype.render = function(view) {
 	// var view = new EPUBJS.View(section);
 	// 

lib/epubjs/spine.js

@@ -54,6 +54,8 @@ EPUBJS.Spine.prototype.get = function(target) {
   } else if(target && target.indexOf("#") === 0) {
     index = this.spineById[target.substring(1)];
   } else if(target) {
+    // Remove fragments
+    target = target.split("#")[0];
     index = this.spineByHref[target];
   }