I have seen this mistake so many times 
Ok, let's start with the basics. All these blocks are located under "on Tick". Which means if there is a new tick - this structure will be executed, starting from the blocks on the top. If any block does what it is supposed to do, it passes and if there is something connected after it - it will be executed as well. The whole process happens in milliseconds, if not less than 1ms. Only if there is a trade to buy, modify or close it can take up until a second. Then, when the next tick comes - everything starts again.
Now you have is, when a tick comes:
Check Condition#1 and if it's true -> create a Sell position. After the position is created, immediately check Condition#4 and it it is true -> go create a Sell position (actually close what we have).
Then do the same starting with Condition#5.
And this all happens because of a single tick!
All I want to say is that there is no time delay between #6 and #7, and between #2 and #4.
Condition#4 will be checked only after a new Sell is created by block #2.
If for some reason Condition#1 and Condition#4 can be true at the same time, you will got a new position that is closed immediately. It there are no moments when both conditions can be true - you will not reach to block #3 at all.
But you maybe have very crazy result - many positions created and closed in no time. This is because your conditions to open a new position does not include the question - is there a position at the moment? And "Buy now" and "Sell now" does not ask this question, they just to what their title is telling until you are out of money.
Hint: You can use "If position is running" or "No position is running". Inside those blocks you can also filter by type (buy or sell)