[mythtv-commits] Ticket #12033: mythmetadatalookup --refresh-all-artwork started by housekeeping no longer runs in master.

[MythTV]

#12033: mythmetadatalookup --refresh-all-artwork started by housekeeping no longer
runs in master.
------------------------------------+-----------------------------
 Reporter:  Bill Meek <keemllib@…>  |          Owner:  dekarl
     Type:  Bug Report - General    |         Status:  closed
 Priority:  minor                   |      Milestone:  0.27
Component:  MythTV - Housekeeper    |        Version:  Master Head
 Severity:  medium                  |     Resolution:  Fixed
 Keywords:                          |  Ticket locked:  0
------------------------------------+-----------------------------
Changes (by bmeek):

 * status:  accepted => closed
 * resolution:   => Fixed
 * milestone:  0.28 => 0.27


Comment:

 Replying to [comment:9 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.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/12033#comment:10>
MythTV <http://www.mythtv.org>
MythTV Media Center