Custom Ban Lengths

[Micolithe]

What I'm trying to do is add additional ban lengths that are less than 1 day. (one hour, six hours, and twelve hours, specifically).

I found in modcp/banning.php where I can add options to the dropdown menu but these options don't seem to do anything, as they just create a ban that is listed as -0 days.

What I want to know is how to define the values H_1, H_6, and H_12 as hours to ban for. I can't seem to find where D_1, etc, are defined otherwise I could just use that as a reference.

If someone can point me in the right direction that would be rad.

[kh99]

If you look in banning.php around line 261, there's this code:

	// check that the number of days is valid
	if ($vbulletin->GPC['period'] != 'PERMANENT' AND !preg_match('#^(D|M|Y)_[1-9][0-9]?$#', $vbulletin->GPC['period']))
	{
		print_stop_message('invalid_ban_period_specified');
	}

you'll have to add h to the (D|M|Y) part to allow it as valid. Also, because it uses the function convert_date_to_timestamp() (in includes/functions_banning.php) to calcualte the lift date from the "period", it looks like you'll have to use lower case h instead of H.

Another issue may be that because the bans are lifted by a scheduled task, you'll also have to make sure that that task runs more often than once a day. ETA: ... turns out it does run once per hour by default.

[Micolithe]

I forgot to mention that I added capital H to the exclusion list in that code but thank you so much, using lowercase H made it work.

Also, I upped the frequency of unban users to 15 minutes.

So the code I added, aside from what was posted in the previous reply, in case anyone happens across this later on, is this:

The ones with stars at the start (which you should obviously exclude) are the ones I added.

file: modcp/banning.php

	// make a list of banning period options
	$periodoptions = array(
		$temporary_phrase => array(
*		    'h_1' => "1 $vbphrase[hour]",
*			'h_6' => "6 $vbphrase[hours]",
*			'h_12' => "12 $vbphrase[hours]",
			'D_1'  => "1 $vbphrase[day]",
			'D_2'  => "2 $vbphrase[days]",
			'D_3'  => "3 $vbphrase[days]",
			'D_4'  => "4 $vbphrase[days]",

[jkotlowski]

Originally Posted by kh99

If you look in banning.php around line 261, there's this code:

	// check that the number of days is valid
	if ($vbulletin->GPC['period'] != 'PERMANENT' AND !preg_match('#^(D|M|Y)_[1-9][0-9]?$#', $vbulletin->GPC['period']))
	{
		print_stop_message('invalid_ban_period_specified');
	}

you'll have to add h to the (D|M|Y) part to allow it as valid. Also, because it uses the function convert_date_to_timestamp() (in includes/functions_banning.php) to calcualte the lift date from the "period", it looks like you'll have to use lower case h instead of H.

Another issue may be that because the bans are lifted by a scheduled task, you'll also have to make sure that that task runs more often than once a day. ETA: ... turns out it does run once per hour by default.

I wanted to bump this and say thanks!

You should consider adding this to the template mods for vBulletin 5. I've been trying to figure this out for hours now and happened to stumble upon your post.