Activiti
  1. Activiti
  2. ACT-1079

TimerEntity#getPersistentState() should include duedate field

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.8
    • Fix Version/s: 5.9
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Similar to MessageEntity, TimerEntity should also contains this method override:

        public Object getPersistentState() {
          Map<String, Object> persistentState = (Map<String, Object>) super.getPersistentState();
          persistentState.put("duedate", getDuedate());
          return persistentState;
        }
      

      The sql 'updateTimer' in Job.xml already includes the duedate, it's just part of the persistentState map in the code.

      I need to programmatically change the duedate and even though I can modify the duedate on the TimerEntity, the change does not trigger a flush to the DB with the new value.

        Activity

        Hide
        Daniel Meyer (camunda) added a comment -

        Actually I stumbled over this also, yesterday. I already added it in
        http://fisheye.codehaus.org/changelog/activiti/?cs=3150

        We can remove it from MessageEntity then, though.

        Show
        Daniel Meyer (camunda) added a comment - Actually I stumbled over this also, yesterday. I already added it in http://fisheye.codehaus.org/changelog/activiti/?cs=3150 We can remove it from MessageEntity then, though.
        Show
        Daniel Meyer (camunda) added a comment - http://fisheye.codehaus.org/changelog/activiti/?cs=3155
        Hide
        Etienne Studer added a comment -

        Great!

        Show
        Etienne Studer added a comment - Great!
        Hide
        Joram Barrez added a comment -

        Nice catch!

        Show
        Joram Barrez added a comment - Nice catch!

          People

          • Assignee:
            Daniel Meyer (camunda)
            Reporter:
            Etienne Studer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: