diff --git a/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSharingController.java b/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSharingController.java
index 3894966..8965dd7 100644
--- a/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSharingController.java
+++ b/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSharingController.java
@@ -37,19 +37,11 @@ public class WorkspaceSharingController {
 	private DialogShareWItem sharingDialog = null;
 
 
-//	/**
-//	 * This dialog doesn't show setting permissions, it use default server permissions
-//	 * @param workspace itemId to share
-//	 */
-//	public WorkspaceSharingController(String itemId) {
-//	}
-//	
-	
 	/**
-	 * This dialog show setting permissions
+	 * This controller instancing sharing dialog
 	 * @param itemId workace item id
-	 * @param shareOnlyOwner if true, only owner can share
-	 * @param defaultPermission ACL_TYPE default permission
+	 * @param if true, only owner can share, otherwise an alert with an error message is displayed
+	 * @param defaultPermission ACL_TYPE default permission, if is null default ACL_TYPE is loaded from server
 	 */
 	public WorkspaceSharingController(String itemId, boolean shareOnlyOwner, ACL_TYPE defaultPermission) {
 		this.workspaceItemId = itemId;
diff --git a/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSmartSharingController.java b/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSmartSharingController.java
index d599aa9..0cddd7c 100644
--- a/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSmartSharingController.java
+++ b/src/main/java/org/gcube/portlets/widgets/workspacesharingwidget/client/WorkspaceSmartSharingController.java
@@ -36,7 +36,7 @@ public class WorkspaceSmartSharingController {
 
 	/**
 	 * 
-	 * @param file a fake file (used to display name of the file or folder)
+	 * @param file a fake file (used to display name of the file or folder) to display the field name
 	 * @param listAlreadySharedContact
 	 */
 	public WorkspaceSmartSharingController(FileModel file, List<CredentialModel> listAlreadySharedContact) {