Amit Dodiya (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/6.0-opw-51082-ado into 
lp:openobject-addons/6.0.

Requested reviews:
  Amit Dodiya (OpenERP) (ado-openerp)
  Vinay Rana (openerp) (vra-openerp)
  Priyesh (OpenERP) (pso-openerp)
  nel (nel-tinyerp)
Related bugs:
  Bug #856780 in OpenERP Addons: "[6.0] Update average price creates incorrect 
entries"
  https://bugs.launchpad.net/openobject-addons/+bug/856780

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-51082-ado/+merge/92951

Hello,

" [FIX] : Update real stock creates journal entries with wrong accounts"

V6 anglo-saxon accounting. Product with average cost, used action button to 
update average price. Entries created were:
DR Stock Variation Account
CR Stock Output Account

There will never be a supplier invoice for this adjustment - CR should go to 
Price Difference Account


Thanks,
Amit
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/6.0-opw-51082-ado/+merge/92951
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/6.0-opw-51082-ado.
=== modified file 'account_anglo_saxon/product.py'
--- account_anglo_saxon/product.py	2011-01-14 00:11:01 +0000
+++ account_anglo_saxon/product.py	2012-02-14 11:27:22 +0000
@@ -19,6 +19,7 @@
 ##############################################################################
 
 from osv import fields, osv
+from tools.translate import _
 
 class product_category(osv.osv):
     _inherit = "product.category"
@@ -86,6 +87,31 @@
     }
 product_template()
 
+class product_product(osv.osv):
+    _inherit = "product.product"
+    
+    def do_change_standard_price(self, cr, uid, ids, datas, context=None):
+        """ Changes the Standard Price of Product and creates an account move accordingly.
+        @param datas : dict. contain default datas like new_price, stock_output_account, stock_input_account, stock_journal
+        @param context: A standard dictionary
+        @return:
+
+        """
+        product_obj=self.browse(cr, uid, ids, context=context)[0]
+        stock_price_diff_account = product_obj.property_account_creditor_price_difference and product_obj.property_account_creditor_price_difference.id or False
+        
+        if not stock_price_diff_account:
+            stock_price_diff_account = product_obj.categ_id.property_account_creditor_price_difference_categ and product_obj.categ_id.property_account_creditor_price_difference_categ.id or False
+        if not stock_price_diff_account:
+            raise osv.except_osv(_('Error!'),_('There is no price diffrent account defined ' \
+                                               'for this product: "%s" (id: %d)') % (product_obj.name, product_obj.id,))
+        datas['stock_input_account'] = stock_price_diff_account
+        datas['stock_output_account'] = stock_price_diff_account
+            
+        return super(product_product, self).do_change_standard_price(cr, uid, ids, datas, context)
+    
+product_product()
+
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to