How to add 2 custom fields to New Thread?

[jaybolt]

I am wanting to add two custom fields to new threads:

latitude
longitude

I want these stored in the thread table in VB

What I have tried:

1. Modified newthread template to include:

<!-- GOOGLE MAP COORDINATES START HERE -->
<fieldset class="fieldset">
<legend>$vbphrase[map_coordinates]</legend>
<div style="padding:$stylevar[formspacer]px">
<div><label for="latitude"><input type="text" class="bginput" size="5" name="latitude" value="$latitude" id="latitude" tabindex="1" /> $vbphrase[latitude]</label></div>
<br />
<div><label for="longitude"><input type="text" class="bginput" size="5" name="longitude" value="$longitude" id="longitude" tabindex="1" /> $vbphrase[longitude]</label></div>
</fieldset>
<!-- GOOGLE MAP COORDINATES END HERE -->

2. Added two new fields to the vb3_thread table - latitude (INT) and longitude (INT)

3. Created hooks in:

threadfpdata_start

$this->validfields['latitude'] = array(TYPE_INT, REQ_NO);
$this->validfields['longitude'] = array(TYPE_INT, REQ_NO);

newpost_process

$this->validfields['latitude'] = array(TYPE_INT, REQ_NO);
$this->validfields['longitude'] = array(TYPE_INT, REQ_NO);

newthread_post_start

$vbulletin->input->clean_array_gpc('p', array('latitude' => TYPE_INT, 'longitude', TYPE_INT));
$newpost['lattitude'] =& $vbulletin->GPC['latitude'];
$newpost['longitude'] =& $vbulletin->GPC['longitude'];

Obviously this isn't working so can anyone help me please? I have searched the forums and that is where I gathered the above information on the hooks and I copied some of the code in the newthread template to try and create the code above.

Thanks very much

Ok, I'm getting there. There is a typo in newpost_process

it now reads:

if ($type == 'thread')
{
    $dataman->setr('thread_dec_lat', $post['thread_dec_lat']);
    $dataman->setr('thread_dec_lon', $post['thread_dec_lon']);
}

It now writes the latitude field to vb_thread but not longitude

I've change the variable names from 'latitude' to 'thread_dec_lat' and 'longitude' to 'thread_dec_long' by the way

Now to figure out why it is only writing one of the variables into the database ...