[mythtv-commits] Ticket #13552: Backend now fails to start as master with a new database
MythTV
noreply at mythtv.org
Tue Jan 14 20:04:09 UTC 2020
#13552: Backend now fails to start as master with a new database
----------------------------------+-------------------------------
Reporter: bib1963 | Owner: Klaas de Waal
Type: Bug Report - General | Status: assigned
Priority: blocker | Milestone: 31.0
Component: MythTV - General | Version: Master Head
Severity: high | Resolution:
Keywords: | Ticket locked: 0
----------------------------------+-------------------------------
Comment (by Klaas de Waal):
Reverted commit 5cc67d5c because of unexpected side effects... and no, I
did not already know this.
In the mythtv-setup "Host Address Backend Setup" page is the field "Master
Backend Name" which shows the value of the database setting
MasterServerName. The database setting value is set to the actual hostname
when the checkbox above it, "This server is the Master Backend" is
checked.
The "Master Backend Name" is shown in a GlobalTextEditSetting field,
which, as the name suggests, allows editing, but here it is not the
intention that the field should be edited.
There is no possibility to make a field read-only in the MythTV UI code
but it is possible to disable a field which also makes it readonly.
However, a disabled field is nowadays also greyed-out. This is useful for
fields that are not applicable or relevant for a given situation and that
should be ignored. Examples of this can be found in the video playback
settings.
So the reason to make the now reverted change was to make the "Master
Backend Name" label visible again instead of greyed out. For this the
field was changed from a GlobalTextEditSetting into a GroupSetting, which
turned to out have the unexpected and unwanted side effect of the hostname
not being entered into the database settings.
--
Ticket URL: <https://code.mythtv.org/trac/ticket/13552#comment:6>
MythTV <http://www.mythtv.org>
MythTV Media Center
More information about the mythtv-commits
mailing list