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

Error 500 when opening bug details: Property or field 'appId' cannot be found #447

Closed
johan12345 opened this issue Aug 30, 2023 · 4 comments
Labels

Comments

@johan12345
Copy link

Next issue after #446 is fixed: When I want to look at one of the reported bugs, I get an error 500 with the following stack trace:

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'bugView' defined in file [/application/BOOT-INF/classes/com/faendir/acra/ui/view/bug/BugView.class]: Failed to instantiate [com.faendir.acra.ui.view.bug.BugView]: Constructor threw exception
        [...]
        at com.faendir.acra.ui.component.grid.LayoutPersistingFilterableGrid$addOnClickNavigation$1$1.invoke(LayoutPersistingFilterableGrid.kt:76) ~[classes/:2.0.0-beta04]
        [...]
Caused by: java.lang.IllegalArgumentException: Failed to evaluate expression 'returnObject == null || hasViewPermission(returnObject.appId)'
        [...]
        at com.faendir.acra.persistence.bug.BugRepository$$SpringCGLIB$$0.find-VSoB3VY(<generated>) ~[classes/:2.0.0-beta04]
        at com.faendir.acra.ui.view.bug.BugView.<init>(BugView.kt:46) ~[classes/:2.0.0-beta04]
        [...]
Caused by: org.springframework.expression.spel.SpelEvaluationException: EL1008E: Property or field 'appId' cannot be found on object of type 'com.faendir.acra.persistence.bug.Bug' - maybe not public or not valid?
        [...]
@johan12345
Copy link
Author

similar issues (`Property or field 'appId' cannot be found) also occur on other actions, e.g. when trying to delete a version.

@F43nd1r
Copy link
Owner

F43nd1r commented Aug 30, 2023

Okay so in a past release method security wasn't working at all. It seems now it is active but not updated for current usage patterns so I guess I gotta make a bigger check

@F43nd1r F43nd1r added the bug label Aug 30, 2023
@potatoru
Copy link

potatoru commented Sep 1, 2023

I can report the same issue on v2.0.0-beta04.

@F43nd1r F43nd1r closed this as completed in ade38f5 Sep 2, 2023
@F43nd1r
Copy link
Owner

F43nd1r commented Sep 2, 2023

I found a great testing library (https://github.com/mvysny/karibu-testing) to replace vaadin testbench (which I had to remove in the past because I did not have access to a license anymore).

Implementing tests for all pages will take a while, but the specific issues mentioned here are tested and fixed now.

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

No branches or pull requests

3 participants