From 1eb59c47606c733be40cbf8923fb00b3210cc667 Mon Sep 17 00:00:00 2001 From: joey Date: Thu, 8 Feb 2007 19:48:00 +0000 Subject: * Add "reverse" option to inline to invert sort orders. --- doc/patchqueue/inline_-_sort_by_old_first.mdwn | 29 -------------------------- 1 file changed, 29 deletions(-) delete mode 100644 doc/patchqueue/inline_-_sort_by_old_first.mdwn (limited to 'doc/patchqueue/inline_-_sort_by_old_first.mdwn') diff --git a/doc/patchqueue/inline_-_sort_by_old_first.mdwn b/doc/patchqueue/inline_-_sort_by_old_first.mdwn deleted file mode 100644 index 9054beadc..000000000 --- a/doc/patchqueue/inline_-_sort_by_old_first.mdwn +++ /dev/null @@ -1,29 +0,0 @@ -Quick hack to be able to list old entries first : - - - --- inline.pm 2007-01-18 18:21:27.000000000 +0100 - +++ /usr/share/perl5/IkiWiki/Plugin/inline.pm 2007-01-18 18:23:01.000000000 +0100 - @@ -92,6 +92,9 @@ - elsif (! exists $params{sort} || $params{sort} eq 'age') { - @list=sort { $pagectime{$b} <=> $pagectime{$a} } @list; - } - + elsif (! exists $params{sort} || $params{sort} eq 'old') { - + @list=sort { $pagectime{$a} <=> $pagectime{$b} } @list; - + } - else { - return sprintf(gettext("unknown sort type %s"), $params{sort}); - } - - -this patch adds a new type of sort "old". - -> Hmm, take another look at your patch above. Comapre the added lines with -> the "age" lines just above them. It's identical; age already does what -> you want. --[[Joey]] - ->> 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 - -However, I wonder if it would be better to add another option for inline like "sortorder" with "desc" or "asc" keywords ? - -> That might be cleaner, since it would allow sorting pages in reverse by -> title. If someone ever had a reason to do that. --[[Joey]] -- cgit v1.2.3