unepwcmc/SAPI — pull request #237 by agnessa (2014-02-26 13:53:39+00:00)
Title: split trade/exports_controller and trade_shipments_controller

The admin trade controllers are currently protected by http auth, and in the future we will hopefully have app level authentication on that namespace for simplicity. Therefore, the public reports need to be served by separate controllers. Separating them produced little duplication, because in fact the old controllers were working quite differently for public / admin web outputs.
An important thing follows from this change in that we no longer rely on passing the 'internal' parameter to unlock admin features. The admin controller will set that flag and the public one won't, which simplifies things in terms of access rights.
This change affects all the reports, both public and admin, both web and csv. May the force be with us.