Everything works flawless and looks good except, when I applied for LoA as a test pilot, I logged out, cleared cookies and cache and then logged back in as admin. I went in and viewed said pilots request and hit delete. After delete, it said are you sure. i clicked delete and nothing happens it just stays on the very same page. goes nowhere and pilot is not deleted from the database. If I click no, that I do not want to delete the request, it also stays on that page and does nothing. that’s as far as it will go. I did test all errors that were testable and all worked such as already exists, date is to long etc etc.
I am going to go back now and try to approve a pilots leave of absence. I will come report back if approval is successful.
On a side note, all database connection errors are gone as well as deprecated code errors.
I found a very minor error in the module in the admin panel.
When a pilot sends in a request and you click view, it is supposed to display the pilots name and profile link and it does not. Also on confirm deletion of LoA request, it does not display the pilot id neither.
However, it does display the pilots name and pilot id number properly on LoA Admin Index page
Quote
You are viewing the Leave of Absence Request for pilot (missing PID and link)
The pilot’s leave started on the following date: 12/17/2016
The pilot’s leave will end on the following date: 01/30/2017
The pilot has specified the following reason for submitting the LoA request:
Hi Ray testing email
Quote
Delete A Leave of Absence Request
Are you sure you want to delete a LoA Request for pilot (Missing PID and Link)
hey real quick while I have this file open and before I forget what I was going to ask, in this piece of code :
private static $maxDuration = 61;
I change the 61 to what ever number I wish such as 90 for 90 days correct?
I may have a slightly newer version than you Ray. At the end of that line, mine states: // Edit this value to set the max LOA Duration
I think (as in I’m not positive) you would set it to the number of days - plus 1 day. So if you limit an LOA to 90 days, it will expire on day 91. This provides the pilot a full 90 days of LOA.
We probably have the same version if you got the new one. I was on skype with Sava when he released this and I grabbed it. I do have that comment as well in mine I just failed to post it. sorry about that. I was just sure about the 1 extra day thing on there.
I appreciate the reply Pro. Thanks man. I’ll go fix that right quick before I forget.
Hey guys, without looking at the code, I think I entered to account for the difference in time.
A huge part of the module is still legacy code from when I was still learning. I’ll get the update done soon. (And send it for you to review Ray :P)
I don;t know if it is a problem on my end but sava’s loa repo is not opening. I am getting these errors on client side:
Deprecated: Non-static method LoAData::CheckPilotID() should not be called statically, assuming $this from incompatible context in /storage/ssd3/285/3946285/public\_html/core/modules/LoA/LoA.php on line 66 Deprecated: Non-static method LoAData::AddLoa() should not be called statically, assuming $this from incompatible context in /storage/ssd3/285/3946285/public\_html/core/modules/LoA/LoA.php on line 76
And these on the admin side:
Deprecated: Non-static method LoAData::GetAllRequests() should not be called statically, assuming $this from incompatible context in /storage/ssd3/285/3946285/public\_html/admin/modules/LoA/LoA.php on line 35 Warning: fgetcsv() expects parameter 1 to be resource, boolean given in /storage/ssd3/285/3946285/public\_html/admin/modules/LoA/LoA.php on line 83 Warning: fclose() expects parameter 1 to be resource, boolean given in /storage/ssd3/285/3946285/public\_html/admin/modules/LoA/LoA.php on line 84
I would really like to use this module as im running Classic VMS 5.5 (PHP version) but sadly none of the download links work could anyone provide me with a download link?
I would really like to use this module as im running Classic VMS 5.5 (PHP version) but sadly none of the download links work could anyone provide me with a download link?
Looks like a fantastic addition to my site.
Kind Regards Harry.
I packaged mine for you. You will need to change the classes to match your css.