Bug #98977 for RT-Extension-SLA: Due and Priority changes in TransactionBatch after SLA modification

Fri Sep 19 03:18:45 2014 andrius.kulbis [...] gmail.com - Ticket created

Subject:	Due and Priority changes in TransactionBatch after SLA modification
Date:	Fri, 19 Sep 2014 10:18:30 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

I noticed that after modifying SLA CustomField value, the Due and Priority are changed in TransactionBatch not TransactionCreate mode. [58456] [Fri Sep 19 07:10:16 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) Set,Status,SetWatcher,CustomField for txn #1787231 on ticket #120949 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [58456] [Fri Sep 19 07:10:16 2014] [debug]: Skipping Scrip #249 because it didn't Prepare (/opt/rt4/sbin/../lib/RT/Scrips.pm:361) [58456] [Fri Sep 19 07:10:16 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) Set for txn #1787236 on ticket #120949 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [58456] [Fri Sep 19 07:10:16 2014] [debug]: Skipping Scrip #249 because it didn't Prepare (/opt/rt4/sbin/../lib/RT/Scrips.pm:361) [58456] [Fri Sep 19 07:10:16 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) Set for txn #1787237 on ticket #120949 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [58456] [Fri Sep 19 07:10:16 2014] [debug]: Skipping Scrip #249 because it didn't Prepare (/opt/rt4/sbin/../lib/RT/Scrips.pm:361) The first line shows the batch of transactions (changed subject, status, owner, some customfields - also SLA). Third and fifth lines show the Set - Due and Set - Priority transactions, fired by an SLA change. If I compare the Due and Priority values in process of first and fifth lines, they are different. Should it act like so? Because I can't work out the "AfterAllTicketChanges" scrip. Regards.

Fri Sep 19 09:29:04 2014 falcone [...] bestpractical.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Fri, 19 Sep 2014 09:28:53 -0400
To:	Andrius Kulbis via RT <bug-RT-Extension-SLA [...] rt.cpan.org>
From:	Kevin Falcone <falcone [...] bestpractical.com>

On Fri, Sep 19, 2014 at 03:18:45AM -0400, Andrius Kulbis via RT wrote: Show quoted text

> I noticed that after modifying SLA CustomField value, the Due and > Priority are changed in TransactionBatch not TransactionCreate mode.

RT-Extension-SLA doesn't ship TransactionBatch rt-extension-sla (master=)$ ag -i batch rt-extension-sla (master=)$ Your logs don't make terribly much sense. I can tell you're trying to explain something, but you don't see to be using the core module. Show quoted text

> If I compare the Due and Priority values in process of first and fifth > lines, they are different. Should it act like so? Because I can't work > out the "AfterAllTicketChanges" scrip.

What Scrip is this? rt-extension-sla (master=)$ ag -i AfterAllTicketChanges rt-extension-sla (master=)$ -kevin

Fri Sep 19 09:29:05 2014 The RT System itself - Status changed from 'new' to 'open'

Fri Sep 19 10:16:31 2014 andrius.kulbis [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Fri, 19 Sep 2014 17:16:25 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

I want to make "AfterAllTicketChanges" scrip, which will work in TransactionBatch mode, and fire after all changes to the ticket was made on update off any ticket information. Now, when I change the SLA field, it changes the the Due and the priority of the ticket accordingly. The problem is that the Due and Priority change happens after my "AfterAllTicketChanges" scrip fires. I understand that the TransactionBatch mode scrips are commited after all TransactionCreate Scrips are done. For example: I change the status and the subject of the ticket in one update. Should the scrip in batch mode run after both of these changes are done and the transactions recorded? Or maybe I understand it wrong ant the scrip in Batch mode run after each different transaction? So technicaly should it be like this: *SCENARIO #1* 1. SLA change => transaction #1, 2. Run Batch scrip for transaction #1 3. Change Due because SLA changed => transaction #2 4. Run Batch scrip for transaction #2 5. Change Priority because SLA changed => transaction #3 6. Run Batch scrip for transaction #3 Or like this: *SCENARIO #2* 1. SLA change => transaction #1, 2. Change Due because SLA changed => transaction #2 3. Change Priority because SLA changed => transaction #3 4. Run Batch scrip for transactions in batch (#1, #2, #3) ? Because now I get it working like SCENARIO #1, where in step #2 and step #6, the values of Due and Priority are different, but the value of the SLA is the same (the one I changed to) in both steps. Sorry for the mess in thoughts, I am not native English speaker. 2014-09-19 16:29, falcone@bestpractical.com via RT rašė: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=98977 > > > On Fri, Sep 19, 2014 at 03:18:45AM -0400, Andrius Kulbis via RT wrote:

>> I noticed that after modifying SLA CustomField value, the Due and >> Priority are changed in TransactionBatch not TransactionCreate mode.

> RT-Extension-SLA doesn't ship TransactionBatch > > rt-extension-sla (master=)$ ag -i batch > rt-extension-sla (master=)$ > > Your logs don't make terribly much sense. I can tell you're trying to > explain something, but you don't see to be using the core module. > >

>> If I compare the Due and Priority values in process of first and fifth >> lines, they are different. Should it act like so? Because I can't work >> out the "AfterAllTicketChanges" scrip.

> What Scrip is this? > > rt-extension-sla (master=)$ ag -i AfterAllTicketChanges > rt-extension-sla (master=)$ > > -kevin >

Fri Sep 19 11:06:20 2014 falcone [...] bestpractical.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Fri, 19 Sep 2014 11:06:13 -0400
To:	Andrius Kulbis via RT <bug-RT-Extension-SLA [...] rt.cpan.org>
From:	Kevin Falcone <falcone [...] bestpractical.com>

On Fri, Sep 19, 2014 at 10:16:31AM -0400, Andrius Kulbis via RT wrote: Show quoted text

> Sorry for the mess in thoughts, I am not native English speaker.

As I said on the mailing list. A failing test, or at least an initialdata file to insert a small scrip that shows your problem plus an SLA configuration would make it much easier to look into this. Even better if you show me something that works without the extension but fails with the extension. -kevin

Mon Sep 22 04:45:31 2014 andrius.kulbis [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Mon, 22 Sep 2014 11:44:59 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

Message body is not shown because it is too large.

Mon Sep 22 21:08:51 2014 sunnavy [...] gmail.com - Correspondence added

Hi Andrius Could you send the output of the following sql: select * from Transactions where ObjectId=1 and ObjectType='RT::Ticket'; Thanks sunnavy

Tue Sep 23 03:40:30 2014 andrius.kulbis [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Tue, 23 Sep 2014 10:40:00 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

+----+------------+----------+-----------+-------------+---------------+---------------------+---------------------+----------------------------+--------------+--------------+-----------------------------------------------------------------------------------------------------------+---------+---------------------+ | id | ObjectType | ObjectId | TimeTaken | Type | Field | OldValue | NewValue | ReferenceType | OldReference | NewReference | Data | Creator | Created | +----+------------+----------+-----------+-------------+---------------+---------------------+---------------------+----------------------------+--------------+--------------+-----------------------------------------------------------------------------------------------------------+---------+---------------------+ | 25 | RT::Ticket | 1 | 0 | Create | NULL | NULL | NULL | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:19:28 | | 26 | RT::Ticket | 1 | 0 | SystemError | NULL | NULL | NULL | NULL | NULL | NULL | Sending the previous mail has failed. Please contact your admin, they can find more details in the logs. | 1 | 2014-09-22 08:19:29 | | 27 | RT::Ticket | 1 | 0 | Set | Due | 1970-01-01 00:00:00 | 2014-09-23 07:00:00 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:54 | | 28 | RT::Ticket | 1 | 0 | Set | Subject | testas | testas1 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:54 | | 29 | RT::Ticket | 1 | 0 | Set | FinalPriority | 0 | 9 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:54 | | 30 | RT::Ticket | 1 | 0 | Set | Priority | 0 | 1 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | | 31 | RT::Ticket | 1 | 0 | Status | Status | new | open | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | | 32 | RT::Ticket | 1 | 0 | Set | Owner | 6 | 12 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | | 33 | RT::Ticket | 1 | 0 | SetWatcher | Owner | 6 | 12 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | | 34 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-23 07:00:00 | 2014-09-25 07:00:00 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | | 35 | RT::Ticket | 1 | 0 | Set | Subject | testas1 | testas | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | | 36 | RT::Ticket | 1 | 0 | Set | Priority | 1 | 2 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | | 37 | RT::Ticket | 1 | 0 | Status | Status | open | new | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | | 38 | RT::Ticket | 1 | 0 | Set | Owner | 12 | 6 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | | 39 | RT::Ticket | 1 | 0 | SetWatcher | Owner | 12 | 6 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | | 41 | RT::Ticket | 1 | 0 | Set | Subject | testas | testass | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | | 42 | RT::Ticket | 1 | 0 | Set | Priority | 2 | 1 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | | 43 | RT::Ticket | 1 | 0 | Status | Status | new | open | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | | 44 | RT::Ticket | 1 | 0 | Set | Owner | 6 | 12 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | | 45 | RT::Ticket | 1 | 0 | SetWatcher | Owner | 6 | 12 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | | 46 | RT::Ticket | 1 | 0 | CustomField | 1 | NULL | NULL | RT::ObjectCustomFieldValue | NULL | 1 | NULL | 12 | 2014-09-22 08:35:22 | | 47 | RT::Ticket | 1 | 0 | Set | Subject | testass | testas | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | | 48 | RT::Ticket | 1 | 0 | Set | Priority | 1 | 2 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | | 49 | RT::Ticket | 1 | 0 | Status | Status | open | new | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | | 50 | RT::Ticket | 1 | 0 | Set | Owner | 12 | 6 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | | 51 | RT::Ticket | 1 | 0 | SetWatcher | Owner | 12 | 6 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | | 52 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | RT::ObjectCustomFieldValue | NULL | 2 | NULL | 12 | 2014-09-22 08:35:45 | | 53 | RT::Ticket | 1 | 0 | Set | Starts | 1970-01-01 00:00:00 | 2014-09-22 15:15:00 | NULL | NULL | NULL | NULL | 1 | 2014-09-22 08:35:45 | | 54 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-25 07:00:00 | 2014-09-29 15:00:00 | NULL | NULL | NULL | NULL | 1 | 2014-09-22 08:35:45 | | 55 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | RT::ObjectCustomFieldValue | 2 | 3 | NULL | 12 | 2014-09-22 08:36:12 | | 56 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-29 15:00:00 | 2014-09-24 23:15:00 | NULL | NULL | NULL | NULL | 1 | 2014-09-22 08:36:12 | | 57 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-24 23:15:00 | 2014-09-30 07:00:00 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | | 58 | RT::Ticket | 1 | 0 | Set | Subject | testas | testass | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | | 59 | RT::Ticket | 1 | 0 | Set | Priority | 2 | 1 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | | 60 | RT::Ticket | 1 | 0 | Status | Status | new | open | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | | 61 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-30 07:00:00 | 2014-09-24 23:15:00 | NULL | NULL | NULL | NULL | 1 | 2014-09-22 08:36:12 | | 62 | RT::Ticket | 1 | 0 | Set | Owner | 6 | 12 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | | 63 | RT::Ticket | 1 | 0 | SetWatcher | Owner | 6 | 12 | NULL | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | +----+------------+----------+-----------+-------------+---------------+---------------------+---------------------+----------------------------+--------------+--------------+-----------------------------------------------------------------------------------------------------------+---------+---------------------+ 2014-09-23 04:08, sunnavy via RT rašė: Show quoted text

> select * from Transactions where ObjectId=1 and ObjectType='RT::Ticket';

-- Pagarbiai Andrius Kulbis

Tue Sep 23 22:50:27 2014 sunnavy [...] gmail.com - Correspondence added

Hi Andrius I can explain the extra Batch scrips from the txn: Apparently you did something extra you didn't mention before, see txn #55: 55 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | RT::ObjectCustomFieldValue So you changed custom field 2 and it caused an extra Batch scrip. Also, SLA set Due date when status is changed, that's why there is txn #61 and another Batch scrip. 61 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-30 07:00:00 | 2014-09-24 23:15:00 | NULL You are not supposed to change Due manually if you set SLA for the ticket. Regards sunnavy On Tue Sep 23 03:40:30 2014, andrius.kulbis@gmail.com wrote: Show quoted text

> +----+------------+----------+-----------+------------- > +---------------+---------------------+--------------------- > +----------------------------+--------------+-------------- > +----------------------------------------------------------------------------------------------------------- > +---------+---------------------+ > | id | ObjectType | ObjectId | TimeTaken | Type | Field > | > OldValue | NewValue | ReferenceType > | > OldReference | NewReference | Data | Creator | Created | > +----+------------+----------+-----------+------------- > +---------------+---------------------+--------------------- > +----------------------------+--------------+-------------- > +----------------------------------------------------------------------------------------------------------- > +---------+---------------------+ > | 25 | RT::Ticket | 1 | 0 | Create | NULL > | > NULL | NULL | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:19:28 | > | 26 | RT::Ticket | 1 | 0 | SystemError | NULL > | > NULL | NULL | > NULL > | NULL | NULL | Sending the previous mail has > failed. > Please contact your admin, they can find more details in the logs. > | 1 | 2014-09-22 08:19:29 | > | 27 | RT::Ticket | 1 | 0 | Set | Due > | > 1970-01-01 00:00:00 | 2014-09-23 07:00:00 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:54 | > | 28 | RT::Ticket | 1 | 0 | Set | Subject > | > testas | testas1 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:54 | > | 29 | RT::Ticket | 1 | 0 | Set | > FinalPriority | > 0 | 9 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:54 | > | 30 | RT::Ticket | 1 | 0 | Set | Priority > | > 0 | 1 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | > | 31 | RT::Ticket | 1 | 0 | Status | Status > | > new | open | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | > | 32 | RT::Ticket | 1 | 0 | Set | Owner > | > 6 | 12 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | > | 33 | RT::Ticket | 1 | 0 | SetWatcher | Owner > | > 6 | 12 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:20:55 | > | 34 | RT::Ticket | 1 | 0 | Set | Due > | > 2014-09-23 07:00:00 | 2014-09-25 07:00:00 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | > | 35 | RT::Ticket | 1 | 0 | Set | Subject > | > testas1 | testas | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | > | 36 | RT::Ticket | 1 | 0 | Set | Priority > | > 1 | 2 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | > | 37 | RT::Ticket | 1 | 0 | Status | Status > | > open | new | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | > | 38 | RT::Ticket | 1 | 0 | Set | Owner > | > 12 | 6 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | > | 39 | RT::Ticket | 1 | 0 | SetWatcher | Owner > | > 12 | 6 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:23:03 | > | 41 | RT::Ticket | 1 | 0 | Set | Subject > | > testas | testass | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | > | 42 | RT::Ticket | 1 | 0 | Set | Priority > | > 2 | 1 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | > | 43 | RT::Ticket | 1 | 0 | Status | Status > | > new | open | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | > | 44 | RT::Ticket | 1 | 0 | Set | Owner > | > 6 | 12 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | > | 45 | RT::Ticket | 1 | 0 | SetWatcher | Owner > | > 6 | 12 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:34:50 | > | 46 | RT::Ticket | 1 | 0 | CustomField | 1 > | > NULL | NULL | > RT::ObjectCustomFieldValue > | NULL | 1 | NULL | 12 | 2014-09-22 08:35:22 | > | 47 | RT::Ticket | 1 | 0 | Set | Subject > | > testass | testas | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | > | 48 | RT::Ticket | 1 | 0 | Set | Priority > | > 1 | 2 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | > | 49 | RT::Ticket | 1 | 0 | Status | Status > | > open | new | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | > | 50 | RT::Ticket | 1 | 0 | Set | Owner > | > 12 | 6 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | > | 51 | RT::Ticket | 1 | 0 | SetWatcher | Owner > | > 12 | 6 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:35:22 | > | 52 | RT::Ticket | 1 | 0 | CustomField | 2 > | > NULL | NULL | > RT::ObjectCustomFieldValue > | NULL | 2 | NULL | 12 | 2014-09-22 08:35:45 | > | 53 | RT::Ticket | 1 | 0 | Set | Starts > | > 1970-01-01 00:00:00 | 2014-09-22 15:15:00 | > NULL > | NULL | NULL | NULL | 1 | 2014-09-22 08:35:45 | > | 54 | RT::Ticket | 1 | 0 | Set | Due > | > 2014-09-25 07:00:00 | 2014-09-29 15:00:00 | > NULL > | NULL | NULL | NULL | 1 | 2014-09-22 08:35:45 | > | 55 | RT::Ticket | 1 | 0 | CustomField | 2 > | > NULL | NULL | > RT::ObjectCustomFieldValue > | 2 | 3 | NULL | 12 | 2014-09-22 08:36:12 | > | 56 | RT::Ticket | 1 | 0 | Set | Due > | > 2014-09-29 15:00:00 | 2014-09-24 23:15:00 | > NULL > | NULL | NULL | NULL | 1 | 2014-09-22 08:36:12 | > | 57 | RT::Ticket | 1 | 0 | Set | Due > | > 2014-09-24 23:15:00 | 2014-09-30 07:00:00 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | > | 58 | RT::Ticket | 1 | 0 | Set | Subject > | > testas | testass | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | > | 59 | RT::Ticket | 1 | 0 | Set | Priority > | > 2 | 1 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | > | 60 | RT::Ticket | 1 | 0 | Status | Status > | > new | open | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | > | 61 | RT::Ticket | 1 | 0 | Set | Due > | > 2014-09-30 07:00:00 | 2014-09-24 23:15:00 | > NULL > | NULL | NULL | NULL | 1 | 2014-09-22 08:36:12 | > | 62 | RT::Ticket | 1 | 0 | Set | Owner > | > 6 | 12 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | > | 63 | RT::Ticket | 1 | 0 | SetWatcher | Owner > | > 6 | 12 | > NULL > | NULL | NULL | NULL | 12 | 2014-09-22 08:36:12 | > +----+------------+----------+-----------+------------- > +---------------+---------------------+--------------------- > +----------------------------+--------------+-------------- > +----------------------------------------------------------------------------------------------------------- > +---------+---------------------+ > > > 2014-09-23 04:08, sunnavy via RT rašė:

> > select * from Transactions where ObjectId=1 and > > ObjectType='RT::Ticket';

Wed Sep 24 02:26:02 2014 andrius.kulbis [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Wed, 24 Sep 2014 09:25:45 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

Hello again, Txn #55 is the change of CustomField SLA. I will give you another example. Here I only changed the SLA CF value, and still get 2 times TransactionBatch fired. Shouldn't all these changes made by SLA belong to one batch? [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to prepare scrips for transaction #64 (/opt/rt4/sbin/../lib/RT/Transaction.pm:187) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 4 scrips for TransactionCreate stage with applicable type(s) CustomField for txn #64 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #11 because it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #18 because it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to commit scrips for transaction #64 (/opt/rt4/sbin/../lib/RT/Transaction.pm:210) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #13 on txn #64 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #19 on txn #64 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Last effective non-requestors' reply to ticket #1 is txn #25 (/opt/rt4/local/plugins/RT-Extension-SLA/lib/RT/Action/SLA_SetDue.pm:41) [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to prepare scrips for transaction #65 (/opt/rt4/sbin/../lib/RT/Transaction.pm:187) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 3 scrips for TransactionCreate stage with applicable type(s) Set for txn #65 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #11 because it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #19 because it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to commit scrips for transaction #65 (/opt/rt4/sbin/../lib/RT/Transaction.pm:210) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #13 on txn #65 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) CustomField for txn #64 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #16 on txn #64 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) Set for txn #65 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #16 on txn #65 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) | 64 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | RT::ObjectCustomFieldValue | 3 | 4 | NULL | 12 | 2014-09-24 06:19:29 | | 65 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-24 23:15:00 | 2014-09-29 08:19:28 | NULL | NULL | NULL | NULL | 1 | 2014-09-24 06:19:29 | 2014-09-24 05:50, sunnavy via RT rašė: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=98977 > > > Hi Andrius > > > I can explain the extra Batch scrips from the txn: > > Apparently you did something extra you didn't mention before, see txn #55: > > 55 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | RT::ObjectCustomFieldValue > > So you changed custom field 2 and it caused an extra Batch scrip. > > Also, SLA set Due date when status is changed, that's why there is txn #61 and another Batch scrip. > > 61 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-30 07:00:00 | 2014-09-24 23:15:00 | NULL > > You are not supposed to change Due manually if you set SLA for the ticket. > > Regards > sunnavy

-- Pagarbiai Andrius Kulbis

Wed Sep 24 05:10:28 2014 sunnavy [...] gmail.com - Correspondence added

Nope, the due change is made by another scrip(not from the web ui), which is the case that doesn't fit here. On Wed Sep 24 02:26:02 2014, andrius.kulbis@gmail.com wrote: Show quoted text

> Hello again, > > Txn #55 is the change of CustomField SLA. > > I will give you another example. Here I only changed the SLA CF value, > and still get 2 times TransactionBatch fired. Shouldn't all these > changes made by SLA belong to one batch? > > [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to prepare scrips > for > transaction #64 (/opt/rt4/sbin/../lib/RT/Transaction.pm:187) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 4 scrips for > TransactionCreate stage with applicable type(s) CustomField for txn > #64 > on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #11 because > it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #18 because > it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to commit scrips for > transaction #64 (/opt/rt4/sbin/../lib/RT/Transaction.pm:210) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #13 on > txn > #64 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #19 on > txn > #64 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Last effective > non-requestors' reply to ticket #1 is txn #25 > (/opt/rt4/local/plugins/RT-Extension- > SLA/lib/RT/Action/SLA_SetDue.pm:41) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to prepare scrips > for > transaction #65 (/opt/rt4/sbin/../lib/RT/Transaction.pm:187) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 3 scrips for > TransactionCreate stage with applicable type(s) Set for txn #65 on > ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #11 because > it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Skipping Scrip #19 because > it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: About to commit scrips for > transaction #65 (/opt/rt4/sbin/../lib/RT/Transaction.pm:210) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #13 on > txn > #65 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 1 scrips for > TransactionBatch stage with applicable type(s) CustomField for txn #64 > on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #16 on > txn > #64 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Found 1 scrips for > TransactionBatch stage with applicable type(s) Set for txn #65 on > ticket > #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) > [12524] [Wed Sep 24 06:19:29 2014] [debug]: Committing scrip #16 on > txn > #65 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) > > > | 64 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | > RT::ObjectCustomFieldValue | 3 | 4 | NULL | 12 | 2014-09-24 > 06:19:29 | > | 65 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-24 23:15:00 | > 2014-09-29 08:19:28 | NULL | NULL | NULL | NULL | 1 | 2014-09-24 > 06:19:29 | > > > 2014-09-24 05:50, sunnavy via RT rašė:

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=98977 > > > > > Hi Andrius > > > > > > I can explain the extra Batch scrips from the txn: > > > > Apparently you did something extra you didn't mention before, see txn > > #55: > > > > 55 | RT::Ticket | 1 | 0 | CustomField | 2 > > | NULL | NULL | > > RT::ObjectCustomFieldValue > > > > So you changed custom field 2 and it caused an extra Batch scrip. > > > > Also, SLA set Due date when status is changed, that's why there is > > txn #61 and another Batch scrip. > > > > 61 | RT::Ticket | 1 | 0 | Set | Due > > | 2014-09-30 07:00:00 | 2014-09-24 23:15:00 | NULL > > > > You are not supposed to change Due manually if you set SLA for the > > ticket. > > > > Regards > > sunnavy

Wed Sep 24 05:23:12 2014 andrius.kulbis [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Wed, 24 Sep 2014 12:22:54 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

Yet another use case. All the scrips are disabled, only left SLA scrips and my scrip in TransactionBatch mode which do nothing. *Changing ONLY CustomField SLA value.* [12528] [Wed Sep 24 09:16:39 2014] [debug]: About to prepare scrips for transaction #66 (/opt/rt4/sbin/../lib/RT/Transaction.pm:187) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Found 2 scrips for TransactionCreate stage with applicable type(s) CustomField for txn #66 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Skipping Scrip #18 because it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) [12528] [Wed Sep 24 09:16:39 2014] [debug]: About to commit scrips for transaction #66 (/opt/rt4/sbin/../lib/RT/Transaction.pm:210) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Committing scrip #19 on txn #66 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Last effective non-requestors' reply to ticket #1 is txn #25 (/opt/rt4/local/plugins/RT-Extension-SLA/lib/RT/Action/SLA_SetDue.pm:41) [12528] [Wed Sep 24 09:16:39 2014] [debug]: About to prepare scrips for transaction #67 (/opt/rt4/sbin/../lib/RT/Transaction.pm:187) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Found 1 scrips for TransactionCreate stage with applicable type(s) Set for txn #67 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Skipping Scrip #19 because it isn't applicable (/opt/rt4/sbin/../lib/RT/Scrips.pm:353) [12528] [Wed Sep 24 09:16:39 2014] [debug]: About to commit scrips for transaction #67 (/opt/rt4/sbin/../lib/RT/Transaction.pm:210) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) CustomField for txn #66 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Committing scrip #16 on txn #66 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Found 1 scrips for TransactionBatch stage with applicable type(s) Set for txn #67 on ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:495) [12528] [Wed Sep 24 09:16:39 2014] [debug]: Committing scrip #16 on txn #67 of ticket #1 (/opt/rt4/sbin/../lib/RT/Scrips.pm:306) | 66 | RT::Ticket | 1 | 0 | CustomField | 2 | NULL | NULL | RT::ObjectCustomFieldValue | 4 | 5 | NULL | 12 | 2014-09-24 09:16:39 | | 67 | RT::Ticket | 1 | 0 | Set | Due | 2014-09-29 08:19:28 | 2014-09-23 08:19:28 | NULL | NULL | NULL | NULL | 1 | 2014-09-24 09:16:39 | Scrips: CustomFields: 2014-09-24 12:10, sunnavy via RT rašė: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=98977 > > > Nope, the due change is made by another scrip(not from the web ui), which is the case that doesn't fit here.

-- Pagarbiai Andrius Kulbis

Download jjcfbceb.png
image/png 73.8k

Download aaeficbb.png
image/png 7k

Wed Sep 24 05:29:30 2014 andrius.kulbis [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #98977] Due and Priority changes in TransactionBatch after SLA modification
Date:	Wed, 24 Sep 2014 12:29:15 +0300
To:	bug-RT-Extension-SLA [...] rt.cpan.org
From:	Andrius Kulbis <andrius.kulbis [...] gmail.com>

AAA. You are trying to tell me, that because the action of due change is made by script (SLA setDue scrip in this case) and not the UI, it is taken into another TransactionBatch? 2014-09-24 12:10, sunnavy via RT rašė: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=98977 > > > Nope, the due change is made by another scrip(not from the web ui), which is the case that doesn't fit here.

-- Pagarbiai Andrius Kulbis

Wed Sep 24 10:45:55 2014 sunnavy [...] gmail.com - Correspondence added

Yep! On Wed Sep 24 05:29:30 2014, andrius.kulbis@gmail.com wrote: Show quoted text

> AAA. You are trying to tell me, that because the action of due change > is > made by script (SLA setDue scrip in this case) and not the UI, it is > taken into another TransactionBatch? > > 2014-09-24 12:10, sunnavy via RT rašė:

> > <URL: https://rt.cpan.org/Ticket/Display.html?id=98977 > > > > > Nope, the due change is made by another scrip(not from the web ui), > > which is the case that doesn't fit here.