MobileLibraryBrowserPresenter now creates a new MobileLibraryBrowserV…

…iew when necessary. Related to issue #408.
ycastonguay · Feb 20, 2013 · 974c281 · 974c281
1 parent 95f1847
commit 974c281
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 6 deletions.
diff --git a/MPfm/MPfm.MVP/Navigation/MobileNavigationManager.cs b/MPfm/MPfm.MVP/Navigation/MobileNavigationManager.cs
@@ -203,7 +203,8 @@ public virtual IMobileLibraryBrowserView CreateMobileLibraryBrowserView(MobileNa
                 // The view list can be accessed from different threads.
                 lock (_locker)
                 {
-                    var presenter = Bootstrapper.GetContainer().Resolve<IMobileLibraryBrowserPresenter>(new NamedParameterOverloads() {{"tabType", tabType}});
+                    var presenter = Bootstrapper.GetContainer().Resolve<IMobileLibraryBrowserPresenter>(new NamedParameterOverloads() 
+                        {{"tabType", tabType }, { "browserType", browserType}});
                     presenter.BindView((IMobileLibraryBrowserView) view);
                     _mobileLibraryBrowserList.Add((IMobileLibraryBrowserView) view, presenter);
                 }
@@ -212,7 +213,6 @@ public virtual IMobileLibraryBrowserView CreateMobileLibraryBrowserView(MobileNa
             // Create view and manage view destruction
             IMobileLibraryBrowserView newView = null;
             newView = Bootstrapper.GetContainer().Resolve<IMobileLibraryBrowserView>(new NamedParameterOverloads() { { "onViewReady", onViewReady } });
-            newView.BrowserType = browserType; // TODO: Shouldn't this be in the presenter instead...? browserType + filter (can be artist or album)
             newView.OnViewDestroy = (view) =>
             {
                 // The view list can be accessed from different threads.

diff --git a/MPfm/MPfm.MVP/Presenters/MobileLibraryBrowserPresenter.cs b/MPfm/MPfm.MVP/Presenters/MobileLibraryBrowserPresenter.cs
@@ -36,17 +36,19 @@ public class MobileLibraryBrowserPresenter : BasePresenter<IMobileLibraryBrowser
 	{
         private readonly MobileNavigationManager _navigationManager;
 	    private readonly MobileNavigationTabType _tabType;
+	    private readonly MobileLibraryBrowserType _browserType;
 	    private readonly ITinyMessengerHub _messengerHub;
         private readonly ILibraryService _libraryService;
         private readonly IAudioFileCacheService _audioFileCacheService;
 	    private List<LibraryBrowserEntity> _items;
 
 	    public AudioFileFormat Filter { get; private set; }
 
-        public MobileLibraryBrowserPresenter(MobileNavigationTabType tabType, ITinyMessengerHub messengerHub, MobileNavigationManager navigationManager,
+        public MobileLibraryBrowserPresenter(MobileNavigationTabType tabType, MobileLibraryBrowserType browserType, ITinyMessengerHub messengerHub, MobileNavigationManager navigationManager,
                                              ILibraryService libraryService, IAudioFileCacheService audioFileCacheService)
 		{
             _tabType = tabType;
+            _browserType = browserType;
             _messengerHub = messengerHub;
             _navigationManager = navigationManager;
             _libraryService = libraryService;
@@ -74,6 +76,20 @@ private void AudioFileCacheUpdated(AudioFileCacheUpdatedMessage audioFileCacheUp
 
 	    private void OnItemClick(int i)
 	    {
+            // PLAYLIST TAB: Playlists --> Player
+            // ARTISTS TAB: Artists --> Albums --> Songs --> Player
+            // ALBUMS TAB: Albums --> Songs --> Player
+            // SONGS TAB: Songs --> Player
+
+            // Check if another MobileLibraryBrowser view needs to be pushed
+            if (_tabType == MobileNavigationTabType.Artists || _tabType == MobileNavigationTabType.Albums)
+            {
+                var browserType = (_browserType == MobileLibraryBrowserType.Artists) ? MobileLibraryBrowserType.Albums : MobileLibraryBrowserType.Songs;
+                var newView = _navigationManager.CreateMobileLibraryBrowserView(_tabType, browserType);
+                _navigationManager.PushTabView(_tabType, newView);
+                return;
+            }
+
             // Make sure the view was binded to the presenter before publishing a message
 	        Action<IBaseView> onViewBindedToPresenter = (theView) => _messengerHub.PublishAsync<MobileLibraryBrowserItemClickedMessage>(new MobileLibraryBrowserItemClickedMessage(this)
 	            {

diff --git a/MPfm/MPfm.MVP/Views/IMobileLibraryBrowserView.cs b/MPfm/MPfm.MVP/Views/IMobileLibraryBrowserView.cs
@@ -26,9 +26,6 @@ namespace MPfm.MVP.Views
 	/// </summary>
     public interface IMobileLibraryBrowserView : IBaseView
 	{
-        MobileLibraryBrowserType BrowserType { get; set; }
-        string Filter { get; set; }
-
         Action<int> OnItemClick { get; set; }
 
         void RefreshLibraryBrowser(IEnumerable<LibraryBrowserEntity> entities);