]> sipb.mit.edu Git - ikiwiki.git/blob - doc/patchqueue/inline_-_sort_by_old_first.mdwn
web commit by hb
[ikiwiki.git] / doc / patchqueue / inline_-_sort_by_old_first.mdwn
1 Quick hack to be able to list old entries first :
2
3     
4     --- inline.pm   2007-01-18 18:21:27.000000000 +0100
5     +++ /usr/share/perl5/IkiWiki/Plugin/inline.pm   2007-01-18 18:23:01.000000000 +0100
6     @@ -92,6 +92,9 @@
7             elsif (! exists $params{sort} || $params{sort} eq 'age') {
8                     @list=sort { $pagectime{$b} <=> $pagectime{$a} } @list;
9             }
10     +       elsif (! exists $params{sort} || $params{sort} eq 'old') {
11     +               @list=sort { $pagectime{$a} <=> $pagectime{$b} } @list;
12     +       }
13             else {
14                     return sprintf(gettext("unknown sort type %s"), $params{sort});
15             }
16     
17
18 this patch adds a new type of sort "old".
19
20 > Hmm, take another look at your patch above. Comapre the added lines with
21 > the "age" lines just above them. It's identical; age already does what
22 > you want. --[[Joey]]
23
24 >> Well, $b and $a are switched in 'old' comparing to 'age'... According to the documentation, *The default, "age" is to sort newest pages first*. I just want the reverse, and that's why I just switched the $a and $b args of the sort func.... --HB
25
26 However, I wonder if it would be better to add another option for inline like "sortorder" with "desc" or "asc" keywords ?
27
28 > That might be cleaner, since it would allow sorting pages in reverse by
29 > title. If someone ever had a reason to do that. --[[Joey]]