Add hook_order_pane_alter(&$panes) [#828374]

A couple of different sites that I've maintained have had this issue, and I've decided to post a patch for it in hopes of it becoming standard in Ubercart. In attempts to make changes to the content panes on pages which have no forms to make use of hook_form_alter(), I've had to looked for increasingly more interesting ways to edit and make changes to content panes.

After speaking to togospizza, we came up with the idea of creating our own hook_order_pane_alter(), which will allow you make changes to the panes content before it is submitted and created.

This has been most helpful in allowing me to overload certain ubercart methods which do 99% of what I would like, but the code needs to be slightly tweaked for my clients needs.

Comment	File	Size	Author
	drupal_6-ubercart_2.2-uc_order.order_pane.inc_.patch	700 bytes	xrampage16

Comments

Comment #1

derjochenmeyer commented 4 October 2010 at 11:27

Version:

6.x-2.2

» 6.x-2.x-dev

+1 for this feature.

Patch tested with ubercart 2.x-dev

Comment #2

longwave

he/him

English

commented 4 October 2010 at 14:57

Status:

Patch (to be ported)

» Reviewed & tested by the community

Trivial patch, can see its usefulness.

Comment #3

Island Usurper commented 25 October 2010 at 15:16

Status:

Reviewed & tested by the community

» Needs work

Needs to be documented in hooks.php.

Also, I want to see what you guys think about putting the alter after 'enabled' and 'weight' have been set. It kind of comes down to who should take precedence: other modules or the end user? I can't think of a specific example, but I think alter hooks can usually override a user setting. Maybe this usually happens with the setting form being altered.