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.

Comments

EgonO’s picture

Status: Active » Reviewed & tested by the community

the 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...

fago’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new15.73 KB

ok, 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.

fago’s picture

StatusFileSize
new12.35 KB

for people who don't know how to patch but want to help: here is the (so far) fully patched module..

(rename to .tar.gz)

fago’s picture

StatusFileSize
new28.35 KB
new12.05 KB

ok, 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!

andiwelt’s picture

StatusFileSize
new975 bytes

Hello 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.

Tim99’s picture

I 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?

Tim99’s picture

StatusFileSize
new8.72 KB

I 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?

Tim99’s picture

StatusFileSize
new4.11 KB

Here 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.

Tim99’s picture

I found an error:

tipping.module Line 759

Wrong:
if module_exist...

Has to be changed into:
if module_exists

fago’s picture

StatusFileSize
new31.1 KB
new14.98 KB

ok, 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.

Tim99’s picture

Does 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.

fago’s picture

try to contact sime and/or read http://drupal.org/node/251466

Tim99’s picture

I 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.

Tim99’s picture

StatusFileSize
new16.17 KB

Attached 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.

fago’s picture

StatusFileSize
new49.71 KB
new14.65 KB

thanks, 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.

Tim99’s picture

Did someone perform a test? Just to be sure that there are no surprises on Saturday...

sime’s picture

Hi 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

gs12345’s picture

If 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?

gs12345’s picture

Category: task » bug
Priority: Normal » Critical

Something 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.

gs12345’s picture

Priority: Critical » Normal

Also, 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.

gs12345’s picture

StatusFileSize
new85.88 KB

here's a screenshot of the EM block in IE 6

summit’s picture

Subscribing, greetings, Martijn

fago’s picture

Version: » 5.x-1.x-dev
Category: bug » task
Status: Needs review » Fixed

thanks 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.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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