metadata: Properly clean inetref when searching for collection

Also reduce duplicated code.
Remove code handling of title as inetref. The issue if it exists must be resolved elsewhere

(cherry picked from commit 7a0fb8ecbfec5a73300df8837ea0ed68877374b9)
metadata: Properly separate TV title/subtitle search from inetref/subtitle search

This prevent likely errors of parsing a title as an inetref, in particular if it contains a ‘:’

(cherry picked from commit a013a2e0b212526994263e3489a90bf5a85f5265)
metadata: if video type not known, try both movie and television grabber

In case the type couldn’t be identified, it would assume it was only TV

(cherry picked from commit 738a7016e657d49fa5f49f2ec46cea7fa61ff9c6)
Bump ABI
CoreContext: Fix IsThisHost

IsThisHost can be called with a hostname, and not just an address as the arguments’ name suggest

(cherry picked from commit feb2ddd5ff2ca37efdd05257ee8b725b25f6a183)
metadata: do not use script name for artwork file name

Clear inetref from the script name.

(cherry picked from commit ee7470d8bc0b843b80d13f4d9d92a6c3b06c3b2b)
CoreContext: Fix IsThisHost

Wrong parenthesis placement.

(cherry picked from commit 3cb8de1a6feaac3b1320331d3bdfd5a93ae148af)
metadata: uses _ as separator in inetref in place of :

the inetref is used in the artwork filename, but the : character isn’t permissible with all file systems (FATx, VFAT, exFAT, HFS, and AmigaOS in particular).
the : used in old inetref is kept for backward compatibility.

Attempt was made to generate a filename not using the inetref as-is, but the change was very disruptive as there are about 12 functions across various classes that generate those filenames. Likelihood of missing a case was too great.

(cherry picked from commit 6fddce08ff196a6b78b066dede0558593ba86f80)
metadata: Fix broken retrieval for screenshots

Test could never have been evaluated to true

(cherry picked from commit a50a24b0f408957885ff928569a061df07e5e993)
metadata: fix stray declaration

(cherry picked from commit eafce8507591311bbbb1d412d28b38113bb9e358)
metadata: Fix GetGrabber

It would never have used the grabber set in the inetref

(cherry picked from commit 61a511562c00858ffb767a24ac9fd586cdf83fe2)
metadata: also set collectionref to new format

(cherry picked from commit 79aea142fc0d23c25a0f79085cf7df0adfb87e07)
metadata: no need to re-run the search when a match has already been found

(cherry picked from commit 43785e4137cc3eaf4174fdfa62e15061765ba300)
metadata: properly handle media search type if we already have an extended inetref.

(cherry picked from commit 30b03cc2b86e949b434b7afc207f12d35e6c6d87)
UPnP: Fix the service IDs for CDS and CMGR which were both incorrect.

(cherry picked from commit 23f098100b01d2143714a196bcbe6dca4058e7e0)
Signed-off-by: Stuart Morgan <smorgan@mythtv.org>
MusicPlayer: On stop clear any temporary one shot metadata

This fixes a problem getting stuck playing the same track after exiting the
import screen if a track was test played while on there.

(cherry picked from commit 31614bf78d8d5740fc4b38385de2d69cc04dffa9)
corecontext: QHostAddress::isNull() appears buggy

It doesn’t reliably returns true when provided an invalid IP address. So get around that
metadata: clear error state once an error has been propagated

Otherwise following successful downloads would have been marked as failed too

(cherry picked from commit 085c65e91273bbeb3e2d70298c795a027406a62b)
metadata: add getStorageGroupName

remove duplicated code

(cherry picked from commit 6954cff4005271b6360fdf25a4e2f7781a8e108a)
metadata: fix storage group access when downloading artworks.

Check for an existing file was all broken, would always download no matter what
Also, make sure only the master backend access the file system directly

(cherry picked from commit 1e844849c6bb6e94e2a4e661ed038848a18ed06d)

Conflicts:
	mythtv/libs/libmythmetadata/metadataimagedownload.cpp