Suppressed payment fields cause error for events requiring approval

[Stuart Parker]

CRM-12279 suppressed the payment fields for registration where an event requires approval:
https://issues.civicrm.org/jira/browse/CRM-12279

However, when someone tries to register for an event, this causes an error on the next screen, which tries to display the fee:

Warning: Invalid argument supplied for foreach() in CRM_Event_Form_Registration_Confirm->buildQuickForm() (line286 of /xxx/sites/all/modules/civicrm/CRM/Event/Form/Registration/Confirm.php).

I have duplicated this error on the demo site by setting up an event "Requires approval". If you use the event link to try to register someone on the event "Requires approval" it comes up with this error.

I think it's better if the person registering can select the fee on the first registration screen, and this would avoid the error. It looks like removing "|| $this->_requireApproval" from line 97 of CRM/Event/Form/Registration/Register.php would sort it out?

If no-one objects to this, I'll raise an issue