First, thanks for this module. Drupal really needs it, IMO.

I'm running the 4.6 version of the module on Drupal CVS from about a month ago. (I'm not going anywhere near the latest CVS because of the form changes).

Tables in this module should default to sorting by date, to show most recent posts first. This seems like an obvious issue, so it may be a bug? If not, it would be a great default to set.

Comments

kbahey’s picture

kbahey commented

Hmmm

The sorting is done by the tablesort_query, and it has:

     array('data' => t('Last Post'), 'field' => 'last_post', 'sort' => 'asc')

So, I am not sure if that takes effect initially, or only after someone clicks on it?

ixis.dylan’s picture

ixis.dylan commented

Shouldn't that be 'desc'?

kbahey’s picture

kbahey commented

That is what I was thinking, but it was getting late.

If you can change it locally and test it, and verify it works, I will commit a change for it.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Tested on 4.6.3. If you edit line 151 of v1.1 to 

      array('data' => t('Last Post'), 'field' => 'last_post', 'sort' => 'desc')

Then the default sort is most recent post.

kbahey’s picture

kbahey commented
Assigned: Unassigned » kbahey
Status: Active » Fixed

Commited for 4.6 and HEAD.

Thanks to both of you.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)