Skip to content

Commit f41c5bc

Browse files
Fixes
1 parent 9371f2e commit f41c5bc

File tree

4 files changed

+7
-10
lines changed

4 files changed

+7
-10
lines changed

src/main/java/main/Session.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
import main.model.dto.*;
1313
import main.model.email.TestRunEmails;
1414

15-
import java.util.ArrayList;
15+
import java.util.Collections;
1616
import java.util.List;
1717
import java.util.stream.Collectors;
1818

@@ -27,6 +27,7 @@ public Session(String sessionId) throws AqualityException {
2727
controllerFactory = new ControllerFactory(user);
2828
}
2929

30+
@Deprecated
3031
public Session(String importToken, int projectId) throws AqualityException {
3132
user = new UserDto();
3233
controllerFactory = new ControllerFactory(user);
@@ -35,9 +36,7 @@ public Session(String importToken, int projectId) throws AqualityException {
3536
projectUser.setProject_id(projectId);
3637
projectUser.setEngineer(1);
3738
projectUser.setViewer(1);
38-
List<ProjectUserDto> projectUsers = new ArrayList<>();
39-
projectUsers.add(projectUser);
40-
user.setProjectUsers(projectUsers);
39+
user.setProjectUsers(Collections.singletonList(projectUser));
4140
controllerFactory = new ControllerFactory(user);
4241
}
4342
}

src/main/java/main/model/db/dao/project/UserDao.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import org.apache.commons.codec.binary.StringUtils;
1313

1414
import java.util.ArrayList;
15+
import java.util.Collections;
1516
import java.util.Date;
1617
import java.util.List;
1718

@@ -90,9 +91,7 @@ private UserDto IsAuthorizedToken(String sessionId) throws AqualityException {
9091
projectUser.setProject_id(projectId);
9192
projectUser.setViewer(1);
9293
projectUser.setEngineer(1);
93-
List<ProjectUserDto> projectUsersList = new ArrayList<>();
94-
projectUsersList.add(projectUser);
95-
user.setProjectUsers(projectUsersList);
94+
user.setProjectUsers(Collections.singletonList(projectUser));
9695
user.setApiSessionProjectId(projectId);
9796
return user;
9897
}

src/main/java/main/model/dto/TestResultDto.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
import java.util.Date;
1111

12-
@ Data@EqualsAndHashCode(callSuper = true)
12+
@Data @EqualsAndHashCode(callSuper = true)
1313
public class TestResultDto extends BaseDto {
1414
private String internalTestId;
1515
@DataBaseName(name="request_project_id")

src/main/java/main/view/BaseServlet.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,9 @@ public class BaseServlet extends HttpServlet{
2525
protected DtoMapperGeneral mapper = new DtoMapperGeneral();
2626

2727
protected Session createSession(HttpServletRequest req) throws AqualityException {
28-
String apiToken = getStringQueryParameter(req, "apiToken");
2928
String importToken = getStringQueryParameter(req, "importToken");
3029
Integer projectId = getIntegerQueryParameter(req, "projectId");
31-
if ((apiToken != null || importToken != null) && projectId != null) {
30+
if (importToken != null && projectId != null) {
3231
return new Session(importToken, projectId);
3332
} else {
3433
return new Session(getSessionId(req));

0 commit comments

Comments
 (0)