5.1p Javadoc: DSRequest.setUserRoles(String userRoles) missing

[Blama]

Hi Isomorphic,

I'm revisiting this thread and wanted to use the advice there.
I noticed while there is RPCManager.setUserRoles(List) and RPCManager.setUserRoles(String), there is only DSRequest.setUserRoles(List). The String version is available in Eclipse, but not documented.
But I'm pretty sure it is fine to use it. If so, could you please add it to the Javadoc?

Best regards
Blama

[Isomorphic]

We made a change yesterday to address this need (should be present in the latest nightly build dated December 22).

Regards
Isomorphic Software

[Blama]

Hi Isomorphic,

thanks for adding. I don't rely on this, but please note that the RPCManager version of the String-method is:

public void setUserRoles(java.lang.String rolesString)
Accepts a comma-separated String representing the list of roles associated with the user who is authenticated for this request....

The DSRequest version is:

public DSRequest setUserRoles(java.lang.String[] roles)
Set the user roles associated with this request.

Is this really correct? This has no priority for me, but I find it suspicious.

Best regards
Blama