Fixing codestyle

This reverts commit 46b03343.

Fixing codestyle

issue #6122

Checks, LDAP base and admin creation were broken

issue #6122

This replaces save_object and execute methods by 3 methods:
readPost - Reads POST data
update - Update object state
render - Render HTML UI

The point is to avoid reading POST and rendering HTML when this is not
 needed (when doing stuff through the webservice for instance).

It’s also more consisent across FD with all classes handling some kind
 of dialog implementing the new interface FusionDirectoryDialog which
 makes sure these 3 methods are implemented.

issue #6072

issue #6054

issue #6046

No more global_un_set, global_delete or delete

issue #6024

No more global_un_set, global_delete or delete

issue #6024

issue #5955

Basic fixes like identation

issue #5955

issue #5956

Not directly related but we might as well clean this one up at the same time.

issue #5951

issue #5919

Signed-off-by: Benoit Mortier <benoit.mortier@opensides.be>

issue #5854

issue #5854

Resolve "Config insertion LDAP errors from setup are not shown"

See merge request fusiondirectory/fd!313

(cherry picked from commit f7036396)

43a2e138  fix(setup) Show errors encountered when saving config from setup

Resolve "Config insertion LDAP errors from setup are not shown"

See merge request fusiondirectory/fd!313

(cherry picked from commit f7036396)

43a2e138  fix(setup) Show errors encountered when saving config from setup

Also tidy up the code a bit.

issue #5850

issue #5831

Move this trailing comment on the previous empty line
Remove the useless trailing whitespaces at the end of this line

Signed-off-by: Benoit Mortier <benoit.mortier@opensides.be>

Move this trailing comment on the previous empty line

Signed-off-by: Benoit Mortier <benoit.mortier@opensides.be>

Fixes #4824 Removed support for tabs in config file, allowing several calls to loadPlist with less problems

Fixes #4824 Removed support for tabs in config file, allowing several calls to loadPlist with less problems

Conflicts:
	include/class_CopyPasteHandler.inc
	include/simpleplugin/class_simplePlugin.inc

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>

Conflicts:
	include/class_acl.inc
	include/simpleplugin/class_simplePlugin.inc
	plugins/admin/groups/class_groupManagement.inc