From d2874a96b8ef944f78f177fde8cfc52f8b2d4735 Mon Sep 17 00:00:00 2001
From: Bob Trahan <bob.trahan@gmail.com>
Date: Thu, 18 Apr 2013 17:22:05 -0700
Subject: [PATCH] DERP 2 with my last diff - shouldn't have pulled this outside
 the conditional

---
 .../conpherence/controller/ConpherenceController.php          | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/applications/conpherence/controller/ConpherenceController.php b/src/applications/conpherence/controller/ConpherenceController.php
index 764797b26f..600c658911 100644
--- a/src/applications/conpherence/controller/ConpherenceController.php
+++ b/src/applications/conpherence/controller/ConpherenceController.php
@@ -146,10 +146,10 @@ abstract class ConpherenceController extends PhabricatorController {
     if (count($transactions) == $too_many) {
       $last_transaction = end($transactions);
       unset($transactions[$last_transaction->getID()]);
+      $oldest_transaction = end($transactions);
+      $oldest_transaction_id = $oldest_transaction->getID();
     }
     $transactions = array_reverse($transactions);
-    $oldest_transaction = reset($transactions);
-    $oldest_transaction_id = $oldest_transaction->getID();
     $handles = $conpherence->getHandles();
     $rendered_transactions = array();
     $engine = id(new PhabricatorMarkupEngine())