Merged WISE-436-fixErrorChangingRoleInSharing into develop

This commit is contained in:
Paulo Gustavo Veiga 2015-04-12 16:11:36 -03:00
commit 96af3564f7
3 changed files with 12 additions and 6 deletions

View File

@ -412,17 +412,17 @@ public class MindmapController extends BaseController {
throw new IllegalArgumentException(roleStr + " is not a valid role");
}
// Remove from the list of pendings to remove ...
if (collaboration != null) {
collabsToRemove.remove(collaboration);
}
// Is owner ?
final CollaborationRole role = CollaborationRole.valueOf(roleStr.toUpperCase());
if (role != CollaborationRole.OWNER) {
mindmapService.addCollaboration(mindMap, restCollab.getEmail(), role, restCollabs.getMessage());
}
// Remove from the list of pendings to remove ...
if (collaboration != null) {
collabsToRemove.remove(collaboration);
}
}
// Remove all collaborations that no applies anymore ..

View File

@ -37,10 +37,12 @@ import javax.xml.bind.annotation.XmlRootElement;
isGetterVisibility = JsonAutoDetect.Visibility.PUBLIC_ONLY)
public class RestCollaboration {
private long id;
private String email;
private String role;
public RestCollaboration(@NotNull Collaboration collaboration) {
this.id = collaboration.getId();
this.email = collaboration.getCollaborator().getEmail();
this.role = collaboration.getRole().getLabel();
}
@ -59,6 +61,10 @@ public class RestCollaboration {
}
public long getId() {
return id;
}
public String getRole() {
return role;
}

View File

@ -192,7 +192,7 @@ $(function () {
success:function (data, textStatus, jqXHR) {
// Owner roles is the first in the table ...
var collabs = data.collaborations.sort(function (a, b) {
return a.role <= b.role;
return a.id > b.id;
});
// Add all the columns in the table ...