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.

mentioned in issue #5702 (moved)

changed the description

created branch 5705-add-ip-non-mandatory-in-the-systems-plugins

mentioned in commit 43fbc8a2

added 2h of time spent

added To Be Tested label

mentioned in issue #5707 (closed)

Hello,

tested on workstation and server, works correctly

Cheers

closed

added Changed and removed technical discussion labels

removed To Be Tested label

changed title from add ip non mandatory in the systems plugins to Add ip non mandatory in the systems plugins

mentioned in issue #5767 (closed)

changed title from Add ip non mandatory in the systems plugins to Make mandatory IPs a configuration option in the systems plugins