[mythtv] Services API for ChannelServices / Channel
keemllib at gmail.com
Mon Nov 25 06:21:44 UTC 2019
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 10915 bytes
Desc: not available
More information about the mythtv-dev