abhay123lp / tunesremote-plus

Automatically exported from code.google.com/p/tunesremote-plus
0 stars 0 forks source link

MonkeyTunes Shuffle/Repeat Update Bug #64

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Q. What Android hardware device do you own?  (HTC Hero, Samsung Galaxy,
etc)

A: Samsung Galaxy S i9000

Q. On what Android operating system version? (2.1, 2.2, Honeycomb 3.0.1
etc)

A: 2.2

Q. What DACP server, OS, and version? (OSX iTunes 10.1.2, Windows 7
MonkeyTunes 1.5.1 etc)

A: Windows 7, MonkeyTunes

Q. What TunesRemote+ version? (found on Menu->More->About)

A: latest Beta

Please provide any additional information below.

While doing the test of the current Beta and release version, I noticed that if 
I do changes on the computer in terms of Shuffle and Repeat, they won't show on 
TunesRemote.
Example: I change on the computer Shuffle -> On
In TunesRemote it will still be displayed as Off.
Same goes for Repeat.

Also to not open another issue for this small thing, can you add the Search 
Button also to Artist and Playlist lists, as right now there are only the 
Artist and Playlist Menu option, but would be also nice to have the Search 
option in the Menu there.

Thanks :)

Original issue reported on code.google.com by cramermaurice@gmail.com on 25 Jun 2011 at 5:11

GoogleCodeExporter commented 8 years ago
For your two part request I will answer in two parts...

1. Unfortunately the shuffle feature cannot be done.  Why?  Because when you 
hit SHuffle in MediaMonkey there is no notification back to MonkeyTunes that 
the shuffle state has changed.   The only time it will change is when the Track 
changes in MM and then the UI's are notified of the updates.  So essentially 
this is a limitation of MediaMonkey API.

2. Adding a Search menu should be easy.

Original comment by mellowaredev on 25 Jun 2011 at 5:14

GoogleCodeExporter commented 8 years ago
Updated title...

Original comment by mellowaredev on 25 Jun 2011 at 5:14

GoogleCodeExporter commented 8 years ago
Ah ok, figured it would be something like that. Just wanted to make sure and 
write it here first :)
Really gotta give it to you with your quick response and user friendliness. 
Props to you for that.

Let me know when you want me to try out the menu addition. 

Original comment by cramermaurice@gmail.com on 25 Jun 2011 at 5:18

GoogleCodeExporter commented 8 years ago
No problem.  I will post a new BETA on Monday.  I will post here when its ready 
for test.

Original comment by mellowaredev on 25 Jun 2011 at 8:35

GoogleCodeExporter commented 8 years ago
OK adding the Search option to that Screen is more difficult than I thought.  
The way Jeff Sharkey designed the original the Search was tied to just that 
main Menu.  While trying to elegantly call it from the Artist or Playlist list 
it is crashing TunesRemote because of the way it is wired.  It will take some 
significant re-writing of the code to get that to work that I just don't have 
the time to tackle right now!

Original comment by mellowaredev on 27 Jun 2011 at 4:21

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Alright, don't want to add more work then needed. So you can just close this 
issue as WontFix.

Thanks for trying :)

Original comment by cramermaurice@gmail.com on 28 Jun 2011 at 7:48

GoogleCodeExporter commented 8 years ago

Original comment by mellowaredev on 28 Jun 2011 at 7:52

GoogleCodeExporter commented 8 years ago
I think I can fix this.  It looks like in MM3.1 and higher they added 
OnRepeatClick and On ShuffleClick events so I would be able to then trigger a 
UI refresh in TunesRemote.

http://www.mediamonkey.com/wiki/index.php/SDBApplication

Original comment by mellowaredev on 11 Jul 2011 at 6:20

GoogleCodeExporter commented 8 years ago
Yea that sounds nice,if it isnt to much work to add.

Original comment by cramermaurice@gmail.com on 23 Jul 2011 at 11:11

GoogleCodeExporter commented 8 years ago
Try this latest MonkeyTunes.  It now changes when you change Shuffle or Repeat 
on the server.

http://melloware.biz/download/warez/monkeytunes-beta.zip

Original comment by mellowaredev on 28 Aug 2011 at 11:30

GoogleCodeExporter commented 8 years ago
Just tried it out. It turns the shuffle on nicely, but doesn't seem to refresh 
in the app when turning the shuffle back off. It only refreshes for the turning 
off shuffle when you close the app and open it up again.

Also after the update, you can't turn shuffle on or off anymore from the App 
itself at all.

Same goes for Repeat.

Original comment by cramermaurice@gmail.com on 29 Aug 2011 at 6:33

GoogleCodeExporter commented 8 years ago
Whoops... sorry about that try this version...

http://melloware.biz/download/warez/monkeytunes-beta.zip

Original comment by mellowaredev on 29 Aug 2011 at 11:00

GoogleCodeExporter commented 8 years ago
Works great now on both the repeat and the shuffle. Nice job :D

But I would recommend renaming the repeat Button. Currently when repeat is off, 
it says "Repeat Single" and when on "No Repeat". Sounds a bit confusing, I'd 
recommend doing it like on the Shuffle, having "Repeat is on/off". Other than 
that great job, and you can mark it as added feature, and as done ;)

Original comment by cramermaurice@gmail.com on 29 Aug 2011 at 11:16

GoogleCodeExporter commented 8 years ago
Unfortunately this is due to iTunes.  Itunes has 3 states....

Repeat Off
Repeat Single
Repeat All

MediaMonkey only supports Repeat On or Repeat Off.  So you are seeing a side 
effect of me "cramming MM to fit the iTunes model".

Original comment by mellowaredev on 29 Aug 2011 at 11:25

GoogleCodeExporter commented 8 years ago
Ah ok, an you can't differentiate if the commands are coming from iTunes or MM. 
Well that's ok. None the less, it's working well :) Thanks a bunch for the fix.

Original comment by cramermaurice@gmail.com on 29 Aug 2011 at 12:01