variable: Complete Thread ASC or DESC

This pull adds the ability to choose between ASC or DESC order for the
complete thread variable. If `complete_asc` is chosen the system will order
the thread in ASC order, if `complete_desc` is chosen the system will order
the thread in DESC order. This keeps backwards compatibility with just
`complete` as some people might forget to update this for a while.

include/class.thread.php

@@ -2989,7 +2989,7 @@ function __toString() {
     }
 
     function asVar() {
-        return $this->getVar('complete');
+        return $this->getVar('complete_asc');
     }
 
     function getVar($name) {
@@ -3012,7 +3012,10 @@ function getVar($name) {
                 return $entry->getBody();
 
             break;
+        case 'complete_desc':
+            $desc = true;
         case 'complete':
+        case 'complete_asc':
             $content = '';
             $thread = $this;
             ob_start();
@@ -3027,7 +3030,8 @@ function getVar($name) {
 
     static function getVarScope() {
       return array(
-        'complete' => __('Thread Correspondence'),
+        'complete_asc' => __('Thread Correspondence Ascending'),
+        'complete_desc' => __('Thread Correspondence Descending'),
         'original' => array('class' => 'MessageThreadEntry', 'desc' => __('Original Message')),
         'lastmessage' => array('class' => 'MessageThreadEntry', 'desc' => __('Last Message')),
       );

include/client/templates/thread-export.tmpl.php

@@ -11,8 +11,9 @@
             'attachments__thread_entry__type__in' => array_keys($entryTypes)
             ))->all();
 
+$order = $desc ? QuerySet::DESC : QuerySet::ASC;
 $entries = $thread->getEntries();
-$entries->filter(array('type__in' => array_keys($entryTypes)));
+$entries->filter(array('type__in' => array_keys($entryTypes)))->order_by('id', $order);
 ?>
 <style type="text/css">
     div {font-family: sans-serif;}