Details
-
Bug
-
Resolution: Done
-
Major
-
jBPM 6.0.1.Final
-
None
Description
The CDI integration docs (Chapter 20) docs need to be updated to refer to the new CDI producer methods in JAASUserGroupCallbackImpl.java, since the old producer class cannot be used anymore.
Also, I had a problem getting Weld to recognize the new producer methods. I could confirm (via the debugger) that Weld loaded the alternate class containing the producer methods, but it refused to inject the JAAS backed UserGroupCallback.
I experimented and discovered I had to move the @Selectable marker annotation from the class header down to each of the producer methods. From the little I understand of Weld this should NOT have been necessary, as the @Selectable annotation is properly marked as being class and field level, but it was the only way I could get the new producer methods to work. But even my CDI-aware IDE (the amazing IntelliJ) would not recognize the producer methods until I marked them @Selectable, so I am uncertain if this is a Weld bug or a subtle consequence of the CDI spec. Can someone comment on this?
I will submit a pull request showing the patch, but if it indeed is a legitimate fix then the other UserGroupCallback producers will need the same fix.
I will also submit a pull request for the updated docs.