Incompatible with UC VAT [#735988]

If you use UC VAT module to show all prices with a VAT, the value saved in database is without the VAT and it is added on display.

For example, you save a product with a price of 100 and vat 20%, it saves 80. User sees 100.

But because this is in database, apachesolr_ubercart ndexes the price without the VAT, making price facet unusable - user sees a product with a price of 100 in a result filtering 20-85 for example.

apachesolr_ubercart should check if there is a VAT module. I can produce a patch but the question is - should we alter the price during index phase (index with a VAT) or during search phase (construct a facet title "price from 50-150" with a real values "price [40 TO 80]" ?)

Comment	File	Size	Author
#3	apachesolr_ubercart_price.patch	1.09 KB	meba

Comments

Comment #1

meba commented 8 March 2010 at 13:38

Maybe we should generalize a bit with price handlers...

Comment #2

meba commented 8 March 2010 at 14:03

Which leads to obvious - alteration should happen during search result phase.

Comment #3

meba commented 8 March 2010 at 15:14

Status:

Active

» Needs review

Status	File	Size
new	apachesolr_ubercart_price.patch	1.09 KB

Found out that it's impossible to do (or not easy/not-hackish) this during search - we simply don't have all the required information available in 'facet_fields'.

Attaching a patch that does uc_price alteration during indexing phase. Might need update function to force reindexing of products, new file apachesolr_ubercart.install:

function apachesolr_ubercart_update_6001() {
 $ret = array();
 $types = module_invoke_all('product_types')
 $ret[] = update_sql("UPDATE {apachesolr_search_node} SET changed = %d WHERE nid IN (SELECT nid FROM {node} WHERE type IN (" . db_placeholders($types, 'text') . "))", time(), $types);
 return $ret;
}