Bug 14521 - Save Playlist (on current playlist screen) causes reconnecting screen
: Save Playlist (on current playlist screen) causes reconnecting screen
Status: VERIFIED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: SB Server
: 7.4.x
: PC Other
: P1 normal (vote)
: 7.4.1
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-02 06:52 UTC by Wadzinski Tom
Modified: 2009-10-14 12:15 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 Wadzinski Tom 2009-10-02 06:52:35 UTC
I think this will give a bad impression to new baby users so I think it should be in 7.4.1....

I got the following error, then dumped the obj (shown below)
20091002 13:38:47.037 WARN   net.comet - Comet.lua:871 Comet {tom-workpc}: _response, nil id=nil failed: encountered object 'Slim::Schema::Playlist=HASH(0x9ec7054)', but neither allow_blessed nor convert_blessed settings are enabled at C:/svn/server/Slim/Web/Cometd.pm line 638.

20091002 13:38:47.037 WARN   net.comet - Comet.lua:951 Comet {tom-workpc}: _response, unknown error: encountered object 'Slim::Schema::Playlist=HASH(0x9ec7054)', but neither allow_blessed nor convert_blessed settings are enabled at C:/svn/server/Slim/Web/Cometd.pm line 638.





[09-10-02 08:47:20.7328] Data::Dump::dump (100) Warning: do {
  my $a = [
    {
      channel => "/e2ba7fb6/slim/request",
      data => {
            __playlist_id  => 1023,
            __playlist_obj => bless({
                                _column_data       => {
                                                        album              => undef,
                                                        audio              => undef,
                                                        audio_offset       => undef,
                                                        audio_size         => undef,
                                                        bitrate            => undef,
                                                        block_alignment    => undef,
                                                        bpm                => undef,
                                                        channels           => undef,
                                                        content_type       => "ssp",
                                                        cover              => undef,
                                                        disc               => undef,
                                                        drm                => undef,
                                                        endian             => undef,
                                                        extid              => undef,
                                                        filesize           => 1027,
                                                        id                 => 1023,
                                                        lossless           => undef,
                                                        lyrics             => undef,
                                                        musicbrainz_id     => undef,
                                                        musicmagic_mixable => 1,
                                                        primary_artist     => undef,
                                                        remote             => 0,
                                                        replay_gain        => undef,
                                                        replay_peak        => undef,
                                                        samplerate         => undef,
                                                        samplesize         => undef,
                                                        secs               => undef,
                                                        tagversion         => undef,
                                                        timestamp          => "1237046720",
                                                        title              => 123,
                                                        titlesearch        => 123,
                                                        titlesort          => 123,
                                                        tracknum           => undef,
                                                        url                => "file:///Users/tom/Music/Playlists/123.m3u",
                                                        vbr_scale          => undef,
                                                        year               => 0,
                                                      },
                                _dirty_columns     => {},
                                _in_storage        => 1,
                                _inflated_column   => {},
                                _orig_ident        => { id => 1023 },
                                _relationship_data => {},
                                _source_handle     => bless({ schema => "Slim::Schema", source_moniker => "Playlist" }, "DBIx::Class::ResultSourceHandle"),
                                related_resultsets => {
                                                        playlist_tracks => bless({
                                                              _result_class => "Slim::Schema::PlaylistTrack",
                                                              _source_handle => bless({ schema => "Slim::Schema", source_moniker => "PlaylistTrack" }, "DBIx::Class::ResultSourceHandle"),
                                                              attrs => {
                                                                    accessor => "multi",
                                                                    alias => "me",
                                                                    cascade_copy => 1,
                                                                    cascade_delete => 1,
                                                                    join_type => "LEFT",
                                                                    where => { "me.playlist" => 1023 },
                                                                  },
                                                              cond => 'fix',
                                                              count => undef,
                                                              pager => undef,
                                                            }, "Slim::Schema::ResultSet::PlaylistTrack"),
                                                      },
                              }, "Slim::Schema::Playlist"),
          },
      ext => { priority => "" },
      id => 31,
    },
  ];
  $a->[0]{data}{__playlist_obj}{related_resultsets}{playlist_tracks}{cond} = $a->[0]{data}{__playlist_obj}{related_resultsets}{playlist_tracks}{attrs}{where};
  $a;
}
Comment 1 Andy Grundman 2009-10-02 08:50:43 UTC
The playlistSaveCommand was adding __playlist_obj to the response, but this does not appear to be used by anything, so I'm going to remove it.
Comment 2 SVN Bot 2009-10-02 08:52:21 UTC
 == Auto-comment from SVN commit #28756 to the slim repo by andy ==
 == https://svn.slimdevices.com/slim?view=revision&revision=28756 ==

Fixed bug 14521, remote unused and un-JSON-able __playlist_obj return value from playlistSaveCommand
Comment 3 James Richardson 2009-10-14 12:15:38 UTC
Verified fixed in SBS r28848