[Crapaddict]

Excellent mod, our users love it and are posting frenzy just for the cash

Installed .91 few days ago, and have a few comments;

1. (minor)
   After importing the xml, i was searching all over the admincp for the games section.
   It took me a while before i found out i had to refresh the left-frame of the admincp.
   
   
2. (minor)
   I needed to set the sports betting groups to be able to set some games to bet on.
   I first was under the assumption that the admingroup should have the neccessary rights by default. Maybe this could be documented more clearly?
   
   
3. (minor)
   If you bet - for instance - $20 in Blackjack, you have to move your mousecursor to click twice on the same chip because the phrase "place your bet" dissapears, moving all chips aside.
   rather than dissapearing the phrase, just hiding it would be an idea?
   
   
4. I get reports from blackjack taking away money and simply freezing now and then.
   there are some javascript errors reported. (have a look at the attached image.)
   Used browser is IE7
   
   
5. I inserted all matches for the european soccer championship into the sports betting,
   realizing it became a lot of scrolling afterwards. (24 matches times three possible outcomes: team 1 or 2 wins, and a tie)
   I came up with the idea to make a "Future events" group, and by default only fetch events from the database which occur within 21 days from now.
   My simple solution: (sportspool.php)
   PHP Code:

   (line 339 and up:)
    if ($recentevents == 1)
    {
        $time = time();        
        $time2 = time() - (14 * 24 * 60 * 60);        
        $query = "SELECT * FROM casino_sb_events WHERE groupid = $groupid AND eventdate > $time2 AND eventdate < $time ";
    }
    else if ($recentevents == 2)
    {
        $time = time();        
        $query = "SELECT * FROM casino_sb_events WHERE groupid = $groupid AND eventdate < $time";
    }
    else if ($recentevents == 3)
    {
        $time = time();        
        $query = "SELECT * FROM casino_sb_events WHERE groupid = $groupid AND result IS NULL";
    }
    else if ($recentevents == 4)     // Add a fourth event to fetch all events.
    {
        $time = time();        
        $query = "SELECT * FROM casino_sb_events WHERE groupid = $groupid AND result IS NULL";
    }
    else 
    {
        $time = time();    
        // Add a variable to set the treshold for upcoming events
        $time3 = time() + (21 * 24 * 60 * 60);    
        // Alter the query to not fetch events occuring after $time3
        $query = "SELECT * FROM casino_sb_events WHERE groupid = $groupid AND (eventdate > $time AND eventdate < $time3) AND result IS NULL";
    } 

   (Added a recentevents #4, and changed the default query...)
   Maybe you could add something similar in future revisions?
   
   One could bet on future games if one knows the url, but it is less crowdy in the frontend.
   offcourse you can add the link in the casino_sportspool_main template if condition=$canedit so it is easy accessible for usergroups with the appropiate rights.
   
   (just an idea )