Paging was simply wrong, it only followed by 1 page

chipitsine · chipitsine · commit 2773b7edb425 · 2018-07-14T02:09:40.000+05:00
diff --git a/GitHubAnalytics.psm1 b/GitHubAnalytics.psm1
@@ -1052,11 +1052,11 @@ function Get-NextResultPage
         return $null
     }
 
-    $nextLinkString = $jsonResult.Headers.Link.Split(',')[0]
+    $nextLinkString = $jsonResult.Headers.Link.Split(',').Where{ $_ -match 'rel="next"' }
     
     # Get url query for the next page
     $query = $nextLinkString.Split(';')[0].replace('<','').replace('>','')
-    if ($query -notmatch '&page=1')
+    if ($query -notmatch '&page=1>')
     {
         return $query
     }
@@ -1197,4 +1197,4 @@ function Get-WeekDate
     }
 
     return $beginningsOfWeeks
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -1052,11 +1052,11 @@ function Get-NextResultPage`
`1052`	`1052`	`return $null`
`1053`	`1053`	`}`
`1054`	`1054`
`1055`		`- $nextLinkString = $jsonResult.Headers.Link.Split(',')[0]`
	`1055`	`+ $nextLinkString = $jsonResult.Headers.Link.Split(',').Where{ $_ -match 'rel="next"' }`
`1056`	`1056`
`1057`	`1057`	`# Get url query for the next page`
`1058`	`1058`	`$query = $nextLinkString.Split(';')[0].replace('<','').replace('>','')`
`1059`		`- if ($query -notmatch '&page=1')`
	`1059`	`+ if ($query -notmatch '&page=1>')`
`1060`	`1060`	`{`
`1061`	`1061`	`return $query`
`1062`	`1062`	`}`
`@@ -1197,4 +1197,4 @@ function Get-WeekDate`
`1197`	`1197`	`}`
`1198`	`1198`
`1199`	`1199`	`return $beginningsOfWeeks`
`1200`		`-}`
	`1200`	`+}`