Added a default limit on TopCustomers widget

bymayo · Aug 9, 2019 · 29e5c52 · 29e5c52
1 parent b008364
commit 29e5c52
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Commerce Widgets Changelog
 
+## 2.0.18 - 2019-08-09
+
+### Fixed
+- Added a default limit on `TopCustomers` widget to fix issue where it pulls all customers in if the widget already existed on the dashboard
+
 ## 2.0.17 - 2019-08-09
 
 {warning} This plugin now requires Craft CMS 3.2.x or higher due to the way it now deals with trashed orders. It's also recommended to now use a `commerce-widgets.php` config file for the plugin settings since this version doesn't include the ability to modify settings via the CP.

diff --git a/composer.json b/composer.json
@@ -2,7 +2,7 @@
  "name": "bymayo/commerce-widgets",
  "description": "Insightful dashboard widgets for your Craft Commerce 2 store.",
  "type": "craft-plugin",
- "version": "2.0.17",
+ "version": "2.0.18",
  "keywords": [
  "craft",
  "cms",

diff --git a/src/widgets/TopCustomers.php b/src/widgets/TopCustomers.php
@@ -19,7 +19,7 @@ class TopCustomers extends Widget
  public $includeGuests;
  public $orderBy;
  public $groupBy; // Remove
- public $limit = 5;
+ public $limit;
 
  // Static Methods
  // =========================================================================
@@ -106,8 +106,10 @@ public function rules()
  $rules,
  [
  [['includeGuests', 'orderBy'], 'string'],
+ [['limit'], 'integer'],
  ['includeGuests', 'default', 'value' => 'yes'],
- ['orderBy', 'default', 'value' => 'totalRevenue']
+ ['orderBy', 'default', 'value' => 'totalRevenue'],
+ ['limit', 'default', 'value' => 5]
  ]
  );