grahamc/hydra2 - Change Q3FVYVWXHEZ3LFMMCJGS6AIDN4JWXVWNWCX73A7275UDPHL6LT7AC

search: limit results to 50, default to 10

This search query is pretty heavy. Defaulting to 500 has caused Hydra's web UI to appear to be down. Since 500 can take it down, users probably shouldn't be allowed t ask for that many.

Created by Graham Christensen on January 30, 2021

Q3FVYVWXHEZ3LFMMCJGS6AIDN4JWXVWNWCX73A7275UDPHL6LT7AC

Dependencies

[2] HPGWVEFKM3DYLNLPMWLZI5VZXURUSVJ4FM2DPJCPR5HBOEQPASQQC
[3] CRDOVMTCSIIQOA7NSCTB3SDFDAWQ4PIW7VCW3S5I6W4OEAHNRCHAC
[4] NX5BM2OINIVVOHN23N2DWH5LHVQN7AVRWYPUZ5J54WXFWE5B5XIQC
[5] J5UVLXOK6EDIL5I7VKWH4V2QDS4DPD7FHRK6XBWSXFRQS4JKXFZQC
[6] WUCOEIFANBM3WXZ2IWEITZ6FKBKXASVWH2LCYRXB4C22H5ADLWVAC
[7] UICHT2PSN2QOCHP6MGY5QQJUDU4IF6FBIOJLBA55QBJDK6ST566AC

In channels

main

Change contents

Insertion in src/lib/Hydra/Controller/Root.pm at line 16 [5.804]
[4.10]
[2.0]
```
use List::Util qw[min max];
```

Replacement in src/lib/Hydra/Controller/Root.pm at line 442 [5.804]

B:BD[6.1] → [3.0:167]

    my $limit = trim $c->request->params->{"limit"};
    if ($limit eq "") {
        $c->stash->{limit} = 500;
    } else {
        $c->stash->{limit} = $limit;
    }

[6.1]

[7.294]

    my $limit = int(trim ($c->request->params->{"limit"} || "10"));
    $c->stash->{limit} = min(50, max(1, $limit));

search: limit results to 50, default to 10

Dependencies

In channels

Change contents

Insertion in src/lib/Hydra/Controller/Root.pm at line 16 [5.804]

Replacement in src/lib/Hydra/Controller/Root.pm at line 442 [5.804]