spree/spree — pull request #1737 by steveh (2012-07-03 02:50:13+00:00)
Title: Rationalize admin form data-hooks

This PR is more to illustrate an issue and start a discussion than something I expect to be merged as-is.

Currently, a minority of the admin forms have suitable data-hooks to insert new fields, and those that do are not consistently named.

There is admin_mail_method_form_fields (which I standardized on arbitrarily), but also things like admin_shipment_form_details and admin_inside_taxon_form and admin_product_form_additional_fields.

This pain point initially came about when I tried to add a new field to the OptionType form (in both edit and new views), and couldn't find a hook that was clear.

This diff makes more sense if you ignore whitespace as most of it is changing indentation. `git diff 1-1-stable -w`