[mythtv] Services API for ChannelServices / Channel

[Bill Meek]

On 11/24/19 3:12 PM, Klaas de Waal wrote:
> On Sun, 24 Nov 2019 at 16:38, Gary Buhrmaster <gary.buhrmaster at gmail.com> wrote:
>>
>> On Sat, Nov 2, 2019 at 4:23 PM Klaas de Waal <klaas.de.waal at gmail.com> wrote:
>>
>>> I have now added the support for the Services API along the same lines
>>> as previously done for the bouquet_id and the region_id. It is
>>> convenient to assume that the version bump done by Bill yesterday does
>>> also cover this change.
>>
>> That change is necessary, as It provides the
>> appropriate ability for a user (app) of the service
>> API to identify when enable the new features.
>>
>> But that does not provide API backward
>> compatibility when using an existing endpoint
>> where existing codes which do not provide the
>> values do not get existing values overridden by
>> updates that do not provide them (likely using
>> something like the checks that were mentioned
>> by Roger to check if the names were supplied
>> in the post, and if not, do not update those
>> values in a row, or ensuring that existing values
>> are maintained (depending on how you wish
>> to write the code).
>>
>> If you are in progress with these changes I
>> will just wait, but otherwise I can create a
>> number of release blocker tickets so that
>> v31 is not released prematurely.  How do
>> you wish to proceed?
>>
> To be honest, I was not aware that there needs to be done more than
> what I have done already so there is nothing in progress.
> So yes, please do create tickets and  please be explicit and clear
> about what is needed and why it is needed.

I *think* the attached is what Gary/Roger are suggesting. But when I
'signed on', a senior developer said there shouldn't be any SQL in the
API code (but there was already.)

At least it's something to talk about. [And way more than what's
needed for Klaas's changes]

It compiles, but I haven't tested it.

-- 
Bill
-------------- next part --------------
A non-text attachment was scrubbed...
Name: concept.patch
Type: text/x-patch
Size: 10915 bytes
Desc: not available
URL: <http://lists.mythtv.org/pipermail/mythtv-dev/attachments/20191125/70e77688/attachment.bin>