Merge pull request #202 from Soundnode/fixes

code refactoring and fix on formating song duration

Author: weblancaster

app/public/js/common/appCtrl.js

@@ -1,6 +1,6 @@
 'use strict';
 
-app.controller('AppCtrl', function ($rootScope, $scope, $window, $log, ngDialog, hotkeys) {
+app.controller('AppCtrl', function ($rootScope, $scope, $window, $log, ngDialog) {
 
     // Settings sub nav (dropdown)
     $scope.isSettingsVisible = false;
@@ -11,7 +11,7 @@ app.controller('AppCtrl', function ($rootScope, $scope, $window, $log, ngDialog,
         } else {
             $scope.isSettingsVisible = true;
         }
-    }
+    };
 
     // check if track has Art work
     // otherwise replace to Soundnode App logo
@@ -28,17 +28,16 @@ app.controller('AppCtrl', function ($rootScope, $scope, $window, $log, ngDialog,
 
     // Format song duration on tracks
     // for human reading
-    $scope.formatSongDuration = function(duration) {
+    $scope.formatSongDuration = function (duration) {
         var minutes = Math.floor(duration / 60000)
             , seconds = ((duration % 60000) / 1000).toFixed(0);
-        
+
         return minutes + ":" + (seconds < 10 ? '0' : '') + seconds;
     };
 
     /*
     * Navigation back and forward
     */
-
     $scope.goBack = function() {
         $window.history.back();
     };

app/public/js/system/settings.js

@@ -2,4 +2,4 @@
 window.settings = window.settings || {};
 
 // App version
-window.settings.appVersion = '0.5.0';
+window.settings.appVersion = '0.5.1';

app/views/common/tracks.html

@@ -20,7 +20,7 @@ <h2 class="songList_item_song_tit">{{ data.title }}</h2>
 
     <h3 class="songList_item_song_user">
       <a ui-sref="profile({id: {{data.user.id}}})">{{ data.user.username }}</a>
-      <span class="songList_item_song_length"> {{ data.duration }} {{ formatSongDuration (data.duration) }}</span>
+      <span ng-controller="AppCtrl" class="songList_item_song_length"> {{ formatSongDuration (data.duration) }}</span>
     </h3>
 
     <div class="songList_item_song_details">