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

some filters do not work with each other, addFilterCriterion error for certain mixes #1968

Open
smutniak opened this issue Oct 25, 2024 · 3 comments · May be fixed by #1982
Open

some filters do not work with each other, addFilterCriterion error for certain mixes #1968

smutniak opened this issue Oct 25, 2024 · 3 comments · May be fixed by #1982
Labels

Comments

@smutniak
Copy link
Contributor

Following a filter on Status with one for Last Name throws 500 error. Filtering on Last Name followed by a filter on Status does not. Following the Last Name filter with a Status filter followed by a Graduation Semester (List) filter does throw an error.

@smutniak smutniak added the bug label Oct 25, 2024
@cstarcher
Copy link
Member

java.lang.NullPointerException: null
at org.tdl.vireo.controller.SubmissionListController.addFilterCriterion(SubmissionListController.java:237) ~[classes/:na]
at org.tdl.vireo.controller.SubmissionListController$$FastClassBySpringCGLIB$$36c0d017.invoke() ~[classes/:na]

@smutniak
Copy link
Contributor Author

smutniak commented Jan 2, 2025

While this ticket does have somewhat of a workaround it is the most frequent issue I get zendesk tickets for. Can the priority be bumped up a bit for the next sprint?

@jsavell
Copy link
Contributor

jsavell commented Jan 14, 2025

@smutniak Could you test this branch to see if it resolves this issue?

https://github.com/TAMULib/Vireo/tree/predicate-null-check

It sounds similar to something I was working on based on local issue reports.

@jsavell jsavell linked a pull request Jan 23, 2025 that will close this issue
@cstarcher cstarcher moved this from Todo to Review in Vireo Development Sprints Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Review
Development

Successfully merging a pull request may close this issue.

3 participants