In this case the rest of transation can't be parsed by wallet. This means that wallet has to stop parsing this tx. The proper fix will be issued later after talking to Amphibian who created this tx.
There are no issues so far with implementing the fix as in my commit, but yes, this is improper error handling
It further parses after without error
It doesn't fail in this case but there is no reason to parse it futher because there is no way to understand it. Your version of wallet that continues to parse unknown data is vulnerable to some possible attacks.