Opened 19 years ago
Closed 17 years ago
#357 closed enhancement (wontfix)
Patch for MythGame to support diff. Hosts
Reported by: | jochen | Owned by: | greg |
---|---|---|---|
Priority: | minor | Milestone: | 0.21 |
Component: | mythgame | Version: | |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
This Patch enables MythGame? to Select on a GamePlayer? if it is only for the current Host, or for all Hosts!
Attachments (2)
Change History (11)
comment:1 Changed 19 years ago by
You Should use Version 2, with this, you can set the Gameplayer for every host that has already a GamePlayer?.
comment:2 Changed 19 years ago by
Doesn't use existing per-host database style (ie, null for global).
comment:3 Changed 19 years ago by
how can i use the MythTV settings function to set a vaule of NULL??? because i want to make the patch to fit the standart coding style.
Does it work if I change +class PlayerHostName?: public ComboBoxSetting?, public MGSetting { +public: + PlayerHostName?(const MythGamePlayerSettings?& parent): + MGSetting(parent, "hostname") { + setLabel(QObject::tr("Hostname")); + addSelection("(all Clients)", "*"); + addSelection("(current Client)", gContext->GetHostName?());
to
+class PlayerHostName?: public ComboBoxSetting?, public MGSetting { +public: + PlayerHostName?(const MythGamePlayerSettings?& parent): + MGSetting(parent, "hostname") { + setLabel(QObject::tr("Hostname")); + addSelection("(all Clients)", NULL); + addSelection("(current Client)", gContext->GetHostName?());
comment:4 Changed 19 years ago by
sorry to use this for discussion, but i saw i was wrong, that can't work. if anyonle could help.
comment:5 Changed 19 years ago by
Owner: | changed from Isaac Richards to greg |
---|
I believe Isaac was referring to how your patch to dbcheck.cpp
sets that columns default value to '*' instead of NULL.
However I'm rejecting this patch as it is right now not just for that, but because it does not properly address the rest of the application to make this work.
If you want per host emulators then you should only display the games in the gametree that apply to that host(or provide an indicator of what is not available to them). Which means you have to update the gametree code to support this new per host feature, and means changing how the gamescan information is entered in the database as it does not take into account that a emulator might not be available on all frontends.
I wont be applying any patch that doesn't address the rest of this. At this point it's little more than a new setting on a page.
comment:6 Changed 18 years ago by
Type: | defect → enhancement |
---|
This Patch in the second version is not only a config setting.
In the Gametree also only the games for the aviable systems are visible.
comment:7 Changed 18 years ago by
Milestone: | → 0.20 |
---|---|
Status: | new → assigned |
comment:8 Changed 18 years ago by
Milestone: | 0.20 → 0.21 |
---|
comment:9 Changed 17 years ago by
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
Version 1.0 of the Patch.