omit the PHP closing tag at EOF in all PHP files
authorDenis Ovsienko <denis@ovsienko.info>
Thu, 13 Jul 2017 16:09:25 +0000 (17:09 +0100)
committerDenis Ovsienko <denis@ovsienko.info>
Thu, 13 Jul 2017 16:15:18 +0000 (17:15 +0100)
"If a file is pure PHP code, it is preferable to omit the PHP closing tag
at the end of the file. This prevents accidental whitespace or new lines
being added after the PHP closing tag, which may cause unwanted effects
because PHP will start output buffering when there is no intention from
the programmer to send any output at that point in the script."
-- PHP manual

55 files changed:
scripts/cleanup_ldap_cache.php
scripts/syncdomain.php
tests/ConfigVarTest.php
tests/DBMutexTest.php
tests/DictionaryAttributeTest.php
tests/EmptySQLWhereTest.php
tests/EntityLinkTriggerTest.php
tests/GetChildrenListTest.php
tests/GetRowsCountTest.php
tests/LinkTriggerTest.php
tests/ObjectAttributesTest.php
tests/ObjectCircularReferenceTest.php
tests/ObjectLogTest.php
tests/ObjectPortsTest.php
tests/PureFunctionTest.php
tests/RackspaceFunctionsTest.php
tests/RenderDepotTest.php
tests/ScriptFunctionsTest.php
tests/StringInsertHrefsTest.php
tests/TagFunctionsTest.php
tests/TagTreeCircularReferenceTest.php
tests/TestHelper.php
tests/UpgradeTest.php
tests/UserAccountTest.php
tests/bootstrap.php
wwwroot/inc/ajax-interface.php
wwwroot/inc/auth.php
wwwroot/inc/caching.php
wwwroot/inc/code.php
wwwroot/inc/config.php
wwwroot/inc/database.php
wwwroot/inc/deviceconfig.php
wwwroot/inc/dictionary.php
wwwroot/inc/exceptions.php
wwwroot/inc/functions.php
wwwroot/inc/init.php
wwwroot/inc/install.php
wwwroot/inc/interface-8021q.php
wwwroot/inc/interface-cables.php
wwwroot/inc/interface-config.php
wwwroot/inc/interface-lib.php
wwwroot/inc/interface-reports.php
wwwroot/inc/interface.php
wwwroot/inc/navigation.php
wwwroot/inc/ophandlers.php
wwwroot/inc/popup.php
wwwroot/inc/pre-init.php
wwwroot/inc/remote.php
wwwroot/inc/slb-interface.php
wwwroot/inc/slb.php
wwwroot/inc/snmp.php
wwwroot/inc/solutions.php
wwwroot/inc/triggers.php
wwwroot/inc/upgrade.php
wwwroot/index.php

index 77b31da..cacca66 100755 (executable)
@@ -19,5 +19,3 @@ if ($user_auth_src == 'ldap')
        constructLDAPOptions();
        discardLDAPCache ($LDAP_options['cache_expiry']);
 }
-
-?>
index b4a8cb4..221d000 100755 (executable)
@@ -200,4 +200,3 @@ if (! $nolock)
        }
 }
 exit (0);
-?>
index c198012..41b31cb 100644 (file)
@@ -153,5 +153,3 @@ class ConfigVarTest extends PHPUnit_Framework_TestCase
                );
        }
 }
-
-?>
index b356d18..88180bf 100644 (file)
@@ -20,5 +20,3 @@ class DBMutexTest extends PHPUnit_Framework_TestCase
                $this->assertSame (FALSE, releaseDBMutex (get_class() . getmypid()));
        }
 }
-
-?>
index 75dd2fd..b3fd5e5 100644 (file)
@@ -182,5 +182,3 @@ class DictionaryAttributeTest extends PHPUnit_Framework_TestCase
                usePreparedDeleteBlade ('Attribute', array ('id' => array_keys ($this->attr_types)));
        }
 }
-
-?>
index 45d1fcd..a351d23 100644 (file)
@@ -42,5 +42,3 @@ class EmptySQLWhereTest extends PHPUnit_Framework_TestCase
                usePreparedUpdateBlade ('TagTree', NULL, NULL);
        }
 }
-
-?>
index e7aa94d..1930797 100644 (file)
@@ -499,4 +499,3 @@ class EntityLinkTriggerTest extends PHPUnit_Framework_TestCase
                );
        }
 }
-?>
index 3fe411c..e64d392 100644 (file)
@@ -92,4 +92,3 @@ class GetChildrenListTest extends PHPUnit_Framework_TestCase
                $this->assertCount (self::$num_children, $children);
        }
 }
-?>
index 26b8406..b6f5c25 100644 (file)
@@ -42,5 +42,3 @@ class GetRowsCountTest extends PHPUnit_Framework_TestCase
                usePreparedExecuteBlade ('DROP TABLE `' . $this->table_name . '`');
        }
 }
-
-?>
index 2740fcc..bf208dc 100644 (file)
@@ -162,4 +162,3 @@ class LinkTriggerTest extends PHPUnit_Framework_TestCase
                );
        }
 }
-?>
index 077a890..71fd03b 100644 (file)
@@ -84,5 +84,3 @@ class ObjectAttributesTest extends PHPUnit_Framework_TestCase
        }
 
 }
-
-?>
index aba6a9c..1910d17 100644 (file)
@@ -121,4 +121,3 @@ class ObjectCircularReferenceTest extends PHPUnit_Framework_TestCase
                );
        }
 }
-?>
index c821635..6a59969 100644 (file)
@@ -96,4 +96,3 @@ class ObjectLogTest extends PHPUnit_Framework_TestCase
         }
        }
 }
-?>
index 8d1b709..12bd60e 100644 (file)
@@ -192,5 +192,3 @@ class ObjectPortsTest extends PHPUnit_Framework_TestCase
                commitDeleteObject ($this->object_id);
        }
 }
-
-?>
index 61c0a6b..5b92dd7 100644 (file)
@@ -1270,4 +1270,3 @@ class PureFunctionTest extends PHPUnit_Framework_TestCase
                );
        }
 }
-?>
index 85376b9..0efb4b7 100644 (file)
@@ -209,5 +209,3 @@ class RackspaceFunctionsTest extends PHPUnit_Framework_TestCase
                commitDeleteRow ($this->row_id);
        }
 }
-
-?>
index 06a2a0b..8353341 100644 (file)
@@ -39,4 +39,3 @@ class RenderDepotTest extends PHPUnit_Framework_TestCase
         }
        }
 }
-?>
index 0b0dcde..79fa929 100644 (file)
@@ -53,5 +53,3 @@ class ScriptFunctionsTest extends PHPUnit_Framework_TestCase
                saveScript (NULL, NULL);
        }
 }
-
-?>
index 7ca7c67..e55126b 100644 (file)
@@ -55,4 +55,3 @@ class StringInsertHrefsTest extends PHPUnit_Framework_TestCase
                );
        }
 }
-?>
index 0ff37d2..7ddf285 100644 (file)
@@ -64,5 +64,3 @@ class TagFunctionsTest extends PHPUnit_Framework_TestCase
                $this->assertEquals ($this->a_tag_ids, buildTagIdsFromChain (buildTagChainFromIds ($this->a_tag_ids)));
        }
 }
-
-?>
index 3964eeb..d9c8dcb 100644 (file)
@@ -43,4 +43,3 @@ class TagTreeCircularReferenceTest extends PHPUnit_Framework_TestCase
                commitUpdateTag ($this->taga_id, 'unit test tag a', $this->tagc_id, 'yes');
        }
 }
-?>
index c5d22bc..d836eb5 100644 (file)
@@ -13,4 +13,3 @@ class TestHelper
                        throw new Exception ('Test must connect to unit testing database (see tests/README).');
        }
 }
-?>
index fdefdc4..59dd5df 100644 (file)
@@ -62,4 +62,3 @@ class UpgradeTest extends PHPUnit_Framework_TestCase
                }
        }
 }
-?>
index 1154a73..b1ec8d7 100644 (file)
@@ -52,5 +52,3 @@ class UserAccountTest extends PHPUnit_Framework_TestCase
                commitCreateUserAccount ($this->user_name, 'x' . self::REALNAME, sha1 (self::PSWDHASH));
        }
 }
-
-?>
index 0a0cac6..76715c7 100644 (file)
@@ -11,4 +11,3 @@ require_once './TestHelper.php';
 
 // Sanity check DB connection
 TestHelper::ensureUsingUnitTestDatabase ();
-?>
index 7930a0e..2de477f 100644 (file)
@@ -337,4 +337,3 @@ function getAutocompleteListAJAX()
 
        echo json_encode ($rows);
 }
-?>
index 9e92d18..2881dea 100644 (file)
@@ -650,5 +650,3 @@ function authenticated_via_database ($userinfo, $password)
                return FALSE;
        return $userinfo['user_password_hash'] == sha1 ($password);
 }
-
-?>
index 0189a5a..41f991f 100644 (file)
@@ -100,5 +100,3 @@ function HTTPDateToUnixTime ($string)
                return FALSE;
        return gmmktime ($hours, $minutes, $seconds, $month, $day, $year);
 }
-
-?>
index bc0add7..3717703 100644 (file)
@@ -759,5 +759,3 @@ function getRackCodeWarnings ()
        );
        return $ret;
 }
-
-?>
index f7b54ec..622bfd5 100644 (file)
@@ -14,5 +14,3 @@ committers' copies) can run into issues:
 */
 
 define ('CODE_VERSION', '0.20.13');
-
-?>
index f42fd34..494650b 100644 (file)
@@ -6074,5 +6074,3 @@ function releaseDBMutex ($name)
        $row = $result->fetchColumn();
        return $row === '1';
 }
-
-?>
index f80449e..52a1f81 100644 (file)
@@ -3573,5 +3573,3 @@ function iosxr4ReadInterfaceStatus ($input)
        }
        return $result;
 }
-
-?>
index 9758f3f..178a8d7 100644 (file)
@@ -2605,5 +2605,3 @@ $dictionary = array
 # and dictionary updates working properly.
        49999 => array ('chapter_id' => 13, 'dict_value' => '[[RH Fedora%GSKIP%Fedora 15 | http://docs.fedoraproject.org/release-notes/f15/en-US/html/]]'),
 );
-
-?>
index 602c252..29d876f 100644 (file)
@@ -436,5 +436,3 @@ function printException ($e)
        else
                printGenericException ($e);
 }
-
-?>
index 79c21c5..19d27d2 100644 (file)
@@ -6663,5 +6663,3 @@ function syncObjectPorts ($object_id, $desiredPorts)
        $dbxlink->exec ('UNLOCK TABLES');
        showSuccess (sprintf ('Added ports: %u, changed: %u, deleted: %u', count ($to_add), count ($to_update), count ($to_delete)));
 }
-
-?>
index 8be1b15..779bf11 100644 (file)
@@ -162,5 +162,3 @@ $impl_tags = array();
 $target_given_tags = array();
 
 callHook ('initFinished');
-
-?>
index 7b48bfb..c945fe0 100644 (file)
@@ -2325,5 +2325,3 @@ WHERE O.objtype_id = 1562";
        return $query;
        }
 }
-
-?>
index 4fe25c0..44a6816 100644 (file)
@@ -1643,5 +1643,3 @@ function renderEditVlan ($vlan_ck)
 
        finishPortlet();
 }
-
-?>
index d631aee..43bc014 100644 (file)
@@ -295,5 +295,3 @@ function renderPatchCableOIFCompatEditor()
        );
        echo '<br>';
 }
-
-?>
index e59fa42..934c6a3 100644 (file)
@@ -1185,5 +1185,3 @@ function renderMuninServersEditor()
                printNewItemTR();
        echo '</table>';
 }
-
-?>
index 9a2b17c..7344ed5 100644 (file)
@@ -1206,5 +1206,3 @@ function showMySQLWarnings()
        }
        $rtdebug_mysql_warnings = array();
 }
-
-?>
index 68cbe73..d990983 100644 (file)
@@ -980,5 +980,3 @@ function renderDataIntegrityReport ()
        if (! $violations)
                echo '<h2 class=centered>No integrity violations found</h2>';
 }
-
-?>
index 0b98afb..dfceae1 100644 (file)
@@ -6451,5 +6451,3 @@ function renderTableViewer ($columns, $rows, $params = NULL)
        echo '</tbody>';
        echo '</table>';
 }
-
-?>
index e252a4e..5155c9f 100644 (file)
@@ -918,5 +918,3 @@ $ajaxhandler['autocomplete'] = 'getAutocompleteListAJAX';
 $popuphandler['objlist'] = 'renderPopupObjectSelector';
 $popuphandler['portlist'] = 'renderPopupPortSelector';
 $popuphandler['inet4list'] = 'renderPopupIPv4Selector';
-
-?>
index dfe8670..44dee83 100644 (file)
@@ -3850,5 +3850,3 @@ function updateVLANDomain()
        usePreparedUpdateBlade ('VLANDomain', array ('group_id' => $group_id, 'description' => $description), array ('id' => $domain_id));
        showSuccess ("VLAN domain updated successfully");
 }
-
-?>
index fa90c61..b128dee 100644 (file)
@@ -518,4 +518,3 @@ function renderPopupHTML ($contents)
 </html>
 <?php
 }
-?>
index fc545a7..9f5dc74 100644 (file)
@@ -101,5 +101,3 @@ function fileSearchExists ($filename)
        }
        return file_exists ($filename);
 }
-
-?>
index 91d7d5d..9b75db2 100644 (file)
@@ -831,5 +831,3 @@ function ios12ShortenIfName ($ifname)
        $ifname = preg_replace ('/^(e|fa|gi|te|po|xg|lo|ma)\s+(\d.*)/', '$1$2', $ifname);
        return $ifname;
 }
-
-?>
index e319ba1..c945b9e 100644 (file)
@@ -628,5 +628,3 @@ function renderLVSConfig ($object_id)
        echo "</form>";
        echo "<pre>" . buildLVSConfig ($object_id) . "</pre>";
 }
-
-?>
index c20aa49..c9bf820 100644 (file)
@@ -730,5 +730,3 @@ function getRSListInPool ($rspool_id)
        }
        return $ret;
 }
-
-?>
index 71a80b8..b28fcac 100644 (file)
@@ -4856,4 +4856,3 @@ function detectSoftwareType ($objectInfo, $sysDescr)
                        return;
                }
 }
-?>
index 2ac0227..306c29f 100644 (file)
@@ -566,5 +566,3 @@ function printSVGMessageBar ($text = 'lost message', $textattrs = array(), $rect
        echo ">${text}</text>\n";
        echo "</svg>\n";
 }
-
-?>
index 419bff3..649d692 100644 (file)
@@ -344,5 +344,3 @@ function triggerGraphCycleResolver()
        }
        return count (getInvalidNodes ($nodelist)) ? 'attn' : '';
 }
-
-?>
index 101efa3..3f01f3a 100644 (file)
@@ -1548,5 +1548,3 @@ function convertMgmtConfigVars()
        }
        return implode (',', $ret);
 }
-
-?>
index a1a3777..7f04376 100644 (file)
@@ -304,4 +304,3 @@ catch (Exception $e)
        ob_end_clean();
        printException ($e);
 }
-?>