Pilot Financial Report V1.0

[Parkho]

Here is another small add-on for those who want to show pilot's financial reports. It's based on pilot id, so the results are for each of you pilots.Cheers

Install

1. Download the zip from repo.

2. Unzip and upload in the order of folders.

3. Access financial report for a pilot by

<?php echo url('/pfinance') ;?>

-----------------OR----------------------

Access financial report for all pilots by

<?php echo url('/pfinance/pilots') ;?>

4. Enjoy.

Download:PilotFinancialReport_V1.0

Screenshot for a pilot report:

Screenshot for all pilots report:

Change Log:

1. PilotFinance.class.php & pilots_finance.tpl have been added. Those of you who have downloaded this before should re-download.

[mattia]

hi parkho

i have this error:

Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 12

€ 0.00

• 
  
• Your total flight fuel expenses in Italiani Volanti is:
  Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 18
  € 0.00
  
• Your total in flight expenses in Italiani Volanti is:
  Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 24
  € 0.00
  
• Your total flight revenue in Italiani Volanti is:
  Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 31
  

[servetas]

Screenshot?

[mattia]

attach photos

[servetas]

I was talking about the general screen shot of the module in order to preview it...

[Fernando]

Great Work, Parkho

I made some adjustments

[Fernando]

Mattia

you touched on some of the files line by disponibilziados Parkho?

[Parkho]

hi parkho

i have this error:

Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 12

€ 0.00

• 
  
• Your total flight fuel expenses in Italiani Volanti is:
  Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 18
  € 0.00
  
• Your total in flight expenses in Italiani Volanti is:
  Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 24
  € 0.00
  
• Your total flight revenue in Italiani Volanti is:
  Warning: mysql_fetch_assoc(): supplied argument is not a valid MySQL result resource in /var/www/virtual/italianivolanti.it/htdocs/iv/core/templates/pilot_finance.tpl on line 31
  

Are you using any shared server for your site? Cause I think that's where the problem lies not sure though. It's working just fine on this side.

[Fernando]

Parkho,

seems to be a permission problem, what do you think?

[Parkho]

I was talking about the general screen shot of the module in order to preview it...

[mattia]

i copy the file in my site but dont work

[Parkho]

i copy the file in my site but dont work

Strange! did you copy the .tpl in templates or your skin folder?

[mattia]

yes,

pilot_finance.tpl in templates folder and Pfinance folder in modules folder

[SparrowBD]

im getting that mysql error as well. one thing you should be able to do maybe in a future update is allow admins to view all the pilots profitability.

[Fly Star Alliance]

Same here, MySql error

[Parkho]

I think it's fixed

[Fly Star Alliance]

Better now, thanks Parkho, although I still have a couple of little problems to solve....

[Parkho]

I edited the .tpl as well, have you replaced that too?

[CPC900]

Don't forget, if you named your database to have something OTHER than "phpvms_pireps" you will get the mysql errors. Change those lines in your "PilotFinance.class.php" to indicate your database ie "nameofdata_pireps"; whatever yours is.

[Parkho]

Good point. Thanks

[Stealthbird97]

Well Its pointless saying that you've fixed it when you don't tell us what we need to change to fix the sql problem.

[Parkho]

Okay. One thing is to check your phpvms table prefix in your DB. If your DB prefix for tables is not phpvms then that's where the problem is.

[mattia]

now work for me thanks

[SparrowBD]

i would like to point out the reason some may get a sql_assoc() error is due to your pirep tables not being EXACTLY phpvms_pireps. if it is not exactly that then it will throw some errors at you. so double check your table names.

[Guest lorathon]

Just glancing through this.

The queries should be using the TABLE_PREFIX variable that is part of phpVMS. Then all of the MySQL DB errors regarding not finding tables would disappear. Also the queries should be using the API of phpVMS not mysql_fetch_assoc()

IMHO

Data should be returned not echoed - allows you to use it in different ways not just displaying it.

Plus the Auth::$pilotid should be in the module not the class. This allows you to use the same function to get data on another pilot. They way it is written it will ONLY return data on the pilot displaying the page.

I personally would format the data on the template not inside of the class. (FinanceData::FormatMoney)

One other item to note. The query will sum the totals of all pireps. Including rejected ones. I would include a filter in the where for only the accepted pireps.

Example

public function fuelsum($pilotid) {
$qry = 'SELECT SUM(fuelprice) AS total FROM '.TABLE_PREFIX.'pireps WHERE pilotid='.$pilotid.' AND accepted=1';
$row = DB::get_row($sql);
return $row->total;
}

[Fly Star Alliance]

God I wish I had Jeff's programming knowledge.... When something doesn't work I'm simply lost without help.

Anyway, table wise, mines are default phpvms and therefore not the problem.

Hope to get some more suggestions......

[CPC900]

How would I show ALL reports for ALL pilots with this code edited for that??

<?php
$pilotid = Auth::$userinfo->pilotid;
$reps = PIREPData::getLastReports($pilotid, 5);
?>

<table class="balancesheet" width="50%">
<tr class="balancesheet_header">
<td colspan="5">Last 5 Flights</td>
</tr>
<tr>
<td align="center"><strong>Flight #</strong></td>
<td align="center"><strong>Duration</strong></td>
<td align="center"><strong>Pilot Pay</strong></td>
<td align="center"><strong>Revenue</strong></td>
<td align="center"><strong>Status</strong></td>
</tr>
<?php
if(!$reps)
{
?>
<tr><td align="center" colspan="4">No Flights</td></tr>
<?php
}
else
{
$i = 1;
foreach($reps as $rep)
{

?>

[CPC900]

BUMP

[Kalo]

I would love to see CPC900's suggestion also.

Is there a way to show every pilots stats along with their earnings? When you click "Pilots" it only shows how many flights and how many hours...it would be great to beable to categorize the list by people with the most hours. I would also like that page to show pilot finances. Look at my page for an example: http://www.msflights.net/pilots/phpvms/index.php/pilots

[CPC900]

I wanted code so I could see what flights were losing money for all pilots, not just my own flights.