Skip to content

Commit cb2489d

Browse files
committed
Merge remote-tracking branch 'origin/master' into refactor/#83
2 parents 6df4aab + 123708f commit cb2489d

File tree

1 file changed

+3
-3
lines changed
  • src/main/java/com/moplus/moplus_server/domain/problem/domain/problem

1 file changed

+3
-3
lines changed

src/main/java/com/moplus/moplus_server/domain/problem/domain/problem/Problem.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -165,12 +165,10 @@ public void deleteChildProblem(Long childProblemId) {
165165

166166
public boolean isValid() {
167167
return problemCustomId != null
168-
&& practiceTestId != null
169168
&& problemType != null
170169
&& title != null && !title.getTitle().isEmpty()
171170
&& answer != null && !answer.getValue().isEmpty()
172171
&& difficulty != null && difficulty.getDifficulty() != null
173-
&& memo != null && !memo.isEmpty()
174172
&& mainProblemImageUrl != null && !mainProblemImageUrl.isEmpty()
175173
&& mainAnalysisImageUrl != null && !mainAnalysisImageUrl.isEmpty()
176174
&& mainHandwritingExplanationImageUrl != null && !mainHandwritingExplanationImageUrl.isEmpty()
@@ -180,7 +178,9 @@ public boolean isValid() {
180178
&& prescriptionImageUrls.stream().allMatch(url -> url != null && !url.isEmpty())
181179
&& answerType != null
182180
&& conceptTagIds != null && !conceptTagIds.isEmpty()
183-
&& recommendedTime != null;
181+
&& recommendedTime != null
182+
&& recommendedTime.getMinute() != null && recommendedTime.getMinute() >= 0
183+
&& recommendedTime.getSecond() != null && recommendedTime.getSecond() >= 0;
184184
}
185185

186186
public String getTitle() {

0 commit comments

Comments
 (0)