Skip to content

fix : AnnotationStatus of Data not changing appropriatly when submitting #238

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 19, 2024

Conversation

chanyoung1998
Copy link
Contributor

i made a pull request of #234 .
When you have a time for reviewing it, leave me a comment for it! Thank you!

I tested this code in my local develop server and it works appropriatly as you mentioned!

@chanyoung1998
Copy link
Contributor Author

chanyoung1998 commented Apr 19, 2024

@fanyinbo ,

if (ObjectUtil.isNull(sceneId)) {
  ~ 
}else if (ObjectUtil.isNotNull(sceneId) && DataStatusEnum.INVALID.equals(status)) {
 ~ 
} else if (ObjectUtil.isNotNull(sceneId) &&  DataStatusEnum.VALID.equals(status)) {
 ~
}

you suggested like this conditional block in #234. when i tested this code, then Once the scene marked as INVALID, there is no way to enter else if (ObjectUtil.isNotNull(sceneId) && DataStatusEnum.VALID.equals(status)) this conditional block. So i think
if (dataInfoDAO.count(lambdaQueryWrapper) == 0) looks more acceptable! Could you please let me know what you think about it?

@jaggerwang jaggerwang merged commit 973ee1c into xtreme1-io:main Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants