Patch for MythGame to support diff. Hosts

[jochen]

This Patch enables MythGame to Select on a GamePlayer if it is only for the current Host, or for all Hosts!

[jochen]

Version 1.0 of the Patch.

[jochen]

Version 2.

[jochen]

You Should use Version 2, with this, you can set the Gameplayer for every host that has already a GamePlayer.

[Isaac Richards]

Doesn't use existing per-host database style (ie, null for global).

[jochen]

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());

[jochen]

sorry to use this for discussion, but i saw i was wrong, that can't work. if anyonle could help.

[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.

[jochen]

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.