sgraypgh Posted January 8, 2023 Report Share Posted January 8, 2023 I have distance set to NM in admin, but weather pages are showing Visibility in km. Any way to have that use either NM or Miles? Quote Link to comment Share on other sites More sharing options...
DisposableHero Posted January 8, 2023 Report Share Posted January 8, 2023 2 hours ago, sgraypgh said: I have distance set to NM in admin, but weather pages are showing Visibility in km. Any way to have that use either NM or Miles? Weather decoding is not a part of Disposable Addons (neither Disposable Theme nor the modules), it is handled by phpvms v7 itself. As far as I remember, wx visibility is reported in statue miles in US (and FAA rules following countries), rest of the world uses meters. So a conversion to nautical miles (nmi) is not logical at all, it should be either miles or kilometers. (this may be the cause of non-conversion by the core) Safe flights Quote Link to comment Share on other sites More sharing options...
sgraypgh Posted January 8, 2023 Author Report Share Posted January 8, 2023 Thanks, I guess I will need to post this over at the phpvms 7 forum area. 1 Quote Link to comment Share on other sites More sharing options...
sgraypgh Posted January 15, 2023 Author Report Share Posted January 15, 2023 I was able to edit the app/Support/Metar.php to correct always using km and converted it to mi. Quote Link to comment Share on other sites More sharing options...
DisposableHero Posted January 15, 2023 Report Share Posted January 15, 2023 So you made every airport report visibility in miles ? Hope it does not mess up with peoples brains and non-us airport reports. Quote Link to comment Share on other sites More sharing options...
sgraypgh Posted January 15, 2023 Author Report Share Posted January 15, 2023 Given that the existing coding is not correctly displaying it, I didn't really have much of a choice. Also, my VA is based in the US so generally won't be an issue. For the unlikely reason someone does fly outside of the US, I highly doubt they will be using that information for their weather reports anyway. I suppose I could display both, like it is done with Pressure. Quote Link to comment Share on other sites More sharing options...
sgraypgh Posted January 15, 2023 Author Report Share Posted January 15, 2023 I also had to fix the Dashboard weather as well, as it had the same issue. Quote Link to comment Share on other sites More sharing options...
DisposableHero Posted January 15, 2023 Report Share Posted January 15, 2023 Well, I am not using decoded weather and not a member of your VA anyway, so whatever you did on your end is ok for me. I would suggest opening a bug report for phpvms v7 (+your idea about double display mentioned above), maybe the developer (or someone else) fixes it for good. As you know this is not an addon/theme issue and related with phpvms v7 core itself, all should go through github. Developer will not see your problem or request here. Safe flights Quote Link to comment Share on other sites More sharing options...
sgraypgh Posted January 15, 2023 Author Report Share Posted January 15, 2023 Thank you for your continued support, you have a great theme and addons! 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.