Adds caching of album art, making the first retrieval the only slow one

core/exthook.py

@@ -2,15 +2,21 @@
 import pprint
 import musicbrainzngs
 from musicbrainzngs import ResponseError
+from core.models import Album
 pp = pprint.PrettyPrinter(indent=4)
 
-
 def init():
 	musicbrainzngs.set_useragent("mymusiclist","0.1", contact="none")
 	musicbrainzngs.set_rate_limit(limit_or_interval=False, new_requests=1)
 	print("[Exthook] Initialized")
 	
-def getAlbumArtByName(name, artist):
+def verifyAlbumArt(album, artist):
+	if(album.art == None):
+		album.art = fetchAlbumArt(album.album_name, artist)
+    	album.save()
+    #may or may not add checks to see if valid http link
+
+def fetchAlbumArt(name, artist):
 	debug("GET " + name + " " + artist)
 	results = musicbrainzngs.search_release_groups(name, limit = 3, artistname=artist, primarytype="album")
 	#pp.pprint(results['release-group-list'])
@@ -21,9 +27,11 @@ def getAlbumArtByName(name, artist):
 		imagelist = musicbrainzngs.get_release_group_image_list(resultid)
 	except ResponseError:
 		debug("No image found.")
-		return "NaN"
+		return None
+	except NetworkError:
+		debug("Cannot connect to MB server")
+		return None
 	return imagelist['images'][0]['thumbnails']['large']
 
-
 def debug(string):
 	print("[Exthook] " + string)
\ No newline at end of file

core/models.py

@@ -12,6 +12,7 @@ class Album(models.Model):
 	album_name = models.CharField(max_length=64)
 	year = models.DecimalField(max_digits=4, decimal_places=0)
 	artist = models.ForeignKey(Artist)
+	art = models.URLField(null=True, blank=True)
 	def __str__(self):
 		return self.album_name
 

search/views.py

@@ -32,5 +32,5 @@ def album_profile(request, identifier):
     result = Album.objects.filter(id = identifier)
     artist_name = result[0].artist.name
     songs = Song.objects.filter(album = identifier)
-    albumart = exthook.getAlbumArtByName(result[0].album_name, artist_name)
-    return render(request, 'album.html', {'result':result[0], 'songs': songs, 'artist': artist_name, 'image': albumart})
+    exthook.verifyAlbumArt(result[0], artist_name)
+    return render(request, 'album.html', {'result':result[0], 'songs': songs, 'artist': artist_name})
\ No newline at end of file

templates/album.html

@@ -4,7 +4,7 @@
 
 
 {% block sidebar %}
-<img src = "{{image}}" alt= "No album art available">
+<img src = "{{result.art}}" alt= "No album art available">
 <p>
 	<b> {{result.album_name}} </b> ({{result.year}}) <br>
 	{{artist}} <br> <br>