kenny Posted April 15, 2015 Report Share Posted April 15, 2015 Hi man, i would like to have a pilot roser without Hub separations. I would like a single list. I have insert this code in Pilots.php, but i have this error in roster page: Warning: Invalid argument supplied for foreach() in /home/hfrjgrdn/sites/phpvms/core/templates/pilots_list.php on line 33. Can you help me? Quote Link to comment Share on other sites More sharing options...
Members Vangelis Posted April 16, 2015 Members Report Share Posted April 16, 2015 Replace public function index() { // Get all of our hubs, and list pilots by hub $allhubs = OperationsData::GetAllHubs(); if(!$allhubs) $allhubs = array(); foreach($allhubs as $hub) { $this->set('title', $hub->name); $this->set('icao', $hub->icao); $this->set('allpilots', PilotData::findPilots(array('p.hub'=>$hub->icao))); $this->render('pilots_list.tpl'); } $nohub = PilotData::findPilots(array('p.hub'=>'')); if(!$nohub) { return; } $this->set('title', 'No Hub'); $this->set('icao', ''); $this->set('allpilots', $nohub); $this->render('pilots_list.tpl'); } With public function index() { $this->set('allpilots', PilotData::getAllpilots()); $this->render('pilots_list.tpl'); } In /core/modules/Pilots/Pilots.php Quote Link to comment Share on other sites More sharing options...
kenny Posted April 16, 2015 Author Report Share Posted April 16, 2015 Thanks for your reply! Now i see "There are no pilots". Quote Link to comment Share on other sites More sharing options...
kenny Posted April 16, 2015 Author Report Share Posted April 16, 2015 I solved . Thanks sir! Quote Link to comment Share on other sites More sharing options...
flyalaska Posted September 10, 2016 Report Share Posted September 10, 2016 I solved . Thanks sir! I ran into the same issue. How did you fix it? Quote Link to comment Share on other sites More sharing options...
web541 Posted September 10, 2016 Report Share Posted September 10, 2016 Not sure which one he has used, but I have this one working on a 5.5.x version public function index() { $this->set('pilot_list', PilotData::getAllPilots()); $this->render('pilots_list.tpl'); } I believe the code is for the old .tpl version which still had the $allpilots variable whereas the 5.5.x version has the newer $pilot_list variable Quote Link to comment Share on other sites More sharing options...
flyalaska Posted September 11, 2016 Report Share Posted September 11, 2016 Not sure which one he has used, but I have this one working on a 5.5.x version public function index() { $this->set('pilot_list', PilotData::getAllPilots()); $this->render('pilots_list.tpl'); } I believe the code is for the old .tpl version which still had the $allpilots variable whereas the 5.5.x version has the newer $pilot_list variable Can you post your whole Pilots.php? I get an error on my index. Parse error: syntax error, unexpected '$nohub' (T_VARIABLE), expecting function (T_FUNCTION) Line 32 Quote Link to comment Share on other sites More sharing options...
web541 Posted September 11, 2016 Report Share Posted September 11, 2016 Working on my end with the default template file. Pilots.php <?php /** * phpVMS - Virtual Airline Administration Software * Copyright (c) 2008 Nabeel Shahzad * For more information, visit www.phpvms.net * Forums: http://www.phpvms.net/forum * Documentation: http://www.phpvms.net/docs * * phpVMS is licenced under the following license: * Creative Commons Attribution Non-commercial Share Alike (by-nc-sa) * View license.txt in the root, or visit http://creativecommons.org/licenses/by-nc-sa/3.0/ * * @author Nabeel Shahzad * @copyright Copyright (c) 2008, Nabeel Shahzad * @link http://www.phpvms.net * @license http://creativecommons.org/licenses/by-nc-sa/3.0/ */ class Pilots extends CodonModule { public $title = 'Pilots'; /** * Pilots::index() * * @return */ public function index() { // $this->set('allpilots', $pilot_list); # deprecated $this->set('pilot_list', PilotData::getAllPilots()); $this->render('pilots_list.tpl'); } /** * Pilots::reports() * * @param string $pilotid * @return */ public function reports($pilotid='') { if($pilotid == '') { $this->set('message', 'No pilot specified!'); $this->render('core_error.tpl'); return; } $pirep_list = PIREPData::GetAllReportsForPilot($pilotid); $this->set('pireps', $pirep_list); # deprecated $this->set('pireps_list', $pirep_list); $this->render('pireps_viewall.tpl'); } /* Stats stuff for charts */ /** * Pilots::statsdaysdata() * * @param mixed $pilotid * @return */ public function statsdaysdata($pilotid) { $data = PIREPData::getIntervalDataByDays(array('p.pilotid'=>$pilotid), 30); $this->create_line_graph('Past 30 days PIREPs', $data); } /** * Pilots::statsmonthsdata() * * @param mixed $pilotid * @return */ public function statsmonthsdata($pilotid) { $data = PIREPData::getIntervalDataByMonth(array('p.pilotid'=>$pilotid), 3); $this->create_line_graph('Monthly Flight Stats', $data); } /** * Pilots::statsaircraftdata() * * @param mixed $pilotid * @return */ public function statsaircraftdata($pilotid) { $data = StatsData::PilotAircraftFlownCounts($pilotid); if(!$data) $data = array(); include CORE_LIB_PATH.'/php-ofc-library/open-flash-chart.php'; $d = array(); foreach($data as $ac) { OFCharts::add_data_set($ac->aircraft, floatval($ac->hours)); } echo OFCharts::create_pie_graph('Aircraft Flown'); } /** * Pilots::create_line_graph() * * @param mixed $title * @param mixed $data * @return */ protected function create_line_graph($title, $data) { if(!$data) { $data = array(); } $bar_values = array(); $bar_titles = array(); foreach($data as $val) { $bar_titles[] = $val->ym; $bar_values[] = floatval($val->total); } OFCharts::add_data_set($bar_titles, $bar_values); echo OFCharts::create_area_graph($title); } /** * Pilots::RecentFrontPage() * * @param integer $count * @return */ public function RecentFrontPage($count = 5) { $pilot_list = PilotData::getLatestPilots($count); $this->set('pilots', $pilot_list); $this->set('pilot_list', $pilot_list); $this->render('frontpage_recentpilots.tpl'); } } Quote Link to comment Share on other sites More sharing options...
flyalaska Posted September 12, 2016 Report Share Posted September 12, 2016 That worked, Thank you! Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.