[mythtv-firehose] mythtv branch master updated by bmeek. v0.28-pre-3569-gdeceaa4

Git Repo Owner noreply at mythtv.org
Thu Feb 11 19:14:46 UTC 2016


The branch, master has been updated on the
mythtv repository by gitolite user bmeek.
       via  deceaa46293510164e8d63da1c4be6c3e8149800 (commit)
      from  f30488b8a99de46bdc7815c40b6a90a7c237fd64 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit deceaa46293510164e8d63da1c4be6c3e8149800
Author:    Bill Meek <bmeek at mythtv.org> at Thu, 11 Feb 2016 13:11:13 -0600
Committer: Bill Meek <bmeek at mythtv.org> at Thu, 11 Feb 2016 13:11:13 -0600
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=deceaa46293510164e8d63da1c4be6c3e8149800

Services API: Let GetSetting return Keys with NULL hostnames
The following example now works:

    BE:6544/Myth/GetSetting?Key=mythfilldatabaseLastRunStatus&HostName=_GLOBAL_

0.28 Background:
9f7d4d2 let GetSetting retrieve a single mythconverg.setting rather
than optionally returning all settings. The GetSettingList endpoint
was added to get all settings. If the GetSetting HostName parmaeter
isn't sent, the existing code uses the gCoreContext->GetHostName()
result in place of the HostName [a very handy feature.]

This change preserves the operation of the existing endpoint.

Original fix:
    if (sHostName.toUpper() == "IS NULL")
        return gCoreContext->GetSetting(sKey, sDefault);

Unfortunately, gCoreContext->GetSetting(sKey, sDefault) also returns
values for existing settings _with_ hostnames, so the query was added
to the fix.

I considered a new endpoint like: GetNullSetting. And also tried, a
new parameter NullHostname=true (defaulting to false.) Credit to
stuartm for the _GLOBAL_ idea.



-----------------------------------------------------------------------

Summary of changes:
 mythtv/programs/mythbackend/services/myth.cpp |   32 ++++++++++++++++++++----
 1 files changed, 26 insertions(+), 6 deletions(-)

-- 



More information about the mythtv-firehose mailing list