Merge branch 'maintenance-0.20.x' into yandex-prod

Revert "refine a few assertions to test for 'inet4'"

This reverts commit 27ed1188b2c7a0685215c9df3638abd9155200ae.
Those functions are IPv6-aware.

Revert "refine a few assertions to test for 'inet4'"

This reverts commit 27ed1188b2c7a0685215c9df3638abd9155200ae.
Those functions are IPv6-aware.

empty() isn't always the best tool, pt. 2

Use count() to clarify the argument is an array.

* renderVLANInfo()
* formatVLANPackDiff()
* renderIPForObject()

renderEditVlan(): simplify some code

init_database_static(): remove a useless test

Since get_pseudo_file() was switched to return an array instead of plain
text there should normally be no empty elements in the array. But if
there is a bug, don't hide it and just let PDO fail the normal way.

empty() isn't always the best tool, pt. 1

To test for an empty string it does not require a function.

* updateIPNameAJAX()
* commitAddPort()
* commitUpdatePort()
* getConfiguredQuickLinks()
* formatPortLink()

updateAddress(): simplify a boolean expression

tests: cover matchVLANFilter() and filterVLANList()

tests: cover goodModeForVSTRole()

simplify trigger_8021q_configured()

use "&&" and "||" instead of "and" and "or"

This change prevents unexpected side effects, that, for instance, used
to require two workarounds in iptree_markup_collapsion() and one in
transformRequestData(). The root cause of the problem is that "and" and
"or" are standalone operators: though their associativity is the same as
that of "&&" and "||", the order of precedence is not just different but
is lower than that of "=". The code style article now explains this.

* syncdomain.php
* getNetUsageAJAX()
* authenticate()
* permitted()
* gotClearanceForTagChain()
* authenticated_via_ldap()
* queryLDAPServer()
* RackCodeParser::accept()
* RackCodeParser::un_expr()
* checkAutotagName()
* findCtxModWarnings()
* listCells()
* fillNetKids()
* amplifyCell()
* commitLinkEntities()
* commitUpdateEntityLink()
* updateAddress()
* generateEntityAutoTags()
* createIPv4Prefix()
* newPortForwarding()
* getDomainVLANs()
* getDomainVLANList()
* upd8021QPort()
* setConfigVar()
* setUserConfigVar()
* xos12ReadLLDPStatus()
* xos12ReadInterfaceStatus()
* vrpReadLLDPStatus()
* ftos8ReadLLDPStatus()
* eos4ReadLLDPStatus()
* ros11ReadLLDPStatus()
* jun10TranslatePushQueue()
* jun10Read8021QConfig()
* ftos8ReadInterfaceStatus()
* eos4ReadInterfaceStatus()
* ucsReadInventory()
* isInnoDBSupported()
* platform_is_ok()
* stringTrace()
* assertUIntArg()
* assertStringArg()
* assertBoolArg()
* genericAssertion()
* rectHeight()
* highlightObject()
* ip4_mask()
* ip6_mask()
* l2addressForDatabase()
* sortTokenize()
* pokeNode()
* peekNode()
* addTraceToNodes()
* redirectIfNecessary()
* getObjectiveTagTree()
* getCellFilter()
* IPNetworkCmp()
* iptree_fill()
* ip_next()
* ip_prev()
* treeApplyFunc()
* getTerminalNetworks()
* iptree_markup_collapsion()
* getPortListPrefs()
* getUnlinkedPortTypeOptions()
* groupIntsToRanges()
* array_values_same()
* buildVLANFilter()
* matchVLANFilter()
* generate8021QDeployOps()
* filter8021QChangeRequests()
* getEmployedVlans()
* get8021QSyncOptions()
* exec8021QDeploy()
* saveDownlinksReverb()
* initiateUplinksReverb()
* recalc8021QPorts()
* authorize8021QChangeRequests()
* compareDecomposedPortNames()
* searchEntitiesByText()
* buildSearchRedirectURL()
* setMessage()
* isEthernetPort()
* getConfigVar()
* getObjectTypeChangeOptions()
* apply8021qChangeRequest()
* fillIPNetsCorrelation()
* fillIPSpareListBstr()
* isIPNetworkEmpty()
* listConstraint()
* checkTypeAndAttribute()
* init.php
* install.php
* not_already_installed()
* init_config()
* get_process_owner()
* check_config_access()
* init_database_dynamic()
* render8021QOrderForm()
* renderVLANDomainListEditor()
* renderVLANDomainVLANList()
* renderObject8021QPorts()
* getAccessPortControlCode()
* renderTrunkPortControls()
* renderVLANInfo()
* renderObject8021QSync()
* renderObject8021QSyncPreview()
* renderConfigMainpage()
* renderTagRowForEditor()
* printNiftySelect()
* transformRequestData()
* getRenderedIPv4NetCapacity()
* serializeTags()
* printTagTRs()
* renderEntitySummary()
* niftyString()
* showLogoutURL()
* renderRackspace()
* renderRow()
* printObjectDetailsForRenderRack()
* renderEditRackForm()
* renderObject()
* showMessageOrError()
* renderRackSpaceForObject()
* renderIPSpaceRecords()
* renderIPSpace()
* renderIPNetwork()
* renderIPAddress()
* renderIPAddressProperties()
* renderNATv4ForObject()
* renderLocationPage()
* renderEditLocationForm()
* renderLivePTR()
* buildTagCheckboxRows()
* renderCellFilterPortlet()
* renderFile()
* renderFilesPortlet()
* printIPNetInfoTDs()
* renderCell()
* getFilePreviewCode()
* renderDiscoveredNeighbors()
* switchportInfoJS()
* addIPAllocation()
* processGridForm()
* updateObjectAttributes()
* addLotOfObjects()
* save8021QPorts()
* updVSTRule()
* importDPData()
* cleanupUCS()
* getOpspec()
* buildOpspecColumns()
* findSparePorts()
* handlePopupPortLink()
* detectDeviceBreed()
* prepareRealServersTable()
* SLBTriplet::getTriplets()
* MacroParser::addMacro()
* addRStoRSPool()
* commitUpdateRS()
* doSwitchSNMPmining()
* createTrueColorOrThrow()
* renderProgressBar4Image()
* renderImagePreview()
* proxyStaticURI()
* trigger_liveports()
* trigger_object_8021qorder()
* trigger_8021q_configured()
* trigger_anyDP()
* triggerCactiGraphs()
* triggerMuninGraphs()
* getDBUpgradePath()
* renderUpgraderHTML()
* index.php

refine getDomainVLANs()

Assign $self at top and dismiss a variable.

renderProgressBarImage(): use array_fetch()

prepareNavigation(): use array_fetch()

saveScript(): refine error handling

See also commit 0a4f411.

tests: cover validTagName()

decrease direct use of $_REQUEST and $sic, pt. 18

* dispatchImageRequest()

splitNetworkByMask(): fixup formatting

test for empty strings w/o strlen(), pt. 8

* authenticate()
* permitted()
* queryLDAPServer()
* index.php

detectDeviceBreed: better detection of 'jun10'

tests: add express.sh and use it in Travis CI

The script provides a convenient way of running the same set of tests
both in Travis CI and in a working copy. Add per-file tests using the
PHP interpreter.

refine getLocationTrail()

Use implode() and call listCells() only when the static array is empty.

loadUserConfigCache(): refine error handling

Don't specify a default value as in this case missing argument is an
error and there is no gain in suppressing the PHP warning. Explain the
exception.

test for empty strings w/o strlen(), pt. 7

* loadUserConfigCache()
* getRenderedAlloc()
* renderEditRowForm()
* printObjectDetailsForRenderRack()
* renderEditObjectForm()
* renderEditRackForm()
* renderRackInfoPortlet()
* renderObject()
* renderPortsForObject()
* renderIPNetwork()
* renderIPAddress()
* renderIPAddressProperties()
* renderNATv4ForObject()
* renderEditLocationForm()
* renderLivePTR()
* buildTagCheckboxRows()
* renderCellFilterPortlet()
* renderFileSummary()
* printIPNetInfoTDs()
* renderCell()
* renderRouterCell()
* showTabs()
* dynamic_title_decoder_throwing()
* renderDiscoveredNeighbors()
* formatIfTypeVariants()
* formatPortReservation()

Changelog line for 2ca58c

Merge pull request #148 from dmage/maintenance-0.20.x

Use Cmd+Click to inverse tag selection on OS X

Travis CI: also test syncdomain.php

syncdomain.php: make --help a valid option

This way it exits with the return code 0.

Use Cmd+Click to inverse tag selection on OS X

Signed-off-by: Oleg Bulatov <dmage@yandex-team.ru>

Nexus 7700 models added into dictionary

test for empty strings w/o strlen(), pt. 6

* renderObject8021QPorts()
* renderVLANInfo()
* renderObject8021QSyncPreview()
* getImageHREF()
* serializeTags()
* niftyString()
* makeIPAllocLink()
* queryTerminal()
* callScript()
* getUpgradeBatch()
* getDatabaseVersion()
* showUpgradeError()
* renderUpgraderHTML()

decrease direct use of $_REQUEST and $sic, pt. 17

* addRow()
* updateRack()
* addObjectlog()

refine more SQL queries

Request specific columns like in commit af13d78.

* SLBTriplet::getTriplets()
* addSLBPortLink()
* addSLBIPLink()

tests: cover ip_get_arpa()

test for empty strings w/o strlen(), pt. 5

* MacroParser::expand()
* addRStoRSPool()
* commitUpdateRS()
* commitUpdateVS()
* concatConfig()

ros11TranslatePushQueue(): fix Marvell ROS as well

That had been copied from the JunOS code (where it applies) too.

eos4TranslatePushQueue: vlan removing from Arista

A vlan was removed from switch only if it was named. Despite the name
check, it was still removed using the VID, so this check seems
completely useless. Removed it.

rectify makeWhereSQL()

Use implode(), validate input arguments and don't prepend the result
with two extra spaces. Update the tests.

tests: cover makeWhereSQL()

test for empty strings w/o strlen(), pt. 4

* formatLoggedSpan()
* ios12PickSwitchportCommand()
* fdry5PickInterfaceSubcommand()

tests: cover 3 script functions

* saveScript()
* loadScript()
* deleteScript()

test for empty strings w/o strlen(), pt. 3

* assertStringArg()
* assertBoolArg()
* findAllEndpoints()
* extractLayout()
* redirectIfNecessary()
* getCellFilter()
* treeApplyFunc()
* considerGivenConstraint()
* scanRealmByText()
* serializeVLANPack()
* getRackCode()
* guessTableStructure()
* universalOpHandler()
* listConstraint()
* formatEntityName()
* getMgmtProtosConfig()

fixup a few more plurals

* showMessageOrError()

addMultiPorts(): fix a PHP warning

PHP Warning:  Missing argument 7 for commitUpdatePort(), called in
/usr/local/racktables/wwwroot/inc/ophandlers.php on line 935

use mkCellA() even more

This is another follow-up to commit 6d8424f.

* renderDiscoveredNeighbors()
* addMultipleObjects()

rectify adding of new objects

Update renderAddMultipleObjectsForm() supply and extra form parameter as
MASSCOUNT may be different when the form is generated and when it is
processed. Update addMultipleObjects() to use that and dismiss some
unnecessary code.

test for empty strings w/o strlen(), pt. 2

* addPortForwarding()
* addMultiPorts()
* addIPAllocation()
* updateUser()
* addLotOfObjects()
* addRealServers()
* renderVS()
* getPopupSLBConfig()
* updateStickerForCell()
* doSwitchSNMPmining()
* doPDUSNMPmining()

addMultiPorts(): handle invalid input better

When some of the lines in the multi-line input have format errors,
instead of stopping the processing and displaying an exception page
just show a standard one-line message for each such line.

tests: cover serializeVLANPack()

tests: update README and convert to Markdown

test for empty strings w/o strlen(), pt. 1

* commitUpdateObject()
* getRackspaceStats()
* generateEntityAutoTags()
* saveScript()
* deleteScript()
* init.php
* renderSLBEntityCell()
* renderRSPool()
* renderRealServerList()
* renderVirtualService()

decrease direct use of $_REQUEST and $sic, pt. 16

* getPortInfoAJAX()
* index.php

decrease direct use of $_REQUEST and $sic, pt. 15

* updateIPNameAJAX()
* updateIPCommentAJAX()
* updateCableIdAJAX()

tests: cover mergeTagChains()

printNiftySelect(): dismiss an unused parameter

importDPData(): amend exception class

The data in this case comes from the HTTP request hence use IRAE to
signal malformed input.

decrease direct use of $_REQUEST and $sic, pt. 14

* updatePortRsvAJAX()
* importDPData()
* buildOpspecColumns()

nextMACAddress(): add input validation

Use a PCRE of its own as RE_L2_IFCFG matches a wider class of strings.
Add more tests.

move processGridForm() to where it belongs

tests: cover nextMACAddress()

tests: cover l2addressForDatabase()

tests: cover ip4_mask() and ip6_mask()

tests: one more for HTTPDateToUnixTime()

punctuation

decrease direct use of $_REQUEST and $sic, pt. 13

* resolve8021QConflicts()

suppress more unnecessary arguments

decrease direct use of $_REQUEST and $sic, pt. 12

* querySNMPData()
* addOIFCompatPack()
* delOIFCompatPack()
* del8021QOrder()
* unlinkPort()
* clearVlan()
* deleteVlan()

fixup a few plurals

* renderVLANDomainVLANList()
* renderEditVlan()
* renderPortsForObject()
* clearVlan()
* deleteVlan()
* renameObjectPorts()
* renderEditRSPool()
* renderEditVService()
* renderEditVS()

add a missing licensing header

factor interface-8021q.php out

Move interface functions below to a feature-specific file:

* render8021QOrderForm()
* render8021QStatus()
* renderVLANDomainListEditor()
* renderVLANDomain()
* renderVLANDomainVLANList()
* get8021QPortTrClass()
* renderObject8021QPorts()
* getAccessPortControlCode()
* getTrunkPortCursorCode()
* renderTrunkPortControls()
* renderVLANInfo()
* renderVLANIPLinks()
* renderObject8021QSync()
* renderObject8021QSyncSchedule()
* renderObject8021QSyncPreview()
* renderObject8021QSyncPorts()
* renderVSTListEditor()
* renderVSTRules()
* renderVST()
* renderVSTRulesEditor()
* renderDeployQueue()
* formatVLANPackDiff()
* renderEditVlan()

refine a couple SQL queries

Use proper capitalization and request specific columns.

* fetchIPv4AddressNetworkRow()
* fetchIPv6AddressNetworkRow()

decrease direct use of $_REQUEST and $sic, pt. 11

* addLocation()
* deleteRack()
* cleanRack()
* cloneRSPool()

renderEditRSPool(): suppress unnecessary arguments

No need to repeat what getOpLink() already does anyway.

clarify the search query error message

Don't throw an exception in searchHandler() as this function is not an
ophandler and there's no special handling for it besides the last-resort
exception trap, which prints only a minimalistic page. Instead handle
the situation locally and present the user with a standard message bar
on the main page.

refine exception class in newPortForwarding()

Throw InvalidArgException and modify addPortForwarding() to convert to
InvalidRequestArgException.

simplify addLotOfObjects() further

Omit unnecessary "else" after "return" and unnecessary "continue" at the
cycle end, use mkCellA(), reword an error message.

factor TEXTAREA input filtering out

Replace some of the earliest bits of code with a much cleaner function.
Add some tests.

* textareaCooked(): new function
* addMultiPorts(): update
* addLotOfObjects(): update
* addRack(): update

tests: cover iosParseVLANString()

tests: cover listToRanges()

tests: cover buildVLANFilter()

update a comment for buildVLANFilter()

decrease direct use of $_REQUEST and $sic, pt. 10

* save8021QPorts()

rectify the code for rack adding

* renderNewRackForm(): update to signal the request origin with just one
  argument and make it mandatory
* addRack(): reflect the change above, decrease direct use of $_REQUEST,
  clean TEXTAREA processing up, use mkCellA() in status messages

add a Travis CI configuration file

add type guards to array_values_same()

Unless both arguments to array_diff() are arrays, the function returns
an empty array as if the arguments were arrays and the first one was a
subset of the second. In this case (though there would be a PHP
warning) array_values_same() would return TRUE to indicate no
differences between the arguments.

Add a few tests.

tests: make the IAE test n-ary

This avoids the hassle of maintaining a separate test/provider method
pair for every value of n.

refine a few assertions to test for 'inet4'

In the functions below the assertions would be satisfied with an either
IPv4 or IPv6 address whereas those functions are strictly IPv4-specific.
Nail it down to IPv4.

* addRealServer()
* addVService()
* updateRealServer()
* updateVService()

renderRackspaceRowEditor(): add a missing TD

decrease direct use of $_REQUEST and $sic, pt. 9

* updateObjectAllocation()
* save8021QPorts()

rectify acceptable8021QConfig()

Fix the 'trunk' case to actually look into 'native' and 'allowed'. This
provides better error detection for manual 802.1Q synchronization. Add
some tests.

update a few functions to use isCheckSet()

* processGridForm()
* mergeGridFormToRack()
* updateLocation()

renderEditLocationForm(): add a LABEL

Changelog line for commit d3dd017a

fix LLDP info for Cisco IOS in cases when remote name is too long:
 separate known interface names from remote names

decrease direct use of $_REQUEST and $sic, pt. 7

* addPortForObject()
* addRack()

fix a PHP warning after commit 19f4c98

"Missing argument 3 for array_fetch()..."

* setConfigVar()
* setUserConfigVar()
* resetUserConfigVar()

decrease direct use of $_REQUEST and $sic, pt. 6

* addPortForwarding()
* delPortForwarding()
* updPortForwarding()

factor NATv4 protocol options out

$natv4_proto: a global list of NATv4 protocol options
renderNATv4ForObject(): feed the new list into printSelect()
genericAssertion(): add a new 'enum/natv4proto' assertion type

newPortForwarding(): clarify error messages

Was: "Argument 'remoteip_bin' of value is invalid (Non-existent ip)."
Now: "Argument 'remoteip_bin' of value '192.168.0.1' is invalid (address
does not belong to a known network)."