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

NPE prevents reviewing further findings #393

Open
salsolatragus opened this issue May 25, 2018 · 2 comments
Open

NPE prevents reviewing further findings #393

salsolatragus opened this issue May 25, 2018 · 2 comments
Assignees

Comments

@salsolatragus
Copy link
Collaborator

When I upload 22 findings to a review site (ex2) that has default_ex2_review_size = 20, I cannot review finding 21, because there's a NPE

[2018-05-24 14:04:03] mubench.CRITICAL: Error: Call to a member function getReview() on null in .../src/Controllers/ReviewsController.php:51

The NPE occurs here. It's the logic that determines the next/current/previous misuse that returns null for the current misuse. Interestingly, when I set default_ex2_review_size = 21, I can successfully review finding 21, but I (still) cannot review finding 22, so somehow the default impacts the querying here.

@joschli
Copy link
Collaborator

joschli commented Jun 18, 2018

How exactly do you try to access the review page of the finding when you get the error ?
I can only reproduce this error by changing the url to a misuse that isn't shown on my detector page.

@salsolatragus
Copy link
Collaborator Author

  • Fix passing of ex2_review_size in guest mode
  • Fix navigation for "outside of range" (either back -> n - 1, forward -> 1, or back = null, forward = null)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants