mythmetadatalookup --refresh-all-artwork started by housekeeping no longer runs in master.

[Bill Meek <keemllib@…>]

If the hostname for value 'DailyArtworkUpdates?' is set to a host (ofc0 in my case,) then housekeeping doesn't run mythmetadatalookup --refresh-all-artwork.

Changing the value of hostname to NULL re-enables --refresh-all-artwork.

The data field for both values of hostname equals '1'.

[Bill Meek <keemllib@…>]

'Good' log and version: v0.28-pre-739-g7b9d369

[Raymond Wagner <rwagner@…>]

In ffa364019e7d5bc0b88f4c83d21511090735b45d/mythtv:

Make "DailyArtworkUpdates?" a global option

This makes the DailyArtworkUpdates? setting a global setting so it can be
properly configured on any frontend. This patch does not include any
code to fix an existing improperly configured setting, which could cause
problems in the future, as the per-host option will override the global
one and will be uneditable without direct database modification.

Refs #12033

[Raymond Wagner <rwagner@…>]

In eaafcb4a917c1b6055e5fb908a3bc6b747aba1c4/mythtv:

Make "DailyArtworkUpdates?" a global option

This makes the DailyArtworkUpdates? setting a global setting so it can be
properly configured on any frontend. This patch does not include any
code to fix an existing improperly configured setting, which could cause
problems in the future, as the per-host option will override the global
one and will be uneditable without direct database modification.

Refs #12033
(cherry picked from commit ffa364019e7d5bc0b88f4c83d21511090735b45d)

[mythtv@…]

These two commits fix the hostname problem but they change the setting type from string to binary. Was that intended? I'm OK if it was but it made it look a bit weird in mysql when I was checking for the per-host setting.

[Raymond Wagner]

Nope.

[Raymond Wagner <rwagner@…>]

In e45e1ca7bf329e8ecea218f0e47cc080ad736b7e/mythtv:

Use proper string conversion for DailyArtworkUpdates? setting.

Refs #12033

[Raymond Wagner <rwagner@…>]

In 1b7404d64080a69373f3b37fc174a5b62f9143ba/mythtv:

Use proper string conversion for DailyArtworkUpdates? setting.

Refs #12033
(cherry picked from commit e45e1ca7bf329e8ecea218f0e47cc080ad736b7e)

[skd5aner <skd5aner@…>]

I can confirm, that for some reason, I had a hostname in this field and it was preventing any artwork from being retrieved. Once I nulled out that field, everything worked great the next time the housekeeper ran.

If possible, the fix should either account for a field that has a hostname there (and ignore it?) or perhaps even NULL out the field for the users as part of an update.

[Karl Egly]

I'll see if I can come up with a schema update to fix old values in time for fixes/0.28.

[Bill Meek]

Replying to dekarl:

I see 6 things to consider:

DailyArtworkUpdates plus:	Action
No entry in settings	Do nothing
Single NULL host	Do nothing
Single (non NULL) host	Convert hostname to NULL, assume old data is desired
Single host and NULL host	Delete host setting
Multiple hosts	Guess which frontend added the desired setting, convert 1, delete others
Multiple hosts and NULL	Delete all non NULL entries

The final solution for this was: changeset:e45e1ca on Feb 3, 2014.

After 2 years, it's likely that users will have fixed their entry (or configured it properly after the above fixes.)

And any conversion and especially guesses may not have the desired result. I can compare the sum of the non NULL host's data with the number of non NULL hosts and if it were 0 or the the same as the non NULL host count, then INSERT the NULL version of the setting.

I recall significant discussion on the -users list. Mostly from those that didn't want artwork, so automatically turning it on based on earlier (mis)settings wouldn't be good. Even if I add a warning in the release notes.

I'm closing this as fixed, please reopen if you disagree.