replace includeJQueryUI() with 2 nullary functions
authorDenis Ovsienko <denis@ovsienko.info>
Mon, 26 Nov 2018 19:16:23 +0000 (19:16 +0000)
committerDenis Ovsienko <denis@ovsienko.info>
Mon, 26 Nov 2018 19:17:46 +0000 (19:17 +0000)
The only argument to this function was always a constant.

* includeJQueryUI(): mark deprecated
* includeJQueryUIJS(): a new function
* includeJQueryUICSS(): a new function
* enableTagsPicker(): update to use the new functions
* renderRackSortForm(): idem
* renderGridForm: idem
* renderIPForObject: idem
* renderRackSpaceForObject: idem
* renderIPAddressAllocations: idem
* renderPopupPortSelector: idem

[skip ci]

wwwroot/inc/interface-lib.php
wwwroot/inc/interface.php
wwwroot/inc/popup.php

index 0850615..2d802e3 100644 (file)
@@ -1060,11 +1060,22 @@ function getRenderedNetVLAN ($cell)
        return "<div class='vlan'><strong><small>${noun}</small> " . implode (', ', $links) . '</strong></div>';
 }
 
+// DEPRECATED and will be removed in 0.22.0
 function includeJQueryUI ($do_css = TRUE)
 {
-       addJSInternal ('js/jquery-ui-1.8.21.min.js');
+       includeJQueryUIJS();
        if ($do_css)
-               addCSSInternal ('css/jquery-ui-1.8.22.redmond.css');
+               includeJQueryUICSS();
+}
+
+function includeJQueryUIJS()
+{
+       addJSInternal ('js/jquery-ui-1.8.21.min.js');
+}
+
+function includeJQueryUICSS()
+{
+       addCSSInternal ('css/jquery-ui-1.8.22.redmond.css');
 }
 
 function getRenderedIPPortPair ($ip, $port = NULL)
@@ -1222,7 +1233,8 @@ function enableTagsPicker ()
 {
        global $taglist;
        static $taglist_inserted;
-       includeJQueryUI ();
+       includeJQueryUIJS();
+       includeJQueryUICSS();
        addCSSInternal ('css/tagit.css');
        addJSInternal ('js/tag-it.js');
        addJSInternal ('js/tag-it-local.js');
index c66bbc9..00487fc 100644 (file)
@@ -1048,7 +1048,7 @@ function renderRack ($rack_id, $hl_obj_id = 0)
 
 function renderRackSortForm ($row_id)
 {
-       includeJQueryUI (false);
+       includeJQueryUIJS();
        // Heredoc, not nowdoc!
        $js = <<<"JSTXT"
        $(document).ready(
@@ -1320,7 +1320,7 @@ function renderGridForm ($rack_id, $filter, $header, $submit, $state1, $state2)
        // Grid form.
        $is_ro = !rackModificationPermitted ($rackData, 'updateRack');
        startPortlet ($header);
-       includeJQueryUI (FALSE);
+       includeJQueryUIJS();
        addJSInternal ('js/racktables.js');
        $table_id = 'selectableRack';
        addBulkSelectorJS ($table_id);
@@ -1828,7 +1828,8 @@ function renderIPForObject ($object_id)
        {
                global $aat;
 
-               includeJQueryUI (TRUE);
+               includeJQueryUIJS();
+               includeJQueryUICSS();
 
                // Heredoc, not nowdoc!
                addJSText (<<<"JSEND"
@@ -2259,7 +2260,7 @@ function renderRackSpaceForObject ($object_id)
        // the former state of the grid needs to make it to the current form.
        echo "<td class=pcright rowspan=2 height='1%'>";
        startPortlet ('Working copy');
-       includeJQueryUI (false);
+       includeJQueryUIJS();
        addJSInternal ('js/racktables.js');
        echo '<table border=0 cellspacing=10 align=center><tr>';
        foreach ($workingRacksData as $rack_id => $rackData)
@@ -3406,7 +3407,8 @@ function renderIPAddressAllocations ($ip_bin)
        {
                global $aat;
 
-               includeJQueryUI (TRUE);
+               includeJQueryUIJS();
+               includeJQueryUICSS();
 
                addJSText (<<<'JSEND'
                        $(document).ready( function() {
index d51eb29..dfc2f71 100644 (file)
@@ -405,7 +405,8 @@ function renderPopupPortSelector()
        )
                $spare_ports = findSparePorts ($port_info, $filter);
 
-       includeJQueryUI (TRUE);
+       includeJQueryUIJS();
+       includeJQueryUICSS();
 
        // display search form
        echo 'Link ' . formatPort ($port_info) . ' to...';