Fix logic inn detecting if OpenVPN resync needed

Commit https://github.com/pfsense/pfsense/commit/f33dcc5c79c54af7daf91a81cfdd7f489e8cb67c reversed the logic sequence when testing if $resync_needed - the individual tests were changed from "==" to "!=" and so on, but the conjunction also need to be changed - "or" needs to be "and". I had noticed that VPNs on some gateway groups of mine didn't failover recently, but hadn't gone looking for the problem until now.
This might help bug #3243 - it will probably now make the OpenVPN resync on every interface/gateway change for the cases of CARP VIPs and VLANs mentioned in that bug report. At least that will be better than not resyncing at all.
Check for disabled OpenVPN instances in openvpn_resync_if_needed

It is much cleaner if we check first for disabled OpenVPN instances, before diving into all the other checks. Note, the final openvpn_resync() does call other routines that finally check if the instance is disabled, but there are so many checks here for various conditions to be met before calling openvpn_resync that it looks better (safer) to bail out early if the instance is disabled.
Added OVH DynHOST in dynamic DNS services
Remove redundant test for OpenVPN resync_needed
Add Captive Portal Zones privileges definition. Fix #3216
make dashboard responsive
some tidying up
Merge pull request #819 from CharlieMarshall/responsive

make dashboard responsive (theme pfsense_ng_fs)
Merge pull request #817 from stephanel/master

Added OVH DynHOST in dynamic DNS services
Fix #3268 - avoid pf table names conflict:

. Create a list of reserved table names for the hardcoded ones
. Use this list to validate aliases and load balance pool names
. Check if alias names don't conflict with LB pool names and vice-versa
Make sure vlan interface exist when it's being configured, it fixes #3270
Fix #3242 and some code cleanup:

- Only explode '/' and set address_subnet when address is a subnet, it
  fixes issue reported at #3242
- While I'm here, do some cleanup on the way addresses are treated
  - Remove unecessary variable $tracker, we already have $counter set
    with the same value
  - Use if:endif and while:endwhile where there is html block inside the
    block
  - Use more meaning names for variables instead of item, item2, item3
    and item4
  - Use while($counter <= $count()) instead of using foreach referencing
    array using $counter inside
Actually there is no reason to set a variable just to use once
Prevent a possible division by zero. it fixes #3212
Fix #3273

- When you disable a interface, it destroys vlan interface from system.
Do not report error when interface doesn't exist.
- While I'm here, use pfSense_interface_destroy() instead of ifconfig
Fix #3259. Save 'packet loss rate' and 'bucket size' for limiter queues
Limit CIDR choices for IPv4 on GRE interface, fixes #3277
Set id for select elements created dynamicaly created
Fix #3283, use jQuery to change attributes based on id
Optimize DHCPv4 lease display online status for static leases. Do not re-parse complete ARP table for each lease, as it can be slow with large ARP tables.