Maybe i am doing this all wrong then

. This is my first attempt at a product.
any help would be great.
PHP Code:
<templates>
<template name="ginfraction" templatetype="template" date="1209011189" username="virtus" version=".1"><![CDATA[
<div>
<form action="misc.php?do=ginfraction" method="post">
<b>Name of player: </b> <input type="text" name="playername" />
<br />
<b>Forum/Account Name: </b> <input type="text" name="forumname" />
<br />
<b>Time of Violation (GMT): </b> <input type="text" name="time" />
<br />
<b>Date of Violation: </b> <input type="text" name="date" />
<br />
<b>Action Taken: </b>
<input type="radio" name="infractiontype" value="warning" /> <b>Warning</b>
<br />
<input type="text" name="suspenddays" /> <b>Days</b> <input type="radio" name="infractiontype" value="temporary suspension" /> <b>Temp Suspension</b>
<br />
<input type="radio" name="infractiontype" value="permanent suspension" /> <b>Perma Ban</b>
<br />
<b>Violation: </b> <input type="text" name="violation" />
<br />
<b>Evidence: </b> <input type="text" name="evidence" />
<br />
<b>Which ToS did they Violate: </b>
<select name="tosrules">
<option value="choose">Choose an Tos Violation</option>
<option value="5.1">5.1</option>
<option value="5.2">5.2</option>
<option value="5.3">5.3</option>
<option value="5.4">5.4</option>
<option value="5.5">5.5</option>
<option value="5.6">5.6</option>
<option value="5.7">5.7</option>
<option value="5.8">5.8</option>
<option value="5.9">5.9</option>
<option value="5.10">5.10</option>
<option value="5.11">5.11</option>
<option value="5.12">5.12</option>
<option value="5.13">5.13</option>
<option value="5.14">5.14</option>
<option value="5.15">5.15</option>
<option value="5.16">5.16</option>
<option value="5.17">5.17</option>
<option value="5.18">5.18</option>
</select>
<input type="submit" name="submit" value="Submit" />
</form>
</div>
]]>
</template>
</templates>
this is only a small but of the php code:
PHP Code:
<phpcode><![CDATA[
if ($_REQUEST['do'] == "ginfraction")
{
if($vbulletin->userinfo['userid'] != 6){
print_no_permission();
}
if(isset($_POST['submit'])){
if(!isset($_POST['infractiontype']) || !isset($_POST['suspenddays']) || !isset($_POST['playername']) || !isset($_POST['forumname']) || !isset($_POST['time'])
|| !isset($_POST['date']) || !isset($_POST['violation']) || !isset($_POST['evidence']) || $_POST['tos'] == "choose"){
$error = "Please fill out all fields";
}
}]]></phpcode>
</plugin>
</plugins>