Bug 8465 - Can't reconfigure Audioscrobbler account
: Can't reconfigure Audioscrobbler account
Status: RESOLVED INVALID
Product: Logitech Media Server
Classification: Unclassified
Component: Plugin
: 7.2
: PC Windows XP
: P3 normal with 1 vote (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-06-18 00:39 UTC by Philip Meyer
Modified: 2009-07-31 10:23 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
even more logging (3.40 KB, application/octet-stream)
2008-06-26 02:29 UTC, Michael Herger
Details
more logging version (3.92 KB, application/octet-stream)
2008-06-26 23:28 UTC, Michael Herger
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Meyer 2008-06-18 00:39:38 UTC
I had a perfectly working Last.fm Audioscrobbler account.  However, I accidentally deleted the account setting by ticking the checkbox.

I have tried to re-enter the account, but after hitting the Apply button all I see is the message "Changes have been saved."  The account is not listed and I cannot select an account to scrobble with under the player UI Audioscrobbler settings page.
Comment 1 Michael Herger 2008-06-19 08:40:28 UTC
change 20915 - changed the wording and layout of the AudioScrobbler settings page a bit to make it less prone to accidental account deletion.

I wasn't able to reproduce your issue though. No problem entering the information (once I remembered its). Is the above confirmation all you got? I at first got an error message about a wrong login/password.

Can you still reproduce this? The obvious question: what browser?
Comment 2 Philip Meyer 2008-06-19 13:13:04 UTC
Yes, I still can't get my last.fm account settings to stick in the Audioscrobbler plugin settings page.

I have tried FF2, FF3 and IE7.  Using default skin.

I have ensured that I am entering the correct account name and password.

In fact, no matter what I enter as a username or password, I get "Changes have been saved.".

I have checked in SqueezeNetwork - my last.fm account is correctly configured in there, and assigned to my players.

It could have gone wrong because I had an account configured called "Hergest", which didn't work with Last.fm because SC didn't like the upper-case H.  I added another one called "hergest", which worked (Last.fm not case sensitive I guess).

A fix was applied to SC to allow the upper-case letters.

I then deleted both accounts accidentally when I only meant to delete one.  Now I can't add either "Hergest" or "hergest" back.
Comment 3 Michael Herger 2008-06-19 14:06:38 UTC
Can you get some log information?
Comment 4 Philip Meyer 2008-06-19 14:16:03 UTC
I did look yesterday in the log for errors and couldn't see anything.

However, just looked again after trying the settings page in fishbone skin, and I noticed this:

[22:06:09.4585] Slim::Web::Settings::handler (145) Preference names must be prefixed by "pref_" in the page template: accounts (PLUGIN_AUDIOSCROBBLER_MODULE_NAME)

Comment 5 Michael Herger 2008-06-19 22:15:46 UTC
> I did look yesterday in the log for errors and couldn't see anything.

Please enable debugging for plugin.audioscrobbler

> [22:06:09.4585] Slim::Web::Settings::handler (145) Preference names must be
> prefixed by "pref_" in the page template: accounts
> (PLUGIN_AUDIOSCROBBLER_MODULE_NAME)

Sounds like outdated files on your system? What exact revision number do the plugin files have?
Comment 6 Philip Meyer 2008-06-20 00:43:09 UTC
TortoiseSVN reports that every file is up to date.

I have plugin.audioscrobbler already set to debug; the only message I see is:

[08:40:36.2118] Slim::Web::Settings::handler (145) Preference names must be prefixed by "pref_" in the page template: accounts (PLUGIN_AUDIOSCROBBLER_MODULE_NAME)



package Slim::Plugin::AudioScrobbler::Plugin;
# $Id: Plugin.pm 20483 2008-06-07 16:44:42Z andy $
Comment 7 Michael Herger 2008-06-20 01:04:57 UTC
Can you take a quick look at its Settings.pm to see whether there's the following line:

$params->{pref_accounts} = $prefs->get('accounts') || [];

Do you get this message whenever you open the settings page or only when changing?
Comment 8 Philip Meyer 2008-06-20 01:27:10 UTC
Yes, I have that line.

I think I only get the warning when I hit apply (or press return in Username or password fields).  Can't check at the moment - will do when I am home tonight.
Comment 9 Michael Herger 2008-06-20 04:49:12 UTC
Probably related case in the forums: http://forums.slimdevices.com/showthread.php?t=48851#7

"The Last.fm plugin was enabled, as was the Last.fm Audioscrobbler plugin.

The problem I encountered was as follows:

Setup > Advanced > Last.fm Audioscrobbler
Enter Last.fm Account and password and click on Apply
Reports back that 'Changes have been saved.

Setup > Player > Last.fm Audioscrobbler
Can't enable submit as it is 'globally disabled.'

Setup > Advanced > Last.fm Audioscrobbler
Account has not been saved.

As it won't save my Account it doesn't seem to allow scrobbling from the player."
Comment 10 Adam Slim 2008-06-22 06:09:05 UTC
I get this, exactly the same as Phil.  Using 7.1 beta (21008); had it with the last nightly I used too.  Extract from log file:

[13:53:16.6768] Slim::Web::Settings::handler (145) Preference names must be prefixed by "pref_" in the page template: accounts (PLUGIN_AUDIOSCROBBLER_MODULE_NAME)
[13:53:43.4758] Slim::Plugin::AudioScrobbler::Plugin::_handshakeOK (281) Invalid Last.fm username or password.

(lots of username/pw fails, but it's right!)

I think the first error occurs when I try to save the settings; the second when it tries to submit a track.

Comment 11 Michael Herger 2008-06-23 01:34:30 UTC
Is anyone of you running the Windows binary? If so: would uninstalling/re-installing help? We've encountered an issue where _some_ files (most importantly: squeezecenter.exe) would not be replace during an up-/downgrade). 
Comment 12 Michael Herger 2008-06-23 02:18:14 UTC
change 21034 - I think the pref_accounts item wasn't correctly passed along validation. This change should improve this, plus add a bit more logging. Please give it another try with debugging enabled. Thanks!
Comment 13 Philip Meyer 2008-06-23 17:15:11 UTC
That change doesn't seem to have been merged into 7.2 branch.
Comment 14 Michael Herger 2008-06-23 22:29:28 UTC
they are in 21042 - why do you think they haven't been merged? Don't you see any change in behaviour?
Comment 15 Michael Herger 2008-06-25 05:07:11 UTC
Any news?
Comment 16 Philip Meyer 2008-06-25 07:28:53 UTC
Yes, I do appear to have the change, and no it didn't work!
No change in behaviour, no additional log messages.
Comment 17 Michael Herger 2008-06-25 07:45:29 UTC
If you enable debugging for plugin.audioscrobbler you should get some information whether it fails or not. You might want to add network.squeezenetwork to the mix too.
Comment 18 Philip Meyer 2008-06-25 16:32:05 UTC
I already had plugin.audioscrobbler set to debug level.  I've included network.squeezenetwork:

[00:29:18.5824] Slim::Web::Settings::handler (145) Preference names must be prefixed by "pref_" in the page template: accounts (PLUGIN_AUDIOSCROBBLER_MODULE_NAME)
[00:29:18.6929] Slim::Networking::SqueezeNetwork::PrefSync::prefEvent (419) Global prefEvent to sync: plugin.audioscrobbler / accounts / []
[00:29:48.7052] Slim::Networking::SqueezeNetwork::PrefSync::syncUpGlobal (526) Syncing global prefs up to SN: {
  prefs => {
        "plugin.audioscrobbler.accounts" => { ts => "1214436562", value => [] },
      },
}
[00:29:49.1217] Slim::Networking::SqueezeNetwork::PrefSync::_syncUp_done (556) Sync OK

Comment 19 Michael Herger 2008-06-26 02:29:15 UTC
Created attachment 3495 [details]
even more logging

Could you please give the attache Settings.pm another try? It will spit out a lot of "errors". What do you see in the logs?
Comment 20 Philip Meyer 2008-06-26 12:26:53 UTC
I copied that file over the top of my \server\Slim\Plugin\AudioScrobbler\settings.pm and it doesn't indicate any differences.  Are you sure you attached the modified version?
Comment 21 Michael Herger 2008-06-26 12:38:50 UTC
Argh... sorry. That's indeed the wrong file :-(. And the correct one is at the office. Will update tomorrow.
Comment 22 Michael Herger 2008-06-26 23:28:01 UTC
Created attachment 3498 [details]
more logging version

Too many branches checked out... this should be the right one
Comment 23 Philip Meyer 2008-06-27 00:47:06 UTC
Still don't see any extra logging.

I enter a user name and password, and then press Apply.  All I see in the log is:

[08:44:03.0369] Slim::Web::Settings::handler (145) Preference names must be prefixed by "pref_" in the page template: accounts (PLUGIN_AUDIOSCROBBLER_MODULE_NAME)
[08:44:03.1384] Slim::Networking::SqueezeNetwork::PrefSync::prefEvent (419) Global prefEvent to sync: plugin.audioscrobbler / accounts / []

Comment 24 Philip Meyer 2008-06-27 01:37:03 UTC
I blew away my SVN checkout and recreated.  I deleted my cache folder.  This "bug" has now gone away - sorry.
Comment 25 Michael Herger 2008-06-27 02:04:52 UTC
> I blew away my SVN checkout and recreated.  I deleted my cache folder.  This
> "bug" has now gone away - sorry.

Wow... I wasn't all that wrong then. Thanks for testing!
Comment 26 Rich Maggio 2008-08-01 12:11:13 UTC
I am experiencing this problem after I updated to the 7.1 production release.  I upgraded from 7.01.

It started where I could not configure my players to scrobble - it directed me to the advanced settings, which was enabled.

I deleted my account with the intent of adding it right back to see if that would solve the problem.  This did nothing, but now I can't create my account.  I get the same log messages described by this bug report.

Others in the forums are seeing the same behavior.
Comment 27 elziko 2008-08-04 01:41:14 UTC
I am also seeing this in v7.1 - should this be reopened?
Comment 28 Chris Owens 2009-07-31 10:23:02 UTC
Reduce number of active targets for SC