update fieldsort plugin again; remove obsolete bug + fix note (thanks!)

master
http://smcv.pseudorandom.co.uk/ 2010-04-07 03:03:40 +00:00 committed by Joey Hess
parent 25f3dd8818
commit e12cd5f293
1 changed files with 2 additions and 13 deletions

View File

@ -131,8 +131,8 @@ I think it should just be part of `field` rather than a separate plugin.
error("sort=field requires a parameter");
}
my $left = IkiWiki::Plugin::field::field_get_value($_[2], $_[0]);
my $right = IkiWiki::Plugin::field::field_get_value($_[2], $_[1]);
my $left = IkiWiki::Plugin::field::field_get_value($_[0], $a);
my $right = IkiWiki::Plugin::field::field_get_value($_[0], $b);
$left = "" unless defined $left;
$right = "" unless defined $right;
@ -141,17 +141,6 @@ I think it should just be part of `field` rather than a separate plugin.
1;
-------
Bug report: `field` has an unnecessary `use YAML::Any`, presumably from before
you separated out `ymlfront`. Trivial patch available from
field-etc branch in git://git.pseudorandom.co.uk/git/smcv/ikiwiki.git (gitweb:
<http://git.pseudorandom.co.uk/smcv/ikiwiki.git?a=shortlog;h=refs/heads/field-etc>)
--[[smcv]]
> Can do for the field plugin (delete one line? Easy.) Will push when I get to a better connection. --[[KathrynAndersen]]
>> Done! -K.A.
----
Disclaimer: I've only looked at this plugin and ymlfront, not other related