display log records consistently

Factor renderLogRecordsPortlet() out of renderObject() and also call it
from renderRow(), renderLocationPage() and renderRackPage(). This way
every entity that allows to add log records also displays them on the
default tab.

renderEditAttributeTRs(): escape string values

Call stringForTextInputValue() for string attributes so they do not
interfere with the resulting HTML. This fixes a bug reported by Quentin
HESS.

[skip ci]

update COPYING for 2021 [skip ci]

upgrade CodeMirror from 5.58.1 to 5.58.3 [skip ci]

spell lastInsertID() (once again) [skip ci]

lose a newline

fix some spelling [skip ci]

gateways: use "#!/usr/bin/env perl"

forward-port more props of 0.21.5 [skip ci]

copy the 0.21.5 ChangeLog section [skip ci]

lose some trailing space in README.md

allow to customize tag palette (Mantis #1991)

Make the default array global in a way that allows plugins to modify it.
In getColorSelect() read from that array and apply some filtering.

getColorSelect(): fix code style

upgrade CodeMirror from 5.54.0 to 5.58.1

SNMP: add two more Netgear switches (Mantis#1983)

[skip ci]

snmp.php: fix some indentation issues

doSwitchSNMPmining(): fixup some pattern matching

Spell end-of-string where it is implied, lose a preg_match() that is
no-op after commit 59736b5.

SNMP: update some Netgear code

Reorder the switch declarations and detect the software version.

[skip ci]

dictionary: add a few more Netgear switches

[skip ci]

tests: address most shellcheck warnings in *.sh

Use modern style command substitution, add some double quotes, assert
that "cd" is successful, lose an unused variable, reword an underdefined
logical AND, replace "expr" with arithmetic expansion.

tests: factor a function out in express_pre.sh

Addresses a few format warnings from shellcheck and deduplicate the code.
While at it, get the variable right in testPHPExitCodeAndOutput() because
after commit a62537c it worked only by coincidence.

git-commit: address some shellcheck warnings

Use modern style command substitution and more double quotes.

[skip ci]

renderAttributes(): also display attribute ID

As suggested by Lucas Aimaretto on the racktables-users mailing list.

renderAttributes(): switch to renderTableViewer()

add RHEL 7 to README.md (GH#266)

Verified on RHEL 7.8.

fixup the RackTables web-site URL

Use HTTPS and a trailing slash to avoid unnecessary redirects.

[skip ci]

upgrade CodeMirror from 5.53.2 to 5.54.0

Add SNMP support for HPE 5130 (JG937A) and A5120 (JG237A)

dictionary: add recent Fedora and Ubuntu releases

[skip ci]

update some installation notes

Remove a couple Linux distributions that are more than 5 years old, add
entries for Fedora 32 and Ubuntu 20.04 based on MariaDB, spell MariaDB as
an alternative consistently.

[skip ci]

make release notes consistent after commit ccb8905

[skip ci]

add PHPUnit results cache to .gitignore

The results cache was introduced in PHPUnit 7.3 and became on by default
in PHPUnit 8. The feature makes no difference in a [single-use] Travis CI
environment, but can enable defects-first test order in a developer
working copy, in case anybody needs it. Hence let's keep it enabled and
ignore the cache file.

[skip ci]

upgrade CodeMirror from 5.49.2 to 5.53.2

[skip ci]

Travis CI: use the latest PHPUnit 7 for PHP 7.1

update COPYING for 2020

[skip ci]

Travis CI: add a job for PHP 7.4

tests: do not assume PHP CLI is always installed

switch some documentation URLs to HTTPS

[skip ci]

allow PHP 7.0 as an untested version

As discussed on the mailing list, Ubuntu 16.04 is still around. Let's
consider this again after Ubuntu 20.04 and PHP 7.4 are out.

[skip ci]

Travis CI: use default PHPUnit (8) for PHP 7.[23]

Also explain it better why PHP 7.0 is not in the matrix and PHP 7.1 uses
a custom PHPUnit version.

tests: list PHPUnit 8 as a known version

Leave versions 6 and 7 in place as they run the current tests fine.

tests: rewrite a test without assertArraySubset()

"assertArraySubset() is deprecated and will be removed in PHPUnit 9."

The replacement works with PHPUnit 6, 7 and 8.

tests: use an assertion, not @expectedException

The annotation is deprecated in PHPUnit 8 and causes lots of warnings.
The method (which is the recommended replacement) works since at least
PHPUnit 6.

tests: declare setup/teardown methods void

Prepare for one of the backward incompatible changes of PHPUnit 8:

PHP Fatal error:  Declaration of ...::setUpBeforeClass() must be
compatible with PHPUnit\Framework\TestCase::setUpBeforeClass(): void

However, with this change the tests still pass with PHPUnit 6 and 7.

tests: refine PHPUnit presence checks

Make express_phpunit.sh distinguish between unavailable PHPUnit (not
installed at all or PHPUNIT_BIN pointed to a file that lacks the
executable permission) and PHPUnit that is available, but has an
unsupported version. Direct the error messages to stderr while at it.

[skip ci]

drop support for PHP 5.x and 7.0

As proposed on the mailing list on 4th of July 2019.

add the usual sections for the next release

let mktemp choose the temporary directory

drop the props for PHPUnit 4 and 5

Travis CI: use Ubuntu 18.04 and PHP 7.[123]

At the time of writing PHP supports only the following versions:
* 7.1 (old stable)
* 7.2 (old stable)
* 7.3 (current stable)

This list is likely to change once PHP 7.4 is available, which seems to
be coming.

Also PHPUnit supports only the following versions:
* 7 (soon to be retired), which requires PHP 7.1
* 8, which requires PHP 7.2+

In Travis CI Ubuntu 18.04 comes with PHPUnit 8 by default, and supports
only PHP versions 7.1, 7.2 and 7.3. Switch the master branch to 18.04 to
make it more straightforward to rewrite (see commit f81030a) the tests
for PHPUnit 8.

Leave the maintenance-0.21.x branch on Ubuntu 16.04 with its unsupported
versions of PHP and PHPUnit.

0.21.4: cut the release

dictionary: add a few recent Linux releases

[skip ci]

Travis CI: specify Ubuntu 16.04 explicitly

At this time Travis CI defaults to Ubuntu 16.04, which has PHP versions
5.6, 7.0, 7.1, 7.2 and 7.3. On request it can also provide Ubuntu 18.04,
which has PHP versions 7.1, 7.2 and 7.3. Before Ubuntu 18.04 becomes the
default, nail 16.04 down in .travis.yml to avoid surprise test failures.

upgrade CodeMirror from 5.49.0 to 5.49.2

[skip ci]

Travis CI: bump PHPUnit 7 version up

upgrade CodeMirror from 5.48.2 to 5.49.0

[skip ci]

clarify the previous commit

[skip ci]

Add snmp syncing support for Arista DCS-7050SX-64 (#264)

In addition, this adds a new processor called 'arista-any-QSFP+' which should be
able to get interfaces named like the following:

Ethernet49/1
Ethernet49/2
Ethernet49/3
Ethernet49/4
Ethernet50/1
...

Travis CI: fix the build after Ubuntu upgrade

Travis CI had changed the default Ubuntu version from 14.04 to 16.04,
which had broken the following jobs:
* PHP 5.5 (because Travis does not supply a PHP 5.5 phpenv archive for
  Ubuntu 16.04)
* PHP 7.0 (because PHPUnit was upgraded from 6.4.3 to 7.5.0)
* PHP 7.1 (because PHPUnit was upgraded from 6.4.3 to 8.0.4)

Complete the change started in commit b3a99c9 and specify the latest
PHPUnit 6 for the PHP 7.0 job and the latest PHPUnit 7 for the PHP 7.1
job. Bump PHP 7.2 and 7.3 jobs to the latest PHPUnit 7 too. Remove PHP
5.5 from the build matrix.

Add snmp syncing support for Arista DCS-7050SX-64

In addition, this adds a new processor called 'arista-any-QSFP+' which should be
able to get interfaces named like the following:

Ethernet49/1
Ethernet49/2
Ethernet49/3
Ethernet49/4
Ethernet50/1
...

use vertical writing mode in the 802.1Q report

It used to be inconvenient to view the domain/VLAN matrix on setups with
many VLAN domains. I tried to switch the domain THs to vertical mode
about 7 years ago, but it didn't work well at that time. Nowadays it is
just a matter of a CSS property, it turns out.

* pi.css
* render8021QReport()

[skip ci]

upgrade CodeMirror from 5.46.0 to 5.48.2

[skip ci]

amend SQL to work on MySQL 8 (Mantis#1909)

ROW is a MySQL 8 keyword.

add sections for 0.21.4

[skip ci]

Travis CI: update custom PHPUnit from 7.5.7 to .12

Reduce hard-coding and make the .phar download conditional while at it.

0.21.3: cut the release

dictionary: add recent Fedora versions

[skip ci]

fix an XSS issue in an error message

The issue was reported by Diego Di Nardo.

Quick fix of formatting.

fix hook integration in Cacti and Munin pligins

Commit 011df35 introduced this issue. With "module=image" and either of
the plugins enabled RackTables failed to fail when "img" was set to an
invalid value. This was because the plugins' extension functions always
return TRUE to dispatchImageRequest(), which concluded the work was
always done and thus never raised the default exception. Make the return
in the plugins conditional to fix that.

authorize8021QChangeRequests(): amend PHP array syntax

See Mantis#1905. Although support for PHP 5.3 is not a goal, the rest of
PHP code uses traditional array syntax, so let's make it consistent.

add a log entry limit to the object page (Mantis#769)

* getConfigDefaults(): add OBJECTLOG_PREVIEW_ENTRIES
* install.php: idem
* upgrade.php: idem
* renderObject(): use the option to limit the amount of entries

upgrade CodeMirror from 5.45.0 to 5.46.0

[skip ci]

add sections for 0.21.3

dictionary: update OpenWrt entries

updateObjectAllocation(): fix two PHP messages

Saving an empty working copy would result in the following:

PHP Notice:  Undefined index: rackmulti in ophandlers.php on line 1225
PHP Warning:  Invalid argument supplied for foreach() in ophandlers.php
on line 1225

[skip ci]

0.21.2: cut the release

update COPYING for 2019

[skip ci]

use PHPUnit 7.5.7 instead of 8

Run all tests on all PHP versions again.

enable minimal CI testing of PHP 7.3

fix warnings specific to PHP 7.3

PHP Warning:  "continue" targeting switch is equivalent to "break". Did
you mean to use "continue 2"?

* jun10ReadInterfaceStatus()
* addMultiPorts()
* addRealServers()

upgrade CodeMirror from 5.43.0 to 5.45.0

[skip ci]

add missing quotes to shell scripts

Issue spotted by Mark Brugnoli-Vinten.

spell product numbers for older HP switch models

[skip ci]

fixup the previous commit

[skip ci]

add SNMP support for HP Switches (GH #252)

add SNMP support for Hp Procurve 5406zl  (Mantis#1485)
add SNMP support for HP A5120-24G (Mantis#1257)
add SNMP support for HP V1910-48G Switch JE009A (Mantis#1193)
add SNMP support for HP Procurve 2900 (Mantis#689)

git-commit: remove unnecessary backticks

[skip ci]

git-commit: make a few variables readonly

[skip ci]

tests: disregard PHPUnit 8 for now

In Travis CI PHP 7.2 now comes with PHPUnit 8, which seems to require
rewriting the tests in a way that is not compatible with older PHPUnit
versions. Introduce SKIP_PHPUNIT into the Travis CI matrix to avoid
running PHPUnit 8 on PHP 7.2 until RackTables code is ready (and
potentially to prevent running older PHPUnit versions on older PHP
versions if the switch happens).

fixup the previous commits

Update snmp.php

add SNMP support for Cisco Catalyst 3850-48T (Mantis#1169 and Mantis#1423)
add SNMP support for Cisco 871 (Mantis#1355)
add SNMP support for Cisco Catalyst Model WS-C3850-24T (Mantis#1701)
add SNMP support for Cisco Nexus 3064T (Mantis#1697)

Update dictionary.php

add SNMP support for Cisco Catalyst 3850-48T (Mantis#1169 and Mantis#1423)
add SNMP support for Cisco 871 (Mantis#1355)
add SNMP support for Cisco Catalyst Model WS-C3850-24T (Mantis#1701)
add SNMP support for Cisco Nexus 3064T (Mantis#1697)

function ios15ShortenIfName_real added to handle port-channel interfaces names correctly - ios15 does not support shorten "po" interface name in cli view

upgrade CodeMirror from 5.42.2 to 5.43.0

[skip ci]

remove more trailing spaces

* renderRackspace()
* snmp.php
* GetChildrenListTest.php
* LinkTriggerTest::setUpBeforeClass()
* StringInsertHrefsTest::setUpBeforeClass()

[skip ci]

ios15ReadLLDPStatus(): add a break for consistency

[skip ci]

fixup more code style

Put spacing right and do not use empty() to test for an empty string.

* hpprocurveN1178ReadLLDPStatus()
* hpprocurveN1178ReadInterfaceStatus()
* hpprocurveN1178ReadMacList()
* hpprocurveN1178Read8021QConfig()
* hpprocurveN1178TranslatePushQueue()

[skip ci]

fixup some code style

Remove excess braces, put indentation right and simplify a few if's.

* hpprocurveN1178ReadLLDPStatus()
* hpprocurveN1178ReadInterfaceStatus()
* hpprocurveN1178ReadMacList()
* hpprocurveN1178Read8021QConfig()
* hpprocurveN1178TranslatePushQueue()

[skip ci]

git-commit: say add/update in the default message

The script used to log "update <filename>" regardless if it was a new
file or a change to an existing file. To make it easier to follow the
history, have it log "add <filename>" in the former case. Replace a
long if() wrapper with an early return while at it.

[skip ci]

Update regexp in func hpprocurveN1178Read8021QConfig to match Trk interfaces on HP Procurve (#245)