This is from https://drupal.org/comment/8151919#comment-8151919 for separate tracking. Thanks to @dandaman for starting!

CommentFileSizeAuthor
#2 mp3player-2139711-2.patch1.93 KBmarkie
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

markie’s picture

markie CreditAttribution: markie commented
Issue summary: View changes
markie’s picture

markie CreditAttribution: markie commented
FileSize
mp3player-2139711-2.patch1.93 KB

So now that we are comfortable here. I wrote that serialization so long ago, I don't even know why. I guess documentation would be nice, right? I found in my test site, $item['data'] was always null, so I wonder if I pulled that in from the 7.x branch. I'll take a closer look. Going to go ahead and move that if statement out into the main function which should resolve your issue. That being said.. probably should still know why it's there in the first place, right? Patch attached..

asb’s picture

asb CreditAttribution: asb commented

The patch does not apply cleanly against the dev release from drupal.org, neither with -p0 nor -p1.

$ cat mp3player.module.rej 
--- mp3player.module
+++ mp3player.module
@@ -86,7 +86,6 @@
   $header = array(t('Player Name'), t('Actions'));
   $rows = array();
   $players = mp3player_players();
-  kpr($players);
   foreach (mp3player_players() as $pid => $player) {
     $row = array();
     $row[] = l($player['name'], 'admin/settings/mp3player/player/'. $player['pid']);

Other/related issues: When the module is activated, I get an WSOD when accessing Views pages that include the player, and when accessing ./admin/reports/status. Also, none of the players at ./admin/settings/mp3player/player can be edited. So, as far as I can the tell, the module is totally broken and is not of much use at the moment.