drop support for $localreports
authorDenis Ovsienko <denis@ovsienko.info>
Mon, 12 Nov 2018 10:52:34 +0000 (10:52 +0000)
committerDenis Ovsienko <denis@ovsienko.info>
Mon, 12 Nov 2018 10:55:23 +0000 (10:55 +0000)
10 years ago this feature seemed to be a useful shortcut, but based on
the experience with both free and proprietary plugins since then it
became clear that it is either not used at all, or can be trivially
implemented without the code in RackTables when really required. Remove
renderLocalReports() and trigger_localreports().

ChangeLog
README.md
wwwroot/inc/init.php
wwwroot/inc/interface-reports.php
wwwroot/inc/interface.php
wwwroot/inc/navigation.php
wwwroot/inc/triggers.php
wwwroot/inc/upgrade.php

index b0af24a..ca05b1f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -6,6 +6,7 @@
        update: new modular port type 'SFP28'
        update: allow to delete local user accounts (Mantis#1089)
        update: show names in allocated units when assigning rackspace (Mantis#1711)
+       update: drop support for $localreports
        new feature: "shared router" allocation (GH#210 by Vladimir Ivanov)
        new feature: optional tag descriptions
        new feature: support for HP N.11.78 and IOS 15 (GH #225 by Vasilii Shashkov)
index ad1d9df..b718eec 100644 (file)
--- a/README.md
+++ b/README.md
@@ -164,6 +164,10 @@ and initialize the application.
 
 ## Release notes
 
+### Upgrading to 0.21.2
+This version drops support for the `$localreports` global variable, which is
+trivial to replace in a local plugin if necessary.
+
 ### Upgrading to 0.21.0
 
 From now on the minimum (oldest) release of PHP that can run RackTables is
index 1c6cf3d..9b134d1 100644 (file)
@@ -40,7 +40,6 @@ if (! fileSearchExists ($path_to_secret_php))
 connectDB();
 transformRequestData();
 $configCache = loadConfigDefaults();
-$tab['reports']['local'] = getConfigVar ('enterprise');
 
 if (getConfigVar ('DB_VERSION') != CODE_VERSION)
 {
index 80609bf..5d68b05 100644 (file)
@@ -69,12 +69,6 @@ function renderSystemReports ()
        renderReports ($tmp);
 }
 
-function renderLocalReports ()
-{
-       global $localreports;
-       renderReports ($localreports);
-}
-
 function render8021QReport ()
 {
        if (!count ($domains = getVLANDomainOptions()))
index 46682fa..f3102bc 100644 (file)
@@ -62,10 +62,6 @@ $vtoptions = array
 #      'alien' => 'never touch',
 );
 
-// This may be populated later onsite, report rendering function will use it.
-// See the $systemreport for structure.
-$localreports = array();
-
 $attrtypes = array
 (
        'uint' => '[U] unsigned integer',
index 9b8e47b..180537a 100644 (file)
@@ -707,14 +707,11 @@ $tab['reports']['8021q'] = '802.1Q';
 $tab['reports']['warranty'] = 'Expirations';
 $tab['reports']['integrity'] = 'Data Integrity';
 $tab['reports']['serverconf'] = 'Server configuration';
-$tab['reports']['local'] = 'local'; // this one is set later in init.php
-$trigger['reports']['local'] = 'trigger_localreports';
 $tabhandler['reports']['default'] = 'renderSystemReports';
 $tabhandler['reports']['8021q'] = 'render8021QReport';
 $tabhandler['reports']['warranty'] = 'renderExpirations';
 $tabhandler['reports']['integrity'] = 'renderDataIntegrityReport';
 $tabhandler['reports']['serverconf'] = 'renderServerConfigurationReport';
-$tabhandler['reports']['local'] = 'renderLocalReports';
 $interface_requires['reports-default'] = 'code.php';
 $interface_requires['reports-serverconf'] = 'dictionary.php';
 $interface_requires['reports-*'] = 'interface-reports.php';
index e086fb7..ee3fe30 100644 (file)
@@ -136,12 +136,6 @@ function trigger_passwdchange ()
        return ($user_auth_src == 'database' || 1 === getUserIDByUsername ($remote_username)) ? 'std' : '';
 }
 
-function trigger_localreports ()
-{
-       global $localreports;
-       return count ($localreports) ? 'std' : '';
-}
-
 function trigger_file_editText ()
 {
        $fileInfo = spotEntity ('file', getBypassValue());
index 8d82081..0645fd3 100644 (file)
@@ -175,6 +175,9 @@ ENDOFTEXT
 ,
        '0.21.2' => <<<'ENDOFTEXT'
 "Shared router" allocation type introduced, useful for documenting VRRP-protected addresses.
+
+This version drops support for the $localreports global variable, which is
+trivial to replace in a local plugin if necessary.
 ENDOFTEXT
 ,
 );