Skip to content

Added "offset" to Result. - #130

Open
de-zakauff wants to merge 1 commit into
intacct:masterfrom
de-zakauff:add-offset-to-result-object
Open

Added "offset" to Result.#130
de-zakauff wants to merge 1 commit into
intacct:masterfrom
de-zakauff:add-offset-to-result-object

Conversation

@de-zakauff

Copy link
Copy Markdown

While working with the SDK, I realized that there is no way to get the offset, so if you don't retain that when passing it to the SDK for querying, the value is lost.

  • Added a new "private" field, _offset, which holds the offset value from the result XML, if it is present.
  • Added a public getter for the _offset field.
  • Updated the constructor so that if the Result XML has an offset attribute, it attempts to parse it to an integer and then set it to the _offset field.
  • Updated ResultTest to add tests for the expected offset data.

While working with the SDK, I realized that there is no way to get the offset, so if you don't retain that when passing it to the SDK for querying, the value is lost.

* Added a new "private" field, `_offset`, which holds the offset value from the result XML, if it is present.
* Added a public getter for the `_offset` field.
* Updated the constructor so that if the Result XML has an `offset` attribute, it attempts to parse it to an integer and then set it to the `_offset` field.
* Updated ResultTest to add tests for the expected `offset` data.
@de-zakauff
de-zakauff changed the base branch from master to v2.2.0 May 3, 2024 16:12
@de-zakauff
de-zakauff changed the base branch from v2.2.0 to master May 3, 2024 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant