UnivNautes

Clarify bracketing

to minimize risk of a problem when adding code here in the future.

Allow extended alias inputs #3890

Currently if you enter a space-separated list of subnets in the IP address box when entering an alias, the code reports that the data is invalid. But it does actually expand the list of subnets into multiple rows, and enters the various subnet CIDRs into the CIDR column for the user. The user can press Save a second time and the data is now valid so the code saves it happily. This is rather odd, as reported in redmine #3890....

firewall_aliases_edit UI text changes

If type URL Table then the heading "Description" on the 3rd column gets suppressed (I am not really sure why that is, since the description data entry box still appears - I guess someone intended that the data entry box itself also be suppressed, since URL Table takes just a single line entry, the overall description of the alias should be enough - no need for a per-line description.)...

Remove useless check for alias description matching an interface description

While looking at other checks in the code I noticed this check. It was not effective anyway, because the first line inside "if ($_POST)" below does
unset($input_errors);
which undoes this check anyway....

Be more strict on removing groups checking group id and group name, it avoids issues like happened to users on ticket #3856. While I'm here, replace GET by POST

Be more strict on user removal checking array id and also username to avoid removing wrong users when browser back button is used. It should fix #3856

Fix close button in the info box

Merge pull request #1287 from jean-m-cyr/master

Provide a toggle for apinger debug messages to be logged to syslog. To help with roubleshooting issues

NTP Service GPS page always reverts to 'Custom' GPS type

Remember and correctly display GPS type setting