correct behaviour of map_files_without_tags and rename to map_tracks_…

…without_tags
jonocole · Jun 16, 2009 · c3e92bb · c3e92bb
1 parent f67a5eb
commit c3e92bb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/resolvers/boffin/BoffinDb.h b/resolvers/boffin/BoffinDb.h
@@ -49,10 +49,10 @@ class BoffinDb
 
     // for each track without tags, call f(track_id, artist_sortname, album_sortname, track_sortname)
     template<typename Functor>
-    void map_files_without_tags(Functor f)
+    void map_tracks_without_tags(Functor f)
     {
         sqlite3pp::query qry(m_db, 
-            "SELECT pd.file_join.track, pd.artist.sortname, pd.album.sortname, pd.track.sortname FROM pd.file_join "
+            "SELECT DISTINCT pd.file_join.track, pd.artist.sortname, pd.album.sortname, pd.track.sortname FROM pd.file_join "
             "INNER JOIN pd.artist ON pd.file_join.artist = pd.artist.id "
             "LEFT JOIN pd.album ON pd.file_join.album = pd.album.id "
             "INNER JOIN pd.track ON pd.file_join.track = pd.track.id "

diff --git a/resolvers/boffin/Tagger.cpp b/resolvers/boffin/Tagger.cpp
@@ -105,7 +105,7 @@ int main(int argc, char *argv[])
         BoffinDb db(argv[2], argv[1]);
 
         // build up the request:
-        db.map_files_without_tags(track_out);
+        db.map_tracks_without_tags(track_out);
         ossreq.flush();
         issreq = new istringstream(ossreq.str());