Bug #1827

Youtube doesn't mark items as played

Added by Leonid Protasov almost 7 years ago. Updated almost 7 years ago.

Status:FixedStart date:01/06/2014
Priority:NormalDue date:
Assignee:Fábio Ferreira% Done:

100%

Category:Plugins
Target version:4.4
Found in version:4.3.627 Platform:PS3

Description

Continious plays works good. If you manually mark item, then close plugin, reopen the item list - mark is there...

History

#1 Updated by Andreas Smas almost 7 years ago

  • Assignee changed from Andreas Smas to Fábio Ferreira

The problem here is that canonicalUrl is not correct in the plugin.

When playing an item it opens:

youtube:video:njos57IJf-0

This would also be used as canonicalUrl by directory model and is what it binds the extra data to (playcount, restart position, etc).

However, after following the redirects it's setup like this (in youtube.js):

    function playVideo(page, title, id, video_url) {
        var url = unescape(unescape(video_url));
        var videoParams = {      
            title: unescape(unescape(title)),
            canonicalUrl: PREFIX + ':video:simple:' + title + ':' + id,
            sources: [{    
                url: url  
            }]    
        }

        ...

So, obviously the canonicalUrl does not match and thus the playcount will be registered on PREFIX + ':video:simple:' + title + ':' + id

Changing this code snippet to:

            canonicalUrl: PREFIX + ':video:' + id,

Makes it work for me.

I think the youtube plugin should settle on a standard (and very simple) canonicalUrl like this, just the video id.

#2 Updated by Leonid Protasov almost 7 years ago

  • Subject changed from ST doesn't mark items as played on page.redirect pages to Youtube doesn't mark items as played
  • Category changed from Video playback to Plugins
  • Target version changed from 4.4 to 33
  • Platform changed from RPi to PS3

#3 Updated by Leonid Protasov almost 7 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Fixed in Youtube v2.2.8.

#4 Updated by Andreas Smas almost 7 years ago

  • Target version deleted (33)

#5 Updated by Andreas Smas almost 7 years ago

  • Target version set to 4.4

Also available in: Atom PDF