This is just aesthetics. When you go to the customer fields and change the address 2 from hidden to optional, it overwrites a whole bunch of templates. That is to be expected. But what I found was the new line
&mvte:global:Basket:ship_addr2;<br>
Wouldn't it be better to make that
<mvt:if expr="NOT ISNULL g.basket:ship_addr2">
&mvte:global:Basket:ship_addr2;<br>
</mvt:if>
That way if it was empty it would not leave a blank line in the address.
Also, the customer confirmation email template did not get modified automatically.
&mvte:global:Basket:ship_addr2;<br>
Wouldn't it be better to make that
<mvt:if expr="NOT ISNULL g.basket:ship_addr2">
&mvte:global:Basket:ship_addr2;<br>
</mvt:if>
That way if it was empty it would not leave a blank line in the address.
Also, the customer confirmation email template did not get modified automatically.