[JBoss JIRA] Commented: (JBPM-445) getActorId in ProcessLog broken
by Stefan Grinsted (JIRA)
[ http://jira.jboss.com/jira/browse/JBPM-445?page=comments#action_12421319 ]
Stefan Grinsted commented on JBPM-445:
--------------------------------------
This issue apparently didn't solve itself during the last 12 months.
Please reopen this issue.
Thanks, Stefan
> getActorId in ProcessLog broken
> -------------------------------
>
> Key: JBPM-445
> URL: http://jira.jboss.com/jira/browse/JBPM-445
> Project: JBoss jBPM
> Issue Type: Bug
> Components: Core Engine
> Affects Versions: jBPM 3.0.1, jBPM 3.0.2, jBPM 3.1 alpha 2
> Reporter: michael lawley
> Assigned To: Tom Baeyens
>
> ProcessLog.getActorId() always delegates to its parent (if any) and the comment says that getActorId is overridden in AuthenticationLog. However there is no AuthenticationLog and there is no subclass of ProcessLog that overrides getActorId so this method _always_ returns null.
> Perhaps a better approach would be to grav the actorId using Authentication.getAuthenticatedActorId(); in the ProcessLog constructor. Naturally this would require adding a new instance variable to ProcessLog.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 6 months
[JBoss JIRA] Created: (JBPM-1201) Resurrect MemLeakTest
by Thomas Diesler (JIRA)
Resurrect MemLeakTest
---------------------
Key: JBPM-1201
URL: http://jira.jboss.com/jira/browse/JBPM-1201
Project: JBoss jBPM
Issue Type: Task
Security Level: Public (Everyone can see)
Reporter: Thomas Diesler
[15:54:19] Thomas Diesler: there seems to be a circular dependency in modules identity and jpdl/jar
[15:54:42] ... how should it be?
[15:54:52] Tom Baeyens: good question
[15:55:20] ... jpdl jar should not need identity
[15:55:38] ... but for building the suite, the identity is needed
[15:55:40] Thomas Diesler: yes, I found that
[15:55:54] Tom Baeyens: building the suite should be in dist
[15:56:01] Thomas Diesler: yes, I found that too
[15:56:24] ... I am swithcing the build to maven
[15:56:32] Tom Baeyens: where did you find the circular dep ?
[15:56:52] Thomas Diesler: can't the identity tests live in module identity?
[15:57:19] Tom Baeyens: are there identity tests in jpdl/jar ?
[15:57:30] Thomas Diesler: yes
[15:57:48] Tom Baeyens: probably there is some reason i can't remember now
[15:57:59] ... i'll see if i can quickly re-find it
[15:58:19] Thomas Diesler: ok, but identity depends on jpdl-core, right?
[15:58:32] ... don't worry
[15:58:40] ... I'll sort it out
[15:59:30] Tom Baeyens: MemLeakTest ?
[15:59:50] Thomas Diesler: yes
[16:00:17] Tom Baeyens: just delete the test
[16:00:19] ... :-)
[16:00:46] ... it's a kind of integration test
[16:01:00] ... which i guess is not part of the continuous integration
[16:01:05] Thomas Diesler: with all the other props and stuff?
[16:01:24] Tom Baeyens: other props ?
[16:01:46] Thomas Diesler: next to the test are lots of config files
[16:01:52] Tom Baeyens: right
[16:02:01] Thomas Diesler: delete them too?
[16:02:04] Tom Baeyens: maybe it makes sense to put the perf package into a separate module
[16:02:28] ... you can delete all of it
[16:02:41] ... but in case we ever want to do performance tests
[16:02:51] ... then those could provide a good start
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
16 years, 6 months