Added album-chooser to the interface, fixed some backend bugs

data/interfaces/default/album.html

@@ -1,5 +1,5 @@
 <%inherit file="base.html" />
-<%! 
+<%!
 	from headphones import db, helpers
 	myDB = db.DBConnection()
 %>
@@ -16,7 +16,28 @@
 			%else:
 			<a id="menu_link_retry" href="#" onclick="doAjaxCall('queueAlbum?AlbumID=${album['AlbumID']}&ArtistID=${album['ArtistID']}&new=False', $(this),true);" data-success="Retrying the same version of '${album['AlbumTitle']}'">Retry Download</a>
 			<a id="menu_link_new" href="#" onclick="doAjaxCall('queueAlbum?AlbumID=${album['AlbumID']}&ArtistID=${album['ArtistID']}&new=True', $(this),true);" data-success="Looking for a new version of '${album['AlbumTitle']}'">Try New Version</a>
-			%endif			
+			%endif		
+			<a class="menu_link_edit" id="album_chooser" href="#">Choose Alternate Version</a>
+				<div id="dialog" title="Choose an Alternate Release" style="display:none" class="configtable">
+					<div class="links">
+						<%
+							alternate_albums = myDB.select("SELECT * from allalbums WHERE AlbumID=?", [album['AlbumID']])
+						%>
+						%if not alternate_albums:
+							<p>No alternate versions found. Try refreshing the artist</p>
+						%else:
+							%for alternate_album in alternate_albums:
+								<%
+									if alternate_album['AlbumID'] == alternate_album['ReleaseID']:
+										alternate_album_name = "Headphones Default Release"
+									else:
+										alternate_album_name = alternate_album['AlbumTitle'] + " (" + alternate_album['ReleaseCountry'] + ", " + alternate_album['ReleaseFormat'] + ")"
+								%>
+								<a href="switchAlbum?AlbumID=${album['AlbumID']}&ReleaseID=${alternate_albums['ReleaseID']}">${alternate_album_name}</a><br>
+							%endfor
+						%endif
+					</div>
+				</div>	
 		</div>
 	</div>	
 	<a href="artistPage?ArtistID=${album['ArtistID']}" class="back">&laquo; Back to ${album['ArtistName']}</a>
@@ -144,6 +165,10 @@
 	}
 	
 	$(document).ready(function()	{
+		$('#album_chooser').click(function() {
+			$('#dialog').dialog();
+			return false;
+		});
 		getAlbumInfo();
 		getAlbumArt();
 		initActions();

headphones/__init__.py

@@ -853,7 +853,6 @@ def dbcheck():
         c.execute('SELECT ReleaseID from albums')
     except sqlite3.OperationalError:
         c.execute('ALTER TABLE albums ADD COLUMN ReleaseID TEXT DEFAULT NULL')
-        c.execute('UPDATE TABLE albums SET ReleaseID = AlbumID')
         
     try:
         c.execute('SELECT ReleaseFormat from albums')

headphones/importer.py

@@ -293,6 +293,8 @@ def addArtisttoDB(artistid, extrasonly=False):
         # If there is a release, check the ReleaseID against the AlbumID to see if they differ (user updated)
         if not rg_exists:
             releaseid = rg['id']
+        elif rg_exists and not rg_exists['ReleaseID']:
+            releaseid = rg['id']
         else:
             releaseid = rg_exists['ReleaseID']
         
@@ -300,9 +302,9 @@ def addArtisttoDB(artistid, extrasonly=False):
 
         controlValueDict = {"AlbumID":  rg['id']}
 
-        newValueDict = {"ArtistID":         artistid,
+        newValueDict = {"ArtistID":         album['ArtistID'],
-                        "ArtistName":       artist['artist_name'],
+                        "ArtistName":       album['ArtistName'],
-                        "AlbumTitle":       rg['title'],
+                        "AlbumTitle":       album['AlbumTitle'],
                         "ReleaseID":        album['ReleaseID'],
                         "AlbumASIN":        album['AlbumASIN'],
                         "ReleaseDate":      album['ReleaseDate'],
@@ -572,8 +574,8 @@ def getHybridRelease(fullreleaselist):
         
         release_dict = {
             'hasasin':        bool(release['AlbumASIN']),
-            'asin':           release['AlbumASIN']
+            'asin':           release['AlbumASIN'],
-            'trackscount':    len(release['Tracks'],
+            'trackscount':    len(release['Tracks']),
             'releaseid':      release['ReleaseID'],
             'releasedate':    release['ReleaseDate'],
             'format':         format,
@@ -597,7 +599,7 @@ def getHybridRelease(fullreleaselist):
 
     sortable_release_list.sort(key=lambda x:getSortableReleaseDate(x['releasedate']))
 
-    average_tracks = sum(x['trackscount'] for x in releaselist) / float(len(releaselist))
+    average_tracks = sum(x['trackscount'] for x in sortable_release_list) / float(len(sortable_release_list))
     for item in sortable_release_list:
         item['trackscount_delta'] = abs(average_tracks - item['trackscount'])