pma + tax module

I've got the same issue. Not sure, is the puzzle in Ubercart 2 VAT support module or in this one.

Hi, I've opened an issue http://drupal.org/node/844722

have a look here
http://www.ubercart.org/issue/3442/checkout_calculations_not_same_review...
(fix uc_pma.js , and comment how I think module should be changed)

No, seriously, the problem is line_item for payment method is not set when taxes module apply taxes, I workaround this with this code in uc_pma.module (uc_pma_order func):

if ($adjustment['value'] && ($adjustment['value'] >= $min_adjust || $adjustment['value'] <= -$min_adjust)) {
        uc_order_line_item_add($arg1->order_id, 'payment_method', $adjustment['description'], $adjustment['value'], 3);
		$setdone = FALSE; // workaround for taxes module
		foreach($arg1->line_items as $i => $item) {
			if($item['type']=='payment_method') {
				$arg1->line_items[$i]['amount'] = $adjustment['value'];
				$setdone = TRUE;
				break;
			}
		}
		if(!$setdone) {
			$li = db_fetch_array(db_query("SELECT * FROM {uc_order_line_items} WHERE order_id=%d AND type='payment_method'",$arg1->order_id));
			$arg1->line_items[]=$li;
		}
}

this change $arg1 var, so when it is uc_taxes_order turn, it will find a line_items (there is something I do not like here, and I wonder why shipping module works correctly with taxes, while pma dont ...)

I have also problems with pma + tax on the checkout page in payment pane. The payment adjustment is displayed but without taxes. On the total line in review page is everything correct with taxes.

payment_method_adjustment_description() has no mechanism to alter payment prices with taxes.

if (!strstr($adjustment, '%')) {

    ---->Here must be something to alter payment price to add taxes

    $adjustment = uc_currency_format(str_replace(variable_get('uc_currency_dec', '.'), ".", $adjustment));
}

Could someone help me how to get a price includng taxes?

I think it should be a bug issue because also of the other problems with pma + tax.

I created the D5 version of this module, port to D6 was by Longwave. I'm not very familiar with the workings of this module on D6, but if someone has a clean patch that is tested and ready for commit, I will commit it. The patch in #7 scares me a bit and suggests that the fix doesn't need to happen in this module, but needs to happen in Tax code.

If someone is deeply invested in this module and wished to maintain the D6 branch, that would be fine too.

I figured this problem out with my Recycling Fee module. Just set weight of the module (in "system" table) to something under zero (I set -2).
This caueses, that during review its function for store line_items called earlier that tax function and tax will be calculated corect.

There are two issues at play here. Firstly, uc_pma and uc_taxes both do their work in hook_order, so uc_pma's module weight must be lower than uc_taxes in order for tax to be calculated on the PMA line item. This problem will affect modules other than uc_pma, and is already noted at #603356: uc_taxes module's weight should be set heavier on install. The attached patch sets uc_pma's weight to -1 so it will run before uc_taxes.

Secondly, uc_pma does not update the $order object correctly during uc_pma_order. The patch in #7 would fix this, but in the attached patch I just put the line item data straight into the $order object using the data that is already available rather than hitting the database.

When testing the attached patch, please note you must run update.php to update the module weight.

@checker: yours is a slightly different issue that is only applicable when running uc_vat as well, however I think another issue should be opened for this, as it is more problematic to fix - the adjustment may not be taxed if the user is outside the EU, so we cannot just assume that the adjustment price should be taxed in all cases.

Updated patch attached, which includes support for uc_price() and can dynamically reload the adjustment descriptions during checkout. This handles cases where tax rules may apply to certain countries only, so the price can change depending on the user's country selection at checkout.

yes! it works, I still have a problem with number format, I added $adjustment = str_replace(variable_get('uc_currency_dec', '.'), '.', $adjustment);
at line 233 of patched module ie. in function _payment_method_adjustment:

...
if (strstr($adjustment, '%')) {
$percent = TRUE;
$adjustment = str_replace('%', '', $adjustment);
$adjustment = str_replace(variable_get('uc_currency_dec', '.'), '.', $adjustment);
$adjustment /= 100;
}

and all work in Italy ;)

The patch in #13 does what it says on the box. RTBC.

However it won't need the hook_update_N() function which lowers uc_pma's module weight as #603356: uc_taxes module's weight should be set heavier on install has been committed.