Bug 16556 - Error when Removing a WiMP account while in the app on a device
: Error when Removing a WiMP account while in the app on a device
Status: VERIFIED FIXED
Product: MySqueezebox.com
Classification: Unclassified
Component: WiMP
: Test
: PC Windows XP
: P3 normal (vote)
: ---
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-09-16 13:42 UTC by Kris Murphy
Modified: 2010-09-21 15:24 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kris Murphy 2010-09-16 13:42:15 UTC
Steps to repro:

1. Set up WiMP account info in mysqueezebox.com
2. Enable WiMP app for a device
3. On device navigate to a track in WiMP (do not play it)
4. In mysqueezebox.com remove WiMP account info
5. Play track on device
Results on Duet:  Error message "malformed JSON string, neither array, object, number string or atom, at character offset"
Results on Baby:  Problem Connecting screen with "Failed to parse"

Can we do a couple of things here?
1.  Hide that error message from the user
2.  Make the resulting message more intuitive so the user knows what's going on?

I don't know how often this scenario will pop up but I think hope we can do a better job messaging the cause of the error.
Comment 1 SVN Bot 2010-09-17 05:37:30 UTC
 == Auto-comment from SVN commit #8858 to the network repo by mherger ==
 == http://svn.slimdevices.com/network?view=revision&revision=8858 ==

Fixed Bug: 16556
Description: catch failure to create WiMP session.

Unfortunately this is a generic issue which can arise in many places under many circumstances. Thus the generic error handling. If it happens during navigation, the user will be asked to check his WiMP credentials. If it happens when starting playback, I have little influence on the message, as the controlling piece of software doesn't receive the actual error, but is expecting a URL - which it doesn't get. Thus the generic "failed to load URL" or similar message in this case.
Comment 2 Kris Murphy 2010-09-17 16:19:37 UTC
So was there a change that I should be validating?  

Like this part:
"If it happens during
navigation, the user will be asked to check his WiMP credentials."
Comment 3 Michael Herger 2010-09-19 21:39:30 UTC
> So was there a change that I should be validating?  

I've addressed your point 1: "hide that error message from the user". Just repeat your steps as reported in the first comment.
Comment 4 Kris Murphy 2010-09-21 15:24:10 UTC
Michael's fix satisfies the request from PQA.  Moving to VERIFIED.