Agila
  1. Agila
  2. AGILA-23

Initial Support for Actor Functionality

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: BPM Engine
    • Labels:
      None

      Description

      Inital working draft of support for "actor" assignments from business process definitions. All current code hard-codes task assignments in the Task code and ignores the /process-model/graph/nodes/node/actors/actor element. This code provides one working approach to supporting this originally intended but unimplemented functionality.

      The basic approach here can be summarized as follows:

      • Define a new service: ActorResolverService responsible for interpreting the simple Actor name declared in the process. An instance implementing this interface is given to the TaskService as part of its configuration.
      • Modify the TaskService interface to add a method assignTaskToActors which accepts an Actor[] and recursively resolves all references to either a UserID or GroupID and creates tasks for each resolved entity.

      An alternative to modifying the TaskService would be to force each concrete Task to handle the resolution and assignment itself. I have initially rejected this approach because it seems out of the scope of concern of the task to handle this resolution, especially when the time comes to implement instance-level fields as actors (such as "self" or "initiator").

      Note that this patch removes the special actor "Self" from the Leave Application sample which is not yet implemented (and apparently not supported by the engine yet)

      1. ActorResolverServiceImpl.java
        1 kB
        Geoff Howard
      2. ActorResolverService.java
        1 kB
        Geoff Howard
      3. actor.patch
        14 kB
        Geoff Howard
      4. actor.patch
        12 kB
        Geoff Howard

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        5d 15h 6m 1 Chris Lim 27/Nov/05 21:12
        Resolved Resolved Closed Closed
        120d 54m 1 Geoff Howard 27/Mar/06 23:07
        Geoff Howard made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Geoff Howard added a comment -

        Thanks for applying the patch quickly. Didn't realize I had forgotten to close the issue out.

        Show
        Geoff Howard added a comment - Thanks for applying the patch quickly. Didn't realize I had forgotten to close the issue out.
        Chris Lim made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Chris Lim added a comment -

        Patch applied.

        Show
        Chris Lim added a comment - Patch applied.
        Geoff Howard made changes -
        Attachment ActorResolverServiceImpl.java [ 12320852 ]
        Geoff Howard made changes -
        Attachment ActorResolverService.java [ 12320851 ]
        Chris Lim made changes -
        Assignee Chris Lim [ clim ]
        Geoff Howard made changes -
        Attachment actor.patch [ 12320848 ]
        Hide
        Geoff Howard added a comment -

        Updated patch with required change in servlet, additional typo fix, and removed "System.out" call. Replaces previous attachment entirely.

        Show
        Geoff Howard added a comment - Updated patch with required change in servlet, additional typo fix, and removed "System.out" call. Replaces previous attachment entirely.
        Geoff Howard made changes -
        Field Original Value New Value
        Attachment actor.patch [ 12320842 ]
        Geoff Howard created issue -

          People

          • Assignee:
            Chris Lim
            Reporter:
            Geoff Howard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development