Compared to other tables within ubercart uc_cart_view_table isn't alterable via TAPIr API because it's not build using tapir_get_table. So the corresponding hooks aren't called.
Due to the fact that I'm completely new to ubercart I'm not 100% sure if my attached patch has any side effects, but for me it seems that somebody simply forgot to use tapir_get_table.
| Comment | File | Size | Author |
|---|---|---|---|
| uc_cart.module.tapir_table.patch | 464 bytes | mkalkbrenner |
Comments
Comment #1
Island Usurper commentedTAPIr tables are now form elements, so they can be modified by hook_form_alter() when they are actually part of a form. tapir_get_table() is used in the place of drupal_get_form() when all of the extra form API isn't needed (validation, submitting, caching, etc.).
I think the reason to do this is the fact that the table is so integrated into the form that you have to have the entire $form array available to modify the table successfully. When you add columns, you might need to modify the form's theme function to render them, or something.
Comment #2
mkalkbrenner@Island Usurper:
You might be right, but in this case the usage of tapir tables is inconsistent. These modules integrate a tapir table in a form using tapir_get_table:
uc_payment => uc_payment_by_order_form
uc_order => uc_order_edit_products_form
Comment #3
Island Usurper commentedOh, well, never mind. I probably did just forget to use it there. Committed.