need some doc around service?methods

[reven]

im wanting to know the service?method calls for the following

1. Get the comskip times for a recording
   
2. Schedule a recording, e.g. "All New on this channel" etc
   

the xbmc addon only shows you how to schedule a single recording instance, and doesn't support comskip (well i didnt see it in the source code).

if they're not currently available sub can you please add them. Preferable comskip should return an array of 2 digit values (start, end) in seconds, and schedule a recording should take a enum value of type, e.g. "All New Recordings" could be type 4, and "This day this timeslot" could be 3 (but match the actual enum values to the nextpvr gui, I dont have access to nextpvr atm to get the actual names/indexes ). I rather set a recording based on an enum value so I'm 100% certain when my app schedules a recording its the same as scheduling in nextpvr, and if nextpvr slightly changes how it schedules my app also schedules the same way. Also please add a call to get the enum values, e.g. service?method=Setting.RecordingScheduleTypes which returns a int/enum value and a name for it. So if new recording options are added/removed in the future, my app automatically gets the updated recording options.

[mvallevand]

The comskip files is filename.edl (tab delimited) but it is currently not supported by port 8866 web server. You could fake it for testing by copying the edl to .txt and asking for it.

Martin

[reven]

oh i thought it was available, assumed some clients were already using this.

i rather not do any parsing on the client side, I would prefer it if an array of values were returned. i have no clue how slow older bluray/tvs are and i dont want to do any unnecessary processing on the client side.

also, since im bugging sub to add stuff, is there any chance you could add json support as well as xml, json is faster and a lot easier to consume in javascript.

[sub]

The most recent alphas of xbmc have added support for retrieving the comskip edl, so ill be adding support for these to nextpvr. I haven't looked into exactly what xbmc requires for this, but I'll check it out when I'm back in New Zealand.

I'll just give you a "maybe" on adding json. Time-permitting, I'm not adverse to the idea. I'd leave the xbmc client using the XML though.

[reven]

cheers sub, the standard way to return json would be to use the header

Code:

Accept: application/json

would also be nice if there was a &json=[true|1] parameter (easier to test in browser than setting a header). default should be left as xml just so nothing breaks.

when are you back in nz?

[mvallevand]

Sub, the edl structure is loaded internally in xbmc so you can send what you want.

Code:

typedef enum
  {
    PVR_EDL_TYPE_CUT = 0, /*!< @brief cut (completly remove content) */
    PVR_EDL_TYPE_MUTE = 1, /*!< @brief mute audio */
    PVR_EDL_TYPE_SCENE = 2, /*!< @brief scene markers (chapter seeking) */
    PVR_EDL_TYPE_COMBREAK = 3 /*!< @brief commercial breaks */
  } PVR_EDL_TYPE;

  typedef struct PVR_EDL_ENTRY
  {
    int64_t start; // ms
    int64_t end; // ms
    PVR_EDL_TYPE type;
  } ATTRIBUTE_PACKED PVR_EDL_ENTRY;

Martin

[sub]

when are you back in nz?

I'm in the states. I'm back in NZ in two and a half weeks

[reven]

is there any recording methods available via "service?method" ? xbmc has a very very very basic one,

Code:

/service?method=recording.save&name=%s&channel=%d&time_t=%d&duration=%d

but is there a better one, where you can pass in a epg_event id?

other clients do use this service correct? its not just xbmc? or is xbmc using every service method?

[sub]

From memory there are some other calls but I'm away from my laptop for the weekend. I'll look tomorrow.

[reven]

cheers sub.