Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExternalPlayer::getTime() function is broken #16183

Open
1 of 7 tasks
bigretromike opened this issue May 21, 2019 · 0 comments
Open
1 of 7 tasks

ExternalPlayer::getTime() function is broken #16183

bigretromike opened this issue May 21, 2019 · 0 comments
Labels
Triage: Needed (managed by bot!) issue that was just created and needs someone looking at it

Comments

@bigretromike
Copy link
Contributor

Bug report

Describe the bug

Here is a clear and concise description of what the problem is:

When using external player via playercorefactory.xml before Leia Kodi would report 1.0 as watched (https://forum.kodi.tv/showthread.php?tid=331041). Right now even when isExternalPlayer() returns true the position with getTime() returns 0 all the time with our without setting <playcountminimumtime> in playercorefactory.xml

Expected Behavior

Here is a clear and concise description of what was expected to happen:
When using ExternalPlayer getTime() should return 1.0 when default <playcountminimumtime> or user defined value would be reached.

Actual Behavior

Using ExternalPlayer and looping with getTime() after onPlayBackStarted while checking isPlayingVideo results always with 0.

Possible Fix

Probably somewhere around here: yol@fc14587 someone forgot to add that 1.0 mark that was working before.

To Reproduce

Steps to reproduce the behavior:

  1. Configure external player with playcountminimumtime set to small number 10/20
  2. Use any addon that play video
  3. Start looping with printing player.getTime() to log
  4. Start playing video
  5. Wait until you reach the minimumtime plus some extra time to be double sure

Debuglog

The debuglog can be found here:
Debug.log

Screenshots

Here are some links or screenshots to help explain the problem:

Additional context or screenshots (if appropriate)

Here is some additional context or explanation that might help:

Your Environment

Used Operating system:

  • Android

  • iOS

  • Linux

  • OSX

  • Raspberry-Pi

  • Windows

  • Windows UWP

  • Operating system version/name:

  • Kodi version: 18.2 stable

note: Once the issue is made we require you to update it with new information or Kodi versions should that be required.
Team Kodi will consider your problem report however, we will not make any promises the problem will be solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triage: Needed (managed by bot!) issue that was just created and needs someone looking at it
Projects
None yet
Development

No branches or pull requests

1 participant