Re: [Banshee-List] Deriving from PlaylistModel

["Josiah Ritchie" <josiah ritchie gmail com>]

Please consider submitting this to bugzilla so that it is in a usable
form and make sure you include a changelog entry.

http://bugs.gnome.org/

JSR/

On 9/1/07, LCID Fire <lcid-fire gmx net> wrote:
> Hi.
> Today I hit rock bottom on my plugin. I need to modify PlaylistModel -
> which currently is not possible. So I wrote the following patch so
> people can change the PlaylistModel behaviour.
> Would be nice if someone could apply it.
>
> Regards
> Andreas
>
>
>
> --- /tmp/tmp-PlaylistModel.cs-BASE-u9QlvU.cs    2007-09-01
> 19:41:06.000000000 +0200
> +++ /home/andreas/Projects/banshee/src/Core/Banshee/PlaylistModel.cs
> 2007-09-01 19:33:07.000000000 +0200
> @@ -67,25 +67,30 @@
>
>          public PlaylistModel() : base(typeof(TrackInfo))
>          {
> -            SourceManager.ActiveSourceChanged +=
> delegate(SourceEventArgs args) {
> -                ReloadSource();
> -            };
> -
> -            PlayerEngineCore.EventChanged += delegate(object o,
> PlayerEngineEventArgs args) {
> -                switch(args.Event) {
> -                    case PlayerEngineEvent.StartOfStream:
> -                        playingIter = TreeIter.Zero;
> -                        return;
> -                }
> -            };
> +               RegisterEvents();
>          }
>
> -        public void AddTrack(TrackInfo ti)
> +       protected virtual RegisterEvents()
> +       {
> +               SourceManager.ActiveSourceChanged += delegate(SourceEventArgs args) {
> +                       ReloadSource();
> +               };
> +
> +               PlayerEngineCore.EventChanged += delegate(object o,
> PlayerEngineEventArgs args) {
> +                       switch(args.Event) {
> +                               case PlayerEngineEvent.StartOfStream:
> +                               playingIter = TreeIter.Zero;
> +                               return;
> +                       }
> +               };
> +       }
> +
> +        public virtual void AddTrack(TrackInfo ti)
>          {
>              AddTrack(ti, true);
>          }
>
> -        public void AddTrack(TrackInfo ti, bool raiseUpdate)
> +        public virtual void AddTrack(TrackInfo ti, bool raiseUpdate)
>          {
>              if(ti == null)
>                  return;
> @@ -98,7 +103,7 @@
>              }
>          }
>
> -        public void ReloadSource()
> +        public virtual void ReloadSource()
>          {
>              ClearModel();
>
> @@ -117,7 +122,7 @@
>
>          // --- Helper Methods ---
>
> -        public TrackInfo IterTrackInfo(TreeIter iter)
> +        public virtual TrackInfo IterTrackInfo(TreeIter iter)
>          {
>              object o = GetValue(iter, 0);
>              if(o != null) {
> @@ -127,7 +132,7 @@
>              return null;
>          }
>
> -        public TrackInfo PathTrackInfo(TreePath path)
> +        public virtual TrackInfo PathTrackInfo(TreePath path)
>          {
>              TreeIter iter;
>
> @@ -139,14 +144,14 @@
>
>          private bool can_save_sort_id = true;
>
> -        public void ClearSortOrder()
> +        public virtual void ClearSortOrder()
>          {
>              can_save_sort_id = false;
>              SetSortColumnId(-1, SortType.Ascending);
>              can_save_sort_id = true;
>          }
>
> -        public void RestoreSortOrder()
> +        public virtual void RestoreSortOrder()
>          {
>              SetSortColumnId(SourceManager.ActiveSource.SortColumn,
> SourceManager.ActiveSource.SortType);
>          }
> @@ -168,7 +173,7 @@
>
>          // --- Playback Methods ---
>
> -        public void PlayPath(TreePath path)
> +        public virtual void PlayPath(TreePath path)
>          {
>              TrackInfo ti = PathTrackInfo(path);
>              if(ti == null)
> @@ -178,7 +183,7 @@
>              GetIter(out playingIter, path);
>          }
>
> -        public void PlayIter(TreeIter iter)
> +        public virtual void PlayIter(TreeIter iter)
>          {
>              TrackInfo ti = IterTrackInfo(iter);
>              if(ti == null)
> @@ -195,22 +200,22 @@
>
>          // --- IPlaybackModel
>
> -        public void PlayPause()
> +        public virtual void PlayPause()
>          {
>
>          }
>
> -        public void Advance()
> +        public virtual void Advance()
>          {
>              ChangeDirection(true);
>          }
>
> -        public void Regress()
> +        public virtual void Regress()
>          {
>              ChangeDirection(false);
>          }
>
> -        public void Continue()
> +        public virtual void Continue()
>          {
>              Advance();
>          }
> @@ -324,7 +329,7 @@
>              }
>          }
>
> -        public int Count()
> +        public virtual int Count()
>          {
>              return IterNChildren();
>          }
> @@ -382,7 +387,7 @@
>              return IterNthChild(out iter, randIndex);
>          }
>
> -        public void ClearModel()
> +        public virtual void ClearModel()
>          {
>              totalDuration = new TimeSpan(0);
>              playingIter = TreeIter.Zero;
> @@ -393,7 +398,7 @@
>              }
>          }
>
> -        public void RemoveTrack(ref TreeIter iter, TrackInfo track)
> +        public virtual void RemoveTrack(ref TreeIter iter, TrackInfo track)
>          {
>              TrackInfo ti = track;
>              if(ti == null) {
> @@ -411,7 +416,7 @@
>              RaiseUpdated(this, new EventArgs());
>          }
>
> -        public void RemoveTrack(TrackInfo track)
> +        public virtual void RemoveTrack(TrackInfo track)
>          {
>              TreeIter iter = track.TreeIter;
>
> @@ -422,7 +427,7 @@
>              }
>          }
>
> -        public int GetIterIndex(TreeIter iter)
> +        public virtual int GetIterIndex(TreeIter iter)
>          {
>              TreePath path = GetPath(iter);
>              return path == null ? - 1 : path.Indices[0];
> @@ -437,14 +442,14 @@
>                  handler(o, args);
>          }
>
> -        public TimeSpan TotalDuration
> +        public virtual TimeSpan TotalDuration
>          {
>              get {
>                  return totalDuration;
>              }
>          }
>
> -        public TreePath PlayingPath
> +        public virtual TreePath PlayingPath
>          {
>              get {
>                  try {
> @@ -456,7 +461,7 @@
>              }
>          }
>
> -        public TreeIter PlayingIter {
> +        public virtual TreeIter PlayingIter {
>              set {
>                  playingIter = value;
>              }
> @@ -466,7 +471,7 @@
>              }
>          }
>
> -        public RepeatMode Repeat {
> +        public virtual RepeatMode Repeat {
>              set {
>                  repeat = value;
>              }
> @@ -476,7 +481,7 @@
>              }
>          }
>
> -        public bool Shuffle {
> +        public virtual bool Shuffle {
>              set {
>                  shuffle = value;
>              }
> @@ -486,7 +491,7 @@
>              }
>          }
>
> -        public TrackInfo FirstTrack {
> +        public virtual TrackInfo FirstTrack {
>            get {
>                TreeIter iter = TreeIter.Zero;
>                if(GetIterFirst(out iter) && !iter.Equals(TreeIter.Zero))
> _______________________________________________
> Banshee-list mailing list
> Banshee-list gnome org
> http://mail.gnome.org/mailman/listinfo/banshee-list
>


-- 
Our Mission
Technology and Hospitality for God's Workmen
http://missions.ritchietribe.net