Custom Fields in Collections

Author: omkelderman

App/Interfaces/Controls/IBeatmapListingPresenter.cs

@@ -81,16 +81,17 @@ public void SetBeatmaps(Beatmaps beatmaps)
 
         public void SetCollection(ICollection collection)
         {
+            CurrentCollection = collection;
             if (collection == null)
             {
                 SetBeatmaps(null);
-                CurrentCollection = collection;
-                return;
             }
-            CurrentCollection = collection;
-            var maps = new Beatmaps();
-            maps.AddRange(collection.AllBeatmaps());
-            SetBeatmaps(maps);
+            else
+            {
+                var maps = new Beatmaps();
+                maps.AddRange(collection.AllBeatmaps());
+                SetBeatmaps(maps);
+            }
         }
 
         public void FilterBeatmaps(string text)

App/Models/Controls/BeatmapListingModel.cs

@@ -6,25 +6,11 @@
 
 namespace App.Presenters.Controls
 {
-    public class BeatmapListingPresenter: IBeatmapListingPresenter
+    public class BeatmapListingPresenter
     {
         readonly IBeatmapListingView _view;
         readonly IBeatmapListingModel _model;
 
-        private Beatmaps _beatmaps;
-
-        public Beatmaps Beatmaps
-        {
-            get
-            {
-                return _beatmaps;
-            }
-            set
-            {
-                _beatmaps = value;
-                _view.SetBeatmaps(value);
-            }
-        }
         public BeatmapListingPresenter(IBeatmapListingView view, IBeatmapListingModel model)
         {
             _view = view;
@@ -36,11 +22,11 @@ public BeatmapListingPresenter(IBeatmapListingView view, IBeatmapListingModel mo
             _view.BeatmapOperation += (s, a) => _model.EmitBeatmapOperation(a);
 
             _model = model;
-            _model.BeatmapsChanged += _model_BeatmapsChanged;
+            _model.BeatmapsChanged += (_, _) => RefreshBeatmapsInViewFromModel();
             _model.FilteringStarted+=ModelOnFilteringStarted;
             _model.FilteringFinished += _model_FilteringFinished;
             _view.SetFilter(_model.GetFilter());
-            Beatmaps = _model.GetBeatmaps();
+            RefreshBeatmapsInViewFromModel();
         }
 
         private void _model_FilteringFinished(object sender, EventArgs e)
@@ -63,9 +49,15 @@ private void ViewOnSearchTextChanged(object sender, EventArgs eventArgs)
             _model.FilterBeatmaps(_view.SearchText);
         }
 
-        private void _model_BeatmapsChanged(object sender, System.EventArgs e)
+        private void RefreshBeatmapsInViewFromModel()
         {
-            Beatmaps = _model.GetBeatmaps();
+            _view.SetBeatmaps(_model.GetBeatmaps());
+            _view.ClearCustomFieldDefinitions();
+            var curCol = _model.CurrentCollection;
+            if(curCol != null && curCol.CustomFieldDefinitions != null)
+            {
+                _view.SetCustomFieldDefinitions(curCol.CustomFieldDefinitions);
+            }
         }
 
 

App/Presenters/Controls/BeatmapListingPresenter.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Collections.Generic;
+using System.Linq;
 
 namespace CollectionManager.DataTypes
 {
@@ -13,5 +15,42 @@ public class BeatmapExtension : Beatmap
         public string UserComment { get; set; } = "";
 
         #endregion
+
+        #region Custom Field Stuff
+
+        private Dictionary<string, object> _customFields;
+
+        public void SetCustomFieldValues(BeatmapExtension other)
+        {
+            _customFields = other._customFields == null ? null : new Dictionary<string, object>(other._customFields);
+        }
+
+        public void SetCustomFieldValue(string key, object value)
+        {
+            _customFields ??= new Dictionary<string, object>();
+            _customFields[key] = value;
+        }
+
+        public object GetCustomFieldValue(string key)
+        {
+            if(_customFields == null ) return null;
+            return _customFields.TryGetValue(key, out var value) ? value : null;
+        }
+
+        public IEnumerable<string> GetStringCustomFieldValues()
+        {
+            if (_customFields == null) yield break;
+            foreach(var customField in _customFields)
+            {
+                if(customField.Value is string stringValue) yield return stringValue;
+            }
+        }
+
+        public IEnumerable<KeyValuePair<string, object>> GetCustomFields()
+        {
+            return _customFields ?? Enumerable.Empty<KeyValuePair<string, object>>();
+        }
+
+        #endregion
     }
 }

CollectionManagerDll/DataTypes/BeatmapExtension.cs

@@ -58,6 +58,8 @@ public virtual int NumberOfBeatmaps
 
         public int Id { get; set; }
 
+        public IReadOnlyCollection<CustomFieldDefinition> CustomFieldDefinitions { get; set; }
+
         public void SetLoadedMaps(MapCacher instance)
         {
             if (instance == null)
@@ -161,6 +163,7 @@ public void AddBeatmapByMapId(int mapId)
         private void ProcessAdditionalProps(BeatmapExtension src, BeatmapExtension dest)
         {
             dest.UserComment = src.UserComment;
+            dest.SetCustomFieldValues(src);
         }
         protected virtual void ProcessNewlyAddedMap(BeatmapExtension map)
         {
@@ -236,6 +239,5 @@ public IEnumerator GetEnumerator()
         {
             return this.AllBeatmaps().GetEnumerator();
         }
-
     }
 }

CollectionManagerDll/DataTypes/Collection.cs

@@ -0,0 +1,11 @@
+using CollectionManager.Enums;
+
+namespace CollectionManager.DataTypes
+{
+    public class CustomFieldDefinition
+    {
+        public string Key { get; set; }
+        public CustomFieldType Type { get; set; }
+        public string DisplayText { get; set; }
+    }
+}

CollectionManagerDll/DataTypes/CustomFieldDefinition.cs

@@ -53,6 +53,8 @@ public interface ICollection
 
         int Id { get; set; }
 
+        IReadOnlyCollection<CustomFieldDefinition> CustomFieldDefinitions { get; }
+
         void SetLoadedMaps(MapCacher instance);
         IEnumerable<BeatmapExtension> AllBeatmaps();
         IEnumerable<BeatmapExtension> NotKnownBeatmaps();

CollectionManagerDll/DataTypes/ICollection.cs

@@ -0,0 +1,21 @@
+namespace CollectionManager.Enums
+{
+    public enum CustomFieldType
+    {
+        String,
+        Boolean,
+        GameMode,
+        Grade,
+        UInt8,
+        UInt16,
+        UInt32,
+        UInt64,
+        Int8,
+        Int16,
+        Int32,
+        Int64,
+        DateTime,
+        Single,
+        Double
+    }
+}