Bug 206028 - add irkick-compatible relative seek(sec) operation
Summary: add irkick-compatible relative seek(sec) operation
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: 2.3-GIT
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-02 19:41 UTC by Andrew Gaydenko
Modified: 2009-12-09 11:34 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Gaydenko 2009-09-02 19:41:10 UTC
Version:            (using KDE 4.3.0)
OS:                Linux
Installed from:    Ubuntu Packages

Please, add irkick-compatible relative seek(sec) operation, consuming both positive and negative int argument 'seconds':

- seeking 'too early' jumps to the beginning of the current track,
- seeking 'too late' starts next track at case last one can be determined.
Comment 1 Edward Hades 2009-09-07 20:01:27 UTC
Fixed by commit 3bff88683f0abeba6a03909f05a130fb4871738c
Author: Michael Zanetti <michael_zanetti@gmx.net>
Date:   Mon Sep 7 12:19:42 2009 +0200

    Added DBus functions Forward(int) and Backward(int)to Player

 src/dbus/PlayerDBusHandler.cpp                  |   12 ++++++++++++
 src/dbus/PlayerDBusHandler.h                    |    3 +++
 src/dbus/org.freedesktop.MediaPlayer.player.xml |    8 ++++++++
 3 files changed, 23 insertions(+), 0 deletions(-)
Comment 2 Myriam Schweingruber 2009-09-07 20:16:41 UTC
*** Bug 206660 has been marked as a duplicate of this bug. ***
Comment 3 ciro.scognamiglio 2009-09-07 20:28:32 UTC
Sorry, I fail to see how the two bugs are related.
The original bug is about D-Bus methods, mine is about the signals amarok sends.
I took a look at your commit and I think it's completely irrelevant to the bug.
Comment 4 Andrew Gaydenko 2009-09-19 02:19:13 UTC
Unfortunately, opening the issue. Have got A2 2.1.85 (under Kubuntu Karmic testing), both functions - Forward and Backward - just seek 1-2 sec back. For both I have set the parameter as 20 in IRKick config.
Comment 5 Edward Hades 2009-09-21 19:20:13 UTC
Try 20000, they're in milliseconds, IINM.
Comment 6 Andrew Gaydenko 2009-09-21 19:27:35 UTC
For both functions max value is 99.
Comment 7 Michael Zanetti 2009-10-18 22:15:43 UTC
(In reply to comment #6)
> For both functions max value is 99.

Is this still an issue? Should be fixed in latest kdelirc.
Comment 8 Andrew Gaydenko 2009-10-19 06:36:35 UTC
Michael, I'm able to set, say, 20000 now, thanks!

The problem is, seeking doesn't work for me even with mouse (playing stops for few seconds, seek bar jumps back and such). So, it seems like another bug takes place (A2 2.20 is in use under Kubuntu Karmic testing), and lirc related one - this bug - can be closed.

With RC seek bar behavior is now the same as with seeking via mouse (stopping and jumping back and such).

I'll report at case any lirc-related problem will nevertheless be discovered after A2 seeking be cured.
Comment 9 Michael Zanetti 2009-10-19 09:59:33 UTC
OK. So I'll close this bug as the relative seek functions are implemented in amarok and the integer limit to 99 is fixed in kdelirc.

If there are other problems with amarok please report them a separate bugs to give appropriate developers a chance to find bug reports by just reading the subject line.
Comment 10 Ben Adler 2009-10-23 02:31:41 UTC
I think that Forward(int) and Backward(int) are rather problematic. It's not clear whether it'll seek (X seconds? X milliseconds?) or forward/rewind X tracks in the playlist. I know there's next() and prev(), but thats not a good enough excuse for having such undescriptive seeking methodnames.

How about allowing negative numbers and using "Seek(int)"? Or maybe SeekMS(int)?

Just an idea.
Comment 11 Michael Zanetti 2009-10-23 09:46:32 UTC
I can't change the value from ms to s bechause all other functions work with ms too. That's because it is defined like this in the MPRIS spec. All other players following the MPRIS spec (VLC, XMMS etc) also work with ms.

I implemented these functions at first as "seek(int)" but the amarok developers told me that they'd like to have it as Backward() and Forward().

I'll update the amarok profile in kdelirc to have a template argument of 10000. So it should be clear for everyone to use ms here (if there is a need to change the default value).