Forum Discussion
Power Query - Number must be either non-negative and less than or equal to Int32.MaxValue or -1.
I am running an Rest API through Power Query, using List.Generate. It was working very happily until suddenly it is pulling through pages with null values, resulting in the following message:
"Number must be either non-negative and less than or equal to Int32.MaxValue or -1."
I have a function fJobs:
let
Source = (Page as number)=>
let
Source = Json.Document(Web.Contents("https://harvest.greenhouse.io/v1/jobs?page="&Number.ToText(Page)))
in
Source
in
Source
and then the normal query with the list generate
let
List = List.Generate(()=>
//Where to start and which page we are on //if error we want it to return a null (Try)
[Result = try fJobs(1) otherwise null , Page = 1],
//We want this to keep running until the result is null
each [Result] <> null,
//condition for creating the row and itterate throug until it is null
each [Result = try fJobs([Page]+1) otherwise null , Page = [Page] + 1],
// create the output
each [Result]),
#"Converted to Table" = Table.FromList(List, Splitter.SplitByNothing(), null, null, ExtraValues.Error),
#"Expanded Column1" = Table.ExpandListColumn(#"Converted to Table", "Column1"),
#"Expanded Column2" = Table.ExpandRecordColumn(#"Expanded Column1", "Column1", {"id", "name", "requisition_id", "notes", "confidential", "is_template", "copied_from_id", "status", "created_at", "opened_at", "closed_at", "updated_at"}, {"id", "name", "requisition_id", "notes", "confidential", "is_template", "copied_from_id", "status", "created_at", "opened_at", "closed_at", "updated_at"}),
#"Filtered Rows" = Table.SelectRows(#"Expanded Column2", each ([id] <> null))
in
#"Filtered Rows"
Any ideas why this is now pulling throug null values? It is happy until I "Close & Apply" the refresh triggering an issue with the null values.
It is Greenhouse Applicant Tracking System that I am connecting to
As you can imagine it is not possible to help much more without access to the API. Try adding more robustness to the exit check in the List.Generate part. In addition to checking for null you should also check for an empty list.
7 Replies
- lbendlin
Super User
Looks benign. The fJobs function can be severely refactored
(Page)=>Json.Document(Web.Contents("https://harvest.greenhouse.io/v1/jobs?page=" & Text.From(Page)))You should use the [Query = [page = Page]] attribute. Otherwise this may not work in the service
- AnonymousNot applicable
lbendlin I think I have made your amendments as suggested
(Page)=> Json.Document(Web.Contents("https://harvest.greenhouse.io/v1/jobs" as text, [Query = [page = Number.ToText(Page)]]) )This works ok in the PowerQuery screen, as the previous function did too. It is only once loaded that I receive the same error message again:
Thanks for your help, any other tips that might help? I did read in the API documentation:API methods that return a collection of results are always paginated. Paginated results will include a Link (see RFC-5988) response header with the following information.
- next. The corresponding URL is the link to the next page.
- prev. The corresponding URL is the link to the previous page.
- last. The corresponding URL is the link to the last page.
Note that when this header is not set, there is only one page, the first page, of results.
But I am not sure how to adjust the query for this- lbendlin
Super User
Don't use Number.ToText. Use Text.From
- AnonymousNot applicable
Thank you lbendlin I was able to refactor the function, but not sure what you mean with the [Query = [page = Page]] attribute?
Once I have refactored the function, the same error still appears when I load it in desktop. - AnonymousNot applicable
Thank you lbendlin you are a legend! That solved it 🙂
each List.IsEmpty([Result]) <> true,let List = List.Generate(()=> //Where to start and which page we are on //if error we want it to return a null (Try) [Result = try fJobsPosts(1) otherwise null , Page = 1], //We want this to keep running until the result is null each List.IsEmpty([Result]) <> true, //condition for creating the row and itterate throug until it is null each [Result = try fJobsPosts([Page]+1) otherwise null , Page = [Page] + 1], // create the output each [Result]), #"Converted to Table" = Table.FromList(List, Splitter.SplitByNothing(), null, null, ExtraValues.Error), #"Expanded Column1" = Table.ExpandListColumn(#"Converted to Table", "Column1"), #"Expanded Column2" = Table.ExpandRecordColumn(#"Expanded Column1", "Column1", {"id", "active", "live", "first_published_at", "title", "location", "internal", "external", "job_id", "content", "internal_content", "updated_at", "created_at"}, {"id", "active", "live", "first_published_at", "title", "location", "internal", "external", "job_id", "content", "internal_content", "updated_at", "created_at"}), #"Expanded location" = Table.ExpandRecordColumn(#"Expanded Column2", "location", {"name"}, {"Location"}), #"Changed Type" = Table.TransformColumnTypes(#"Expanded location",{{"updated_at", type datetime}, {"created_at", type datetime}, {"first_published_at", type datetime}}), #"Renamed Columns" = Table.RenameColumns(#"Changed Type",{{"title", "Job Title"}, {"first_published_at", "First Published"}, {"id", "Job Post ID"}, {"active", "Active"}, {"live", "Live"}, {"internal", "Internal"}, {"external", "External"}, {"content", "Job Content"}}) in #"Renamed Columns"
Thanks so much for your help 🙂