LaMonte Forthun

The solution is to modify the way the ‘sport’ sorts teams. The ‘Sports’ are templates that are designed to be user modifiable. If you want to sort on name, you’d just change the function.

Here’s the function in the ‘baseball.php’ file:

/**
 * rank Teams
 *
 * @param array $teams
 * @return array of teams
 */
function rankTeams( $teams )
{
	foreach ( $teams AS $key => $row ) {
		$won[$key] = $row->won_matches;
		$lost[$key] = $row->lost_matches;
	}
	array_multisort( $won, SORT_DESC, $lost, SORT_ASC, $teams );
	return $teams;
}

Here’s what I did with ‘basketball.php’:

/**
 * rank Teams
 *
 * @param array $teams
 * @return array of teams
 */
function rankTeams( $teams )
{
	foreach ( $teams AS $key => $row ) {
		$points[$key] = $row->points['plus']+$row->add_points;
		$done[$key] = $row->done_matches;
		$diff[$key] = $row->diff;
		$GA[$key] = round((($row->points2_plus) > 0 ? (($row->points2_minus) > 0 ? ((($row->points2_plus)/($row->points2_minus))*1000) : 1000) : 0),0);
		$WinPerc[$key] =  round((($row->won_matches) > 0 ? (($row->done_matches) > 0 ? ((($row->won_matches)/($row->done_matches))*100) : 100) : 0));
	}
	array_multisort( $points, SORT_DESC, SORT_NUMERIC, $WinPerc, SORT_DESC, SORT_NUMERIC, $GA, SORT_DESC, SORT_NUMERIC, $teams );
	return $teams;
}

Going in and changing team numbers and such in the database will kill the entire set up as those are the numbers everything else is based off…

Are you using ‘baseball’ as your sport or something else?

The way to add things like home/away to standings is to create a template that adds the column. Go into the template folder, grab the ‘standings-extended.php’ file, copy it, rename it something like ‘standings-HA.php’ and then you can modify that to get what you want.

When you want to call it from the shortcode you’d use

[standings league_id=1 template=HA]

Any chance of adding it to the shortcode? Even if I add something to the shortcode it still has to be added to a template, those are user customizable. If you built a template you wouldn’t need it in the shortcode, it would just be there.

Yeah, there’s something going on because that’s not normal and I’ve never seen that in any version of LeagueManager I’ve used.

Just to be clear, what version do you have installed?

I’m going to look at a few things this morning, then I’m completely out of touch for about 36 hours, but will get back on it Sunday afternoon. If I find a solution this morning, I’ll post something.

I posted this a couple of months ago about this issue:

**************************
This problem is based on a PHP update (introduced in 5.3.9) that introduced a setting to help prevent hash collision attacks. The setting is “max_input_vars” and is defaulted to be set at 1,000.

The problem with updating an entire season, or even a few weeks worth, of matches is there are a lot of variables per game and those variables add up quickly when you’re limited to 1,000.

The best solution would be to re-write the code to work on a few lines at a time instead of grabbing all the code at once and then updating game by game, and I’m looking at it to see if I can do that (keeping in mind that League Manager isn’t my plug in).

In the mean time, there are two solutions:

1). Make sure you have match days defined in the season (how many matches per season), then make sure when you add a match you have a match day assigned to the match. When you go to update matches, filter by match day so you are only working with game 12 or whatever, so that the number of updates and variables used stays under the limit. If you’ve got 36 teams in a league and you’re trying to update 18 games at a time, you might run into the problem anyway and you’ll need to see solution 2. There are a lot of reasons that this is the best option, but the best reason is that it doesn’t involved anything like code changing or server tweaking…

2). This isn’t the ideal solution because it’s bypassing a setting that’s been added to PHP to avoid a vulnerability. The solution is to increase the number of variables that can be set at one time from 1,000 to some number higher. If you have access to your server’s php.ini file and know how to modify files on a server, you can add or modify the max_input_vars setting like this:

max_input_vars = 2,500

You can also change this setting in an .htaccess file in your local word press directory like this (they are different, if you put the above code in an htaccess file you’ll get a server error):

php_value max_input_vars 2500

I’ve tested this and it does work. If you find that the number isn’t high enough, add another 500 or 1,000, but I wouldn’t recommend throwing 10,000 in right away as the purpose of the restriction was to close a PHP vunerability.

If I can figure out how to update the code to work with a group of matches at a time so that the problem goes away, I’ll update the code here.
*****************************

Changing the basketball.php file will only help those that are using basketball as their sport, otherwise that file is never called. Personally, I do as that’s what my site is, basketball, but it won’t help anyone with a soccer site, etc.

Deleting the ‘!isset($_POST[‘doaction2′])’ code isn’t a solution or part of the problem. That block is part of an if-then-else that has multiple options.

Thanks for the heads up on the ‘clas’ typos, I haven’t had time to go through all the files for errors like that, so please do let me know of anything you (or anyone else) find…

@intellicents, do you have a specific need for the calendar? Did you look at the link @ben52 provided to see if that’s what you need as well?

I can’t and don’t take any credit for the plugin, but hope we can all continue to make it better and better. Any and all ideas are welcome. No promises, other than to consider the idea and go from there!

That’s a template that would have to be built, but yes, could be done…

What error are you getting?