From e032705c9ab73a4ef71479a1f6b6ffebbbca696a Mon Sep 17 00:00:00 2001 From: Joseph Date: Fri, 26 Jul 2019 08:47:20 +0000 Subject: [PATCH] [HaveIBeenPwnedBridge] Add item limit parameter, set default limit to 20 (#1219) * Add `item_limit` parameter to allow user to control number of item returned by bridge. Suggested by @triatic and @somini (code). --- bridges/HaveIBeenPwnedBridge.php | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/bridges/HaveIBeenPwnedBridge.php b/bridges/HaveIBeenPwnedBridge.php index 1eb62e32..96dc7b2a 100644 --- a/bridges/HaveIBeenPwnedBridge.php +++ b/bridges/HaveIBeenPwnedBridge.php @@ -13,6 +13,11 @@ class HaveIBeenPwnedBridge extends BridgeAbstract { 'Date added to HIBP' => 'dateAdded', ), 'defaultValue' => 'dateAdded', + ), + 'item_limit' => array( + 'name' => 'Limit number of returned items', + 'type' => 'number', + 'defaultValue' => 20, ) )); @@ -109,6 +114,12 @@ class HaveIBeenPwnedBridge extends BridgeAbstract { */ private function createItems() { + $limit = $this->getInput('item_limit'); + + if ($limit < 1) { + $limit = 20; + } + foreach ($this->breaches as $breach) { $item = array(); @@ -118,6 +129,10 @@ class HaveIBeenPwnedBridge extends BridgeAbstract { $item['content'] = $breach['content']; $this->items[] = $item; + + if (count($this->items) >= $limit) { + break; + } } } }