New smackdown form shows up instead of voting page

colan - December 4, 2007 - 19:18
Project:Smackdown
Version:5.x-2.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:colan
Status:closed
Description

Voting pages don't show up with the current logic, given that the output of array_rand() is used improperly. See http://ca.php.net/manual/en/function.array-rand.php for details.

If you are picking only one entry, array_rand() returns the key for a random entry.

Here's a patch. Please test!

AttachmentSize
smackdown.patch476 bytes

#1

colan - December 4, 2007 - 19:22
Title:Improper use of array_rand()» New smackdown form shows up instead of voting page

#2

sirkitree - December 6, 2007 - 20:09
Status:needs review» fixed

Tested. Thanks!

#3

Anonymous - December 20, 2007 - 20:12
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.