Shrink output to prevent 'Database server has gone away' if output is…

… too large
tigron · Jan 18, 2023 · 1bf42c3 · 1bf42c3
1 parent 8b2411a
commit 1bf42c3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/lib/Skeleton/Transaction/Log.php b/lib/Skeleton/Transaction/Log.php
@@ -95,7 +95,7 @@ public static function create(
 		$log = new self();
 		$log->transaction_id = $transaction->id;
 		$log->failed = $failed;
-		$log->output = $output;
+		$log->output = substr($output, 0, 16777215);
 
 		if (isset($t) === true) {
 			$log->exception = substr(print_r($t, true), 0, 16777215);

diff --git a/lib/Skeleton/Transaction/Transaction.php b/lib/Skeleton/Transaction/Transaction.php
@@ -209,7 +209,6 @@ public function unlock() {
 	 * @access public
 	 */
 	public function mark_failed($output, $exception, $date = null) {
-		Database::reset();
 		Log::create($this, true, $output, $exception, $date);
 
 		$this->failed = true;
@@ -230,7 +229,6 @@ public function mark_failed($output, $exception, $date = null) {
 	 * @param string $date
 	 */
 	public function mark_completed($output, $date = null) {
-		Database::reset();
 		Log::create($this, false, $output, $date);
 
 		// Don't mark this transaction as completed if it has been rescheduled.