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)

fixup the previous commit

[skip ci]

add SNMP support for HP switches (GH #243)

add SNMP support for HP 1810-24G (J9450A) (Mantis#1379)
add SNMP support for HP 1810-8G (J9449A) (Mantis#1377)
add SNMP support for HP 1810-24G v2 (J9803A) (Mantis#1285)

git-commit: clarify the usage for -a and -d

[skip ci]

fixed typo for breed io15 in function queryTerminal

tests: fixup the previous commit

tests: make the express test sequence safer

git-commit: try to clean up on a commit failure

[skip ci]

git-commit: factor print_error() out

[skip ci]

git-commit: add a check for FILEPATH

Test that the file is not outside of the repository directory. While at
it, move the non-empty check into git_commit_or_exit() as it actually
belongs there.

[skip ci]

git-commit: add support for -M (replace|append)

[skip ci]

git-commit: fixup the help text for -u

[skip ci]

git-commit: print the usage text on -h only

On an option error instead of printing the full text only suggest the -h
option as that is more appropriate for stderr.

[skip ci]

git-commit: fixup issues from previous commits

[skip ci]

upgrade CodeMirror from 5.41.0 to 5.42.2

[skip ci]

git-commit: add an early check for -o

spell the product number for HP 1810G-24

[skip ci]

git-commit: add -v for git verbosity level

Make it possible for the calling PHP code to receive the git command(s)
output on request. Make the current quiet mode the default to preserve
some backward compatibility.

git-commit: add an error check for mkdir

[skip ci]

snmp: add support for a variant of HP JE006A (Mantis #1293)

HP JE006A is a 3Com 3CRBSG2893.

[skip ci]

git-commit: add -h for "help"

[skip ci]

git-commit: add more error checks

Test that the provided <repodir> is good before making any changes. Test
that git is available. Test that "git add" and "git commit" worked.

Move the default value for COMMITMSG right into the git command. Update
the usage text.

[skip ci]

spell the product number for HP 1910-24G

Clarify the end result of the following series of commits (in
chronological order):

e7705e2a add HP ProCurve 1910-24G
8df12894 finalize the previous commit
8db81368 dict: remove duplicate item HP ProCurve 1910-24G
e6562cdb SNMP: fix dict_key for HP 1910-24G

[skip ci]

git-commit: implement -d for git commit --date

[skip-ci]

git-commit: make the actions more fine-grained

Introduce an optional -o ("only") option to request only a specific
step of the pull-commit-push workflow. This is useful for updating many
files in a batch -- any changes will still end up one commit per file,
but the workflow will be pull-commit-commit-...-commit-commit-push, which
will not multiply the delay accessing a slow git remote before and after
every attempted file.

Without the option the script will work same as before this change.

[skip ci]

git-commit: move the code into smaller functions

[skip ci]

fixup the previous commit

Put HP JG311A ports right and fix some regexp issues.

[skip ci]

Update snmp.php

add SNMP support for HP 1950 48G 2SFP+ 2XGT Switch (Mantis#1599)
add SNMP support for HPE FlexFabric 5900AF 48XG 4QSFP+ Switch (Mantis#1841)
add SNMP support for HPE 5500-24G-4SFP Switch (Mantis#1839)
add SNMP support for HP A5800AF-48G Switch with 2 Processors (JG225A) (Mantis#1843)
add SNMP support for HP 1810-8G v2 (J9802A) Switch (Mantis#1287)

Update dictionary.php

add SNMP support for HP 1950 48G 2SFP+ 2XGT Switch (Mantis#1599)
add SNMP support for HPE FlexFabric 5900AF 48XG 4QSFP+ Switch (Mantis#1841)
add SNMP support for HPE 5500-24G-4SFP Switch (Mantis#1839)
add SNMP support for HP A5800AF-48G Switch with 2 Processors (JG225A) (Mantis#1843)
add SNMP support for HP 1810-8G v2 (J9802A) Switch (Mantis#1287)

callScript(): fixup an error message

[skip ci]

git-commit: quote $SUDOUSER too

git-commit: support custom author for the commit

git-commit: switch to getopts

This breaks backward compatibility, but makes it easier to add new
options in future.

git-commit: prepend all messages with own filename

gateways/git-commit: handle new files better

The script works as expected updating existing files, but to add new
files to the repository things need to be done slightly differently.

[skip ci]

display file comment in the SELECT (Mantis#1419)

* getAllUnlinkedFiles(): include the beginning of the comment into the
  result and return the result as it is
* renderFilesForEntity(): add a round to build the printSelect() input

simplify getFilesOfEntity()

squeeze some blank lines

[skip ci]

syncdomain.php: just use basename()

fixup indentation in snmp.php

[skip ci]

refine platform tests further

In platform_is_ok() convert the PHP extension tests to declarations and
describe the impact of a failed test iff the test is not critical for the
core RackTables features to work in the default configuration. Ibid,
display the impact of an optional test in the output table when it fails
(this did not work before because platform_function_test() ignored one of
its parameters). Ibid, for the test just use extension_loaded(). Remove
platform_function_test(), which now has no purpose.

[skip ci]

platform_is_ok(): add a non-critical test for cURL

[skip ci]

simplify renderObjectHistory()

Just use renderTableViewer(), this among other things yields a better
formatted table.

[skip ci]

add basic integrity checks for config variables

Use the configuration defaults data in two new checks in
renderDataIntegrityReport(). One of the checks immediately fails because
the data in getConfigDefaults() did not include ENABLE_BULKPORT_FORM, so
fix that right away.

make the configuration defaults data shareable

Move the array from resetUIConfig() to a new function,
getConfigDefaults(). Place the new function in dictionary.php as that is
the best way to make it available in the installer (which still does not
use it though). Update renderUIResetForm() to use a new opcode that
works better with the hardcoded require_once in index.php.

tests: update RenderReportsTest

Assert that the integrity report prints the "all clear" message.

Remove some dead code: the class props are no longer required since
commit 4829189; the test condition is always true because
renderLocalReports() does not exist since commit df8739f.

renderTableViewer(): quote the TD class value

This way specifying more than one class in the calling function isn't
more difficult than it should be.

[skip ci]

make RE_L2_SOLID uppercase-only

This is a fixup to commit 97792a8.

Update dictionary.php

add SNMP support for HP Aruba 2530 48 PoE+ Switch and HP Aruba 2530 24 PoE+ Switch (Mantis#1641)

Update snmp.php

add SNMP support for HP Aruba 2530 48 PoE+ Switch and HP Aruba 2530 24 PoE+ Switch (Mantis#1641)

renderObject(): align some TDs in the NATv4 tables

[skip ci]

eliminate a global variable

Rewrite addPageHeader() so it does not need the helper array.

rename getObjectClass() to getCellClass()

This completes the change started in the previous commit -- the function
input may come from realms other than 'object'. Update the functions
below.

* printObjectDetailsForRenderRack()
* renderRack()
* renderDepot()
* renderIPSpaceRecords()
* renderAtomGrid()
* renderCell()
* renderRouterCell()
* renderSLBEntityCell()

fix colour handling in user account search results

setEntityColors() caused the following log messages when the search
results included any local user accounts:

PHP Notice:  Undefined index: etags in .../inc/interface-lib.php on
line 1344
PHP Warning:  Invalid argument supplied for foreach() in
.../inc/interface-lib.php on line 1344

This was caused by getAccountSearchResult() constructing and returning
a mock user cell structure, which through the search results eventually
ended up in renderCell(). The latter used to have a workaround for that
specific case (see commit 573214e), but only after the call to
setEntityColors(), which needs the 'etags' data, even when it is an
empty list.

Remove the workaround from renderCell() and remove the unnecessary
custom code from getAccountSearchResult(), which now just calls
spotEntity() like other similar search functions. That delivers 'etags'
to renderCell(), which now has input for getObjectClass(), which now
yields another log message:

PHP Notice:  Undefined index: id in .../interface-lib.php on line 1302

To fix that update getObjectClass() to use the proper index for user
cells like mkCellA() does.

[skip ci]

replace includeJQueryUI() with 2 nullary functions

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]

spell PHP FALSE in upppercase

[skip ci]

deprecate escapeString()

[skip ci]

fixup the previous commit

Move RE_STATIC_URI from solutions.php to functions.php and address
assorted non-critical issues:

* isUri(): use the same regexp as the chrome module does (update the
  tests respectively)
* addPageHeader(): a new helper function with the common code
* addJSExternal(): simplify by using the above function
* addJSInternal(): idem
* addJSText(): idem
* addCSSExternal(): idem
* addCSSInternal(): idem
* addCSSText(): idem
* checkColumnOfRadios8021Q(): update to make a clean call to addJSText()
* renderRackCodeViewer(): idem
* renderRackCodeEditor(): idem
* addParentNodeOptionsJS(): idem
* getOptionTree(): idem
* renderLocationFilterPortlet(): idem

Put some lettercase and punctuation right and add some missing comments
while at it.

enable CSS and JavaScript from external URLs

tests: default to PHPUnit versions 6 and 7

Versions 4 and 5 still work (that's what is in Travis CI PHP versions
5.5 and 5.6 respectively) with the right bootstrap file, but they are
not available anymore on the system where I happen to run phpunit
without express.sh.

[skip ci]

rename VIRTUAL_OBJ_LISTSRC to VIRTUAL_OBJ_CSV

After commit 8dc2a6c the value is no longer a RackCode string, so change
the name for clarity.

improve the object NATv4 tab trigger

* getNATv4CountForObject(): a new helper function
* trigger_natv4(): update to work like trigger_ip(), i.e. keep the tab
  available so long as the object has associated items

Add some tests while at it.

replace remaining niftyString() occurrences

This seems to be the last commit in the series, the previous one was
commit a0bdf6d.

* formatVLANAsLabel()
* formatVLANAsPlainText()
* formatVLANAsShortLink()
* formatPatchCableHeapAsPlainText()
* dynamic_title_decoder_throwing()

[skip ci]

fix allObjectLogs() after the previous commit

Make the <BR> work again, though keep escaping the user name.

[skip ci]

fix object log TD escaping after commit 274f160

In allObjectLogs() it is the comment column that renderTableViewer()
ought not escape, not the date/user column.

[skip ci]

tests: use myString() and myStringStatic() more

* EmptySQLWhereTest
* LinkTriggerTest
* RenderDepotTest
* TagTreeCircularReferenceTest

tests: skip symlinks in tests/

This removes bootstrap.php from the list and the totals.

tests: check init.php for syntax errors

In express.sh move the checks code out to functions and leave the
simplified main loops to decide which check to run and to count the
errors. Have init.php checked at least syntax-wise.

While at it, update a comment and increase the format width to
accommodate the recently added long file name.

drop support for $localreports

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().

add and use RTTestCase::myStringStatic()

Even after the introduction of RTTestCase::myString() some tests
continued to use fixed strings for the database props naming in their
static setup methods. When such a test failed in a way that prevented the
teardown method to run, that would fail subsequent attempts to run the
setup method until the props were manually removed or the unit test
database contents was reset.

Introduce a static counterpart of the existing solution and update the
following classes to use it:

* EntityLinkTriggerTest
* GetChildrenListTest
* ObjectCircularReferenceTest
* ObjectLogTest

renderTagStats(): remove an unused global

fixup some language in the installer messages

* not_already_installed()
* check_config_access()

[skip ci]

gateways/git-commit: accept a full commit message

The script used to prepend the first line to the provided message, which
isn't always convenient. Change that and make the message parameter
optional.

[skip ci]

renderServerConfigurationReport(): handle InnoDB failures better

The InnoDB test fails trying to create a trigger when binary logging is
enabled in the database server (in particular, that is required for the
old-style master-slave replication setup). This used to result in an
unhandled PDOException, which effectively blinded all other tests.

Use the exception to generate a separate specific error message and to
skip the InnoDB test only.