[MetalChris]

I believe it is failing to grab the mediaId variable correctly in the IFRAME module. Then that causes the 400 Bad Request error when the add-on tries to open the furl. When I try to access the smil file in my browser for the ISU Pairs live event, I see I'm geoblocked though, so I'm stuck there.

<param name="exception" value="GeoLocationBlocked"/>

[jbinkley60]

I believe it is failing to grab the mediaId variable correctly in the IFRAME module. Then that causes the 400 Bad Request error when the add-on tries to open the furl. When I try to access the smil file in my browser for the ISU Pairs live event, I see I'm geoblocked though, so I'm stuck there.

<param name="exception" value="GeoLocationBlocked"/>

That's exactly why it is failing.  I uploaded v0.998 which checks for an invalid mediaId now and give a notification if it is missing.  I was trying to determine why so many events were suddenly missing the mediaId.  I also looked at the Hockey Night pages and see where the normal parsing won't work because it doesn't contain a single video playback window like the other pages.  It does have links for each of the Hockey Night events.  I did a capture of the raw page and will see if there is a way to parse out the links and match them to the various events.  It would be much nicer if the CBC Sports JSON for each event would point to the correct page vs. a generic page for multiple events. 

I too am geo-location blocked so I work off the detailed logs as best I can.


Jeff

[kodaksmith]

Thank you jbinkley60 for migrating this add-on to Matrix.

Currently I'm using Kodi Leia on my RPis.  In the process of back porting some of your improvements to Leia I noticed that the addon.py code was concatenating the aired date with the plot text.  Because I've noticed other add-ons that display a separate aired date I did a little more digging.  What I discovered is that if one not only sets the "Aired": date in infoLabels, but also sets "mediatype": 'episode' then the Aired date will be automatically displayed when one uses the Info button on a specific video.

This seems to be a little known feature, though I did come across some earlier discussion in Development > Add-ons > Is there an example of "mediatype" info label.

[jbinkley60]

Thank you jbinkley60 for migrating this add-on to Matrix.

Currently I'm using Kodi Leia on my RPis.  In the process of back porting some of your improvements to Leia I noticed that the addon.py code was concatenating the aired date with the plot text.  Because I've noticed other add-ons that display a separate aired date I did a little more digging.  What I discovered is that if one not only sets the "Aired": date in infoLabels, but also sets "mediatype": 'episode' then the Aired date will be automatically displayed when one uses the Info button on a specific video.

This seems to be a little known feature, though I did come across some earlier discussion in Development > Add-ons > Is there an example of "mediatype" info label.

The concatenation was a simple way to provide a formatted date and description in the description section of the skin window.  I don't currently set the mediatype and thus there is no info option in the listitem context menu.  I typically have done this with other addons.  I am heading out of town for a few days and will play around with this again when I get back.  I think I had tried different mediatype options before but may not have tried episode.  I'd prefer to use the native skin variables vs. what I am currently doing with the concatenation.  It will work better across the different skins and view types.  If you need any help back porting the changes, I am happy to help.

Thanks,

Jeff

[kodaksmith]

Thank you Jeff for the offer of help. I was successful with back porting your improvements to Kodi Leia.

For Kodi Matrix the changes to enable the skin to display the Aired date are:

• In the VIDEOS function set aired format to '%Y-%m-%d' (I'm not sure if this is required, I just copied another add-on):
  
  aired = datetime.fromtimestamp(pubDate / 1000).strftime('%Y-%m-%d')
  
  
• In the addDir2 function add Aired, aired_text, Year, and mediatype:
  
  
  plot_text = ''
  if aired and plot:
          plot_text = '[COLOR blue]Aired: ' + aired + '\n\n' + '[COLOR blue]Description: ' + plot
  aired_text = ''
  if aired:
          aired_text = aired
  infoLabels={ "Title": name,
               "Aired": aired_text,
               "Year": aired_text.split('-',1)[0],
               "mediatype": 'episode',
               "Duration": duration,
               "Plot": plot_text }
  liz.setInfo( "video", infoLabels)
  
  
  Note mediatype set to episode.
  

I'm sure you'd easily figure this out but thought I might post how I did it so others might try and perhaps learn a bit about video add-on development.

I tested this with the Confluence and Estuary skins. Note that there might be other values for mediatype that are more appropriate. I simply stopped when I discovered that a value of episode enabled the skins to display an Aired date.

[barney63]

@jbinkley60

Championship Figure Skating: Italian Grand Prix - Women's Free Program - Error No mediaId from the CBC Sports website

2021 Skate Ontario Sectional Championships Rink #1 is working

Log file

Edit: Road to the Olympic Games: FIG Rhythmic Gymnastics World Championships - Error No mediaId from the CBC Sports website

[barney63]

Will test the following four events in order and then post log file

2021 Skate Ontario Sectional Championships Rink #1 is working
Road to the Olympic Games: FIG Rhythmic Gymnastics World Championships - Error No mediaId from the CBC Sports website
UCI Track Champions League on CBC: Mallorca is working
Championship Figure Skating: Italian Grand Prix - Pair's Free Program - Error No mediaId from the CBC Sports website

Log file

Edit:
Canadian Hockey League: Saint John Sea Dogs vs. Drummondville Voltigeurs - Error No mediaId from the CBC Sports website

Championship Figure Skating: Italian Grand Prix - Mens's Free Program - Error No mediaId from the CBC Sports website

[jbinkley60]

Thank you Jeff for the offer of help. I was successful with back porting your improvements to Kodi Leia.

For Kodi Matrix the changes to enable the skin to display the Aired date are:

• In the VIDEOS function set aired format to '%Y-%m-%d' (I'm not sure if this is required, I just copied another add-on):
  
  aired = datetime.fromtimestamp(pubDate / 1000).strftime('%Y-%m-%d')
  
  
• In the addDir2 function add Aired, aired_text, Year, and mediatype:
  
  
  plot_text = ''
  if aired and plot:
          plot_text = '[COLOR blue]Aired: ' + aired + '\n\n' + '[COLOR blue]Description: ' + plot
  aired_text = ''
  if aired:
          aired_text = aired
  infoLabels={ "Title": name,
               "Aired": aired_text,
               "Year": aired_text.split('-',1)[0],
               "mediatype": 'episode',
               "Duration": duration,
               "Plot": plot_text }
  liz.setInfo( "video", infoLabels)
  
  
  Note mediatype set to episode.
  

I'm sure you'd easily figure this out but thought I might post how I did it so others might try and perhaps learn a bit about video add-on development.

I tested this with the Confluence and Estuary skins. Note that there might be other values for mediatype that are more appropriate. I simply stopped when I discovered that a value of episode enabled the skins to display an Aired date.

Thanks for the edits.  I've incorporated the infolabels changes into v0.999 which I just uploaded.  I also adjusted the aired date format to match what you have.  The skins will change it to whatever the local country type / format is.  It's nice having the info item now working on the context menu.  I did take a quick peak at the JSON data which is returned to see if there was any other information which would be interesting.  The only thing which caught my eye was the resolution in case folks want to see SD / HD etc.. icons displayed.  If so I can add that code.


Jeff

[kodaksmith]

Thank you Jeff for resurrecting this add-on for Matrix.

With the 0.999 version, I think mine says it is 0.997 in addon.xml.  Perhaps a typo when updating from 0.998?

[jbinkley60]

Thank you Jeff for resurrecting this add-on for Matrix.

With the 0.999 version, I think mine says it is 0.997 in addon.xml.  Perhaps a typo when updating from 0.998?

Sorry about that.  I just posted version 1.0.0 .  It should show up correctly now.  This version adds the video resolution and aspect ratios for the on demand videos.  It also standardizes the language file and image file formats to the current Kodi Matrix standards.


Jeff

[barney63]

@jbinkley60

After updating to 1.0.0, clicking on Live Schedule gives the Check the Log error

[jbinkley60]

@jbinkley60

After updating to 1.0.0, clicking on Live Schedule gives the Check the Log error

So sorry about that.  Please download again.  I have fixed 1.0.0.  It was a simple 1 character fat finger on my part.


Jeff

[barney63]

That's a relief

[jbinkley60]

That's a relief

I looked through the logs you posted.  Thanks for taking the time to post them and making me aware of the issue above.  All of the failures you posted were no mediaIds coming from the CBC Sports website.  I am looking to see if there is another field they might be sending that can be used to play the videos if they don't send the mediaId.  I now need to find one which works for me that I can debug while it is playable to see if another field will also work. 


Jeff

[barney63]

@jbinkley60

Not sure if I am looking at the correct thing, but all four events for today have a media Id when the "Coming Up" video is right clicked at the website.  The addon (Live Schedule) is giving the no mediaId is available error

FIVB Men's and Women's Beach Volleyball World Tour on CBC: Pool Play - Itapema  
mediaId 1970697283888

International Swimming League on CBC: Playoffs  
mediaId 1972005955815

Championship Figure Skating on CBC: Grand Prix of Japan - Pairs Short Program
mediaId 1971029059671

Championship Figure Skating on CBC: Grand Prix of Japan - Ice Dance Rhythm Dance
mediaId 1971029059724

https://www.cbc.ca/player/play/1970697283888