Skip to content

Commit

Permalink
Fix top value when top is not provided (Azure#14051)
Browse files Browse the repository at this point in the history
* Fix top value when max is not provided

* Fix changelog.md error

* Update ChangeLog.md

Co-authored-by: Namrata Jagasia <najagasi@microsoft.com>
Co-authored-by: msJinLei <leijin@microsoft.com>
Co-authored-by: Jin Lei <54836179+msJinLei@users.noreply.github.com>
  • Loading branch information
4 people authored Jan 29, 2021
1 parent a037261 commit 1695bb0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/PolicyInsights/PolicyInsights/ChangeLog.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
- Additional information about change #1
-->
## Upcoming Release
* Retracted changes made in powershell that increased request row limit. Removed incorrect statement of supporting paging

## Version 1.4.0
* Added support for returning paginated results for Get-AzPolicyState
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ public override void Execute()
{
var queryOptions = new RestApiModels.QueryOptions
{
Top = MyInvocation.BoundParameters.ContainsKey("Top") ? (int?)Top : int.MaxValue,
Top = MyInvocation.BoundParameters.ContainsKey("Top") ? (int?)Top : null,
OrderBy = OrderBy,
Select = Select,
FromProperty = MyInvocation.BoundParameters.ContainsKey("From") ? (DateTime?)From : null,
Expand Down

0 comments on commit 1695bb0

Please sign in to comment.