aboutsummaryrefslogtreecommitdiff
path: root/doc/bugs/Hyperestraier_search_plug-in_defective.mdwn
blob: aec5dbc3fef1525e9fa417320d12369efba58b09 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
The map() function used in the hyperestraier search plug-in doesn't work as intended as ilustrated by this simple script:

    #!/usr/bin/perl -w
    use strict;
    
    my @foo = (
    	[ qw/foo bar baz/ ],
    	[ qw/fee faa fum/ ],
    	);
    
    # similar to current ikiwiki code (defective):
    my @bar = map { "/path/to/$_" foreach @{$_} } @foo;
    
    # this works:
    #my @bar = map { map { "/path/to/$_" } @{$_} } @foo;
    
    foreach (@bar) {
    	print "$_\n";
    }

Expected output:

    /path/to/foo
    /path/to/bar
    /path/to/baz
    /path/to/fee
    /path/to/faa
    /path/to/fum

Current output:

    Useless use of string in void context at perl-map.pl line 10.

The patch below fixes this issue:

    --- IkiWiki/Plugin/search.pm.orig       Thu Feb  1 23:52:03 2007
    +++ IkiWiki/Plugin/search.pm    Thu Feb  1 23:52:41 2007
    @@ -64,8 +64,9 @@
            debug(gettext("updating hyperestraier search index"));
            estcmd("gather -cm -bc -cl -sd",
                    map {
    -                       Encode::encode_utf8($config{destdir}."/".$_)
    -                               foreach @{$renderedfiles{pagename($_)}};
    +                       map {
    +                               Encode::encode_utf8($config{destdir}."/".$_)
    +                       } @{$renderedfiles{pagename($_)}};
                    } @_
            );
            estcfg();

[[bugs/done]] ; thanks for the patch. Suprised it worked at all since the
bad code was added (did it?) --[[Joey]]

Thank you for accepting my patch. I can't see how it could ever have worked with the previous code, no. --[[Brix]]