Put checks in on init to deal with empty decks, fixes imakewebthings#51

Author: imakewebthings

core/deck.core.js

@@ -233,7 +233,9 @@ that use the API provided by core.
 				});
 			});
 
-			updateStates();
+			if (slides.length) {
+				updateStates();
+			}
 
 			// Show deck again now that slides are in place
 			$container.removeClass(options.classes.loading);

extensions/hash/deck.hash.js

@@ -65,9 +65,10 @@ slide.
 
 	$d.bind('deck.init', function() {
 	   var opts = $[deck]('getOptions');
-		$internals = $();
+		$internals = $(),
+		slides = $[deck]('getSlides');
 
-		$.each($[deck]('getSlides'), function(i, $el) {
+		$.each(slides, function(i, $el) {
 			var hash;
 
 			/* Hand out ids to the unfortunate slides born without them */
@@ -96,7 +97,9 @@ slide.
 		}
 
 		/* Set up first id container state class */
-		$[deck]('getContainer').addClass(opts.classes.onPrefix + $[deck]('getSlide').attr('id'));
+		if (slides.length) {
+			$[deck]('getContainer').addClass(opts.classes.onPrefix + $[deck]('getSlide').attr('id'));
+		};
 	})
 	/* Update permalink, address bar, and state class on a slide change */
 	.bind('deck.change', function(e, from, to) {

test/fixtures/empty.html

@@ -0,0 +1,19 @@
+<div class="deck-container">
+	<a href="#" class="deck-prev-link">Previous</a>
+	<a href="#" class="deck-next-link">Next</a>
+	
+	<p class="deck-status">
+		<span class="deck-status-current"></span>
+		/
+		<span class="deck-status-total"></span>
+	</p>
+	
+	<form action="." method="get" class="goto-form">
+		<label for="goto-slide">Go to slide:</label>
+		<input type="number" name="slidenum" id="goto-slide" value="">
+		<input type="submit" value="Go">
+	</form>
+	
+	<a href="#" class="deck-permalink">#</a>
+	<a href="#custom-id" id="internal-test">Internal Test Link</a>
+</div>

test/spec.core.js

@@ -415,4 +415,15 @@ describe('Deck JS', function() {
 		  expect($.deck('getSlide').find('iframe').data('src')).toBeUndefined();
 		});
 	});
+	
+	describe('empty deck', function() {
+		beforeEach(function() {
+			loadFixtures('empty.html');
+			$.deck('.slide');
+		});
+		
+		describe('getSlide()', function() {
+			it('should not error on init', $.noop);
+		});
+	});
 });