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.

Sign up for GitHub

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

Jump to bottom

NPE If a null Fact is injected through annotation #139

Closed
bpoussin opened this issue Mar 1, 2018 · 1 comment
Closed

NPE If a null Fact is injected through annotation #139

bpoussin opened this issue Mar 1, 2018 · 1 comment
Assignees
@bpoussin
Labels
bug
Milestone
3.2.0

Comments

@bpoussin
Copy link
Contributor

bpoussin commented Mar 1, 2018

When we use annotation in pojo for argument and in facts i push null for one or more fact, there is NPE when easy-rules try to call method.

This issue is corrected in PR #137
bpoussin added a commit to bpoussin/easy-rules that referenced this issue Mar 1, 2018
@bpoussin
add test for issue j-easy#139 (NPE If i add Fact with null value and …
7b7113b 
…use annotation)
fmbenhassine pushed a commit that referenced this issue Mar 27, 2018
@bpoussin @fmbenhassine
add test for issue #139 (NPE If i add Fact with null value and use an…
62f49a7 
…notation)
@fmbenhassine fmbenhassine added this to the v3.2 milestone Mar 27, 2018
@fmbenhassine
Copy link
Member

Hi @bpoussin

Thank you for reporting and fixing this issue 👍 Congrats on your First PR!
I added you to the contributors list: cf66bd6

I'm releasing v3.2 soon. Your fix will be included. Stay tuned 😉

Kind regards,
Mahmoud

@fmbenhassine fmbenhassine changed the title NPE If i add Fact with null value and use annotation NPE If a null Fact is injected through annotation Apr 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bpoussin bpoussin

Labels
bug
Milestone
3.2.0
Development

No branches or pull requests
2 participants
@fmbenhassine @bpoussin