Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fetchxml with top= tag #98

Closed
pyro-mm opened this issue Jun 18, 2021 · 3 comments
Closed

fetchxml with top= tag #98

pyro-mm opened this issue Jun 18, 2021 · 3 comments
Assignees
Labels

Comments

@pyro-mm
Copy link

pyro-mm commented Jun 18, 2021

When adding the top tag to the fetchxml string I always get the error
"Error: The top attribute can't be specified with paging attribute page"

I just added to line 875
if (pageNumber < 0) replacementString = '$1 ';
When I call executeFetchXml(entityname, sFetchXML, null, -1, null) it seems to work.

@AleksandrRogov
Copy link
Owner

hi @pyro-mm , I remember I had to include a paging number in every fetch request because I wanted to retrieve a paging cookie and for some reason it was not doing it back in 8.2 without adding a "paging" attribute to fetch xml, I don't know if it's fixed right now, did not test it.

To be honest, I did not take into account the fact that devs can use "top" inside a fetch xml as well. So, I am going to make some changes and publish a patch asap.

Thanks for submitting the issue!

@AleksandrRogov
Copy link
Owner

ok. it should be fixed in 1.7.4. let me know if it works for you.

@pyro-mm
Copy link
Author

pyro-mm commented Jun 22, 2021

Thank you so much. It works like a charm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants