Page 1 of 1

"Our plans are on hold" - archive bug?

Posted: Sun Apr 28, 2013 9:40 am
by Manhattan Glutton
http://www.songfight.org/songpage.php?k ... re_on_hold

Problem: "winner: The Wankers" should be "winner: glennny"

The win also isn't showing up on glennny's artist page. I would venture that this is due to the common typo of forgetting the third n in glennny.

Re: "Our plans are on hold" - archive bug?

Posted: Sun Apr 28, 2013 10:43 am
by Lunkhead
My guess is it's a copy/paste error as The Wankers were the winners of the previous fight. Could be the previous fight's archive data was copied, pasted, and modified, to create the "Our Plans Are On Hold" data, and the winning artist name was not updated from what was copied and pasted.

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 12:04 pm
by Spud
Well, no idea how it happened, but it is fixed.

SPUD

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 12:05 pm
by Spud
Manhattan Glutton wrote: I would venture that this is due to the common typo of forgetting the third n in glennny.
Whatever it was, it wasn't that.

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 12:07 pm
by Spud
Lunkhead wrote:My guess is it's a copy/paste error as The Wankers were the winners of the previous fight. Could be the previous fight's archive data was copied, pasted, and modified, to create the "Are Plans Are On Hold" data, and the winning artist name was not updated from what was copied and pasted.
It probably wasn't that, either, as that part of the process is completely automated. No cutting and pasting involved. In fact, I NEVER type the artist's name at any point during the process, unless they neglect to send it to me in the body of their submission email.

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 12:17 pm
by Manhattan Glutton
Strangely enough, the SFJukebox wasn't affected. That reads directly from the text file, right Sam?

Do both songfight.org and jukebox calculate the winner from the archive text the same way? Is the winner its own dedicated piece of data, or does it look to the vote total?

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 12:29 pm
by Lunkhead
I noticed the Jukebox wasn't effected, which was surprising to me. It does read straight from the secret text file "database". The winning artist name is split out into its own line in the text file, after the fightmaster's summary line. I think I have some janky stuff in my code that may be getting the winner from somewhere else while the fight is still "current" though (e.g. not in the archive yet). I don't have the code handy so I don't remember right now.

By the way, I have my code in BitBucket and could share it with you if you want to sign up (BitBucket is like GitHub only with free unlimited private repos, if you haven't heard of it). Not that I think you're that interested but I feel like more eyes on it could definitely help with its quality. ;)

Just looked at the code, I'm really not sure how the Jukebox didn't get effected. I parse out the line with the winning artist name and when I get to the song with that artist's name I mark the song as a winning song. So I would think if the data had been wrong all along the Jukebox would have wound up with no winning song for the fight. Maybe the data was right initially then got broken? Weird.

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 1:06 pm
by Manhattan Glutton
I actually am kind of interested since I've got my own project of bringing Nur Ein into a SQL database, so I'll have to sign up for that someday...

Back to the topic at hand, perhaps the songfight.org page generation got confused?

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 3:42 pm
by Spud
Lunkhead wrote:Maybe the data was right initially then got broken?
You may be on to something there, Sam.

Re: "Our plans are on hold" - archive bug?

Posted: Mon Apr 29, 2013 3:56 pm
by Manhattan Glutton
Well, whatever happened, thanks for fixing it spud! I'm sure glennny's especially happy.