fire noneLeft on single page scenario

fieg · fieg · commit 3eb308b08f34 · 2018-03-20T21:01:12.000+01:00
diff --git a/src/jquery-ias.js b/src/jquery-ias.js
@@ -388,8 +388,12 @@
 
     this.nextUrl = this.getNextUrl();
 
+    if (!this.nextUrl) {
+      this.fire('noneLeft', [this.getLastItem()]);
+    }
+
     // start loading next page if content is shorter than page fold
-    if (currentScrollOffset >= scrollThreshold) {
+    if (this.nextUrl && currentScrollOffset >= scrollThreshold) {
       this.next();
 
       // flag as initialized when rendering is completed
@@ -498,6 +502,10 @@
     if (event === 'ready' && this.isInitialized) {
       $.proxy(callback, this)();
     }
+    // same applies to noneLeft
+    else if (event === 'noneLeft' && !this.nextUrl) {
+      $.proxy(callback, this)();
+    }
 
     return this;
   };
@@ -567,7 +575,6 @@
 
           if (!self.nextUrl) {
             self.fire('noneLeft', [self.getLastItem()]);
-            self.listeners['noneLeft'].disable(); // disable it so it only fires once
           }
 
           self.resume();
diff --git a/test/02-listeners-test.js b/test/02-listeners-test.js
@@ -189,4 +189,29 @@ describe("IAS", function () {
 
     return deferred.promise;
   });
+
+  it("should call noneLeft listeners when content is short without next", function() {
+    var deferred = when.defer();
+    var spy1 = this.spy();
+
+    jQuery.ias('destroy');
+
+    loadFixture("short-without-next.html", function() {
+      jQuery.ias({
+        container : '.listing',
+        item: '.post',
+        pagination: '.navigation',
+        next: '.next-posts a',
+        initialize: true
+      })
+          // register listener
+          .on('noneLeft', spy1);
+
+      expect(spy1).toHaveBeenCalledOnce();
+
+      deferred.resolve();
+    });
+
+    return deferred.promise;
+  });
 });
diff --git a/test/buster.js b/test/buster.js
@@ -32,6 +32,7 @@ config["My tests"] = {
     { path: "/page3.html",  file: "test/fixtures/page3.html" },
     { path: "/framed.html", file: "test/fixtures/framed.html" },
     { path: "/short.html",  file: "test/fixtures/short.html" },
+    { path: "/short-without-next.html",  file: "test/fixtures/short-without-next.html" },
     { path: "/ajax1.html",   file: "test/fixtures/ajax1.html" },
     { path: "/ajax2.html",   file: "test/fixtures/ajax2.html" },
     { path: "/multiple1.html",   file: "test/fixtures/multiple1.html" },
diff --git a/test/fixtures/short-without-next.html b/test/fixtures/short-without-next.html
@@ -0,0 +1,29 @@
+<!DOCTYPE html>
+<html>
+<head>
+    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
+    <title>Very short Page 1</title>
+</head>
+<body>
+<div id="content">
+    <div class="listing">
+        <div class="post">
+            <strong>item 1</strong>
+            <p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
+        </div>
+
+        <div class="post">
+            <strong>item 2</strong>
+            <p>Pellentesque habitant morbi tristique senectus et netus et malesuada fames ac turpis egestas. Vestibulum tortor quam, feugiat vitae, ultricies eget, tempor sit amet, ante. Donec eu libero sit amet quam egestas semper. Aenean ultricies mi vitae est. Mauris placerat eleifend leo. Quisque sit amet est et sapien ullamcorper pharetra. Vestibulum erat wisi, condimentum sed, commodo vitae, ornare sit amet, wisi. Aenean fermentum, elit eget tincidunt condimentum, eros ipsum rutrum orci, sagittis tempus lacus enim ac dui. Donec non enim in turpis pulvinar facilisis. Ut felis.</p>
+        </div>
+    </div>
+
+    <div class="navigation">
+        <ul>
+            <li>1</li>
+        </ul>
+    </div>
+</div>
+
+</body>
+</html>
