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.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | pickem-1139520-11-confidence_box_admin_page.patch | 1.76 KB | kiel2155 |
| #7 | confidence.patch | 17.7 KB | kiel2155 |
| #3 | confidence.patch | 15.86 KB | kiel2155 |
Comments
Comment #1
jvandervort commentedThat would be fun.
Comment #2
jvandervort commentedMoving new features to the latest dev version.
Comment #3
kiel2155 commentedHey 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 :)
Comment #4
jvandervort commented@kiel2155,
Can you rebase your patch on the latest 7.x-1.x head? The patch no longer applies.
Comment #5
kiel2155 commentedYep, give me a day or two and I'll repost.
Comment #6
jvandervort commented@kiel2155,
Great, I think I'll start a feature branch for it: 7.x-2.x
Comment #7
kiel2155 commentedTry this one.
Comment #8
jvandervort commentedCommitted to new 7.x-2.x branch. Need to add checking for duplicate confidence points.
Comment #9
kiel2155 commentedThe 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.
Comment #10
jvandervort commentedYep, 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.
Comment #11
kiel2155 commentedI 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.
Comment #12
jvandervort commentedCommitted to dev.