I'm a big believer in the Keep It Simple Stupid (KISS) principle. Assuming you have to keep the basic design (i.e. the table with accordion expander), one of the main things you have to consider is the workflow of your users.
If the majority of the time users would only be expanding the selection to view it, then I would leave it as is. On the other hand, if the majority of the time users are expanding the selection because they want to edit it, then I would consider changing this.
In your question you say: "For this example, expect the user to view 50% and Edit 50%."
I'm not entirely sure what you meant by this, but if you mean that half your users will be editing and half will only be viewing, OR if you mean that typically only half the fields will be edited and half won't be, then I guess from a workflow point of view things are evenly divided. If so, then I would leave it as is because the way you have it now minimises the risk of someone editing something by mistake.
However, one of the factors of a good user interface design is that your design doesn't behave in a way the majority of users don't expect it to. So, despite everything I've said above, if the clear majority of your users expect that expanding the selection would allow them to edit it, then you would have to seriously consider changing it.
Personally, if it was me, I would leave it as is, but I'm not one of the users.