In your project I don't understand why you have "If trade" blocks instead of "No trade". "If trade" passes when at least 1 trade exists, but initially you have 0, so how can it ever pass?
Block 41 and block 2 - you can't have both crossovers at the same time. In block 41 better use > instead of x>. When MA5 is > MA231, this means that the last cross was x>, and you don't really care when that happened.