[mythtv] [mythtv-commits] mythtv/master commit: 5bf377b55 by Robert McNamara (rmcnamara)

Robert McNamara robert.mcnamara at gmail.com
Sun Apr 10 14:48:42 UTC 2011


Hi David,

On Sun, Apr 10, 2011 at 7:16 AM, David Blain <mythtv at theblains.net> wrote:
> Robert,
>
> I haven't looked at ChannelUtil to see what edits are being done, but wanted
> to remind you that calls to these or any method without all the parameters
> will work.  The missing parameters will be assigned default values based on
> the type.

Yeah, I was bearing that in mind with the warnings above.  In HTML
setup we'll be able to ensure that all values are submitted.  At least
as far as users of the API that we control, it can be done safely.

> So if an update call is made with only a channel ID, it will in effect
> update all columns with 0's and empty strings.

Right.

> Let me know if you want to discuss possibly changing this default behavior
> or maybe making it optional per method.

Ultimately we need *all* the info to safely update the channel-- now,
making all the parameters for these functions required might be a nice
thing for safety, if that's possible.

Robert


More information about the mythtv-dev mailing list