Make mandatory IPs a configuration option in the systems plugins

Description

It bugs me is having the IP field mandatory only in some cases, we should put it either mandatory for all systems or none, or have an option in the configuration back-end.

If it’s non-mandatory by default we can make it mandatory when some tabs are activated (argonautClient for instance needs it).

FusionDirectory Version

1.3-dev

Plugin with the defect

plugin-systems

PHP version used

5.6

Origin of php packages

debian

Steps to Reproduce

1. Create a system
2. try to save without an ip
3. doesnt work

Expected behavior:

• Make it non mandatory by default, code wise.

• Have an option in configuration back-end for each of the plugin thats need ip address.

• Active the option by default, to not change the fonctional usage of people already using it but allowing people to change it if needed.

Actual behavior:

ip is mandatory on all systems and we cannot change that

Reproduces how often:

100%

Additional Information

Make it a configuration option since we changed this too many times already.

Multi-valuated configuration field listing the types for which IP is mandatory.

Note that ipHost have the field ipHostNumber as mandatory so we’ll need to add code to remove this objectClass when the field is empty. This is already what we are doing for phones.