Getting fields through webservice on configuration plugins tab returns no fields

Webservice and subtabs are not working correctly together.

changed milestone to %FusionDirectory 1.4

added PJ1802-0188 plugin-webservice labels

We have 3 cases of subtabs in FusionDirectory:

1. Services in servicesManagement, this is a really specific case which should be left alone, and a specific webservice endpoint should be added if these needs to be accessed through it.
2. Fuse modules in "Argonaut Fuse" service, this is because each one can be (de)activated. It could be replaced by a combination of ObjectClassBooleanAttribute and managed attributes. It does not use multiPlugin and is highly specific.
3. Plugins small configuration sections in configuration Plugins tab.

(3) is the only use of multiPlugin class. It does not integrate correctly with the webservice.

I think we should modify multiPlugin to make it simpler and closer to simplePlugin behavior so that it integrates better. The idea is to switch ownership of all attributes and sections to multiPlugin instance right after constructor, and behave like a normal simplePlugin instance afterwards. This means that smallConfig classes will not be able to have code outside of constructor/getAttributesInfo, but none of them do anyway.

Once this is done the multiPlugin tab should behave fine through webservice without any modification to the webservice code.

mentioned in issue fd#6166 (closed)

mentioned in commit 75872989

mentioned in commit 72716414

added 3h of time spent at 2021-06-03

marked this issue as related to fd#6166 (closed)

mentioned in commit 2d6d6588

added 30m of time spent

added To Be Tested label

hello @cchilliet , What are the test we can do to check this, plase give some exemple

Cheers

added 5m of time spent

changed due date to August 05, 2021

@bmortier Checking that GET /fusiondirectory/rest.php/v1/types/configuration/pluginsConfigInLdap correctly returns section and fields list.

I just tested on my local instance and it works fine, if that is enough for you we can close this.

added 5m of time spent

removed due date

@cchilliet ok

closed

removed To Be Tested label

added Fixed label