How about a pick'em league option with confidence points. Straight up, pick the winner of the game, but with confidence points for the players to spread around.

Comments

jvandervort’s picture

That would be fun.

jvandervort’s picture

Version: 6.x-2.x-dev » 7.x-1.x-dev

Moving new features to the latest dev version.

kiel2155’s picture

StatusFileSize
new15.86 KB

Hey all,

First, thanks to @jvandervort, @tlreiners and everyone else that has contributed to this project.

I've attached a patch that should add this confidence points functionality. Apply it to the latest dev version (Wed, 15 Aug 2012 04:19:24 +0000 (23:19 -0500)).

Let me know if it works or not. This is my first contribution to an open source project :)

jvandervort’s picture

@kiel2155,
Can you rebase your patch on the latest 7.x-1.x head? The patch no longer applies.

kiel2155’s picture

Yep, give me a day or two and I'll repost.

jvandervort’s picture

@kiel2155,
Great, I think I'll start a feature branch for it: 7.x-2.x

kiel2155’s picture

StatusFileSize
new17.7 KB

Try this one.

jvandervort’s picture

Status: Active » Fixed

Committed to new 7.x-2.x branch. Need to add checking for duplicate confidence points.

kiel2155’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev

The check to make sure confidence values are not out of range (ie. less than 1 or more than the number of games that week) and the values are not duplicated is implemented in pickem.mypicks.page.inc in function theme_mypicks_week_form_submit around line 170. If you submit an invalid value, the page stores a NULL and displays an error.

jvandervort’s picture

Yep, the hole I saw was for duplicate points (ie 1, 2, 2, 3, etc..).
I added a check for it and moved it to the _validate function so it highlights the boxes in red. Thanks a ton for the help. Definitely check out the new dev release on the 7.x-2.x branch. After we get some more eyes on the 2.x branch I hope to merge it back into the mainline.

kiel2155’s picture

Status: Fixed » Needs review
StatusFileSize
new1.76 KB

I noticed that I didn't update the admin picks page. This patch should add the confidence box in and make sure it can be edited.

jvandervort’s picture

Status: Needs review » Fixed

Committed to dev.

Status: Fixed » Closed (fixed)

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