From 7ff92215def3ad11b90f517461fa7db440a1218f Mon Sep 17 00:00:00 2001
From: Paul Arthur <paul.arthur@flowerysong.com>
Date: Thu, 20 Sep 2012 11:44:42 -0400
Subject: FS#142 - Clicking on song results in downloading

Change the title link in a song row so it links to the song details.

While there's no real value in obfuscating the fact that a stream
can be downloaded, there is value in a consistent interface. The
song title being a stream link is inconsistent with other parts of
the row (such as album and artist), inconsistent with other times we
display songs (such as Recently Played), and doesn't consistently match
the currently selected playback method.
---
 templates/show_song_row.inc.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'templates')

diff --git a/templates/show_song_row.inc.php b/templates/show_song_row.inc.php
index 5913e64a..2e55076d 100644
--- a/templates/show_song_row.inc.php
+++ b/templates/show_song_row.inc.php
@@ -30,7 +30,7 @@
 <td class="cel_add">
 	<?php echo Ajax::button('?action=basket&type=song&id=' . $song->id,'add', T_('Add'),'add_' . $song->id); ?>
 </td>
-<td class="cel_song"><a href="<?php echo Song::play_url($song->id); ?>" title="<?php echo scrub_out($song->title); ?>"><?php echo $song->f_title; ?></a></td>
+<td class="cel_song"><?php echo $song->f_link; ?></a></td>
 <td class="cel_artist"><?php echo $song->f_artist_link; ?></td>
 <td class="cel_album"><?php echo $song->f_album_link; ?></td>
 <td class="cel_tags"><?php echo $song->f_tags; ?></td>
-- 
cgit