Pirep updates Arrival ICAO with Dep ICAO?

[Alex]

.930

My pilots are reporting that even though they file the Pirep correctly giving a correct Dept ICAO and Arr ICAo when it arrives at admin for approval the Arrival filed hs been populated with the Departure ICAO.

I haven't changed anything in the pirep tpl's.

It means that the Fleet table also is not being updated correctly with the last filed pirep position.

Any Clues or is this a BUG!

Thanks

Alex

[mark1million]

Yes i had a pilot report this as well, i just thought it was a one off and reset his destination from the admin pireps

[Alex]

Yes i had a pilot report this as well, i just thought it was a one off and reset his destination from the admin pireps

It started as just a single Pilot, Now it has migrated and is affecting two. Nasty Bug!! I have a PC Flu

[mark1million]

Im on 932 so if it happens again i will post back here, but its only been the 1 pilot so for and i have quite a few flights a day go through my va.

[Nabeel]

It's a bug in the template, still in 930? I thought I fixed it for 930..

If you look in the database, is the field changed there? Or is it correct there

[mark1million]

Im on 932 and nothing since, i have only had the one but will keep a look out.

[Nabeel]

Im on 932 and nothing since, i have only had the one but will keep a look out.

Can you check in the database and see what it says

[Alex]

Because i have edited it to be correct, i'll have to wait until it pops up again, then i'll check the database. Out of curiosity what was the bug?

[Nabeel]

I put depicao twice in the template by accident

[Alex]

Nabeel, which template? pirep_new?

[Alex]

This bug is getting worse it seems all pilots are now ariving at the same deprture point.

I have checked the tpl's that i think may be affecting this but don't see any double depicao entries.

[Nabeel]

Have you modified any templates?

And my question wasn't answered - are they the same when you look in the database? How are you submitting the PIREP?

[Alex]

i havent modified any templates other than using obsessblue. The database shows it as being in the correct place, but the fleet list shows departure icao as the current location.

The pirep is submitted manually

[Nabeel]

i havent modified any templates other than using obsessblue. The database shows it as being in the correct place, but the fleet list shows departure icao as the current location.

The pirep is submitted manually

Fleet list? Which template? Then instead of being depicao it should be arricao. Sounds like a typo in the mod

[Alex]

This is the Fleet List tpl. looking at it i dont think there is an error?

h1 align="center"><?php echo SITE_NAME?>Fleet</h1>
<!-- Fleet Table Version 2.0.- - fleet_table.tpl - By Mitchell W
http://forum.phpvms.net/topic/1522-fleet-table/
CHANGES FROM 1.1:
Added image
Added download
-->
<table border="1">
 <!--DWLayoutTable-->
<thead>  
<tr>
<td height="42"><b>Image</b></td>
<td><b>ICAO</b></td>
<td><b>Type</b></td>
<td><b>Registration</b></td>
<td><b>Range</b></td>
<td><b>Cruise Kts</b></td>
<td><b>Max Pax</b></td>
<td><b>Max Cargo</b></td>
<td><b>Total Hours</b></td>
<td width="79"><p><b>Current Location</b></p>  </td>
</tr>
</thead>
<tbody>
<?php foreach ($fleet as $aircraft) 
{
?>
<tr>
<td height="36"><img src="<?php echo $aircraft->imagelink; ?>"></td>
<td><?php echo $aircraft->icao; ?> </td>
<td><?php echo $aircraft->name; ?> </td>
<td><?php echo $aircraft->registration; ?></td>
<td><?php echo $aircraft->range; ?> </td>
<td><?php echo $aircraft->cruise; ?></td>
<td><?php echo $aircraft->maxpax; ?></td>
<td><?php echo $aircraft->maxcargo; ?></td>
<td><?php echo $aircraft->totaltime; ?></td>
<td valign="top"><?php echo $current_location.'-'.$current_location2; ?></td>
<?php
$params = (array('a.registration'=>$aircraft->registration, 'p.accepted'=>PIREP_ACCEPTED));

$pirep = PIREPData::findPIREPS($params);
$current_location = $pirep[0]->arricao;
$current_location2 = $pirep[0]->arrname;
?>
</tr>
<?php } ?>
</tbody>
</table>

[Nabeel]

This should be flipped:

<td valign="top"><?php echo $current_location.'-'.$current_location2; ?></td>
<?php
$params = (array('a.registration'=>$aircraft->registration, 'p.accepted'=>PIREP_ACCEPTED));

$pirep = PIREPData::findPIREPS($params);
$current_location = $pirep[0]->arricao;
$current_location2 = $pirep[0]->arrname;
?>

To

<?php
$params = (array('a.registration'=>$aircraft->registration, 'p.accepted'=>PIREP_ACCEPTED));

$pirep = PIREPData::findPIREPS($params);
$current_location = $pirep[0]->arricao;
$current_location2 = $pirep[0]->arrname;
?>
<td valign="top"><?php echo $current_location.'-'.$current_location2; ?></td>

Otherwise it's looking at the previous iteration

[Alex]

That did the trick, Thanks Nabeel for your help.