OnlineMetadataFixer: removed workaround for FSharp.Data bug

Load method is already fixed in the newest version of FSharp.Data

Dap.Bluetooth: Update addin dependencies

Addin dependencies are not calculated recursively...

Dap.Bluetooth: Correct button order

Easily seen from the packing function, the volume buttons were the wrong
way round...

Dap.Bluetooth: Remove unused static string

OnlineMetadataFixer: checking whether user's API key is valid

OnlineMetadataFixer: block plugin usage if user doesn't enter API key

Dap.Bluetooth: Rename file, add Popover

Since the widget is now really just a button, we rename it as such while
adding the Popover class.

The popover consists of a single constructor; functions for setting
position and changing the relative_to GtkWidget are not implemented, nor
are signals.

Dap.Bluetooth: Retire MediaControlWidget

After updating to Gnome 3.12, I wanted to use a Popover for the media
controls. Unfortunately they aren't available in GTK# yet so this patch
reduces the wasted space by abusing a GtkMenu activated by a toggle
button.

While requiring a little more work to make it look good, as a first
refactoring step this gives us a checkpoint we can return to.

Dap.Bluetooth: path restore fix

The path was not being restored correctly after a dropped connection. In
this patch we make use of the robust crawl function and clear the path
before attempting to navigate.

Dap.Bluetooth: Threading update

In this commit we reduce the verbosity of threading statements.

OnlineMetadataFixer: fixed file names in plugin description file

Dap.Bluetooth: MediaControlWidget, minor refactor

Refactor repetitive widget creation to a function, as done in
DeviceWidget, also ensure main thread when refreshing.

Dap.Bluetooth: Ensure icons are all the same size

Gnome 3.12 brought a size difference between the default icon size for
buttons compared to 3.10. This patch states that LargeToolbar is correct
to use throughout the DeviceWidget, ensuring that buttons line up as if
in a grid.

We leave the MediaControlWidget with smaller buttons, as it is a
sub panel of the DeviceWidget.

Dap.Bluetooth: Cover failure cases in GnomeRfkill

- if the device has no killswitch, any operation succeeds;
- if the device is hard blocked, any operation fails;
- all other cases are success.

Dap.Bluetooth: Initialise Rfkill PropertyManager

To ensure that we have up to date properties from the start, instead of
whatever default that particular property type may return.

Dap.Bluetooth: Extract Gnome specific component

All AdapterManager really needs is a function to set and unset the
killswitch. We extract this to GnomeRfkill, although both types are
still placed in the same file, and pass an instance's Set member into
AdapterManager at construction time.

The value passed must have signature (bool -> bool), at the moment the
return is unchecked but in future will be used to present a Hyena
Warnf message.

Dap.Bluetooth: organise/rename

Bluetooth DAP is rather heavy on files, this patch corrects the file
name specified in an initial comment block (GnomeApi.fs) and takes the
opportunity to create a solution folder for the smaller building blocks.

Dap.Bluetooth: Update for Gnome 3.12

gnome-bluetooth-3.12 dropped support for the kill switch, which is now
provided by a dbus object in the org.gnome.SettingsDaemon namespace.

This patch adds support for that interface and removes the unrequired
submodule.

OnlineMetadataFixer: added gnome-keyring dependency checking

OnlineMetadataFixer: renamed *Source class to *Solver if inherits
from a Fixup.Solver class

OnlineMetadataFixer: extended status info with "Settings" button

Dap.Bluetooth: break if condition then stmt

Dap.Bluetooth: Don't force sync if AutoSync active

If AutoSync is active, the sync process will start during PostLoad. In
these cases, we can't remove the source. However, with
e.g. transfer, errors displayed as a child of the primary source,
perhaps we shouldn't remove it following a scheduled sync anyway.

Dap.Bluetooth: Use TimeSpan.FromSeconds for Sleep

Using TimeSpan calculation when sleeping is more semantically clear
than using a literal number of milliseconds.

Dap.Bluetooth: Synchronous sync

In the context of a scheduled connection and synchronisation we need to
know when the sync completes in order to be able to remove the source at
the correct time.

Ideally, DapSync would provide us with a synchronous method. In this
patch, we use Reflection to get at the private method and call it,
falling back to the asynchronous call if we're not successful.

Dap.Bluetooth: fix build against banshee master

This patch adds the force parameter to the DeviceInitialize call for
BluetoothSource.

SongKick.GeoLocation: improve notifyUser function

If number of gigs in the user city is a lot Banshee will try
to show many notifications.
What we're doing now is if number of gigs exceeds 3 gigs
then instead of showing all of them, we show one notification
with next text: "SongKick found many gigs near you,
check out all of them in Banshee!"
If we can't find any gigs we also notify user about it.

OnlineMetadataFixer: handled unexcepted exceptions

OnlineMetadataFixer: minors - typo, fixed slashes in project file

OnlineMetadataFixer: use "not" in more readable way

OnlineMetadataFixer: don't run metadata sender if AcoustID API key is empty

OnlineMetadataFixer: made api key invisible in a preferences window

OnlineMetadataFixer: minor - more accurate name for API key helper class

OnlineMetadataFixer: added preferences section

Dap.Bluetooth: Separate schedule and auto sync

This patch allows the user to have auto sync disabled when doing a
scheduled sync.

Dap.Bluetooth: Crawler, extract Drop to function

This patch extracts the member function, Drop, to a let binding over our
state.

It is used under failure conditions to request that BlueZ
discards the session and allows reconnect in a single
connection scenario (the scenario I test in) where the original
connection is dead, but BlueZ hasn't become aware.

Dap.Bluetooth: Sleep between root requests

The last few patches made it quite easy to trigger a stack overflow
due to the retry being too fast.

When attempting to restore the
connection under these conditions it makes sense to sleep between
recursive calls to allow time for the connection to come back with a
grace time of up to 30 seconds, as originally intended.

Dap.Bluetooth: Adds scheduled connection

As a first stab at automatic sync, this patch adds the capability to
connect a bluetooth device at a specified time of day, e.g. when you get
in from work or while your PC is on at night and your phone is on charge.

To enable auto sync, the usual Dap preferences checkbox is employed at
the moment as starting sync from elsewhere will require some threading
concerns, i.e. making sure that sync doesn't start while the source is
still loading tracks.

SongKick: display city name in "Find music events by Location"

We should provide city name in SearchBar if
"Find music events by Location" is selected
and SongKick.GeoLocation is started.

Dap.Bluetooth: Redundancy and Match Improvements

The first attempt at a redundant file crawler gave us connection retry,
this patch should go a little further by making sure the path is also
restored once the connection returns. To achieve this, Crawler's
navigation functions had to be, almost entirely, extracted.

In addition, the LoadCache is ported and included, along with the
necessary override to prevent DapSource purging our tracks.

A whole raft of matchers are included. Directories are matched
against "Artist/Album" or "Artist - Album". Then file names against:

- "01. Track"
- "01 - Track"
- "Artist - Album - 01 - Track"
- "Album - 01 - Track"

Last, a whole lot of Infof messages have been downgraded to Debugf.

OnlineMetadataFixer: sending metadata on one of the fields was modified

OnlineMetadataFixer: fixed background job implementation

OnlineMetadataFixer: submitting metadata to an AcoustID service improvement:

 * more logs added
 * submission status in debug info printed

OnlineMetadataFixer: used AcoustIDSender in plugin's background job

OnlineMetadataFixer: implemented acoustid metadata submitter

OnlineMetadataFixer: fixed warnings

OnlineMetadataFixer: added gnome-keyring based password manager

Dap.Bluetooth: Now with Hyena.Log support

The F# way, with statically checked format strings :)

FanArt: make log message more informative

Radiostationfetcher: Migrate to GTK3 and re-enable

FanArt: fix GSlice memory error

In some situations Banshee crashed with next error:
"GSlice: MemChecker: attempt to release block with invalid size..."
(more: https://gist.github.com/dimart/a79a4b18193f6db64d15)

FanArt: add dynamic recovery cache when the last was deleted

After FanArt has retrieved some logos, if you quit banshee,
remove the cache folder, and run banshee again,
it doesn't retrieve the logos again.
So, now we check image path existence first,
and then try to redownload missed image.

FanArt: fix infinite iterations when there is no Internet connection

If there's no internet connection, IterateCore keeps running
and printing logs messages in the console, unnecessarily.

FanArt: refactoring in FanArtQueryJob.Lookup()

This doesn't change behaviour.

FanArt: make sql commands more readable

FanArt: add ability to get ArtistName by MusicBrainzID

SongKick.GeoLocation: make cache folder name in lower

CacheService: check whether directory exist on add

FanArt: unsubscribe from ExtensionChanged event when ext is disabled

SongKick.GeoLocation: simplify methods

SongKick.GeoLocation: set last scan time to null when ext is disabled

FanArt: cache server reply

FanArt: fix build and change project reference to just reference

FanArt: add reference to CacheService

FanArt: throw away generic catch blocks

FanArt: remove FanArtMetadataProvider when extension disabled

FanArt: fix SQL errors (bgo#733558)

Disabling FanArt extension causes SQL errors.
This happened because ArtistImageJob was not cancelled correctly.

SongKick.GeoLocation: detach Service from LocationProvider in Dispose()

If user turn on and turn off SongKick.GeoLocation extension
many times we should properly subscribe and unsubscribe to
LocationProviderManager.

SongKick: no need to specify path for FSharp.Core

The FIXME then is deleted, because there's no need for it.

FanArt: fix Cairo Surface leaking problem

We need to be sure that artistImage is always disposed correctly.
So we used 'using'.

FanArt: refactoring in ArtistColumnCell.Render()

This doesn't change behaviour.

SongKick: fix obscure error message

SongKick: fix types in Search

It was a typo.

FanArt: make exception catching more specific

It was unclear what type of exception we are afraid of.

SongKick: check query string for null or empty before use

CacheService: check keys for null or empty

Check keys for empty or null before using it.

SongKick.GeoLocation: Empty artist names shall not pass!

If somehow we get empty artist name in recommendations
then we should exclude it from the flow.

SongKick.GeoLocation: implement ICityNameObserver in Service

It's better to update RunTimeout for the first time
when city name is available.

SongKick.GeoLocation: fix refresh time issue

If runtime of RefreshRecommendedGigs() takes more than one second,
then it will be invoked many times (due to tiny refresh time).
So, if it's needed we call RefreshRecommendedGigs and return
delay time as next retry time so that RefreshRecommendedGigs
should have time to perform.

LiveRadio: Migrated to GTK3 and re-enable

Karaoke: fix build and re-enable

Ampache: Migrate to GTK3 and re-enable

Removed the auto-generated Preferences GUI and manually created it.
Changed from Source to PrimarySource to prevent crash when switching to
Ampache source.

SongKick.GeoLocation: fix typo

SongKick.GeoLocation: improve refresh logic

Now it saves last update time and updates gigs list
only when 12 hours pass. (Even if Banshee has been closed).

Awn: Migrated to GTK3 and re-enabling

AppIndicator: Migrated to GTK3 to compile

The appindicator is still based on GTK2 so the menu does not work. The
extension compiles and runs, notifications work with new notify-sharp-3.0

There is still some work to be done, so not re-enabling the extension

Streamrecorder: fixed build and re-enable

Lyrics: Migrated to GTK3, but did not re-enable as WebBrowser is not working

Telepathy: Migrated to GTK3

Coverwallpaper: fixed build by not taking CS1701 warning as error and re-enable

LCD: Migrated to GTK3 and re-enabled

Jamendo: Updated to latest Hyena logging convention and re-enabled

Hyena has deprecated logging "Exception", so now logging as "Warning"

FolderSync: Migrated to GTK3 and re-enabled

Alarmclock: migrated to GTK3 and re-enabled extension

SongKick.GeoLocation: cleanup code and get rid of mutables

I've also made some methods private, chose better names,
made code better structured in general.

CacheService: minor changes

FanArt: last (hopefully) Fanart->FanArt renaming

FanArt: minor code cleanup

FanArt: more Fanart->FanArt renaming, this time in namespaces

FanArt: rename most "Fanart" artifacts to "FanArt" name

The name "Fanart" doesn't say anything in itself, it's better to
rename it to use PascalNotation like other extension names.