Hello everyone, so I’m trying to make a module for the pilots to send suggestions of routes. But when I click to send it it does not send the email.
My core/templates/routesubmit_mainform.php - MY BUTTON CODE
\<input type="submit" class="btn btn-primary btn-block btn-flat" name="submit" value='Submit Route'\>
My core/modules/RouteSubmit.php - MY SENDEMAIL CODE
\<?php class RouteSubmit extends CodonModule { public function index() { require\_once CORE\_LIB\_PATH.'/recaptcha/recaptchalib.php'; $this-\>set('title', 'Add Schedule'); $this-\>set('allairlines', OperationsData::GetAllAirlines()); $this-\>set('allaircraft', OperationsData::GetAllAircraft()); $this-\>set('allairports', OperationsData::GetAllAirports()); //$this-\>set('airport\_json\_list', OperationsData::getAllAirportsJSON()); $this-\>set('flighttypes', Config::Get('FLIGHT\_TYPES')); if(isset($\_POST['submit'])) { $this-\>SendEmail(); } else { $this-\>ShowForm(); } $this-\>render('routesubmit/routesubmit\_mainform'); } protected function ShowForm() { //Google reCaptcha //updated to Google noCaptcha 1/15 $this-\>set('sitekey', "MyCaptchaKey"); $this-\>set('lang', 'pt'); } public function SendEmail() { $name = $\_POST['name']; $email = $\_POST['email']; $depp = $\_POST['depp']; $arr = $\_POST['arr']; $route = $\_POST['route']; $aircraft = $\_POST['Aircraft']; $fl = $\_POST['fl']; $dtime = $\_POST['dtime']; $atime = $\_POST['atime']; $ftime = $\_POST['ftime']; $message = "From: $name \n Depp: $depp \n Arr: $arr \n Route: $route \n Aircraft: $aircraft \n Fl: $fl \n Dtime: $dtime \n Atime: $atime \n Ftime: $ftime \n"; $email\_to = "MYEMAIL"; $subject = "Pilot Submitted Route"; $headers = "From: $email \r\n"; Util::SendEmail($email\_to, $subject, $message, $mailheader); } }
in your send_email I see: $email = $_POST[‘email’];
then later I see $email_to = “MYEMAIL”; which you then use to send the email
I would delete the second line and rewrite the first one to $email_to = $_POST[‘email’]; and see where that goes.
$email is the pilot’s email. The pilot who is sending the route request.
$email_to is where the route request is sent to.
I do not believe that this is an issue. First of all, instead of using this:
$\_POST['email']
you will have to use this:
$this-\>post-\>email
You will have to do so with all the variables in the SendEmail and index function. Also, your form should include a captcha but I can see that there is not any captcha validation in the SendEmail function. I would suggest removing it from your form or validating in on the form submit. Personally, I would suggest you to add several random echo’s in your code. As soon as you run your code, you will be able to understand (based on the echo’s) which part of the code run and which part of the code did not run.
Did you mean like that?
public function SendEmail() { $name = $this-\>post-\>name; $email\_from = $this-\>post-\>email; $depp = $this-\>post-\>depp; $arr = $this-\>post-\>arr; $route = $this-\>post-\>route; $Aircraft = $this-\>post-\>Aircraft; $fl = $this-\>post-\>fl; $dtime = $this-\>post-\>dtime; $atime = $this-\>post-\>atime; $ftime = $this-\>post-\>ftime; $message = "From: $name \n Depp: $depp \n Arr: $arr \n Route: $route \n Aircraft: $aircraft \n Fl: $fl \n Dtime: $dtime \n Atime: $atime \n Ftime: $ftime \n"; $email = "MYEMAIL"; $subject = "Pilot Submitted Route"; $headers = "From: $email\_from \r\n"; Util::SendEmail($email, $subject, $message, $headers); }
If you want to see my entire code: https://github.com/carlosmfreitas2409/RouteSubmit
No, if you want to see my whole code, it’s in the message above
Thank you, but I’ll test it soon, I’m not on the computer now
Now I fully updated the codes, and it’s still not working, I looked in google, I did everything, but nothing … (Using Gmail and PhpVMS 5.5 Simpilot)
Code: https://github.com/carlosmfreitas2409/RouteSubmit
I am also looking into it as it is not working on my server aswell
Can you tell me when it works? I’m trying to do everything I can to fix it, too. Thank you. If I do, I’ll tell you…
Sorry if I wrote something wrong, because I’m using google translate. I am Brazilian
I have made a few changes:
core/modules/RouteSubmit/RouteSubmit.php: First of all, the “RouteSubmit.php” file should be placed in a folder titled “RouteSubmit” under the modules section. Replaces its contents with the following:
\<?php class RouteSubmit extends CodonModule { public $title = "RouteSubmit Request"; public function index() { if (!Auth::LoggedIn()) { $this-\>set('error', 'You are not logged in.'); $this-\>show('core\_error'); } else { $this-\>set('allairlines', OperationsData::GetAllAirlines()); $this-\>set('allaircraft', OperationsData::GetAllAircraft()); $this-\>set('allairports', OperationsData::GetAllAirports()); //$this-\>set('airport\_json\_list', OperationsData::getAllAirportsJSON()); $this-\>set('flighttypes', Config::Get('FLIGHT\_TYPES')); $this-\>render('routesubmit/routesubmit\_index'); } } public function submit() { if ($this-\>post-\>arr == '') { $this-\>set('error', 'You haven\'t specified a reason for your leaeve of absence.'); $this-\>render('routesubmit/routesubmit\_error'); $this-\>render('routesubmit/routesubmit\_index'); } else { $this-\>sendmail($data); } } protected function sendmail($data) { // Form arrays $name = $this-\>post-\>name; $from = $this-\>post-\>email; $depp = $this-\>post-\>depp; $arr = $this-\>post-\>arr; $route = $this-\>post-\>route; $aircraft = $this-\>post-\>Aircraft; $fl = $this-\>post-\>fl; $dtime = $this-\>post-\>dtime; $atime = $this-\>post-\>atime; $ftime = $this-\>post-\>ftime; // Send email to admin $subject\_admin = SITE\_NAME.' A pilot has submitted a Route Request'; $email\_admin = 'your\_email\_or\_ADMIN\_EMAIL'; $message = "From: $name | $from \n Departure: $depp \n Arrival: $arr \n Route: $route \n Aircraft: $aircraft \n Flight level: $fl \n Departure time: $dtime \n Arrival time: $atime \n Flight time: $ftime \n Essa mensagem automatica, por favor não responder"; if ($\_POST['submit']) { Util::SendEmail($email\_admin, $subject\_admin, $message); // Show the template $this-\>render('routesubmit/routesubmit\_submitted'); } } }
Also, put the template files within a “routesubmit” folder too within the core/templates folder. Also, update the routesubmit_index.php file with this:
\<?php if(!defined('IN\_PHPVMS') && IN\_PHPVMS !== true) { die(); } ?\> \<body class="hold-transition login-page" style="background-color: #222222;"\> \<form action="\<?php echo url('/routesubmit/submit');?\>" method="post"\> \<div class="login-box"\> \<div class="login-logo"\> \<a href="" style="color: white;"\>Route\<b\>Submit\</b\>\</a\> \</div\> \<!-- /.login-logo --\> \<div class="login-box-body"\> \<p class="login-box-msg" style="font-size: 13px;"\>Please complete this form to send your route.\</p\> \<form method="post" action="\<?php echo url('/routesubmit/submit');?\>"\> \<div class="form-group" align="center"\> \<tr\> \<td\>\<strong\>Name:\</strong\>\</td\> \<td\> \<?php if(Auth::LoggedIn()) { echo Auth::$userinfo-\>firstname .' '.Auth::$userinfo-\>lastname; echo '\<input type="hidden" name="name" value="'.Auth::$userinfo-\>firstname .' '.Auth::$userinfo-\>lastname.'" /\>'; } else { ?\> \<input type="text" name="name" value="" /\> \<?php } ?\> \</td\> \</tr\> \</div\> \<div class="form-group" align="center"\> \<tr\> \<td width="1%" nowrap\>\<strong\>Endereço de E-mail:\</strong\>\</td\> \<td\> \<?php if(Auth::LoggedIn()) { echo Auth::$userinfo-\>email; echo '\<input type="hidden" name="name" value="'.Auth::$userinfo-\>email.'" /\>'; } else { ?\> \<input type="text" name="email" value="" /\> \<?php } ?\> \</td\> \</tr\> \</div\> \<div class="form-group"\> \<input type="text" name="depp" class="form-control" value="\<?php echo Vars::POST('lastname');?\>" placeholder="Departure ICAO"\> \<?php if($departure\_error == true) echo '\<p class="error"\>Please enter your departure ICAO\</p\>'; ?\> \</div\> \<div class="form-group"\> \<input type="text" name="arr" class="form-control" placeholder="Arrival ICAO"\> \<?php if($arrival\_error == true) echo '\<p class="error"\>Please enter your arrival ICAO\</p\>'; ?\> \</div\> \<div class="form-group"\> \<input type="text" name="route" class="form-control" placeholder="Route"\> \<?php if($route\_error == true) echo '\<p class="error"\>Please enter your route\</p\>'; ?\> \</div\> \<div class="form-group"\> \<select name="Aircraft" class="form-control"\> \<option value=" "\>Select your aircraft\</option\> \<option value="2"\>Airbus\</option\> \<option\>Airbus A318-111\</option\> \<option\>Airbus A319-111\</option\> \<option\>Airbus A320-211\</option\> \<option\>Airbus A321-211\</option\> \<option value="2"\>Boeing\</option\> \<option\>Boeing 737-800/W\</option\> \<option\>Boeing 737-900/W\</option\> \<option\>Boeing 747-400\</option\> \<option\>Boeing 747-400F\</option\> \<option\>Boeing 747-800I\</option\> \<option\>Boeing 757-200\</option\> \<option\>Boeing 767-300\</option\> \<option\>Boeing 777-200LR\</option\> \<option\>Boeing 777-300ER\</option\> \<option\>Boeing 777-200F\</option\> \<option value="3"\>De Havilland\</option\> \<option\>DHC-8-400\</option\> \<option value="4"\>McDonnell Douglas\</option\> \<option\>MD-11\</option\> \<option\>MD-11F\</option\> \</select\> \<?php if($aircraft\_error == true) echo '\<p class="error"\>Please enter your aircraft\</p\>'; ?\> \</div\> \<div class="form-group"\> \<input type="text" name="fl" class="form-control" placeholder="Flight level"\> \<?php if($flightlevel\_error == true) echo '\<p class="error"\>Please enter your flight level\</p\>'; ?\> \</div\> \<div class="form-group"\> \<input type="text" name="dtime" class="form-control" placeholder="Departure Time"\> \<?php if($departuretime\_error == true) echo '\<p class="error"\>Please enter your departure time\</p\>'; ?\> \</div\> \<div class="form-group"\> \<input type="text" name="atime" class="form-control" placeholder="Arrival Time"\> \<?php if($arrivaltime\_error == true) echo '\<p class="error"\>Please enter your arrival time\</p\>'; ?\> \</div\> \<div class="form-group"\> \<input type="text" name="ftime" class="form-control" placeholder="Flight Time"\> \<?php if($flighttime\_error == true) echo '\<p class="error"\>Please enter your flight time\</p\>'; ?\> \</div\> \<input type="hidden" name="loggedin" value="\<?php echo (Auth::LoggedIn())?'true':'false'?\>" /\> \<input id="submit" type="submit" class="btn btn-success" class="btn btn-primary btn-block btn-flat" name="submit" value='Submit Route'\> \</td\> \<!-- /.col --\> \</tr\> \</form\> \</div\> \<!-- /.login-box-body --\> \</div\> \<!-- /.login-box --\> Copyright © \<?php echo date(Y); ?\> CCBS Virtual Airlines - All Rights Reserved \<!-- iCheck --\> \<script src="\<?php echo SITE\_URL?\>/lib/skins/crewcenter/plugins/iCheck/icheck.min.js"\>\</script\> \<script\> $(function () { $('input').iCheck({ checkboxClass: 'icheckbox\_square-blue', radioClass: 'iradio\_square-blue', increaseArea: '20%' // optional }); }); \</script\>
It worked in my localhost server (do not comment the values ):
Quote
From: pilot_submitted_email | Departure: dsadsad Arrival: sadsadsa Route: dsad Aircraft: Flight level: dsadsa Departure time: dsa Arrival time: dsads Flight time: adsa Essa mensagem automatica, por favor não responder
Let us know in case you have any issues.
Try to add echoes in several part of the code. Different echoes. After that, run the module and check when echoes stopped showing. That is the best I can suggest in order to troubleshoot the reason why it is not running correctly. Does your website generally sends emails without issues?
Yes, send, all emails that have been configured by phpVMS or addon’s are sent
This is the my new code:
core/modules/RouteSubmit/RouteSubmit.php
\<?php class RouteSubmit extends CodonModule { public $title = "RouteSubmit Request"; public function index() { if (!Auth::LoggedIn()) { $this-\>set('error', 'You are not logged in.'); $this-\>show('core\_error'); } else { $this-\>set('allairlines', OperationsData::GetAllAirlines()); $this-\>set('allaircraft', OperationsData::GetAllAircraft()); $this-\>set('allairports', OperationsData::GetAllAirports()); //$this-\>set('airport\_json\_list', OperationsData::getAllAirportsJSON()); $this-\>set('flighttypes', Config::Get('FLIGHT\_TYPES')); $this-\>render('routesubmit/routesubmit\_index'); } } public function submit() { if(isset($\_POST['submit'])) { if ($this-\>post-\>arr == '') { $this-\>set('error', 'You haven\'t specified a reason for your leaeve of absence.'); $this-\>render('routesubmit/routesubmit\_error'); $this-\>render('routesubmit/routesubmit\_index'); } else { $this-\>sendmail(); } } } protected function sendmail() { // Form arrays $name = $this-\>post-\>name; $from = $this-\>post-\>email; $depp = $this-\>post-\>depp; $arr = $this-\>post-\>arr; $route = $this-\>post-\>route; $aircraft = $this-\>post-\>Aircraft; $fl = $this-\>post-\>fl; $dtime = $this-\>post-\>dtime; $atime = $this-\>post-\>atime; $ftime = $this-\>post-\>ftime; // Send email to admin $subject = SITE\_NAME.' A pilot has submitted a Route Request'; $email = '[email protected]'; $message = 'From: $name | $from \n Departure: $depp \n Arrival: $arr \n Route: $route \n Aircraft: $aircraft \n Flight level: $fl \n Departure time: $dtime \n Arrival time: $atime \n Flight time: $ftime \n Esta mensagem é automatica, por favor não responder'; $headers = 'From:' . $from; Util::SendEmail($email, $subject, $message. $headers); // Show the template $this-\>render('routesubmit/routesubmit\_submitted'); } } ?\>
Did you try to add different echoes in your code in order to check where the code stops running?