Move hook_field_views_data to commerce_customer.views.inc

Hi guys,

Following up with #1319810: Improve Entity Relationships in Views, I was brought to look closer at the Product Reference and Customer modules.

Currently, both modules seem to implement hook_field_views_data in different places and I was wondering if it would be possible to make the code consistent:

• Product Reference seems to implement hook_field_views_data in commerce_product_reference.views.inc, line 11:
  function commerce_product_reference_field_views_data($field)
  which I would assume would be the right file for this hook's implementation (the .views.inc file).
• Customer, which has its own reference field commerce_customer_profile_reference defined in commerce_customer.module, line 759, also seems to implement hook_field_views_data in commerce_customer.module, line 423:
  function commerce_customer_field_views_data($field)
  which I was a little surprised to find in this file, since I had initially been looking at commerce_product_reference.views.inc. So logically, I looked at commerce_customer.views.inc but couldn't find it there.

Would there be any particular reason why hook_field_views_data would be implemented in commerce_customer.module rather than commerce_customer.views.inc?
 

This may sound pretty minor, but I think it would be a rather easy/simple change that could get modules' code aligned with the same consistency.
I would greatly appreciate to have your feedback on that and if you could let me know if I overlooked or missed anything in any of these modules' implementations or APIs.

Feel free to let me know if you would have any questions, comments or concerns on any aspects of the discussed implementations, I would be glad to explain in more details.

Thanks very much to all, in advance, for your comments, feedback, and insight.
Cheers!