[Rt-commit] rt branch, 4.2/record-attachments-dropping, repushed
? sunnavy
sunnavy at bestpractical.com
Mon Aug 18 20:10:24 EDT 2014
The branch 4.2/record-attachments-dropping was deleted and repushed:
was cac2bcff502dff07d679bcb19d08b93347857929
now 5d1089ff1bcb8653cd9f205c712d067a5ce3e68c
1: 15dddd2 = 1: 15dddd2 record attachments' dropping/truncation as txns
2: 1545237 = 2: 1545237 test attachment dropping/truncation
3: 4aaa132 = 3: 4aaa132 make attachment truncate/drop and error txns more noticeable
4: cac2bcf ! 4: 5d1089f refactor email sent failure system error txn to use the same tech as Attachment Truncate/Drop.
@@ -2,7 +2,8 @@
refactor email sent failure system error txn to use the same tech as Attachment Truncate/Drop.
- it's more simple and secure: by specifying ActivateScrips to 0, it won't cause infinite-loop.
+ it's more simple and secure: by specifying ActivateScrips to 0, it won't cause infinite-loop,
+ which could happen if the system error txn triggers some scrips to send mails.
diff --git a/lib/RT/Interface/Email.pm b/lib/RT/Interface/Email.pm
--- a/lib/RT/Interface/Email.pm
@@ -16,7 +17,7 @@
- Content => "Sending the previous mail has failed. Please contact your admin, they can find more details in the logs.",
+ $ticket->_NewTransaction(
+ Type => "SystemError",
-+ Data => "Sending the previous mail has failed. Please contact your admin, they can find more details in the logs.",
++ Data => "Sending the previous mail has failed. Please contact your admin, they can find more details in the logs.", #loc
+ ActivateScrips => 0,
);
return 1;
More information about the rt-commit
mailing list