Closed (fixed)
Project:
Tipping
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 May 2008 at 15:17 UTC
Updated:
18 Jun 2008 at 09:42 UTC
Jump to comment: Most recent file
I just tried the HEAD version for drupal 5.x and fixed some/most outstanding issues. -> Patch attached.
I would like to use it for the euro2008, but I could need some help get it adopted.
We need to:
1. edit the number of groups to 4 and eliminate the round of 16
2. edit the data to fit for the EM
If I could get some help for 2., I'd look at 1.
| Comment | File | Size | Author |
|---|---|---|---|
| #21 | tippspielscreenshot.jpg | 85.88 KB | gs12345 |
| #15 | tipping.tar_.gz | 14.65 KB | fago |
| #15 | tipping_neu.patch | 49.71 KB | fago |
| #14 | tipping.zip | 16.17 KB | Tim99 |
| #10 | tipping_3.tar_.gz | 14.98 KB | fago |
Comments
Comment #1
EgonO commentedthe patch seems to work - but the module needs some more coding - i try to get it work for EM only until EM starts - but after it works for EM we should try to make it more modular so users can include rules for the country-specific leaques, EM, WM, Champions Leaque and such things...
Comment #2
fagook, here is an updated patch:
* added uninstall hook
* fixed the own date form element
* changed number of groups to fit for the EM
TODO:
* renaming groups. best edit tipping.install and insert the group names and their abbreviations. group A is number 1, B is 2 and so on..
* probably the easiest way to get this done, is to edit the matches via the admin UI - which should work. You can fix the match-Ids only by using phpmyadmin or something similar. Then make a dump of the tipping_matches table and append it to this issue - then I can try to integrate it into the installation procedure.
Comment #3
fagofor people who don't know how to patch but want to help: here is the (so far) fully patched module..
(rename to .tar.gz)
Comment #4
fagook, I have done some more.
* entered the team names
* fixed the matches in the db, so that they are editable in the admin interface
* added the possibility to specify the match id in the UI
-> I have entered about 15 matches, but I could definitely need help!
Here you can find the matches:
http://www.euro2008.uefa.com/tournament/matches/index.html
To get the latest db changes (matches!) you need to reinstall the module. If you have input some matches, export the table "tipping_matches" via phpmyadmin or so and attach it to this issue - thanks!
Comment #5
andiwelt commentedHello fago,
I entered the last 16 matches and exported the table `tripping_matches` with phpmyadmin. The 3rd place match should be removed. There is no 3rd place match for the Euro 2008.
Comment #6
Tim99 commentedI installed tipping_2.tar.gz (Drupal 5) and manually created a role "tipping-admin" (I am the only member). I activated the tipping block (which I can see). Unfortunately I can't see anything else. No configuration, nothing. How do I access them?
Comment #7
Tim99 commentedI made a german translation file. You have to create a folder po inside the module directory and put the de.po into that folder. Perhaps you have to reinstall the module.
Could somebody please integrate the updated schedule into the module and the translation file as well so that there is a fully working module without a game for 3rd place?
Comment #8
Tim99 commentedHere are the flags of all teams. The names correspond to their default abbreviations (GER.gif, SWE.gif, RUS.gif,...). Perhaps those flags could be integrated?
I have some ideas that could be integrated into the module:
- Goal difference:
Right now it is: Correct result --> 3 points, correct winner/draw: 1 point. What about correct difference --> 2 points? Example:
Result: 2:0
User 1 said: 2:0 --> 3 points
User 2 said: 5:0 --> 1 point
User 3 said: 3:1 --> 2 points (correct difference)
This is not applicable for draws, because it won't make sense in that case.
I believe this is can easily be done by adjusting the function tipping_update_scores($matches) (line 1167).
- Bonus questions like:
-- Favourite team (Player get's 1 point for a draw by "his" team and 3 points for a win.)
-- Group winner(s) (2 points for each correct tip)
-- Player with most goals (1 extra point for each goal)
Unfortunately I'm not very experienced with Drupal coding. But if you give me clear instructions which function I have to change and what shall be the result, then I'm doing my best to contribute to the module.
Comment #9
Tim99 commentedI found an error:
tipping.module Line 759
Wrong:
if module_exist...
Has to be changed into:
if module_exists
Comment #10
fagook, I've put that all together and updated the patch/module.
I've fixed the above mentioned mistake and also the permissions (untested but it should work) and removed the game for the 3rd place.
Now we should have all together - so let's make sure everything is working as desired and test betting, scores, ..
@flags: Yep, that would be cool. Are this flags gpl? If not, we can't include it, but we can make it possible that one just needs to drop in the images and when they are there they are used.
@bonus: hm, I don't think that this is desired by most of the people. So we need to make this configurable or just stay with the common behaviour. However you can adopt it for yourself, of course.
edit: hm, the .po file is missing in the patch.
Comment #11
Tim99 commentedDoes anyone know how to become a Developer of that module? So that we can have SVN/CVS-Access? That whole attachment/patch think isn't very comfortable ;-) ...and nobody except us will notice the progress of the module.
Comment #12
fagotry to contact sime and/or read http://drupal.org/node/251466
Comment #13
Tim99 commentedI wrote a mail to sime. I'd like to propose that we use different 'issues' for different features. I wrote one for my wish regarding the 2-point-rule. The issues can be seen here:
http://drupal.org/project/issues/tipping
Currently I'm improving the translation, because not all strings have been translatable. Some are hard-coded. I'm working on that and will release it soon.
Comment #14
Tim99 commentedAttached you will find an updated version of the module:
Changes:
- all hard-coded strings have been made translatable
- de.po contains a complete German translation
- Tipping block has been slightly re-formatted
Now I think it would be nice to integrate the flags.
Comment #15
fagothanks, next round of changes:
* improved strings (and translation) to talk about the european championchips not the world cup, as the shipped matches are for the EM
* improved the css, so that the tipping page looks better in garland. should also work fine in other themes.
* also changed the block slightly
so, far I think the module is now usable for the EM.
Comment #16
Tim99 commentedDid someone perform a test? Just to be sure that there are no surprises on Saturday...
Comment #17
simeHi all.
Seems the best thing is to give fago commit access, as I know his work well, he already has CVS access, and he appears well and truly involved. (fago, bit of an assumption, but if you have any concerns please let me know, and if you want me to add any other specific maintainers I'm more than happy to take your recommendations.)
I'm really happy to see this module being used as a platform for more tipping! Please feel free to get it working for your needs. I'm pretty tied up with my business so I've been having trouble being a good maintainer. If there is anything I can do to clear the way for more commits, just let me know.
Cheers
.s
Comment #18
gs12345 commentedIf you're not registered/logged in, the first page gives this as a login link
http://test.mysite.de/%3Cem%3E/user/login?destination=tipping%3C/em%3E
it puts em tags before and after the link... it's in the module.tipping file, but I wasn't able to correct it so far. Changing the de.po file to a hardcoded /user/register and user/login wasn't the solution either because the redirect after the login is missing. help?
Comment #19
gs12345 commentedSomething else, which is much more problematic: I've reset the times to betatest the script. Now, what I can see is that for a game that has kick-off at 09:03 MET can be tipped for up to now (it's 09:17). 20 minutes ago, changed the script from 1h to 5min before kick-off, but there's no reaction from the script. It's unusable this way, I'm afraid.
Comment #20
gs12345 commentedAlso, in IE 6 and the default Garland theme the EM block set to appear on the right side isn't displayed properly, but beneath the center block.
Comment #21
gs12345 commentedhere's a screenshot of the EM block in IE 6
Comment #22
summit commentedSubscribing, greetings, Martijn
Comment #23
fagothanks a lot, sime.
I've just committed this and created a 5.x-1.x branch, suited for the euro. So tonight there should be a generated dev release.
Please open up separate issues for reporting bugs and file further patches against this branch.
Unfortunately I won't have time to actively maintain yet another module, so I've noted on the project page that this is unsupported.
Comment #24
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.