Skip to content
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

Đoạn check thừa trong API logs #34

Closed
Belikhun opened this issue Nov 28, 2019 · 1 comment
Closed

Đoạn check thừa trong API logs #34

Belikhun opened this issue Nov 28, 2019 · 1 comment
Assignees
Labels
backend Process data~! bug Something isn't working help wanted Extra attention is needed
Milestone

Comments

@Belikhun
Copy link
Owner

Belikhun commented Nov 28, 2019

Mô tả lỗi
Đoạn check thừa khi thay đổi tình trạng chấm từ Đang Chấm tới Chấm Xong
Gây ra lỗi item Đang Chấm không được loại bỏ khi bài đã chấm xong

Để gây ra lỗi
Các bước thực hiện để gây ra lỗi:

  1. Nộp bài làm

Hành vi dự kiến
Khi chấm xong một bài, tình trạng Đang Chấm phải bị loại bỏ

Ảnh chụp màn hình
không

Thông tin thêm
Tại /api/contest/logs.php:96

if ($item["name"] === $data["file"]["name"] && file_exists($log) && (int)$item["lastmtime"] < (int)filemtime($log))

Đoạn check cần được loại bỏ:

(int)$item["lastmtime"] < (int)filemtime($log))
@Belikhun Belikhun added bug Something isn't working help wanted Extra attention is needed backend Process data~! labels Nov 28, 2019
@Belikhun Belikhun self-assigned this Nov 28, 2019
@Belikhun Belikhun added this to the v0.5.4 milestone Nov 28, 2019
Belikhun added a commit that referenced this issue Dec 18, 2019
remove unnecessary check
@Belikhun
Copy link
Owner Author

fixed in commit 1bdf9b8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Process data~! bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant