[Beta 1.5.3]Alternative MessageIndex sort

[Daniel D.]

Doesn't work. Installed it (altMI_MOD01.zip) on unmodded 1.5.3 & localhost with pics. First page is sorted, if I click on [2], the remain in the standard order.

[A.M.A.]

Doesn't work. Installed it (altMI_MOD01.zip) on unmodded 1.5.3 & localhost with pics. First page is sorted, if I click on [2], the remain in the standard order.

Fixed: Sorry for that .. I do not know how I missed a whole step! Please try loading it again.

[Anguz]

THANK YOU!!! this is a mod a need a lot!  

a couple of questions:

when you're viewing a topic that's in a forum sorted with your mod, if you hit the NEXT button, will it take you to the next one in the new order?

is it possible to sort a certain forum by subject as default? so that will be the first order it'll be displayed in when opening it

[Daniel D.]

Fixed: Sorry for that .. I do not know how I missed a whole step! Please try loading it again.

Thx !

[A.M.A.]

when you're viewing a topic that's in a forum sorted with your mod, if you hit the NEXT button, will it take you to the next one in the new order?

Yes, if you choose the second ZIPFILE in the first post.

is it possible to sort a certain forum by subject as default? so that will be the first order it'll be displayed in when opening it

I guess it can be done 'hard-coded'.

[David]

Having problems with it sorting by thread starter.  It is instead sorting by the last poster.  The issue is the query is:

Code Select Expand

$result = mysql_query("
               SELECT t.ID_TOPIC
               FROM {$db_prefix}topics AS t, {$db_prefix}messages AS m
               WHERE m.ID_MSG=t.ID_LAST_MSG
                       AND t.ID_BOARD=$currentboard
               ORDER BY $stickyOrder $ordrsql
               LIMIT $start,$maxindex") or database_error(__FILE__, __LINE__);

So the ordersql of 'm.posterName ASC' is doing that on the last message instead of the first.  I can't come up with a good solution to fix this without a subselect for the order by to get the thread starters membername instead of the id number.

[A.M.A.]

Change this

Code Select Expand


 else if ($ordrmindex == 3)
      $ordrsql = 'm.posterName ASC' ;
 else if ($ordrmindex == 4)
      $ordrsql = 'm.posterName DESC' ;
With

Code Select Expand


 else if ($ordrmindex == 3)
    $ordrsql = 't.ID_MEMBER_STARTED ASC' ;
 else if ($ordrmindex == 4)
    $ordrsql = 't.ID_MEMBER_STARTED DESC' ;

I hope this will fix it.

[David]

That would sort it by date the thread starter registered their account.

[A.M.A.]

Here is another try.. this may be a fix tho I am not sure of the effect that it may cause!
Look for:

Code Select Expand

   // Grab the appropriate topic information
 if ($ordrmindex == 1)
    $ordrsql = 'm.subject ASC' ;
 else if ($ordrmindex == 2)
    $ordrsql = 'm.subject DESC' ;
 else if ($ordrmindex == 3)
    $ordrsql = 'm.posterName ASC' ;
 else if ($ordrmindex == 4)
    $ordrsql = 'm.posterName DESC' ;
 else if ($ordrmindex == 5)
    $ordrsql = 't.numReplies ASC' ;
 else if ($ordrmindex == 6)
    $ordrsql = 't.numReplies DESC' ;
 else if ($ordrmindex == 7)
    $ordrsql = 't.numViews ASC' ;
 else if ($ordrmindex == 8)
    $ordrsql = 't.numViews DESC' ;
 else if ($ordrmindex == 9)
    $ordrsql = 'm.posterTime ASC' ;
 else if ($ordrmindex == 10)
    $ordrsql = 'm.posterTime DESC' ;
 else
    $ordrsql = 'm.posterTime DESC' ;
   
   $stickyOrder = ($modSettings['enableStickyTopics'] == 1 ? 't.isSticky DESC,' : '');
   $result = mysql_query("
      SELECT t.ID_TOPIC
      FROM {$db_prefix}topics AS t, {$db_prefix}messages AS m
      WHERE m.ID_MSG=t.ID_LAST_MSG
         AND t.ID_BOARD=$currentboard
      ORDER BY $stickyOrder $ordrsql
      LIMIT $start,$maxindex") or database_error(__FILE__, __LINE__);
   $topics = array();
and replace it with

Code Select Expand

   // Grab the appropriate topic information
 $PostrOnly='t.ID_LAST_MSG';
 if ($ordrmindex == 1)
    $ordrsql = 'm.subject ASC' ;
 else if ($ordrmindex == 2)
    $ordrsql = 'm.subject DESC' ;
 else if ($ordrmindex == 3)
    {
    $ordrsql = 'm.posterName ASC' ;
    $PostrOnly='t.ID_FIRST_MSG';
    }
 else if ($ordrmindex == 4)
    {
    $ordrsql = 'm.posterName DESC' ;
    $PostrOnly='t.ID_FIRST_MSG';
    }
 else if ($ordrmindex == 5)
    $ordrsql = 't.numReplies ASC' ;
 else if ($ordrmindex == 6)
    $ordrsql = 't.numReplies DESC' ;
 else if ($ordrmindex == 7)
    $ordrsql = 't.numViews ASC' ;
 else if ($ordrmindex == 8)
    $ordrsql = 't.numViews DESC' ;
 else if ($ordrmindex == 9)
    $ordrsql = 'm.posterTime ASC' ;
 else if ($ordrmindex == 10)
    $ordrsql = 'm.posterTime DESC' ;
 else
    $ordrsql = 'm.posterTime DESC' ;
   
   $stickyOrder = ($modSettings['enableStickyTopics'] == 1 ? 't.isSticky DESC,' : '');
   $result = mysql_query("
      SELECT t.ID_TOPIC
      FROM {$db_prefix}topics AS t, {$db_prefix}messages AS m
      WHERE m.ID_MSG=$PostrOnly
         AND t.ID_BOARD=$currentboard
      ORDER BY $stickyOrder $ordrsql
      LIMIT $start,$maxindex") or database_error(__FILE__, __LINE__);
   $topics = array();

if ($ordrmindex == 3)
    $ordrsql = 'mname ASC';
else if ($ordrmindex == 4)
    $ordrsql = 'mname DESC';

[David]

Real question is why is the messages table in that query at all, we are only selecting from the topics table.

[A.M.A.]

so.. did it work!

Real question is why is the messages table in that query at all, we are only selecting from the topics table.

I did not create any query. It is already in there.

[dthomas]

First let me say it is a great mod and thanks for the work you put into it. Now to my question I was wanting the ability to sort by original date posted.  I had this on another board I was using and everyone seemed to like that. I run a photo board and many peoples post gets burried because it sorts by replies and not the start date. Can this be done?

[Keng]

english.lng

Code Select Expand

$txt['sort'] = 'Sort';
$txt['sort1'] = ' -> ASC';
$txt['sort2'] = ' <- DESC';

all_altMI.mod
Look for:

Code Select Expand

       <form name="AscdesDD"><font size="1">Sort:</font>
         <select name="DDAD">
           <option value="' . $cgi . '&ordrmindex=1">Subject -> ASC</option>
           <option value="' . $cgi . '&ordrmindex=2">Subject <- DESC</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=3">Started by -> ASC</option>
           <option value="' . $cgi . '&ordrmindex=4">Started by <- DESC</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=5">Replies -> ASC</option>
           <option value="' . $cgi . '&ordrmindex=6">Replies <- DESC</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=7">Views -> ASC</option>
           <option value="' . $cgi . '&ordrmindex=8">Views <- DESC</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=9">Last post -> ASC</option>
           <option value="' . $cgi . '&ordrmindex=10">Last post <- DESC</option>

replace it with:

Code Select Expand

       <form name="AscdesDD"><font size="1">' . $txt[sort] . ':</font>
         <select name="DDAD">
           <option value="' . $cgi . '&ordrmindex=1">' . $txt[70] . '' . $txt[sort1] . '</option>
           <option value="' . $cgi . '&ordrmindex=2">' . $txt[70] . '' . $txt[sort2] . '</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=3">' . $txt[109] . '' . $txt[sort1] . '</option>
           <option value="' . $cgi . '&ordrmindex=4">' . $txt[109] . '' . $txt[sort2] . '</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=5">' . $txt[110] . '' . $txt[sort1] . '</option>
           <option value="' . $cgi . '&ordrmindex=6">' . $txt[110] . '' . $txt[sort2] . '</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=7">' . $txt[301] . '' . $txt[sort1] . '</option>
           <option value="' . $cgi . '&ordrmindex=8">' . $txt[301] . '' . $txt[sort2] . '</option>
           <option value="">---------------------</option>
           <option value="' . $cgi . '&ordrmindex=9">' . $txt[111] . '' . $txt[sort1] . '</option>
           <option value="' . $cgi . '&ordrmindex=10">' . $txt[111] . '' . $txt[sort2] . '</option>

[A.M.A.]

dthomas: I guess It can be done .. it may be something similar to what I did for David .. tho am not sure!

Keng: Thanks for adding the text strings, this way it will work with other translations.

[sensovision]

thank you for the great mod!