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

MtApi5: rename OrderCloseALL() -> PositionCloseALL() #107

Closed
KptKuck opened this issue Mar 25, 2018 · 1 comment
Closed

MtApi5: rename OrderCloseALL() -> PositionCloseALL() #107

KptKuck opened this issue Mar 25, 2018 · 1 comment

Comments

@KptKuck
Copy link

KptKuck commented Mar 25, 2018

I found a litle missmatch .
In mt5 orders and positions are treated separately.

Please rename the function

@vdemydiuk
Copy link
Owner

I have added function
int PositionCloseAll()
that closes all open positions and return count of them.

Function OrderCloseAll set as deprecated. It makes no sense in MT5 and it was copied from MtApi MT4.
It's not possible to close an Order in MT5 since Orders become (or add/subtract to) Positions.

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

No branches or pull requests

2 participants