From baa33f40b3b806abdd31f2567e379b474545994c Mon Sep 17 00:00:00 2001
From: James Turner <zakalawe@mac.com>
Date: Mon, 14 Jan 2019 12:16:22 +0000
Subject: [PATCH] Airport-list: allow filtering by type for helipads

Patch by Daniel Wickstrom, requires a corresponding FLightGear change
to extend the special search function used by this dialog to add
support for a type-prefix.
---
 gui/dialogs/airports.xml | 51 ++++++++++++++++++++++++++++++++++++----
 1 file changed, 46 insertions(+), 5 deletions(-)

diff --git a/gui/dialogs/airports.xml b/gui/dialogs/airports.xml
index 26fa2f308..4231206c6 100644
--- a/gui/dialogs/airports.xml
+++ b/gui/dialogs/airports.xml
@@ -70,6 +70,10 @@
         setprop("/sim/gui/dialogs/airports/display-tower", "1");
       }
 
+      if (getprop("/sim/gui/dialogs/airports/show-helipads") == "") {
+        setprop("/sim/gui/dialogs/airports/show-helipads", "1");
+      }
+
       # Start with the closest airport
       var airport_id = airportinfo().id;
 
@@ -288,10 +292,6 @@
             <command>dialog-apply</command>
             <object-name>input</object-name>
           </binding>
-          <binding>
-            <command>dialog-update</command>
-            <object-name>airport-list</object-name>
-          </binding>
         </input>
 
 <!--
@@ -330,6 +330,32 @@
             <command>dialog-apply</command>
             <object-name>input</object-name>
           </binding>
+          <binding>
+            <command>nasal</command>
+            <script>
+              var apt_type = "airport:";
+              var heli_type = "heliport:";
+              var search_term = getprop("/sim/gui/dialogs/airports/list");
+
+              # strip off airport type prefix
+              if (string.match(search_term,heli_type)) {
+                search_term = string.substr(search_term,size(heli_type));
+              }
+              else if (string.match(search_term,apt_type)) {
+                search_term = string.substr(search_term,size(apt_type));
+              }
+              var new_value = "";
+
+              # add new airport type prefix based off helipad checkbox
+              if (getprop("/sim/gui/dialogs/airports/show-helipads")) {
+                new_value = heli_type ~ search_term;
+              }
+              else {
+                new_value = apt_type ~ search_term;
+              }
+              setprop("/sim/gui/dialogs/airports/list", new_value);
+            </script>
+          </binding>
           <binding>
             <command>dialog-update</command>
             <object-name>airport-list</object-name>
@@ -346,7 +372,12 @@
           <binding>
             <command>nasal</command>
             <script>
-              var airports = findAirportsWithinRange(100);
+              # change airport type to heliport if checkbox is set
+              var apt_type = "airport";
+              if (getprop("/sim/gui/dialogs/airports/show-helipads")) {
+                 apt_type = "heliport";
+              }
+              var airports = findAirportsWithinRange(100,apt_type);
 
               var list = dlg.getNode("close-airports", 1);
 
@@ -363,6 +394,16 @@
           </binding>
         </button>
 
+        <checkbox>
+          <name>show-helipads</name>
+          <label>Helipads</label>
+          <property>/sim/gui/dialogs/airports/show-helipads</property>
+          <binding>
+            <command>dialog-apply</command>
+            <object-name>show-helipads</object-name>
+          </binding>
+        </checkbox>
+
       </group>
 
       <group>