Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

Commit d65f619

Browse files
authored
Merge pull request #452 from appirio-tech/dev
fix for adding member to direct project problem (#448)
2 parents f7ffe1c + e6d5d5e commit d65f619

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/java/main/com/topcoder/direct/services/view/action/project/edit/SaveCockpitProjectSettingAction.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -731,14 +731,14 @@ public String saveProjectPermissionsAndNotifications() {
731731

732732
boolean add = true;
733733
long permissionTypeId;
734-
735-
if (pp.getPermission().toLowerCase().equals("full")) {
734+
735+
if ("full".equalsIgnoreCase(pp.getPermission())) {
736736
permissionTypeId = PermissionType.PERMISSION_TYPE_PROJECT_FULL;
737-
} else if (pp.getPermission().toLowerCase().equals("write")) {
737+
} else if ("write".equalsIgnoreCase(pp.getPermission())) {
738738
permissionTypeId = PermissionType.PERMISSION_TYPE_PROJECT_WRITE;
739-
} else if (pp.getPermission().toLowerCase().equals("read")) {
739+
} else if ("read".equalsIgnoreCase(pp.getPermission())) {
740740
permissionTypeId = PermissionType.PERMISSION_TYPE_PROJECT_READ;
741-
} else if (pp.getPermission().toLowerCase().equals("report")) {
741+
} else if ("report".equalsIgnoreCase(pp.getPermission())) {
742742
permissionTypeId = PermissionType.PERMISSION_TYPE_PROJECT_REPORT;
743743
} else {
744744
// remove

0 commit comments

Comments
 (0)