Prevent multiple request for the same image

doomhz · Mar 25, 2011 · 13a2858 · 13a2858
1 parent 251094c
commit 13a2858
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/assets/jquery.imageCache.js b/assets/jquery.imageCache.js
@@ -32,13 +32,16 @@
 				var src = $(img).attr('src') || $(img).attr('data-src');
 				if (localStorage) {
 					var localSrc = localStorage[src];
-					if (localSrc) {
+					if (localSrc != null && localSrc != 'false') {
 						$(img).attr('src', localSrc);
 					} else {
-						$.get(self.config.base64ImageEncoderPath + src, function (data) {
-							localStorage[src] = data;
-							$(img).attr('src', data);
-						});
+						$(img).attr('src', src);
+						if (localStorage[src] === null) {
+							localStorage[src] = 'false';
+							$.get(self.config.base64ImageEncoderPath + src, function (data) {
+								localStorage[src] = data;
+							});
+						}
 					}
 				}
 			});

diff --git a/demo.html b/demo.html
@@ -24,6 +24,8 @@ <h1>jQuery Image Cache plugin</h1>
 	<ul>
 	  <li><img src="" data-src="assets/test-image-4.jpg" class="imageCache" /></li>
 	  <li><img src="" data-src="assets/test-image-5.jpg" class="imageCache" /></li>
+	  <li><img src="" data-src="assets/test-image-5.jpg" class="imageCache" /></li>
+	  <li><img src="" data-src="assets/test-image-5.jpg" class="imageCache" /></li>
 	</ul>
 
 	<script type="text/javascript">