Merge lp:~vauxoo/openerp-venezuela-localization/ovl70-rev-modulesdoc-yani into lp:openerp-venezuela-localization

[Yanina Aular (Vauxoo)]

documentation and fixing conflicts with current ovl

[lp:~vauxoo/openerp-venezuela-localization/ovl70-rev-modulesdoc-yani]

828. By Yanina Aular (Vauxoo) on 2013-04-27

[REM] needless files

[Yanina Aular (Vauxoo) (yanina-aular)]

to merge this before than https://code.launchpad.net/~vauxoo/openerp-venezuela-localization/ovl70-rev-taskovl-yani/+merge/161270

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

review: Approve

=== modified file 'l10n_ve_fiscal_requirements/model/invoice.py'
--- l10n_ve_fiscal_requirements/model/invoice.py	2013-04-24 20:27:17 +0000
+++ l10n_ve_fiscal_requirements/model/invoice.py	2013-04-27 02:25:32 +0000
@@ -28,11 +28,10 @@
 class account_invoice(osv.osv):
 
     def _get_journal(self, cr, uid, context=None):
-        '''
-        returns the journal which is 
+        """ Return the journal which is 
         used in the current user's company, otherwise
         it does not exist, return false
-        '''
+        """
         
         if context is None:
             context = {}
@@ -50,11 +49,10 @@
             return super(account_invoice, self)._get_journal(cr, uid, context=context)
     
     def _unique_invoice_per_partner(self, cr, uid, ids, context=None):
-        '''
-        return false when it is found 
+        """ Return false when it is found 
         that the bill is not out_invoice or out_refund,
         and it is not unique to the partner.
-        '''
+        """
         
         if context is None: context={}
         inv_brw = self.browse(cr, uid, ids, context=context)
@@ -109,11 +107,10 @@
          ]
 
     def copy(self, cr, uid, id, default={}, context=None):
-        '''
-        allows you to duplicate a record,
+        """ Allows you to duplicate a record,
         child_ids, nro_ctrl and reference fields are
         cleaned, because they must be unique
-        '''
+        """
         if context is None:
             context = {}
         default.update({

=== modified file 'l10n_ve_fiscal_requirements/model/l10n_ut.py'
--- l10n_ve_fiscal_requirements/model/l10n_ut.py	2013-03-27 02:39:52 +0000
+++ l10n_ve_fiscal_requirements/model/l10n_ut.py	2013-04-27 02:25:32 +0000
@@ -54,12 +54,11 @@
     }
 
     def get_amount_ut(self, cr, uid, date=False, *args):
-        '''
-        returns the value of
+        """ Return the value of
         the tributary unit of the specified date or
         if it's empty return the value to current
         date.
-        '''
+        """
         rate = 0
         date = date or time.strftime('%Y-%m-%d')
         cr.execute("""SELECT amount FROM l10n_ut WHERE date <= '%s'
@@ -69,10 +68,9 @@
         return rate
 
     def compute(self, cr, uid, from_amount, date=False, context=None):
-        '''
-        returns the number of tributary
+        """ Return the number of tributary
         units depending on an amount of money.
-        '''
+        """
         if context is None:
             context = {}
         result = 0.0
@@ -83,10 +81,8 @@
 
     def compute_ut_to_money(self, cr, uid, amount_ut, date=False,
                             context=None):
-        '''
-        transforms from money to
-        tributary units
-        '''
+        """ Transforms from money to tributary units
+        """
         if context is None:
             context = {}
         money = 0.0

=== modified file 'l10n_ve_fiscal_requirements/model/partner.py'
--- l10n_ve_fiscal_requirements/model/partner.py	2013-03-26 23:19:12 +0000
+++ l10n_ve_fiscal_requirements/model/partner.py	2013-04-27 02:25:32 +0000
@@ -41,33 +41,25 @@
     _inherit = 'res.partner'
 
     def _get_country_code(self, cr, uid, context=None):
-        '''
-        return the country code
-        of the user company. If not exists, 
-        return XX.
-        '''
+        """ Return the country code of the user company. If not exists, return XX.
+        """
         context = context or {}
         user_company = self.pool.get('res.users').browse(cr, uid, uid, context=context).company_id
         return user_company.partner_id and user_company.partner_id.country_id \
                 and user_company.partner_id.country_id.code or 'XX'
 
     def default_get(self, cr, uid, fields, context=None):
-        '''
-        load the country code
-        of the user company to form to be
-        created.
-        '''
+        """ Load the country code of the user company to form to be created.
+        """
         context = context or {}
         res = super(res_partner, self).default_get(cr, uid, fields, context=context)
         res.update({'uid_country': self._get_country_code(cr,uid,context=context)})
         return res
 
     def _get_uid_country(self, cr, uid, ids, field_name, args, context=None):
-        '''
-        returns a dictionary of key 
-        ids as invoices, and value the country code
+        """ Return a dictionary of key ids as invoices, and value the country code
         of the user company.
-        '''
+        """
         context = context or {}
         res= {}.fromkeys(ids,self._get_country_code(cr,uid,context=context))
         return res
@@ -82,11 +74,8 @@
     }
 
     def name_search(self,cr,uid,name='',args=[],operator='ilike',context=None,limit=80):
-        '''
-        gets el id of the partner
-        with the vat or the name and return the
-        name
-        '''
+        """ Gets el id of the partner with the vat or the name and return the name
+        """
         if context is None: 
             context={}
         ids= []
@@ -100,11 +89,9 @@
     Required Invoice Address
     '''
     def _check_partner_invoice_addr(self,cr,uid,ids,context={}):
-        '''
-        returns true if the partner
-        is a company of Venezuela and if the
+        """ Return true if the partner is a company of Venezuela and if the
         address is for billing.
-        '''
+        """
         partner_obj = self.browse(cr,uid,ids[0])
         if partner_obj.vat and partner_obj.vat[:2].upper() == 'VE' and not partner_obj.parent_id:
                 res = partner_obj.type == 'invoice'
@@ -117,13 +104,10 @@
         return True
 
     def _check_vat_uniqueness_def(self, cr, uid, ids, current_vat,list_node_tree, context=None):
-        '''
-        receiving nodes in the tree 
-        that are not members of the current partner
-        level, and the remainder of the assembly 
-        to ensure that the partner at the same
+        """ Receiving nodes in the tree that are not members of the current partner
+        level, and the remainder of the assembly to ensure that the partner at the same
         level as the current vat has different.
-        '''
+        """
         
         if context is None: context = {}
         nodes = self.search(cr, uid, [] )
@@ -132,10 +116,8 @@
         return not nodes
 
     def _check_vat_uniqueness(self, cr, uid, ids, context=None):
-        '''
-        check that the vat is unique
-        in the level where the partner in the tree
-        '''
+        """ Check that the vat is unique in the level where the partner in the tree
+        """
         if context is None: context = {}
         
         user_company = self.pool.get('res.users').browse(cr, uid, uid).company_id
@@ -168,7 +150,7 @@
         return True    
 
     def _check_vat_mandatory(self, cr, uid, ids, context=None):
-        '''This method will check the vat mandatoriness in partners
+        """ This method will check the vat mandatoriness in partners
         for those user logged on with a Venezuelan Company
         
         The method will return True when:
@@ -180,7 +162,7 @@
                 +) partner is_company=True AND parent_id is not NULL
                 +) partner with parent_id is NULL 
                 +) partner with parent_id is NOT NULL AND type of address is invoice   
-            '''
+        """
         if context is None: context = {}
         # Avoiding Egg-Chicken Syndrome
         # TODO: Refine this approach this is big exception
@@ -214,9 +196,8 @@
         return True
 
     def _validate(self, cr, uid, ids, context=None):
-        '''
-        validates the fields
-        '''
+        """ Validates the fields
+        """
             
         #In the original orm.py openerp does not allow using
         #context within the constraint because we have to yield 
@@ -259,9 +240,8 @@
     ]
  
     def vat_change_fiscal_requirements(self, cr, uid, ids, value, context=None):
-        '''
-        checks the syntax of the vat
-        '''
+        """ Checks the syntax of the vat
+        """
         if context is None:
             context={}
         if not value:
@@ -280,10 +260,9 @@
             return super(res_partner,self).vat_change(cr, uid, ids, value, context=context)
 
     def check_vat_ve(self, vat, context = None):
-        '''
-        Check Venezuelan VAT number, locally called RIF.
+        """ Check Venezuelan VAT number, locally called RIF.
         RIF: JXXXXXXXXX RIF VENEZOLAN IDENTIFICATION CARD: VXXXXXXXXX FOREIGN IDENTIFICATION CARD: EXXXXXXXXX
-        '''
+        """
         
         if context is None:
             context={}
@@ -294,21 +273,17 @@
         return False
         
     def update_rif(self, cr, uid, ids, context=None):
-        '''
-        load the rif and name of the partner
-        from the database seniat
-        '''
+        """ Load the rif and name of the partner from the database seniat
+        """
         if context is None:
             context = {}
         su_obj = self.pool.get('seniat.url')
         return su_obj.update_rif(cr, uid, ids, context=context)
 
     def button_check_vat(self, cr, uid, ids, context=None):
-        '''
-        is called by the button that load
-        information of the partner from database 
+        """ Is called by the button that load information of the partner from database 
         SENIAT
-        '''
+        """
         if context is None: context = {}
         context.update({'update_fiscal_information':True})
         super(res_partner, self).button_check_vat(cr, uid, ids, context=context)

=== modified file 'l10n_ve_fiscal_requirements/model/res_company.py'
--- l10n_ve_fiscal_requirements/model/res_company.py	2013-04-24 20:27:17 +0000
+++ l10n_ve_fiscal_requirements/model/res_company.py	2013-04-27 02:25:32 +0000
@@ -40,22 +40,20 @@
         }
 
     def create(self, cr, uid, vals, context=None):
-        '''
-        to create a new record,
+        """ To create a new record,
         adds a Boolean field to true
         indicates that the partner is a company
-        '''
+        """
         if context is None:
             context = {}
         context.update({'create_company': True})
         return super(res_company, self).create(cr, uid, vals, context=context)
 
     def write(self, cr, uid, ids, values, context=None):
-        '''
-        to write a new record,
+        """ To write a new record,
         adds a Boolean field to true
         indicates that the partner is a company
-        '''
+        """
         context = context or {}
         context.update({'create_company': True})
         return super(res_company, self).write(cr, uid, ids, values, context=context)

=== modified file 'l10n_ve_fiscal_requirements/model/res_users.py'
--- l10n_ve_fiscal_requirements/model/res_users.py	2013-03-26 23:20:54 +0000
+++ l10n_ve_fiscal_requirements/model/res_users.py	2013-04-27 02:25:32 +0000
@@ -35,21 +35,19 @@
     _inherit = 'res.users'
         
     def create(self, cr, uid, vals, context=None):
-        '''
-        to create a new record,
+        """ To create a new record,
         adds a Boolean field to true 
         indicates that the partner is a company
-        '''
+        """
         if context is None: context = {}
         context.update({'create_company':True})
         return super(res_users, self).create(cr, uid, vals, context=context)
         
     def write(self, cr, uid, ids, values, context=None):
-        '''
-        to write a new record,
+        """ To write a new record,
         adds a Boolean field to true 
         indicates that the partner is a company
-        '''
+        """
         context = context or {}
         context.update({'create_company':True})
         return super(res_users, self).write(cr, uid, ids, values, context=context)

=== modified file 'l10n_ve_fiscal_requirements/model/seniat_url.py'
--- l10n_ve_fiscal_requirements/model/seniat_url.py	2013-04-25 20:11:20 +0000
+++ l10n_ve_fiscal_requirements/model/seniat_url.py	2013-04-27 02:25:32 +0000
@@ -33,8 +33,7 @@
 
 
 class seniat_url(osv.osv):
-    """
-    OpenERP Model : seniat_url
+    """ OpenERP Model : seniat_url
     """
     _name = 'seniat.url'
     _description = "Seniat config needed to run auto-config partner"
@@ -107,10 +106,8 @@
         return False
 
     def _load_url(self, retries, url):
-        '''
-        check that the seniat url is loaded
-        correctly
-        '''
+        """ Check that the seniat url is loaded correctly
+        """
         str_error = '404 Not Found'
         while retries > 0:
             try:
@@ -127,10 +124,8 @@
         return str_error
 
     def _buscar_porcentaje(self, cr, uid, rif):
-        '''
-        finds the percentage
-        to withhold the withholding agent.
-        '''
+        """ Finds the percentage to withhold the withholding agent.
+        """
         url_obj = self.browse(cr, uid, self.search(cr, uid, []))[0]
         url = url_obj.url_seniat + '%s'
         context = {}
@@ -146,10 +141,8 @@
             return 0.0
 
     def _parse_dom(self, cr, uid, dom, rif, url_seniat, context={}):
-        '''
-        this function extracts the information
-        partner of the string and returns
-        '''
+        """ This function extracts the information partner of the string and returns
+        """
         rif_aux = dom.childNodes[0].getAttribute('rif:numeroRif')
         name = dom.childNodes[0].childNodes[0].firstChild.data
         wh_agent = dom.childNodes[0].childNodes[
@@ -165,18 +158,14 @@
                 'wh_iva_agent': wh_agent}
 
     def _print_error(self, error, msg):
-        '''
-        shows an error on the screen
-        '''
+        """ Shows an error on the screen
+        """
         raise osv.except_osv(error, msg)
 
     def _eval_seniat_data(self, xml_data, vat, context={}):
-        '''
-        returns false when there was
-        no error in the query in url SENIAT and
-        return true when there was error in the
-        query.
-        '''
+        """ Returns false when there was no error in the query in url SENIAT and
+        return true when there was error in the query.
+        """
         if context is None:
             context = {}
         if not context.get('all_rif'):
@@ -196,10 +185,8 @@
                 return False
 
     def _get_rif(self, cr, uid,  vat, url1, url2, context=None):
-        '''
-        partner information transforms
-        XML to string and returns.
-        '''
+        """ Partner information transforms XML to string and returns.
+        """
         if context is None:
             context = {}
 
@@ -209,11 +196,10 @@
             return self._parse_dom(cr, uid, dom, vat, url2, context=context)
 
     def _dom_giver(self, cr, uid, vat, context=None):
-        '''
-        check and validates that the vat is a passport,
+        """ Check and validates that the vat is a passport,
         id or rif, to send information to SENIAT and returns the
         partner info that provides.
-        '''
+        """
         if context is None:
             context = {}
 
@@ -243,18 +229,14 @@
                 return self._get_rif(cr, uid, vat, url1, url2, context=context)
 
     def _update_partner(self, cr, uid, id, context=None):
-        '''
-        indicates that the partner was
-        updated with information provided by seniat
-        '''
+        """ Indicates that the partner was updated with information provided by seniat
+        """
         rp_obj = self.pool.get('res.partner')
         rp_obj.write(cr, uid, id, {'seniat_updated': True})
 
     def update_rif(self, cr, uid, ids, context={}):
-        '''
-        updates the partner info if it have a vat,
-
-        '''
+        """ Updates the partner info if it have a vat
+        """
         aux = []
         rp_obj = self.pool.get('res.partner')
         if context.get('exec_wizard'):
@@ -278,11 +260,9 @@
         return True
 
     def connect_seniat(self, cr, uid, ids, context={}, all_rif=False):
-        '''
-        adds true value to the field
-        all_rif to denote that rif was charged with
+        """ Adds true value to the field all_rif to denote that rif was charged with
         SENIAT database
-        '''
+        """
         ctx = context.copy()
         if all_rif:
             ctx.update({'all_rif': True})

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_debit.py'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_debit.py	2013-03-01 17:48:52 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_debit.py	2013-04-27 02:25:32 +0000
@@ -40,6 +40,8 @@
     }
 
     def _get_journal(self, cr, uid, context=None):
+        """ Return partner journal depending of the invoice type
+        """
         obj_journal = self.pool.get('account.journal')
         if context is None:
             context = {}
@@ -61,6 +63,8 @@
     }
 
     def fields_view_get(self, cr, uid, view_id=None, view_type=False, context=None, toolbar=False, submenu=False):
+        """ Change fields position in the view 
+        """
         if context is None:
             context = {}
         
@@ -82,8 +86,7 @@
         return res
 
     def _get_orig(self, cr, uid, inv, ref, context=None):
-        """
-        Return  default origin value
+        """ Return default origin value
         """
         if context is None:
             context = {}
@@ -94,11 +97,10 @@
         return orig
 
     def compute_debit(self, cr, uid, ids, context=None):
-        """
-        @param cr: the current row, from the database cursor,
-        @param uid: the current user’s ID for security checks,
-        @param ids: the account invoice refund’s ID or list of IDs
-
+        """ Create a debit note
+        @param cr: The current row, from the database cursor,
+        @param uid: The current user’s ID for security checks,
+        @param ids: The account invoice refund’s ID or list of IDs
         """
         inv_obj = self.pool.get('account.invoice')
         reconcile_obj = self.pool.get('account.move.reconcile')
@@ -216,6 +218,8 @@
         return result
 
     def invoice_debit(self, cr, uid, ids, context=None):
+        """ Call method compute_debit
+        """
         if context is None:
             context = {}
         return self.compute_debit(cr, uid, ids, context=context)

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_parent.py'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_parent.py	2013-04-18 15:22:33 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_parent.py	2013-04-27 02:25:32 +0000
@@ -43,6 +43,8 @@
     }
 
     def _get_partner(self, cr, uid, context=None):
+        """ Return invoice partner
+        """
         inv_obj = self.pool.get('account.invoice')
         if context is None:
             context = {}
@@ -57,11 +59,10 @@
     }
 
     def fields_view_get(self, cr, uid, view_id=None, view_type=False, context=None, toolbar=False, submenu=False):
-        
+        """ Change fields position in the view 
+        """
         if context is None:
             context = {}
-        
-        
         journal_obj = self.pool.get('account.journal')
         res = super(account_invoice_parent,self).fields_view_get(cr, uid, view_id=view_id, view_type=view_type, context=context, toolbar=toolbar, submenu=submenu)
         if view_type == 'form':
@@ -105,12 +106,20 @@
         return res
 
     def default_get(self, cr, uid, fields, context=None):
+        """ Change value for default of the type field
+        """
         res = super(account_invoice_parent, self).default_get(cr, uid, fields, context=context)
         if context.get('op_type', False):
             res.update({'type': context.get('op_type', 'modify')})
         return res
 
     def get_window(self, cr, uid, ids, xml_id, module, op_type, partner_id, parent_id=False, context=None):
+        """ Update values (op_type, partner_id and parent_id) in the window  
+        @param xml_id:
+        @param op_type:
+        @param partner_id:
+        @param parent_id:
+        """
         mod_obj = self.pool.get('ir.model.data')
         act_obj = self.pool.get('ir.actions.act_window')
         #we get the model
@@ -125,16 +134,25 @@
         return result
 
     def check_sure(self, cr, uid, ids, ok, context=None):
+        """ Checks if the user is sure 
+        """
         if not ok:
             raise osv.except_osv(_('User Error'), _('Assign parent invoice, Please check the box to confirm that you agree!'))
         return True
 
     def check_recursion(self, cr, uid, ids, child_id,parent_id, context=None):
+        """ Checks that have not recursion between parent and children   
+        @param child_id: child id
+        @param parent_id: parent id
+        """
         if child_id == parent_id:
             raise osv.except_osv(_('User Error'), _('Current invoice is the same father invoice, Credit or debit note have to be diferent of parent invoice, Please choise another one!'))
         return True
 
     def check_grandfather(self, cr, uid, ids, parent_id, context=None):
+        """ Check that parent_id having parent
+        @param parent_id:
+        """
         inv_obj = self.pool.get('account.invoice')
         inv_parent_brw = inv_obj.browse(cr, uid, parent_id, context=context)
         if inv_parent_brw.parent_id:
@@ -142,6 +160,9 @@
         return True
 
     def action_assigned(self, cr, uid, ids, form, context=None):
+        """ Check that credit or debit note having assigned invoice
+        @param form: fields values
+        """
         self.check_sure(cr, uid, ids, form['sure'], context)
         active_id = context.get('active_id', False)
         parent_id = form.get('parent_id', False)
@@ -163,6 +184,9 @@
 
 
     def action_unlink(self, cr, uid, ids, form, context=None):
+        """ Remove the parent of the partner
+        @param form: fields values parent_id and partner_id
+        """
         self.check_sure(cr, uid, ids, form['sure'], context)
         active_id = context.get('active_id', False)
         parent_id = form.get('parent_id', False)
@@ -178,6 +202,9 @@
         return result
 
     def action_modify(self, cr, uid, ids, form, context=None):
+        """ Modify parent of the partner
+        @param form: fields values parent_id and partner_id  
+        """
         self.check_sure(cr, uid, ids, form['sure'], context)
         active_id = context.get('active_id', False)
         parent_id = form.get('parent_id', False)
@@ -196,6 +223,8 @@
 
 
     def invoice_operation(self, cr, uid, ids, context=None):
+        """ General method that calls a function depending of the data['type']
+        """
         if context is None:
             context = {}
         data = self.read(cr, uid, ids)[0]

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py	2013-04-17 03:11:25 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py	2013-04-27 02:25:32 +0000
@@ -26,8 +26,8 @@
 from openerp import netsvc
 
 class account_invoice_refund(osv.osv_memory):
-
     """Refunds invoice"""
+
     _inherit = 'account.invoice.refund'
     
     def default_get(self, cr, uid, fields, context=None):
@@ -69,6 +69,8 @@
             }
 
     def _get_journal(self, cr, uid, context=None):
+        """ Return journal depending of the invoice type
+        """
         obj_journal = self.pool.get('account.journal')
         if context is None:
             context = {}
@@ -79,7 +81,8 @@
         return journal and journal[0] or False
 
     def fields_view_get(self, cr, uid, view_id=None, view_type=False, context=None, toolbar=False, submenu=False):
-        
+        """ Depending on context, options are displayed in the selection field
+        """
         if context is None:
             context = {}
         
@@ -97,8 +100,7 @@
         return res
 
     def _get_orig(self, cr, uid, inv, ref, context={}):
-        """
-        Return  default origin value
+        """ Return  default origin value
         """
         nro_ref = ref
         if inv.type == 'out_invoice':
@@ -108,6 +110,8 @@
 
     
     def cn_iva_validate(self, cr, uid,invoice,context=None):
+        """ Validates if retentions have been changes to move the state confirmed and done
+        """
         if context is None:
             context={}
         ret_iva_id=False
@@ -138,11 +142,11 @@
         return True
 
     def compute_refund(self, cr, uid, ids, mode='refund', context=None):
-        """
+        """ 
         @param ids: the account invoice refund’s ID or list of IDs
-
         """
         wzd_brw = self.browse(cr,uid,ids[0],context=context)
+        brw = self.browse(cr,uid,ids[0],context=context)
         inv_obj = self.pool.get('account.invoice')
         reconcile_obj = self.pool.get('account.move.reconcile')
         account_m_line_obj = self.pool.get('account.move.line')
@@ -318,8 +322,7 @@
             return result
 
     def validate_total_payment_inv(self, cr, uid, ids, context=None):
-        """
-        Method that validate if invoice is totally paid.
+        """ Method that validate if invoice is totally paid.
         @param ids: list of invoices.
         return: True: if invoice is paid.
                 False: if invoice is not paid.
@@ -331,8 +334,7 @@
         return res
 
     def validate_wh(self, cr, uid, ids, context=None):
-        """
-        Method that validate if invoice has non-yet processed withholds.
+        """ Method that validate if invoice has non-yet processed withholds.
 
         return: True: if invoice is does not have wh's or it does have and those ones are validated.
                 False: if invoice is does have and those wh's are not yet validated.
@@ -343,8 +345,7 @@
         return True
 
     def unreconcile_paid_invoices(self, cr, uid, invoiceids, context=None):
-        """
-        Method that unreconcile the payments of invoice.
+        """ Method that unreconcile the payments of invoice.
         @param invoiceids: list of invoices.
         return: True: unreconcile successfully.
                 False: unreconcile unsuccessfully.
@@ -374,6 +375,8 @@
         return res
 
     def invoice_refund(self, cr, uid, ids, context=None):
+        """ Create a invoice refund
+        """
         if context is None:
             context = {}
         inv_obj = self.pool.get('account.invoice')

=== modified file 'l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py'
--- l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py	2013-04-27 02:25:32 +0000
@@ -35,6 +35,8 @@
     }
 
     def search_partner_seniat(self, cr, uid, ids, context=None):
+        """ Check vat of the partner and update iva rate
+        """
         if context is None:
             context={}
         this = self.browse(cr, uid, ids)[0]

=== modified file 'l10n_ve_fiscal_requirements/wizard/update_info_partner.py'
--- l10n_ve_fiscal_requirements/wizard/update_info_partner.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/wizard/update_info_partner.py	2013-04-27 02:25:32 +0000
@@ -24,12 +24,11 @@
 from openerp.tools.translate import _
 
 class update_info_partner(osv.osv_memory):
-    """
-    OpenERP osv memory wizard : update_info_partner
-    """
     _name = 'update.info.partner'
     
     def update_info(self, cr, uid, ids, context={}):
+        """ OpenERP osv memory wizard : update_info_partner
+        """
         aux=[]
         seniat_url_obj = self.pool.get('seniat.url')
         cr.execute('''SELECT id FROM res_partner WHERE vat ilike 'VE%';''')

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_invoice_nro_ctrl.py'
--- l10n_ve_fiscal_requirements/wizard/wizard_invoice_nro_ctrl.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_invoice_nro_ctrl.py	2013-04-27 02:25:32 +0000
@@ -39,7 +39,10 @@
     }
 
     def action_invoice_create(self, cr, uid, ids, wizard_brw,inv_brw,context=None):
-
+        """ If the invoice has control number, this function is responsible for passing the bill to damaged paper
+        @param wizard_brw: nothing for now 
+        @param inv_brw: damaged paper
+        """
         invoice_line_obj = self.pool.get('account.invoice.line')
         invoice_obj = self.pool.get('account.invoice')
         acc_mv_obj = self.pool.get('account.move')
@@ -78,9 +81,8 @@
         return inv_brw.id
 
     def new_open_window(self,cr,uid,ids,list_ids,xml_id,module,context=None):
-        '''
-        Generate new window at view form or tree
-        '''
+        """ Generate new window at view form or tree
+        """
         mod_obj = self.pool.get('ir.model.data')
         act_obj = self.pool.get('ir.actions.act_window')
         result = mod_obj._get_id(cr, uid, module, xml_id) 
@@ -90,6 +92,8 @@
         return result
 
     def create_invoice(self, cr, uid, ids, context=None):
+        """ Create a invoice refund
+        """
         if context==None:
             context={}
         wizard_brw = self.browse(cr, uid, ids, context=None)

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_nro_ctrl.py'
--- l10n_ve_fiscal_requirements/wizard/wizard_nro_ctrl.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_nro_ctrl.py	2013-04-27 02:25:32 +0000
@@ -31,6 +31,8 @@
     _description = "Wizard that changes the invoice control number"
 
     def set_noctrl(self, cr, uid, ids, context=None):
+        """ Change control number of the invoice
+        """
         if context is None:
             context={}
         data = self.pool.get('wiz.nroctrl').read(cr, uid, ids)[0]

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_update_name.py'
--- l10n_ve_fiscal_requirements/wizard/wizard_update_name.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_update_name.py	2013-04-27 02:25:32 +0000
@@ -31,6 +31,8 @@
     _description = "Wizard that changes the partner name"
 
     def set_name(self, cr, uid, ids, context):
+        """ Change value of the name field
+        """
         data = self.pool.get('wiz.updatename').read(cr, uid, ids)[0]
         print ids
         if not data['sure']:
@@ -48,6 +50,8 @@
     }
 
     def _get_name(self, cr, uid, context=None):
+        """ Get name field value
+        """
         if context is None:
             context = {}
         partner_obj = self.pool.get('res.partner')

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_url_seniat.py'
--- l10n_ve_fiscal_requirements/wizard/wizard_url_seniat.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_url_seniat.py	2013-04-27 02:25:32 +0000
@@ -28,6 +28,8 @@
 
 
     def _get_url(self,cr,uid,ids,context=None):
+        """ Get seniat web page
+        """
         url= self.pool.get('seniat.url')
         url_ids = url.search(cr, uid,[])
         if len(url_ids)>1:

=== modified file 'l10n_ve_sale_purchase/product.py'
--- l10n_ve_sale_purchase/product.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_sale_purchase/product.py	2013-04-27 02:25:32 +0000
@@ -44,12 +44,11 @@
 
 
 class product_product(osv.osv):
-
     _inherit = "product.product"
-    '''
-    Function that adds a default concept for products that are not service
-    '''
+    
     def onchange_product_type(self, cr, uid, ids, prd_type, context=None):
+        """ Function that adds a default concept for products that are not service
+        """
         domain = {}
         if prd_type != 'service':
             concept_obj = self.pool.get('islr.wh.concept')

=== modified file 'l10n_ve_sale_purchase/purchase.py'
--- l10n_ve_sale_purchase/purchase.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_sale_purchase/purchase.py	2013-04-27 02:25:32 +0000
@@ -42,9 +42,8 @@
     }
     
     def product_id_change(self, cr, uid, ids, pricelist, product, qty, uom, partner_id, date_order=False, fiscal_position=False, date_planned=False, name=False, price_unit=False, notes=False):
-        '''
-        This method loads the withholding concept to a product automatically
-        '''        
+        """ This method loads the withholding concept to a product automatically
+        """        
         def get_concept():
             concept_obj = self.pool.get('islr.wh.concept')
             concept_id = concept_obj.search(cr, uid, [('withholdable','=',False)])
@@ -65,6 +64,8 @@
     _inherit = 'purchase.order'
     
     def _prepare_inv_line(self, cr, uid, account_id, order_line, context=None):
+        """ Update concept_id field to the prepare line
+        """
         data = super(purchase_order, self)._prepare_inv_line( cr, uid, account_id, order_line, context=context)
         data.update({'concept_id':order_line and order_line.concept_id and order_line.concept_id.id   })
         return data

=== modified file 'l10n_ve_sale_purchase/sale.py'
--- l10n_ve_sale_purchase/sale.py	2013-04-24 19:47:03 +0000
+++ l10n_ve_sale_purchase/sale.py	2013-04-27 02:25:32 +0000
@@ -56,6 +56,8 @@
 
     #~method that adds to the original in concept retention lines for sale, is called by action_invoice_create() in sale.order
     def invoice_line_create(self, cr, uid, ids, context={}):
+        """ Assing concept_id to the invoice lines
+        """
         create_ids = super(sale_order_line, self).invoice_line_create(cr, uid, ids, context)
         invoice_line_brws = self.pool.get('account.invoice.line').browse(cr, uid, create_ids)
         order_line_sale_brws = self.pool.get('sale.order.line').browse(cr, uid, ids)

=== modified file 'l10n_ve_sale_purchase/stock.py'
--- l10n_ve_sale_purchase/stock.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_sale_purchase/stock.py	2013-04-27 02:25:32 +0000
@@ -33,10 +33,10 @@
 
 class stock_picking(osv.osv):
     _inherit = 'stock.picking'
-    '''
-    Function that adds the concept of retention to the invoice_lines from a purchase order or sales order with billing method from picking list
-    '''
     def action_invoice_create(self, cursor, user, ids, journal_id=False,group=False, type='out_invoice', context=None):
+        """ Function that adds the concept of retention to the invoice_lines from 
+        a purchase order or sales order with billing method from picking list
+        """
         if context is None:
             context = {}
         data = super(stock_picking, self).action_invoice_create(cursor, user, ids, journal_id, group, type, context)

=== modified file 'l10n_ve_split_invoice/installer.py'
--- l10n_ve_split_invoice/installer.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_split_invoice/installer.py	2013-04-27 02:25:32 +0000
@@ -29,23 +29,23 @@
 import openerp.addons
 
 class split_invoice_config(osv.osv_memory):
-    """
-    Fiscal Requirements installer wizard
+    """ Fiscal Requirements installer wizard
     """
     _name = 'split.invoice.config'
     _inherit = 'res.config'
     _description= __doc__
 
     def default_get(self, cr, uid, fields_list=None, context=None):
+        """ Default value to the config_logo field
+        """
         defaults = super(split_invoice_config, self).default_get(cr, uid, fields_list=fields_list, context=context)
         logo = open(addons.get_module_resource('l10n_ve_split_invoice', 'images', 'puente-maracaibo.jpg'), 'rb')
         defaults['config_logo'] = base64.encodestring(logo.read())
         return defaults
 
     def execute(self, cr, uid, ids, context=None):
-        '''
-        In this method I will configure the maximum number of lines in your invoices.
-        '''
+        """ In this method I will configure the maximum number of lines in your invoices.
+        """
         wiz_data = self.browse(cr, uid, ids[0])
         if wiz_data.name < 1 :
              raise osv.except_osv(_('Error !'), _('The number of customer invoice lines must be at least one'))

=== modified file 'l10n_ve_split_invoice/invoice.py'
--- l10n_ve_split_invoice/invoice.py	2013-03-01 17:48:52 +0000
+++ l10n_ve_split_invoice/invoice.py	2013-04-27 02:25:32 +0000
@@ -31,9 +31,8 @@
     _inherit = 'account.invoice'
 
     def split_invoice(self, cr, uid, ids):
-        '''
-        Split the invoice when the lines exceed the maximum set for the company
-        '''
+        """ Split the invoice when the lines exceed the maximum set for the company
+        """
         for inv in self.browse(cr, uid, ids):
             inv_id =False
             if inv.company_id.lines_invoice < 1:
@@ -75,6 +74,8 @@
         return True
 
     def action_date_assign(self, cr, uid, ids, *args):
+        """ Return assigned dat
+        """
         data = super(account_invoice, self).action_date_assign(cr, uid, ids, *args)
         self.split_invoice(cr,uid,ids)
         return True

=== modified file 'l10n_ve_withholding/model/account.py'
--- l10n_ve_withholding/model/account.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_withholding/model/account.py	2013-04-27 02:25:32 +0000
@@ -73,15 +73,14 @@
     _inherit = "account.period"
 
     def find_fortnight(self, cr, uid, dt=None, context=None):
-        '''
-        This Function returns a tuple composed of 
+        """ This Function returns a tuple composed of 
             *) period for the asked dt (int)
             *) fortnight for the asked dt (boolean):
                 -) False: for the 1st. fortnight
                 -) True: for the 2nd. fortnight.
             Example:
             (3,True) => a period whose id is 3 in the second fortnight
-        '''
+        """
         if context is None: context = {}
         if not dt:
             dt = time.strftime('%Y-%m-%d')

=== modified file 'l10n_ve_withholding/model/account_move_line.py'
--- l10n_ve_withholding/model/account_move_line.py	2013-04-17 17:47:36 +0000
+++ l10n_ve_withholding/model/account_move_line.py	2013-04-27 02:25:32 +0000
@@ -31,9 +31,8 @@
     _inherit = 'account.move.line'
     
     def _models_retencion_get(self, cr, uid, context={}):
-        '''
-        List document for link with account entry
-        '''
+        """ List document for link with account entry
+        """
         obj = self.pool.get('ir.model.fields')
         wh_doc_obj = self.pool.get('account.wh.doc')
         wh_doc_ids = wh_doc_obj.search(cr, uid, [])
@@ -49,9 +48,8 @@
         return res
     
     def _document_get(self, cr, uid, ids, field_name, arg, context=None):
-        '''
-        Link document with account entry
-        '''
+        """ Link document with account entry
+        """
         res = {}
         obj = self.pool.get('ir.model.fields')
 

=== modified file 'l10n_ve_withholding/model/invoice.py'
--- l10n_ve_withholding/model/invoice.py	2013-03-26 03:26:02 +0000
+++ l10n_ve_withholding/model/invoice.py	2013-04-27 02:25:32 +0000
@@ -37,10 +37,9 @@
                             pay_journal_id, writeoff_acc_id, 
                             writeoff_period_id, writeoff_journal_id, date, 
                             name, context=None):
-        '''
-        Function openerp is rewritten for adaptation in
+        """ Function openerp is rewritten for adaptation in
         the ovl
-        '''
+        """
         if context is None: context = {}
         return []
         
@@ -48,9 +47,8 @@
                             period_id, pay_journal_id, writeoff_acc_id, 
                             writeoff_period_id, writeoff_journal_id, date, 
                             name, to_wh, context=None):
-        '''
-        make the payment of the invoice 
-        '''
+        """ Make the payment of the invoice 
+        """
         if context is None:
             context = {}
         
@@ -109,9 +107,8 @@
 
     
     def ret_payment_get(self, cr, uid, ids, *args):
-        '''
-        returns payments associated with this bill
-        '''
+        """ Return payments associated with this bill
+        """
         for invoice in self.browse(cr, uid, ids):
             moves = self.move_line_id_payment_get(cr, uid, [invoice.id])
             src = []
@@ -135,11 +132,10 @@
     
     
     def check_tax_lines(self, cr, uid, inv, compute_taxes, ait_obj):
-        '''
-        check if no tax lines are created. If 
+        """ Check if no tax lines are created. If 
         existing tax lines, there are checks on the invoice 
         and match the tax base.
-        '''
+        """
         if not inv.tax_line:
             for tax in compute_taxes.values():
                 ait_obj.create(cr, uid, tax)
@@ -174,10 +170,9 @@
     }
 
     def compute(self, cr, uid, invoice_id, context=None):
-        '''
-        calculates the amount, base, tax amount,
+        """ Calculate the amount, base, tax amount,
         base amount of the invoice
-        '''
+        """
         
         tax_grouped = {}
         tax_obj = self.pool.get('account.tax')

=== modified file 'l10n_ve_withholding_islr/model/installer.py'
--- l10n_ve_withholding_islr/model/installer.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_withholding_islr/model/installer.py	2013-04-27 02:25:32 +0000
@@ -37,12 +37,16 @@
     _description = __doc__
 
     def default_get(self, cr, uid, fields_list=None, context=None):
+        """ Default value config_logo field
+        """
         defaults = super(wh_islr_config, self).default_get(cr, uid, fields_list=fields_list, context=context)
         logo = open(addons.get_module_resource('l10n_ve_withholding_islr', 'images', 'playa-medina.jpg'), 'rb')
         defaults['config_logo'] = base64.encodestring(logo.read())
         return defaults
 
     def _create_journal(self, cr, uid, name, type, code):
+        """ Create journal account
+        """
         self.pool.get("account.journal").create(cr, uid, { 
             'name': name,
             'type': type,
@@ -51,6 +55,10 @@
         )
 
     def _update_concepts(self, cr, uid, sale, purchase,context={}):
+        """ Update sale and purchase concepts
+        @param sale: sale concept
+        @param purchase: purchase concept
+        """
         concept_pool = self.pool.get("islr.wh.concept")
         concept_pool.write(cr, uid, concept_pool.search(cr, uid, [],context=context), {
             'property_retencion_islr_payable': purchase,
@@ -59,10 +67,14 @@
         return True
 
     def _set_wh_agent(self, cr, uid):
+        """ Set if is withholding agent or not
+        """
         company = self.pool.get('res.users').browse(cr, uid, uid).company_id
         self.pool.get('res.partner').write(cr, uid, [company.partner_id.id], {'islr_withholding_agent': True})
 
     def execute(self, cr, uid, ids, context=None):
+        """ Create journals and determinate if is withholding agent or not
+        """
         wiz_data = self.read(cr, uid, ids[0],context=context)
         if wiz_data['journal_purchase']:
             self._create_journal(cr, uid, wiz_data["journal_purchase"], 'islr_purchase', 'ISLRP')

=== modified file 'l10n_ve_withholding_islr/model/invoice.py'
--- l10n_ve_withholding_islr/model/invoice.py	2013-04-23 21:06:58 +0000
+++ l10n_ve_withholding_islr/model/invoice.py	2013-04-27 02:25:32 +0000
@@ -33,9 +33,8 @@
 
 
 class account_invoice_line(osv.osv):
-    '''
-    It adds a field that determines if a line has been retained or not
-    '''
+    """ It adds a field that determines if a line has been retained or not
+    """
     _inherit = "account.invoice.line"
     _columns = {
         'apply_wh': fields.boolean('Withheld',
@@ -55,10 +54,18 @@
                           type='out_invoice', partner_id=False,
                           fposition_id=False, price_unit=False,
                           currency_id=False, context=None, company_id=None):
-        '''
-        Onchange to show the concept of retention associated with the product
+        """ Onchange information of the product invoice line
         at once in the line of the bill
-        '''
+        @param product: new product for the invoice line
+        @param uom: new measuring unit of product
+        @param qty: new quantity for the invoice line
+        @param name: new description for the invoice line
+        @param type: invoice type
+        @param partner_id: partner of the invoice
+        @param fposition_id: fiscal position of the invoice
+        @param price_unit: new Unit Price for the invoice line
+        @param currency_id: 
+        """
         if context is None:
             context = {}
         data = super(
@@ -78,10 +85,9 @@
         return data
 
     def create(self, cr, uid, vals, context=None):
-        '''
-        initialilizes the fields wh_xml_id and apply_wh,
+        """ Initialilizes the fields wh_xml_id and apply_wh,
         when it comes to a new line
-        '''
+        """
         if context is None:
             context = {}
 
@@ -122,18 +128,16 @@
 ## BEGIN OF REWRITING ISLR
 
     def check_invoice_type(self, cr, uid, ids, context=None):
-        '''
-        This method check if the given invoice record is from a supplier
-        '''
+        """ This method check if the given invoice record is from a supplier
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         inv_brw = self.browse(cr, uid, ids[0], context=context)
         return inv_brw.type in ('in_invoice', 'in_refund')
 
     def check_withholdable_concept(self, cr, uid, ids, context=None):
-        '''
-        Check if the given invoice record is ISLR Withholdable
-        '''
+        """ Check if the given invoice record is ISLR Withholdable
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         iwdi_obj = self.pool.get('islr.wh.doc.invoices')
@@ -141,10 +145,9 @@
 
     def _create_doc_invoices(self, cr, uid, ids, islr_wh_doc_id,
                              context=None):
-        '''
-        This method link the invoices to be withheld
+        """ This method link the invoices to be withheld
         with the withholding document.
-        '''
+        """
         # TODO: CHECK IF THIS METHOD SHOULD BE HERE OR IN THE ISLR WH DOC
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
@@ -156,9 +159,8 @@
         return iwhdi_ids
 
     def _create_islr_wh_doc(self, cr, uid, ids, context=None):
-        '''
-        Function to create in the model islr_wh_doc
-        '''
+        """ Function to create in the model islr_wh_doc
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
 
@@ -201,10 +203,9 @@
         return islr_wh_doc_id
 
     def copy(self, cr, uid, id, default=None, context=None):
-        '''
-        inicializes the fields islr_wh_doc and status
+        """ Inicializes the fields islr_wh_doc and status
         when the line is duplicated
-        '''
+        """
         if default is None:
             default = {}
 
@@ -222,9 +223,8 @@
                                                  context)
 
     def _refund_cleanup_lines(self, cr, uid, lines, context=None):
-        '''
-        initializes the fields of the lines of a refund invoice
-        '''
+        """ Initializes the fields of the lines of a refund invoice
+        """
         data = super(account_invoice, self)._refund_cleanup_lines(
             cr, uid, lines, context=context)
         list = []
@@ -240,8 +240,7 @@
         return list
 
     def validate_wh_income_done(self, cr, uid, ids, context=None):
-        """
-        Method that check if wh income is validated in invoice refund.
+        """ Method that check if wh income is validated in invoice refund.
         @params: ids: list of invoices.
         return: True: the wh income is validated.
                 False: the wh income is not validated.
@@ -265,6 +264,16 @@
     def _get_move_lines(self, cr, uid, ids, to_wh, period_id, pay_journal_id,
                         writeoff_acc_id, writeoff_period_id, writeoff_journal_id, date,
                         name, context=None):
+        """ Generate move lines in corresponding account                            
+        @param to_wh: whether or not withheld                                   
+        @param period_id: Period                                                
+        @param pay_journal_id: pay journal of the invoice                       
+        @param writeoff_acc_id: account where canceled                          
+        @param writeoff_period_id: period where canceled                        
+        @param writeoff_journal_id: journal where canceled                      
+        @param date: current date                                               
+        @param name: description
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         res = super(account_invoice, self)._get_move_lines(cr, uid, ids, to_wh,

=== modified file 'l10n_ve_withholding_islr/model/islr_wh_concept.py'
--- l10n_ve_withholding_islr/model/islr_wh_concept.py	2013-03-20 21:35:20 +0000
+++ l10n_ve_withholding_islr/model/islr_wh_concept.py	2013-04-27 02:25:32 +0000
@@ -34,9 +34,8 @@
 
 
 class islr_wh_concept(osv.osv):
-    '''
-    Model to create the withholding concepts
-    '''
+    """ Model to create the withholding concepts
+    """
     _name='islr.wh.concept'
     _description = 'Income Withholding Concept'
 

=== modified file 'l10n_ve_withholding_islr/model/islr_wh_doc.py'
--- l10n_ve_withholding_islr/model/islr_wh_doc.py	2013-04-19 20:10:44 +0000
+++ l10n_ve_withholding_islr/model/islr_wh_doc.py	2013-04-27 02:25:32 +0000
@@ -37,18 +37,16 @@
 class islr_wh_doc(osv.osv):
 
     def _get_type(self, cr, uid, context=None):
-        '''
-        returns type of invoice or returns in_invoice
-        '''
+        """ Return type of invoice or returns in_invoice
+        """
         if context is None:
             context = {}
         type = context.get('type', 'in_invoice')
         return type
 
     def _get_journal(self, cr, uid, context=None):
-        '''
-        returns a islr journal depending on the type of bill
-        '''
+        """ Return a islr journal depending on the type of bill
+        """
         if context is None:
             context = {}
         journal_obj = self.pool.get('account.journal')
@@ -64,9 +62,8 @@
             return False
 
     def _get_currency(self, cr, uid, context):
-        '''
-        returns the currency of the current company
-        '''
+        """ Return the currency of the current company
+        """
         user = self.pool.get('res.users').browse(cr, uid, [uid])[0]
         if user.company_id:
             return user.company_id.currency_id.id
@@ -74,9 +71,8 @@
             return self.pool.get('res.currency').search(cr, uid, [('rate', '=', 1.0)])[0]
 
     def _get_amount_total(self, cr, uid, ids, name, args, context=None):
-        '''
-        returns the cumulative amount of each line
-        '''
+        """ Return the cumulative amount of each line
+        """
         res = {}
         for rete in self.browse(cr, uid, ids, context):
             res[rete.id] = 0.0
@@ -138,10 +134,9 @@
     }
 
     def check_income_wh(self, cr, uid, ids, context=None):
-        '''
-        checks invoices to be retained and have
+        """ Check invoices to be retained and have
         their fair share of taxes.
-        '''
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         obj = self.browse(cr, uid, ids[0], context=context)
@@ -167,21 +162,17 @@
         return True
 
     def check_auto_wh(self, cr, uid, ids, context=None):
-        '''
-        tells us if the process already
-        checked and everything was fine.
-        '''
+        """ Tell us if the process already checked and everything was fine.
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         obj = self.browse(cr, uid, ids[0], context=context)
         return obj.automatic_income_wh or False
 
     def check_auto_wh_by_type(self, cr, uid, ids, context=None):
-        '''
-        tells us if the process already
-        checked and everything was fine in case of a
-        in_invoice or in_refund
-        '''
+        """ Tell us if the process already checked and everything was 
+        fine in case of a in_invoice or in_refund
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         brw = self.browse(cr, uid, ids[0], context=context)
@@ -190,10 +181,9 @@
         return brw.automatic_income_wh or False
 
     def compute_amount_wh(self, cr, uid, ids, context=None):
-        '''
-        calculates the total withholding each invoice
+        """ Calculate the total withholding each invoice
         associated with this document
-        '''
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         iwdi_obj = self.pool.get('islr.wh.doc.invoices')
@@ -217,10 +207,9 @@
         return False
 
     def action_done(self, cr, uid, ids, context=None):
-        '''
-        calls the functions in charge of
-        preparing the document to pass the state done
-        '''
+        """ Call the functions in charge of preparing the document 
+        to pass the state done
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         self.action_number(cr, uid, ids, context=context)
@@ -234,10 +223,8 @@
         return True
 
     def action_cancel_process(self, cr, uid, ids, context=None):
-        '''
-        deletes all withholding lines and reverses
-        the process of islr
-        '''
+        """ Delete all withholding lines and reverses the process of islr
+        """
         if not context:
             context = {}
         line_obj = self.pool.get('islr.wh.doc.line')
@@ -274,10 +261,8 @@
         return True
 
     def retencion_seq_get(self, cr, uid, context=None):
-        '''
-        determinates the next sequence for
-        islr withhold and returns.
-        '''
+        """ Determinate the next sequence for islr withhold and returns.
+        """
         pool_seq = self.pool.get('ir.sequence')
         cr.execute(
             "select id,number_next,number_increment,prefix,suffix,padding from ir_sequence where code='islr.wh.doc' and active=True")
@@ -290,10 +275,10 @@
         return False
 
     def onchange_partner_id(self, cr, uid, ids, type, partner_id, context=None):
-        '''
-        unlink all taxes whean change the partner in the
-        document
-        '''
+        """ Unlink all taxes whean change the partner in the document.
+        @param type: invoice type
+        @param partner_id: partner id was changed
+        """
         context = context or {}
         acc_id = False
         inv_ids = []
@@ -350,11 +335,9 @@
             'invoice_ids': res_wh_lines}}
 
     def create(self, cr, uid, vals, context=None, check=True):
-        '''
-        When you create a new document, this function
-        is responsible for generating the sequence
-        code for the field
-        '''
+        """ When you create a new document, this function is responsible 
+        for generating the sequence code for the field
+        """
         if not context:
             context = {}
         code = self.pool.get('ir.sequence').get(cr, uid, 'islr.wh.doc')
@@ -362,10 +345,9 @@
         return super(islr_wh_doc, self).create(cr, uid, vals, context)
 
     def action_confirm(self, cr, uid, ids, context=None):
-        '''
-        is responsible for checking if the provider
-        allows retention is automatically verified and checked
-        '''
+        """ This checking if the provider allows retention is 
+        automatically verified and checked
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         check_auto_wh = self.browse(cr, uid, ids[0],
@@ -374,10 +356,9 @@
                                             'automatic_income_wh': check_auto_wh}, context=context)
 
     def action_number(self, cr, uid, ids, context=None):
-        '''
-        Is responsible for generating a number for
-        the document if it does not have one
-        '''
+        """ Is responsible for generating a number for the document 
+        if it does not have one
+        """
         context = context or {}
         obj_ret = self.browse(cr, uid, ids)[0]
         cr.execute('SELECT id, number '
@@ -393,9 +374,8 @@
         return True
 
     def action_cancel(self, cr, uid, ids, context={}):
-        '''
-        The operation is canceled and not allows automatic retention
-        '''
+        """ The operation is canceled and not allows automatic retention
+        """
         #~ if self.browse(cr,uid,ids)[0].type=='in_invoice':
             #~ return True
         self.pool.get('islr.wh.doc').write(
@@ -406,9 +386,8 @@
         return True
 
     def cancel_move(self, cr, uid, ids, *args):
-        '''
-        Retention cancel documents
-        '''
+        """ Retention cancel documents
+        """
         context = {}
         ret_brw = self.browse(cr, uid, ids)
         account_move_obj = self.pool.get('account.move')
@@ -426,16 +405,14 @@
         return True
 
     def action_cancel_draft(self, cr, uid, ids, *args):
-        '''
-        Back to draft status
-        '''
+        """ Back to draft status
+        """
         self.write(cr, uid, ids, {'state': 'draft'})
         return True
 
     def action_move_create(self, cr, uid, ids, context=None):
-        '''
-        build account moves related to withholding invoice
-        '''
+        """ Build account moves related to withholding invoice
+        """
         wh_doc_obj = self.pool.get('islr.wh.doc.line')
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
@@ -503,7 +480,17 @@
         return True
 
     def wh_and_reconcile(self, cr, uid, ids, invoice_id, pay_amount, pay_account_id, period_id, pay_journal_id, writeoff_acc_id, writeoff_period_id, writeoff_journal_id, context=None, name=''):
-
+        """ retain, reconcile and create corresponding journal items
+        @param invoice_id: invoice to retain and reconcile
+        @param pay_amount: amount payable on the invoice
+        @param pay_account_id: payment account
+        @param period_id: period for the journal items
+        @param pay_journal_id: payment journal
+        @param writeoff_acc_id: account for reconciliation
+        @param writeoff_period_id: period for reconciliation
+        @param writeoff_journal_id: journal for reconciliation
+        @param name: withholding voucher name
+        """
         inv_obj = self.pool.get('account.invoice')
         ret = self.browse(cr, uid, ids)[0]
         if context is None:
@@ -592,6 +579,8 @@
     }
 
     def copy(self, cr, uid, id, default=None, context=None):
+        """ Initialized id by duplicating
+        """
         if default is None:
             default = {}
         default = default.copy()
@@ -607,9 +596,8 @@
     _description = 'Document and Invoice Withheld Income'
 
     def _amount_all(self, cr, uid, ids, fieldname, args, context=None):
-        '''
-
-        '''
+        """ Return all amount relating to the invoices lines
+        """
         res = {}
         for ret_line in self.browse(cr, uid, ids, context):
             res[ret_line.id] = {
@@ -640,9 +628,8 @@
     _rec_rame = 'invoice_id'
 
     def _get_concepts(self, cr, uid, ids, context=None):
-        '''
-        Gets a list of withholdable concepts (concept_id) from the invoice lines
-        '''
+        """ Get a list of withholdable concepts (concept_id) from the invoice lines
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         inv_obj = self.pool.get('account.invoice')
@@ -654,8 +641,9 @@
         return list(concept_set)
 
     def _withholdable_invoices(self, cr, uid, ids, context=None):
-        '''Given a list of invoices return only those
-        where there are withholdable concepts'''
+        """ Given a list of invoices return only those
+        where there are withholdable concepts
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         res_ids = []
@@ -666,9 +654,9 @@
         return res_ids
 
     def _get_wh(self, cr, uid, ids, concept_id, context=None):
-        '''
-        Returns a dictionary containing all the values of the retention of an invoice line.
-        '''
+        """ Return a dictionary containing all the values of the retention of an invoice line.
+        @param concept_id: Withholding reason
+        """
         # TODO: Change the signature of this method
         # This record already has the concept_id built-in
         context = context or {}
@@ -735,10 +723,9 @@
         return True
 
     def load_taxes(self, cr, uid, ids, context=None):
-        '''
-        load taxes to the current invoice,
+        """ Load taxes to the current invoice,
         and if already loaded, it recalculates and load.
-        '''
+        """
         context = context or {}
         ids = isinstance(ids, (int, long)) and [ids] or ids
         ixwl_obj = self.pool.get('islr.xml.wh.line')
@@ -824,9 +811,10 @@
         return True
 
     def _get_partners(self, cr, uid, invoice):
-        '''
-        Se obtiene: el id del vendedor, el id del comprador de la factura y el campo booleano que determina si el comprador es agente de retencion.
-        '''
+        """ Se obtiene: el id del vendedor, el id del comprador de la 
+        factura y el campo booleano que determina si el comprador es 
+        agente de retencion.
+        """
         if invoice.type == 'in_invoice' or invoice.type == 'in_refund':
             vendor = invoice.partner_id
             buyer = invoice.company_id.partner_id
@@ -837,10 +825,11 @@
         return (vendor, buyer, buyer.islr_withholding_agent)
 
     def _get_residence(self, cr, uid, vendor, buyer):
-        '''
-        Se determina si la direccion fiscal del comprador es la misma que la del vendedor, con el fin de luego obtener la tasa asociada.
-        Retorna True si es una persona domiciliada o residente. Retorna False si es, no Residente o No Domicialiado.
-        '''
+        """ Se determina si la direccion fiscal del comprador es la misma 
+        que la del vendedor, con el fin de luego obtener la tasa asociada.
+        Retorna True si es una persona domiciliada o residente. Retorna 
+        False si es, no Residente o No Domicialiado.
+        """
         vendor_address = self._get_country_fiscal(cr, uid, vendor)
         buyer_address = self._get_country_fiscal(cr, uid, buyer)
         if vendor_address and buyer_address:
@@ -851,9 +840,9 @@
         return False
 
     def _get_nature(self, cr, uid, partner_id):
-        '''
-        Se obtiene la naturaleza del vendedor a partir del RIF, retorna True si es persona de tipo natural, y False si es juridica.
-        '''
+        """ Se obtiene la naturaleza del vendedor a partir del RIF, retorna 
+        True si es persona de tipo natural, y False si es juridica.
+        """
         if not partner_id.vat:
             raise osv.except_osv(_('Invalid action !'), _(
                 "Impossible income withholding, because the partner '%s' has not vat associated!") % (partner_id.name))
@@ -865,11 +854,11 @@
                 return False
 
     def _get_rate(self, cr, uid, concept_id, residence, nature, context):
-        '''
-        Se obtiene la tasa del concepto de retencion, siempre y cuando exista uno asociado a las especificaciones:
-            La naturaleza del vendedor coincida con una tasa.
-            La residencia del vendedor coindica con una tasa.
-        '''
+        """ Se obtiene la tasa del concepto de retencion, siempre y 
+        cuando exista uno asociado a las especificaciones:
+        La naturaleza del vendedor coincida con una tasa.
+        La residencia del vendedor coindica con una tasa.
+        """
         ut_obj = self.pool.get('l10n.ut')
         rate_brw_lst = self.pool.get(
             'islr.wh.concept').browse(cr, uid, concept_id).rate_ids
@@ -884,9 +873,9 @@
         return ()
 
     def _get_country_fiscal(self, cr, uid, partner_id, context=None):
-        '''
-        Get the country of the partner
-        '''
+        """ Get the country of the partner
+        @param partner_id: partner id whom consult your country
+        """
         # TODO: THIS METHOD SHOULD BE IMPROVED
         # DUE TO OPENER HAS CHANGED THE WAY PARTNER
         # ARE USED FOR ACCOUNT_MOVE
@@ -898,6 +887,9 @@
             return partner_id.country_id.id
 
     def _get_xml_lines(self, cr, uid, ail_brw, context=None):
+        """ Extract information from the document to generate xml lines
+        @param ail_brw: invoice of the document
+        """
         context = context or {}
         vendor, buyer, wh_agent = self._get_partners(
             cr, uid, ail_brw.invoice_id)
@@ -958,6 +950,8 @@
     _description = 'Lines of Document Income Withholding'
 
     def _retention_rate(self, cr, uid, ids, name, args, context=None):
+        """ Return the retention rate of each line
+        """
         res = {}
         for ret_line in self.browse(cr, uid, ids, context=context):
             if ret_line.invoice_id:

=== modified file 'l10n_ve_withholding_islr/model/islr_xml_wh.py'
--- l10n_ve_withholding_islr/model/islr_xml_wh.py	2013-04-19 19:16:47 +0000
+++ l10n_ve_withholding_islr/model/islr_xml_wh.py	2013-04-27 02:25:32 +0000
@@ -41,6 +41,8 @@
     _description = 'Generate XML'
 
     def _get_amount_total(self,cr,uid,ids,name,args,context=None):
+        """ Return withhold total amount
+        """
         res = {}
         for xml in self.browse(cr,uid,ids,context):
             res[xml.id]= 0.0
@@ -49,6 +51,8 @@
         return res
 
     def _get_amount_total_base(self,cr,uid,ids,name,args,context=None):
+        """ Return base total amount
+        """
         res = {}
         for xml in self.browse(cr,uid,ids,context):
             res[xml.id]= 0.0
@@ -83,6 +87,8 @@
     }
 
     def period_return(self,cr,uid,context=None):
+        """ Return current period
+        """
         period_obj = self.pool.get('account.period')
         fecha = time.strftime('%m/%Y')
         period_id = period_obj.search(cr,uid,[('code','=',fecha)])
@@ -92,6 +98,9 @@
             return False
 
     def search_period(self,cr,uid,ids,period_id,context=None):
+        """ Return islr lines associated with the period_id
+        @param period_id: period associated with returned islr lines
+        """
         if context is None:
             context = {}
         res ={'value':{}}
@@ -103,6 +112,8 @@
                 return res
                 
     def name_get(self, cr, uid, ids, context={}):
+        """ Return id and name of all records
+        """
         if not len(ids):
             return []
         
@@ -110,21 +121,28 @@
         return res
 
     def action_anular1(self, cr, uid, ids, context={}):
+        """ Return the document to draft status
+        """
         return self.write(cr, uid, ids, {'state':'draft'})
 
     def action_confirm1(self, cr, uid, ids, context={}):
+        """ Passes the document to state confirmed
+        """
         return self.write(cr, uid, ids, {'state':'confirmed'})
 
     def action_done1(self, cr, uid, ids, context={}):
+        """ Passes the document to state done
+        """
         root = self._xml(cr,uid,ids)
         self._write_attachment(cr,uid,ids,root,context)
         self.write(cr, uid, ids, {'state':'done'})
         return True
 
     def _write_attachment(self, cr,uid,ids,root,context):
-        '''
-        Codify the xml, to save it in the database and be able to see it in the client as an attachment
-        '''
+        """ Codify the xml, to save it in the database and be able to 
+        see it in the client as an attachment
+        @param root: data of the document in xml
+        """
         fecha = time.strftime('%Y_%m_%d_%H%M%S')
         name = 'ISLR_' + fecha +'.'+ 'xml'
         self.pool.get('ir.attachment').create(cr, uid, {
@@ -139,6 +157,10 @@
         self.log(cr, uid, ids[0], _("File XML %s generated.") % name)
 
     def indent(self,elem, level=0):
+        """ Return indented text
+        @param level: number of spaces for indentation
+        @param elem: text to indentig
+        """
         i = "\n" + level*"  "
         if len(elem):
             if not elem.text or not elem.text.strip():
@@ -155,6 +177,8 @@
 
 
     def _xml(self, cr,uid,ids):
+        """ Transform this document to XML format
+        """
         root = ''
         for id in ids:
             wh_brw = self.browse(cr,uid,id)
@@ -221,11 +245,16 @@
     }
 
     def onchange_partner_vat(self, cr, uid, ids, partner_id, context={}):
+        """ Changing the partner, the partner_vat field is updated.
+        """
         partner_brw = self.pool.get('res.partner').browse(cr,uid,partner_id)
         return {'value' : {'partner_vat':partner_brw.vat[2:]}} 
         
         
     def onchange_code_perc(self, cr, uid, ids, rate_id, context={}):
+        """ Changing the rate of the islr, the porcent_rete and concept_code fields
+        is updated.
+        """
         rate_brw = self.pool.get('islr.rates').browse(cr,uid,rate_id)
         return {'value' : {'porcent_rete':rate_brw.wh_perc,'concept_code':rate_brw.code}} 
 

=== modified file 'l10n_ve_withholding_islr/model/partner.py'
--- l10n_ve_withholding_islr/model/partner.py	2013-03-21 15:32:55 +0000
+++ l10n_ve_withholding_islr/model/partner.py	2013-04-27 02:25:32 +0000
@@ -46,8 +46,3 @@
         'islr_withholding_agent': lambda *a: True,
     }
 res_partner()
-
-
-
-
-

=== modified file 'l10n_ve_withholding_islr/model/product.py'
--- l10n_ve_withholding_islr/model/product.py	2013-04-24 19:47:03 +0000
+++ l10n_ve_withholding_islr/model/product.py	2013-04-27 02:25:32 +0000
@@ -46,10 +46,15 @@
 class product_product(osv.osv):
 
     _inherit = "product.product"
-    '''
-    Add a default concept for products that are not service type
-    '''
+    
+   
     def onchange_product_type(self, cr, uid, ids, prd_type, context=None):
+        """ Add a default concept for products that are not service type,
+        Returns false if the product type is not a service, and if the
+        product is service, returns the first concept except 'No apply
+        withholding'
+        @param prd_type: product type new
+        """
         concept_id = False
         if prd_type != 'service':
             concept_obj = self.pool.get('islr.wh.concept')
@@ -62,6 +67,3 @@
         return {'value' : {'concept_id':concept_id or False}} 
 
 product_product()
-
-
-

=== modified file 'l10n_ve_withholding_islr/model/rates.py'
--- l10n_ve_withholding_islr/model/rates.py	2013-04-25 20:11:20 +0000
+++ l10n_ve_withholding_islr/model/rates.py	2013-04-27 02:25:32 +0000
@@ -34,17 +34,15 @@
 from openerp.addons import decimal_precision as dp
 
 class islr_rates(osv.osv):
-    '''
-    The module to create the rates of the withholding concepts
-    '''
+    """ The module to create the rates of the withholding concepts
+    """
     _name='islr.rates'
     _description = 'Rates'
 
 
     def _get_name(self,cr,uid,ids, field_name, arg, context):
-        '''
-        Get the name of the withholding concept rate
-        '''
+        """ Get the name of the withholding concept rate
+        """
         res={}
         for rate in self.browse(cr,uid,ids):
             if rate.nature:
@@ -72,6 +70,3 @@
     'concept_id': fields.many2one('islr.wh.concept','Withhold  Concept',help="Withhold concept associated with this rate",required=False, ondelete='cascade'),
     }
 islr_rates()
-
-
-

=== modified file 'l10n_ve_withholding_islr/report/list_wh_islr.py'
--- l10n_ve_withholding_islr/report/list_wh_islr.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_withholding_islr/report/list_wh_islr.py	2013-04-27 02:25:32 +0000
@@ -28,6 +28,7 @@
 from openerp.osv import osv
 
 class list_wh_islr(report_sxw.rml_parse):
+
     def __init__(self, cr, uid, name, context):
         super(list_wh_islr, self).__init__(cr, uid, name, context=context)    
 

=== modified file 'l10n_ve_withholding_islr/report/wh_islr.py'
--- l10n_ve_withholding_islr/report/wh_islr.py	2013-04-25 20:11:20 +0000
+++ l10n_ve_withholding_islr/report/wh_islr.py	2013-04-27 02:25:32 +0000
@@ -29,6 +29,7 @@
 from openerp.tools.translate import _ 
 
 class rep_comprobante_islr(report_sxw.rml_parse):
+    
     def __init__(self, cr, uid, name, context):
         super(rep_comprobante_islr, self).__init__(cr, uid, name, context=context)    
         self.localcontext.update({
@@ -36,6 +37,8 @@
         })
 
     def _get_partner_addr(self, idp=False):
+        """ Return address of the partner
+        """
         if not idp:
             return []
 

=== modified file 'l10n_ve_withholding_islr/wizard/account_invoice_refund.py'
--- l10n_ve_withholding_islr/wizard/account_invoice_refund.py	2013-03-01 17:48:52 +0000
+++ l10n_ve_withholding_islr/wizard/account_invoice_refund.py	2013-04-27 02:25:32 +0000
@@ -28,12 +28,11 @@
 class account_invoice_refund(osv.osv_memory):
 
     """Refunds invoice"""
+
     _inherit = 'account.invoice.refund'
 
     def validate_wh(self, cr, uid, ids, context=None):
-        """
-        Method that validate if invoice has non-yet processed INCOME withholds.
-
+        """ Method that validate if invoice has non-yet processed INCOME withholds.
         return: True: if invoice is does not have wh's or it does have and those ones are validated.
                 False: if invoice is does have and those wh's are not yet validated.
         """

=== modified file 'l10n_ve_withholding_iva/model/generate_txt.py'
--- l10n_ve_withholding_iva/model/generate_txt.py	2013-04-22 17:19:13 +0000
+++ l10n_ve_withholding_iva/model/generate_txt.py	2013-04-27 02:25:32 +0000
@@ -39,6 +39,8 @@
     _name = "txt.iva"
 
     def _get_amount_total(self,cr,uid,ids,name,args,context=None):
+        """ Return total amount withheld of each selected bill
+        """
         context = context or {}
         res = {}
         for txt in self.browse(cr,uid,ids,context):
@@ -51,6 +53,8 @@
         return res
 
     def _get_amount_total_base(self,cr,uid,ids,name,args,context=None):
+        """ Return total amount base of each selected bill 
+        """
         context = context or {}
         res = {}
         for txt in self.browse(cr,uid,ids,context):
@@ -76,8 +80,8 @@
         'date_start': fields.date('Begin Date',required=True,states={'draft':[('readonly',False)]}, help="Begin date of period"),
         'date_end': fields.date('End date', required=True,states={'draft':[('readonly',False)]}, help="End date of period"),
         'txt_ids':fields.one2many('txt.iva.line','txt_id', readonly=True,states={'draft':[('readonly',False)]}, help='Txt field lines of ar required by SENIAT for VAT withholding'),
-        'amount_total_ret':fields.function(_get_amount_total,method=True, digits=(16, 2), readonly=True, string=' Total Monto de Retencion', help="Monto Total Retenido"),
-        'amount_total_base':fields.function(_get_amount_total_base,method=True, digits=(16, 2), readonly=True, string='Total Base Imponible', help="Total de la Base Imponible"),
+        'amount_total_ret':fields.function(_get_amount_total,method=True, digits=(16, 2), readonly=True, string='Withholding total amount', help="Monto Total Retenido"),
+        'amount_total_base':fields.function(_get_amount_total_base,method=True, digits=(16, 2), readonly=True, string='Taxable total amount', help="Total de la Base Imponible"),
     }
     _defaults = {
         'state': lambda *a: 'draft',
@@ -90,6 +94,8 @@
         }
 
     def period_return(self,cr,uid,context=None):
+        """ Return current period
+        """
         context = context or {}
         period_obj = self.pool.get('account.period')
         fecha = time.strftime('%m/%Y')
@@ -100,6 +106,8 @@
             return False
     
     def name_get(self, cr, uid, ids, context=None):
+        """ Return a list with id and name of the current register
+        """
         context = context or {}
         if not len(ids):
             return []
@@ -107,14 +115,20 @@
         return res 
 
     def action_anular(self, cr, uid, ids, context=None):
+        """ Return document state to draft
+        """
         context = context or {}
         return self.write(cr, uid, ids, {'state':'draft'})
 
     def action_confirm(self, cr, uid, ids, context=None):
+        """ Transfers the document status to confirmed
+        """
         context = context or {}
         return self.write(cr, uid, ids, {'state':'confirmed'})
 
     def action_generate_lines_txt(self,cr,uid,ids,context=None):
+        """ Current lines are cleaned and rebuilt
+        """
         context = context or {}
         voucher_obj = self.pool.get('account.wh.iva')
         txt_iva_obj = self.pool.get('txt.iva.line')
@@ -146,6 +160,8 @@
         return True
 
     def action_done(self, cr, uid, ids, context=None):
+        """ Transfer the document status to done
+        """
         context = context or {}
         root = self.generate_txt(cr,uid,ids)
         self._write_attachment(cr,uid,ids,root,context)
@@ -154,6 +170,9 @@
         return True
 
     def get_type_document(self,cr,uid,txt_line):
+        """ Return the document type
+        @param txt_line: line of the current document
+        """
         type= '03'
         if txt_line.invoice_id.type in ['out_invoice','in_invoice']:
             type= '01'
@@ -162,6 +181,9 @@
         return type
 
     def get_document_affected(self,cr,uid,txt_line,context=None):
+        """ Return the reference or number depending of the case
+        @param txt_line: line of the current document
+        """
         context = context or {}
         number='0'
         if txt_line.invoice_id.type in ['in_invoice','in_refund'] and txt_line.invoice_id.parent_id:
@@ -171,6 +193,11 @@
         return number
 
     def get_number(self,cr,uid,number,inv_type,long):
+        """ Return a list of number for document number
+        @param number: list of characters from number or reference of the bill 
+        @param inv_type: invoice type
+        @param long: max size oh the number 
+        """
         if not number:
             return '0'
         result= ''
@@ -184,6 +211,10 @@
         return result[::-1].strip()
 
     def get_document_number(self,cr,uid,ids,txt_line,inv_type,context=None):
+        """ Return the number o reference of the invoice into txt line
+        @param txt_line: One line of the current txt document
+        @param inv_type: invoice type into txt line
+        """
         context = context or {}
         number=0
         if txt_line.invoice_id.type in ['in_invoice','in_refund']:
@@ -196,6 +227,9 @@
         return number
 
     def get_amount_exempt_document(self,cr,uid,txt_line):
+        """ Return total amount not entitled to tax credit and the remaining amounts
+        @param txt_line: One line of the current txt document
+        """
         tax = 0
         amount_doc = 0
         for tax_line in txt_line.invoice_id.tax_line:
@@ -206,6 +240,10 @@
         return (tax,amount_doc)
 
     def get_buyer_vendor(self,cr,uid,txt,txt_line):
+        """ Return the buyer and vendor of the sale or purchase invoice
+        @param txt: current txt document 
+        @param txt_line: One line of the current txt document
+        """
         if txt_line.invoice_id.type in ['out_invoice','out_refund']:
             vendor = txt.company_id.partner_id.vat[2:]
             buyer  = txt_line.partner_id.vat[2:]
@@ -215,6 +253,9 @@
         return (vendor,buyer)
 
     def get_alicuota(self,cr,uid,txt_line):
+        """ Return aliquot of the withholding into line
+        @param txt_line: One line of the current txt document
+        """
         list = []
         for tax_line in txt_line.invoice_id.tax_line:
             if '12' in tax_line.name:
@@ -228,6 +269,8 @@
         return max(list)
 
     def generate_txt(self,cr,uid,ids,context=None):
+        """ Return string with data of the current document
+        """
         context = context or {}
         txt_string = ''
         for txt in self.browse(cr,uid,ids,context):
@@ -257,10 +300,10 @@
         return txt_string
         
     def _write_attachment(self, cr,uid,ids,root,context=None):
+        """ Encrypt txt, save it to the db and view it on the client as an attachment
+        @param root: location to save document
+        """
         context = context or {}
-        '''
-        Encrypt txt, save it to the db and view it on the client as an attachment
-        '''
         fecha = time.strftime('%Y_%m_%d_%H%M%S')
         name = 'IVA_' + fecha +'.'+ 'txt'
         self.pool.get('ir.attachment').create(cr, uid, {

=== modified file 'l10n_ve_withholding_iva/model/installer.py'
--- l10n_ve_withholding_iva/model/installer.py	2013-03-21 15:58:01 +0000
+++ l10n_ve_withholding_iva/model/installer.py	2013-04-27 02:25:32 +0000
@@ -29,14 +29,15 @@
 import base64
 
 class wh_vat_installer(osv.osv_memory):
-    """
-    wh_vat_installer
+    """ wh_vat_installer
     """
     _name='l10n_ve_withholding_iva.installer'
     _inherit = 'res.config.installer'
     _description = __doc__
     
     def default_get(self, cr, uid, fields, context=None):
+        """ Return information relating to the withholding regime 
+        """
         data = super(wh_vat_installer, self).default_get(cr, uid, fields, context=context)
         gaceta = open(addons.get_module_resource('l10n_ve_withholding_iva','files', 'RegimendeRetencionesdelIVA.odt'),'rb')
         data['gaceta'] = base64.encodestring(gaceta.read())
@@ -78,16 +79,15 @@
     }
 
     def _show_company_data(self, cr, uid, context=None):
-        '''
-        We only want to show the default company data in demo mode, otherwise users tend to forget
+        """ We only want to show the default company data in demo mode, otherwise users tend to forget
         to fill in the real company data in their production databases
-        '''
+        """
         return self.pool.get('ir.model.data').get_object(cr, uid, 
                                                             'base',
                                              'module_meta_information').demo
 
     def default_get(self, cr, uid, fields_list=None, context=None):
-        """ get default company if any, and the various other fields
+        """ Get default company if any, and the various other fields
         from the company's fields
         """
         defaults = super(wh_iva_config, self)\
@@ -104,6 +104,11 @@
         return defaults
 
     def _create_journal(self, cr, uid, name, type, code):
+        """ Create a journal
+        @param name: journal name
+        @param type: journal type
+        @param code: code for journal
+        """
         self.pool.get("account.journal").create(cr, uid, { 
             'name': name,
             'type': type,
@@ -112,8 +117,7 @@
         )
 
     def execute(self, cr, uid, ids, context=None):
-        '''
-        In this method I will configure all needs for work out of the box with 
+        """ In this method I will configure all needs for work out of the box with 
         This module,
         First: Setting if The company will be agent of retention.
         Second: Create Minimal Journals.
@@ -121,7 +125,7 @@
         Fourth: Ask if you have internet conexion and you want to connect to 
         SENIAT
         and update all your partners information.
-        '''
+        """
         user=self.pool.get('res.users').browse(cr,uid,[uid],context)
         wiz_data=self.read(cr,uid,ids[0],context)
         p_obj=self.pool.get('res.partner')

=== modified file 'l10n_ve_withholding_iva/model/invoice.py'
--- l10n_ve_withholding_iva/model/invoice.py	2013-04-22 04:01:24 +0000
+++ l10n_ve_withholding_iva/model/invoice.py	2013-04-27 02:25:32 +0000
@@ -32,6 +32,8 @@
 class account_invoice(osv.osv):
     _inherit = 'account.invoice'
     def _retenida(self, cr, uid, ids, name, args, context):
+        """ Verify whether withholding was applied to the invoice 
+        """
         res = {}
         if context is None:
             context = {}
@@ -41,6 +43,8 @@
 
 
     def _get_inv_from_line(self, cr, uid, ids, context={}):
+        """ Return invoice from journal items
+        """
         context = context or {}
         move = {}
         for line in self.pool.get('account.move.line').browse(cr, uid, ids):
@@ -56,6 +60,8 @@
         return invoice_ids
 
     def _get_inv_from_reconcile(self, cr, uid, ids, context={}):
+        """ Return invoice from reconciled lines
+        """
         context = context or {}
         move = {}
         for r in self.pool.get('account.move.reconcile').browse(cr, uid, ids):
@@ -85,7 +91,14 @@
 
     def onchange_partner_id(self, cr, uid, ids, type, partner_id,
             date_invoice=False, payment_term=False, partner_bank_id=False, company_id=False):
-
+        """ Return withholding iva rate of the partner and other data
+        @param type: Invoice type
+        @param partner_id: Partner id of the invoice
+        @param date_invoice: Date invoice
+        @param payment_term: Payment terms
+        @param partner_bank_id: Partner bank id of the invoice
+        @param company_id: Company id
+        """
         data = super(account_invoice, self).onchange_partner_id(cr, uid, ids, type, partner_id,
             date_invoice, payment_term, partner_bank_id, company_id)
         if partner_id:
@@ -95,6 +108,8 @@
 
 
     def create(self, cr, uid, vals, context={}):
+        """ To the create an invoice is saved the withholding iva rate of the partner
+        """
         context = context or {}
         partner_id = vals.get('partner_id',False)
         if partner_id:
@@ -103,6 +118,8 @@
         return super(account_invoice, self).create(cr, uid, vals, context)
 
     def copy(self, cr, uid, id, default=None, context=None):
+        """ Initialized fields to the copy a register
+        """
         context = context or {}
         if default is None:
             default = {}
@@ -112,6 +129,8 @@
         return super(account_invoice, self).copy(cr, uid, id, default, context)
 
     def test_retenida(self, cr, uid, ids, *args):     
+        """ Verify if this invoice is withhold 
+        """
         type2journal = {'out_invoice': 'iva_sale', 'in_invoice': 'iva_purchase', 'out_refund': 'iva_sale', 'in_refund': 'iva_purchase'}
         type_inv = self.browse(cr, uid, ids[0]).type
         type_journal = type2journal.get(type_inv, 'iva_purchase')      
@@ -130,6 +149,8 @@
 
 
     def wh_iva_line_create(self, cr, uid, inv):
+        """ Create line with iva withholding
+        """
         wh_iva_rate = inv.type in ('in_invoice', 'in_refund') and inv.partner_id.wh_iva_rate or inv.type in ('out_invoice', 'out_refund') and inv.company_id.partner_id.wh_iva_rate
         return (0, False, {
             'name': inv.name or inv.number,
@@ -138,6 +159,8 @@
         })
 
     def action_wh_iva_supervisor(self, cr, uid, ids, *args):
+        """ Validate the currencys are equal
+        """
         user_obj= self.pool.get('res.users')
         user_brw= user_obj.browse(cr,uid,uid)
         print 
@@ -154,6 +177,8 @@
 
 
     def action_wh_iva_create(self, cr, uid, ids, *args):
+        """ Create iva whithholding line
+        """
         wh_iva_obj = self.pool.get('account.wh.iva')
         for inv in self.browse(cr, uid, ids):
             if inv.wh_iva_id:
@@ -183,6 +208,8 @@
         return ret_id
 
     def button_reset_taxes_ret(self, cr, uid, ids, context=None):
+        """ Recalculate taxes in invoice 
+        """
         if not context:
             context = {}
 
@@ -196,13 +223,17 @@
         return True
 
     def button_reset_taxes(self, cr, uid, ids, context=None):
+        """ It makes two function calls related taxes reset
+        """
         context = context or {}
         super(account_invoice, self).button_reset_taxes(cr, uid, ids, context)
         self.button_reset_taxes_ret(cr, uid, ids, context)
         
         return True
 
-    def _withholding_partner(self, cr, uid, ids, context=None):  
+    def _withholding_partner(self, cr, uid, ids, context=None):
+        """ I verify that the provider retains or not
+        """
         if context is None:
             context={}
         obj = self.browse(cr, uid, ids[0],context=context)
@@ -214,20 +245,21 @@
         return False
 
     def _withholdable_tax(self, cr, uid, ids, context=None):
+        """ Verify that existing withholding in invoice 
+        """
         if context is None:
             context={}
         return any([line.tax_id.ret for line in self.browse(cr, uid, ids[0], context=context).tax_line])
 
     def check_withholdable(self, cr, uid, ids, context=None):
-        '''
-        This will test for Refund invoice trying to find out
+        """ This will test for Refund invoice trying to find out
         if its regarding parent is in the same fortnight.
         
         return True if invoice is type 'in_invoice'
         return True if invoice is type 'in_refund' and parent_id invoice
                 are both in the same fortnight.
         return False otherwise
-        '''
+        """
         per_obj = self.pool.get('account.period')
         if context is None:
             context={}
@@ -242,6 +274,8 @@
         return False
 
     def check_wh_apply(self, cr, uid, ids, context=None):
+        """ Apply withholding to the invoice
+        """
         if context is None:
             context={}
         invo_brw = self.browse(cr,uid,ids[0],context=context)
@@ -256,6 +290,16 @@
                             pay_journal_id, writeoff_acc_id, 
                             writeoff_period_id, writeoff_journal_id, date, 
                             name, context=None):
+        """ Generate move lines in corresponding account
+        @param to_wh: whether or not withheld
+        @param period_id: Period 
+        @param pay_journal_id: pay journal of the invoice
+        @param writeoff_acc_id: account where canceled
+        @param writeoff_period_id: period where canceled
+        @param writeoff_journal_id: journal where canceled
+        @param date: current date
+        @param name: description
+        """
         if context is None: context = {}
         res = super(account_invoice,self)._get_move_lines(cr, uid, ids, to_wh, period_id, 
                             pay_journal_id, writeoff_acc_id, 
@@ -288,8 +332,7 @@
         return res
 
     def validate_wh_iva_done(self, cr, uid, ids, context=None):
-        """
-        Method that check if wh vat is validated in invoice refund.
+        """ Method that check if wh vat is validated in invoice refund.
         @params: ids: list of invoices.
         return: True: the wh vat is validated.
                 False: the wh vat is not validated.
@@ -316,6 +359,8 @@
     }
 
     def compute_amount_ret(self, cr, uid, invoice_id, context={}):
+        """ Calculate withholding amount 
+        """
         context = context or {}
         res = {}
         inv = self.pool.get('account.invoice').browse(cr, uid, invoice_id, context)

=== modified file 'l10n_ve_withholding_iva/model/partner.py'
--- l10n_ve_withholding_iva/model/partner.py	2013-03-01 10:53:41 +0000
+++ l10n_ve_withholding_iva/model/partner.py	2013-04-27 02:25:32 +0000
@@ -47,6 +47,11 @@
     _inherit = 'seniat.url'
     
     def _parse_dom(self,cr,uid,dom,rif,url_seniat,context=None):
+        """ Extract iva rate and whether is withholding agent or not of the partner
+        @param dom: data from seniat page
+        @param rif: vat to partner to consult
+        @param url_seniat: seniat url
+        """
         su_obj = self.pool.get('seniat.url')
         wh_agent = dom.childNodes[0].childNodes[1].firstChild.data.upper()=='SI' and True or False
         wh_rate = su_obj._buscar_porcentaje(cr, uid, rif)

=== modified file 'l10n_ve_withholding_iva/model/wh_iva.py'
--- l10n_ve_withholding_iva/model/wh_iva.py	2013-03-21 15:58:01 +0000
+++ l10n_ve_withholding_iva/model/wh_iva.py	2013-04-27 02:25:32 +0000
@@ -43,6 +43,9 @@
 class account_wh_iva_line_tax(osv.osv):
     
     def _set_amount_ret(self, cr, uid, id, name, value, arg, ctx=None):
+        """ Change withholding amount into iva line
+        @param value: new value for retention amount
+        """
         if ctx is None: 
             ctx = {}
         if not self.browse(cr,uid,id,context=ctx).wh_vat_line_id.retention_id.type=='out_invoice':
@@ -54,6 +57,8 @@
         return True
         
     def _get_amount_ret(self, cr, uid, ids, fieldname, args, context=None):
+        """ Return withholding amount
+        """
         if context is None: context=None
         res = {}
         
@@ -91,6 +96,9 @@
 class account_wh_iva_line(osv.osv):
 
     def _get_tax_lines(self, cr, uid, tax_id_brw, context=None):
+        """ Return dictionary with tax line data
+        @param tax_id_brw: tax object
+        """
         if context is None: context = {}
         return {
             'inv_tax_id':tax_id_brw.id,
@@ -103,8 +111,11 @@
         }
     
     def load_taxes(self, cr, uid, ids, context=None):
+        """ Clean and load again tax lines of the withholding voucher
+        """
         if context is None: context = {}
         awilt_obj = self.pool.get('account.wh.iva.line.tax')
+        
         for ret_line in self.browse(cr, uid, ids, context):
             lines = []
             if ret_line.invoice_id:
@@ -125,6 +136,8 @@
         return True
 
     def _amount_all(self, cr, uid, ids, fieldname, args, context=None):
+        """ Return amount total each line
+        """
         res = {}
         for ret_line in self.browse(cr, uid, ids, context):
             res[ret_line.id] = {
@@ -157,6 +170,9 @@
     ] 
 
     def invoice_id_change(self, cr, uid, ids, invoice, context=None):
+        """ Return invoice data to assign to withholding vat
+        @param invoice: invoice for assign a withholding vat
+        """
         if context is None:
             context = {}
         if not invoice:
@@ -183,6 +199,8 @@
 class account_wh_iva(osv.osv):
 
     def _amount_ret_all(self, cr, uid, ids, name, args, context=None):
+        """ Return withholding amount total each line
+        """
         res = {}
         for retention in self.browse(cr, uid, ids, context):
             res[retention.id] = {
@@ -196,12 +214,16 @@
         return res
         
     def _get_type(self, cr, uid, context=None):
+        """ Return invoice type
+        """
         if context is None:
             context = {}
         type = context.get('type', 'in_invoice')
         return type
 
     def _get_journal(self, cr, uid, context):
+        """ Return a iva journal depending of invoice type
+        """
         if context is None:
             context = {}
         type_inv = context.get('type', 'in_invoice')
@@ -214,6 +236,8 @@
             return False
 
     def _get_currency(self, cr, uid, context):
+        """ Return currency to use
+        """
         user = self.pool.get('res.users').browse(cr, uid, [uid])[0]
         if user.company_id:
             return user.company_id.currency_id.id
@@ -261,12 +285,15 @@
 
     }
     def action_cancel(self,cr,uid,ids,context={}):
+        """ Call cancel_move and return True
+        """
         self.cancel_move(cr,uid,ids)
         return True
     
     
     def cancel_move(self,cr,uid,ids, *args):
-        
+        """ Delete move lines related with withholding vat and cancel
+        """
         ret_brw = self.browse(cr, uid, ids)
         account_move_obj = self.pool.get('account.move')
         for ret in ret_brw:
@@ -282,12 +309,19 @@
     
         
     def _get_valid_wh(self, cr, uid, amount_ret, amount, wh_iva_rate, offset=0.5,  context=None):
-        '''This method can be override in a way that
-        you can afford your own value for the offset'''
+        """ This method can be override in a way that
+        you can afford your own value for the offset
+        @param amount_ret: withholding amount
+        @param amount: invoice amount
+        @param wh_iva_rate: iva rate
+        @param offset: compensation
+        """
         if context is None: context = {}
         return amount_ret >= amount * (wh_iva_rate - offset)/100.0 and amount_ret <= amount * (wh_iva_rate + offset)/100.0
     
     def check_wh_taxes(self, cr, uid, ids, context=None):
+        """ Check that are valid and that amount retention is not greater than amount 
+        """
         if context is None: context = {}
         res = {}
         note = _('Taxes in the following invoices have been miscalculated\n\n')
@@ -309,6 +343,8 @@
         return True
 
     def check_vat_wh(self, cr, uid, ids, context={}):
+        """ Check whether the bill will need to withhold taxes
+        """
         obj = self.browse(cr, uid, ids[0])
         res = {}
         for wh_line in obj.wh_lines:
@@ -324,9 +360,7 @@
         return True
         
     def check_invoice_nro_ctrl(self, cr, uid, ids, context=None):
-        """
-        Method that check if the control number of the invoice is set
-
+        """ Method that check if the control number of the invoice is set
         Return: True if the control number is set, and raise an exception
         when is not.
         """
@@ -347,9 +381,7 @@
         return True
         
     def write_wh_invoices(self, cr, uid, ids, context=None):
-        """
-        Method that writes the wh vat id in sale invoices.
-    
+        """ Method that writes the wh vat id in sale invoices.
         Return: True: write successfully.
                 False: write unsuccessfully.
         """
@@ -362,9 +394,8 @@
         return True
 
     def _check_partner(self, cr, uid, ids, context={}):
-        '''
-        Determine if a given partner is a VAT Withholding Agent
-        '''
+        """ Determine if a given partner is a VAT Withholding Agent
+        """
         agt = False
         obj = self.browse(cr, uid, ids[0])
         if obj.type in ('out_invoice', 'out_refund') and obj.partner_id.wh_iva_agent:
@@ -383,6 +414,8 @@
 
 
     def wh_iva_seq_get(self, cr, uid, context=None):
+        """ Generate sequences for records of withholding iva
+        """
         pool_seq=self.pool.get('ir.sequence')
         cr.execute("select id,number_next,number_increment,prefix,suffix,padding from ir_sequence where code='account.wh.iva' and active=True")
         res = cr.dictfetchone()
@@ -395,6 +428,8 @@
 
 
     def action_number(self, cr, uid, ids, *args):
+        """ Update records numbers
+        """
         obj_ret = self.browse(cr, uid, ids)[0]
         if obj_ret.type == 'in_invoice':
             cr.execute('SELECT id, number ' \
@@ -410,12 +445,16 @@
         return True
     
     def action_date_ret(self,cr,uid,ids,context=None):
+        """ Undated records will be assigned the current date
+        """
         for wh in self.browse(cr, uid, ids, context):
             wh.date_ret or self.write(cr, uid, [wh.id], {'date_ret':time.strftime('%Y-%m-%d')})
         return True
 
 
     def action_move_create(self, cr, uid, ids, context=None):
+        """ Create movements associated with retention and reconcile
+        """
         inv_obj = self.pool.get('account.invoice')
         user_obj = self.pool.get('res.users')
         per_obj = self.pool.get('account.period')
@@ -464,6 +503,8 @@
         return True
     
     def _withholdable_tax_(self, cr, uid, ids, context=None):
+        """ Return lines with withholdable taxes 
+        """
         if context is None:
             context={}
         account_invo_obj = self.pool.get('account.invoice')
@@ -471,6 +512,11 @@
         return acc_id
 
     def onchange_partner_id(self, cr, uid, ids, type, partner_id,context=None):
+        """ Changing the partner is again determinated accounts and lines retain
+        for document
+        @param type: invoice type
+        @param partner_id: vendor or buyer
+        """
         if context is None: context = {}
             
         acc_id = False
@@ -512,6 +558,9 @@
         return res
 
     def _new_check(self, cr, uid, values, context={}):
+        """ Verify that the partner associated of the invoice is correct
+        @param values: Contain withholding lines, partner id and invoice_id
+        """
         lst_inv = []
 
         if 'wh_lines' in values and values['wh_lines']:
@@ -533,6 +582,8 @@
         return True
 
     def compute_amount_wh(self, cr, uid, ids, context=None):
+        """ Calculate withholding amount each line
+        """
         res = {}
         if context is None:
             context = {}      
@@ -545,6 +596,8 @@
 
 
     def copy(self, cr, uid, id, default=None, context=None):
+        """ Update fields when duplicating
+        """
         if not default:
             default = {}
         if context is None:

=== modified file 'l10n_ve_withholding_iva/report/list_wh_iva.py'
--- l10n_ve_withholding_iva/report/list_wh_iva.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_withholding_iva/report/list_wh_iva.py	2013-04-27 02:25:32 +0000
@@ -52,40 +52,60 @@
         })
         
     def _get_total_amount_doc(self,doc_id):
+        """ Return iva total amount   
+        """
         total=0
         for line in self.pool.get('txt.iva').browse(self.cr,self.uid, doc_id).txt_ids:
             total+=line.invoice_id.amount_total
         return total
         
     def _get_alicuota(self,txt_line):
+        """ Return alicuota
+        """
         return self.pool.get('txt.iva').get_alicuota(self.cr,self.uid,txt_line)
         
     def _get_amount_exempt(self,txt_line):
+        """ Return amount exempt
+        """
         amount_exempt,amount_untaxed = self.pool.get('txt.iva').get_amount_exempt_document(self.cr,self.uid,txt_line)
         self.total_amount_exempt+=amount_exempt
         return amount_exempt
 
     def _get_total_amount_exempt(self):
+        """ Return total amount exempt
+        """
         return self.total_amount_exempt
 
     def _get_amount_untaxed(self,txt_line):
+        """ Return untaxed amount
+        """
         amount_exempt,amount_untaxed = self.pool.get('txt.iva').get_amount_exempt_document(self.cr,self.uid,txt_line)
         self.total_amount_untaxed+=amount_untaxed
         return amount_untaxed
         
     def _get_total_amount_untaxed(self):
+        """ Return untaxed total amount
+        """
         return self.total_amount_untaxed
                 
     def _get_control_number(self,txt_line):
+        """ Return control number
+        """
         return self.pool.get('txt.iva').get_number(self.cr,self.uid,txt_line.invoice_id.nro_ctrl,'inv_ctrl',20)    
 
     def _get_type_document(self,line):
+        """ Return document type
+        """
         return self.pool.get('txt.iva').get_type_document(self.cr,self.uid,line)
 
     def _get_document_number(self,txt_id,txt_line):
+        """ Return document number
+        """
         return self.pool.get('txt.iva').get_document_number(self.cr,self.uid,txt_id,txt_line,'inv_number')
 
     def _get_document_affected(self,line):
+        """ Return affected document
+        """
         return self.pool.get('txt.iva').get_document_affected(self.cr,self.uid,line)
 
 report_sxw.report_sxw(

=== modified file 'l10n_ve_withholding_iva/report/withholding_vat.py'
--- l10n_ve_withholding_iva/report/withholding_vat.py	2013-04-25 20:11:20 +0000
+++ l10n_ve_withholding_iva/report/withholding_vat.py	2013-04-27 02:25:32 +0000
@@ -59,11 +59,14 @@
         })
 
     def _get_user(self):
-        
+        """ Return current user
+        """
         return self.pool.get('res.users').browse(self.cr, self.uid, self.uid)
 
 
     def _get_partner_addr2(self, idp=False):
+        """ Return address2 partner
+        """
         if not idp:
             return []
 
@@ -81,6 +84,8 @@
         return addr_inv 
 
     def _get_tipo_doc(self, tipo=None):
+        """ Return type doc
+        """
         if not tipo:
             return []
 
@@ -90,7 +95,9 @@
 
 
 
-    def _get_totales(self, comp_id):        
+    def _get_totales(self, comp_id):
+        """ Return total amount
+        """
         if not comp_id:
             return []
 
@@ -230,27 +237,41 @@
         self.ttretencion = tot_iva_ret.get('s',0.0) - tot_iva_ret.get('r',0.0)
         return lst_comp
 
-    def _get_tot_gral_compra(self): 
+    def _get_tot_gral_compra(self):
+        """ Return overall total purchase
+        """
         return self.ttcompra
 
-    def _get_tot_gral_compra_scf(self): 
+    def _get_tot_gral_compra_scf(self):
+        """ Return total general not entitled to tax credit
+        """
         return self.ttcompra_sdcf
 
-    def _get_tot_gral_base(self): 
+    def _get_tot_gral_base(self):
+        """ Return total general base
+        """
         return self.ttbase 
         
-    def _get_tot_gral_iva(self): 
+    def _get_tot_gral_iva(self):
+        """ Return total general iva
+        """
         return self.ttiva
 
-    def _get_tot_gral_retencion(self): 
+    def _get_tot_gral_retencion(self):
+        """ Return total general retention
+        """
         return self.ttretencion 
 
     def _get_rif(self, vat=''):
+        """ Return RIF
+        """
         if not vat:
             return []
         return vat[2:].replace(' ', '')
 
-    def _get_tot_linea(self, base, iva): 
+    def _get_tot_linea(self, base, iva):
+        """ Return linea total
+        """
         return base + iva
 
 

=== modified file 'l10n_ve_withholding_iva/wizard/account_invoice_refund.py'
--- l10n_ve_withholding_iva/wizard/account_invoice_refund.py	2013-03-01 17:48:52 +0000
+++ l10n_ve_withholding_iva/wizard/account_invoice_refund.py	2013-04-27 02:25:32 +0000
@@ -31,9 +31,7 @@
     _inherit = 'account.invoice.refund'
 
     def validate_wh(self, cr, uid, ids, context=None):
-        """
-        Method that validate if invoice has non-yet processed VAT withholds.
-
+        """ Method that validate if invoice has non-yet processed VAT withholds.
         return: True: if invoice is does not have wh's or it does have and those ones are validated.
                 False: if invoice is does have and those wh's are not yet validated.
         """

=== modified file 'l10n_ve_withholding_muni/model/invoice.py'
--- l10n_ve_withholding_muni/model/invoice.py	2013-02-28 16:24:39 +0000
+++ l10n_ve_withholding_muni/model/invoice.py	2013-04-27 02:25:32 +0000
@@ -34,6 +34,17 @@
                             pay_journal_id, writeoff_acc_id,
                             writeoff_period_id, writeoff_journal_id, date,
                             name, context=None):
+        """ Generate move lines in corresponding account                            
+        @param to_wh: whether or not withheld                                   
+        @param period_id: Period                                                
+        @param pay_journal_id: pay journal of the invoice                       
+        @param writeoff_acc_id: account where canceled                          
+        @param writeoff_period_id: period where canceled                        
+        @param writeoff_journal_id: journal where canceled                      
+        @param date: current date                                               
+        @param name: description                                                
+        """
+
         context = context or {}
         res = super(account_invoice, self)._get_move_lines(cr, uid, ids, to_wh,
                             period_id, pay_journal_id, writeoff_acc_id,
@@ -63,6 +74,8 @@
         return res
 
     def _retenida_munici(self, cr, uid, ids, name, args, context=None):
+        """ Check that all is well in the log lines
+        """
         context = context or {}
         res = {}
         for id in ids:
@@ -70,6 +83,8 @@
         return res
 
     def test_retenida_muni(self, cr, uid, ids, *args):
+        """ Check that all lines having their share account
+        """
         type2journal = {'out_invoice': 'mun_sale',
                         'out_refund': 'mun_sale',
                         'in_invoice': 'mun_purchase',
@@ -91,6 +106,8 @@
         return ok
 
     def _get_inv_munici_from_line(self, cr, uid, ids, context=None):
+        """ Return invoice from journal items
+        """
         context = context or {}
         move = {}
         aml_brw = self.pool.get('account.move.line').browse(cr, uid, ids)
@@ -108,6 +125,8 @@
         return invoice_ids
 
     def _get_inv_munici_from_reconcile(self, cr, uid, ids, context=None):
+        """ Return invoice from reconciled lines
+        """
         context = context or {}
         move = {}
         amr_brw = self.pool.get('account.move.reconcile').browse(cr, uid, ids)

=== modified file 'l10n_ve_withholding_muni/model/wh_muni.py'
--- l10n_ve_withholding_muni/model/wh_muni.py	2013-04-22 20:09:48 +0000
+++ l10n_ve_withholding_muni/model/wh_muni.py	2013-04-27 02:25:32 +0000
@@ -34,12 +34,17 @@
 class account_wh_munici(osv.osv):
 
     def _get_type(self, cr, uid, context=None):
+        """ Return invoice type 
+        """
         if context is None:
             context = {}
         type = context.get('type', 'in_invoice')
         return type
 
     def _get_journal(self, cr, uid, context=None):
+        """ Return the journal to the journal items that coresspond to local
+        retention depending on the invoice
+        """
         if context is None:
             context = {}
         type_inv = context.get('type', 'in_invoice')
@@ -54,6 +59,8 @@
             return False
 
     def _get_currency(self, cr, uid, context=None):
+        """ Return company currency
+        """
         if context is None:
             context = {}
         user = self.pool.get('res.users').browse(cr, uid, [uid])[0]
@@ -108,6 +115,8 @@
     ]
 
     def action_confirm(self, cr, uid, ids, context=None):
+        """ Verifies the amount withheld and the document is confirmed
+        """
         if context is None:
             context = {}
         obj = self.pool.get('account.wh.munici').browse(cr, uid, ids)
@@ -121,6 +130,8 @@
         return True
 
     def action_number(self, cr, uid, ids, *args):
+        """ Generate sequence for empty number fields in account_wh_munici records
+        """
         obj_ret = self.browse(cr, uid, ids)[0]
         if obj_ret.type == 'in_invoice':
             cr.execute('SELECT id, number '
@@ -136,6 +147,8 @@
         return True
 
     def action_done(self, cr, uid, ids, context=None):
+        """ The document is done
+        """
         if context is None:
             context = {}
         self.action_number(cr, uid, ids)
@@ -143,6 +156,8 @@
         return True
 
     def action_move_create(self, cr, uid, ids, context=None):
+        """ Create movements associated with retention and reconcile 
+        """
         if context is None:
             context = {}
         inv_obj = self.pool.get('account.invoice')
@@ -192,6 +207,10 @@
         return True
 
     def onchange_partner_id(self, cr, uid, ids, type, partner_id):
+        """ Changing the partner is again determinated accounts and lines retain for document                                                      
+        @param type: invoice type                                               
+        @param partner_id: vendor or buyer                                      
+        """        
         acc_id = False
         if partner_id:
             p = self.pool.get('res.partner').browse(cr, uid, partner_id)
@@ -208,6 +227,8 @@
         return result
 
     def _update_check(self, cr, uid, ids, partner_id, context=None):
+        """ Check if the invoices are selected partner
+        """
         if context is None:
             context = {}
         if ids:
@@ -224,6 +245,8 @@
         return True
 
     def _new_check(self, cr, uid, values, context=None):
+        """ Check amount withheld and Check if the invoices are selected partner
+        """
         if context is None:
             context = {}
         lst_inv = []
@@ -249,6 +272,8 @@
         return True
 
     def write(self, cr, uid, ids, vals, context=None, check=True, update_check=True):
+        """ Validate invoices before update records
+        """
         if context is None:
             context = {}
         ret = self.browse(cr, uid, ids[0])
@@ -261,6 +286,8 @@
         return super(account_wh_munici, self).write(cr, uid, ids, vals, context=context)
 
     def create(self, cr, uid, vals, context=None, check=True):
+        """ Validate before create record
+        """
         if context is None:
             context = {}
         if check:
@@ -274,6 +301,8 @@
 class account_wh_munici_line(osv.osv):
 
     def default_get(self, cr, uid, fields, context=None):
+        """ Default for munici_context field
+        """
         if context is None:
             context = {}
         data = super(account_wh_munici_line, self).default_get(cr,
@@ -307,6 +336,9 @@
     ]
 
     def onchange_invoice_id(self, cr, uid, ids, invoice_id, context=None):
+        """ Validate that the bill is no longer assigned to retention
+        @param invoice_id: invoice id
+        """
         if context is None:
             context = {}
         lines = []
@@ -319,8 +351,7 @@
             ok = True
             res = self.pool.get(
                 'account.invoice').browse(cr, uid, invoice_id, context)
-            cr.execute(
-                'select retention_id from account_wh_munici_line where invoice_id=%s',
+            cr.execute('select retention_id from account_wh_munici_line where invoice_id=%s',
                 (invoice_id,))
             ret_ids = cr.fetchone()
             ok = ok and bool(ret_ids)

=== modified file 'l10n_ve_withholding_muni/report/wh_muni_report.py'
--- l10n_ve_withholding_muni/report/wh_muni_report.py	2013-04-22 20:09:48 +0000
+++ l10n_ve_withholding_muni/report/wh_muni_report.py	2013-04-27 02:25:32 +0000
@@ -39,6 +39,8 @@
         })
 
     def _get_partner_addr(self, idp=False):
+        """ Return partner address
+        """
         if not idp:
             return []
 
@@ -51,6 +53,8 @@
 
 
     def _get_rif(self, vat=''):
+        """ Return partner rif
+        """
         if not vat:
             return []
         return vat[2:].replace(' ', '')

=== modified file 'l10n_ve_withholding_src/model/invoice.py'
--- l10n_ve_withholding_src/model/invoice.py	2013-03-21 16:14:09 +0000
+++ l10n_ve_withholding_src/model/invoice.py	2013-04-27 02:25:32 +0000
@@ -33,7 +33,14 @@
      
     def onchange_partner_id(self, cr, uid, ids, type, partner_id,\
             date_invoice=False, payment_term=False, partner_bank_id=False, company_id=False):
-        
+        """ Change invoice information depending of the partner
+        @param type: Invoice type                                                                                                                  
+        @param partner_id: Partner id of the invoice                            
+        @param date_invoice: Date invoice                                       
+        @param payment_term: Payment terms                                      
+        @param partner_bank_id: Partner bank id of the invoice                  
+        @param company_id: Company id  
+        """
         p = self.pool.get('res.partner').browse(cr, uid, partner_id)
         c = self.pool.get('res.partner').browse(cr, uid, uid)
         res = super(account_invoice,self).onchange_partner_id(cr, uid, ids, type, \
@@ -49,6 +56,8 @@
         return res
     
     def _retenida(self, cr, uid, ids, name, args, context):
+        """ Verify whether withholding was applied to the invoice
+        """
         res = {}
         if context is None:
             context = {}
@@ -57,6 +66,8 @@
         return res
     
     def _get_inv_from_line(self, cr, uid, ids, context={}):
+        """ Returns invoice from journal items
+        """
         move = {}
         for line in self.pool.get('account.move.line').browse(cr, uid, ids):
             if line.reconcile_partial_id:
@@ -71,6 +82,8 @@
         return invoice_ids
 
     def _get_inv_from_reconcile(self, cr, uid, ids, context={}):
+        """ Return invoice from reconciled lines
+        """
         move = {}
         for r in self.pool.get('account.move.reconcile').browse(cr, uid, ids):
             for line in r.line_partial_ids:
@@ -84,8 +97,8 @@
         return invoice_ids
     
     def _check_retention(self, cr, uid, ids, context=None):
-        '''This method will check the retention value will be maximum 5%   
-            '''
+        """ This method will check the retention value will be maximum 5%   
+        """
         if context is None: context = {}
         
         invoice_brw = self.browse(cr, uid,ids)
@@ -114,6 +127,16 @@
                             pay_journal_id, writeoff_acc_id, 
                             writeoff_period_id, writeoff_journal_id, date, 
                             name, context=None):
+        """ Generate move lines in corresponding account                            
+        @param to_wh: whether or not withheld                                   
+        @param period_id: Period                                                
+        @param pay_journal_id: pay journal of the invoice                       
+        @param writeoff_acc_id: account where canceled                          
+        @param writeoff_period_id: period where canceled                        
+        @param writeoff_journal_id: journal where canceled                      
+        @param date: current date                                               
+        @param name: description 
+        """
         if context is None: context = {}
         res = super(account_invoice,self)._get_move_lines(cr, uid, ids, to_wh, period_id, 
                             pay_journal_id, writeoff_acc_id, 

=== modified file 'l10n_ve_withholding_src/model/wh_src.py'
--- l10n_ve_withholding_src/model/wh_src.py	2013-04-18 21:03:12 +0000
+++ l10n_ve_withholding_src/model/wh_src.py	2013-04-27 02:25:32 +0000
@@ -34,6 +34,8 @@
 class account_wh_src(osv.osv):
 
     def name_get(self, cursor, user, ids, context=None):
+        """ To generate a name for src record
+        """
         if isinstance(ids, (int, long)):
             ids = [ids]
         if not ids:
@@ -52,11 +54,16 @@
         return res
 
     def _get_uid_wh_agent(self, cr, uid, context=None):
+        """ Return true if current partner is social responsability agent and
+        return false in otherwise
+        """
         context = context or {}
         user_wh_agent = self.pool.get('res.partner').browse(cr, uid, uid, context=context).wh_src_agent
         return user_wh_agent
 
     def _get_partner_agent(self, cr, uid, context=None):
+        """ Return a list of browse partner depending of invoice type
+        """
         context = context or {}
         
         obj_partner = self.pool.get('res.partner')
@@ -73,6 +80,9 @@
         return l
 
     def default_get(self, cr, uid, fields, context=None):
+        """ Update fields uid_wh_agent and partner_list to the create a
+        record        
+        """
         context = context or {}
         res = super(account_wh_src, self).default_get(cr, uid, fields, context=context)
         res.update({'uid_wh_agent': self._get_uid_wh_agent(cr,uid,context=context) })
@@ -81,11 +91,15 @@
         return res
 
     def _get_p_agent(self, cr, uid, ids, field_name, args, context=None):
+        """ Create a dictionary with ids partner and their browse item 
+        """
         context = context or {}
         res= {}.fromkeys(ids,self._get_partner_agent(cr,uid,context=context))
         return res 
 
     def _get_wh_agent(self, cr, uid, ids, field_name, args, context=None):
+        """ Create a dictionary with ids agent partner and their browse item
+        """
         context = context or {}
         res= {}.fromkeys(ids,self._get_uid_wh_agent(cr,uid,context=context))
         return res 
@@ -123,12 +137,14 @@
     } 
     
     def _diario(self, cr, uid, model, context=None):
+        """  Return journal to use in purchase or sale
+        """
         if context is None:
             context={}
         ir_model_data = self.pool.get('ir.model.data')
-        journal_purchase=ir_model_data.search(cr, uid, [('model','=','account.journal'),('module','=','l10n_ve_withholding_src'),('name','=','withholding_scr_purchase_journal')])
-        journal_sale=ir_model_data.search(cr, uid, [('model','=','account.journal'),('module','=','l10n_ve_withholding_src'),('name','=','withholding_src_sale_journal')])
-        ir_model_purchase_brw=ir_model_data.browse(cr, uid, journal_purchase, context=context)
+        journal_purchase = ir_model_data.search(cr, uid, [('model','=','account.journal'),('module','=','l10n_ve_withholding_src'),('name','=','withholding_scr_purchase_journal')])
+        journal_sale = ir_model_data.search(cr, uid, [('model','=','account.journal'),('module','=','l10n_ve_withholding_src'),('name','=','withholding_src_sale_journal')])
+        ir_model_purchase_brw = ir_model_data.browse(cr, uid, journal_purchase, context=context)
         ir_model_sale_brw=ir_model_data.browse(cr, uid, journal_sale, context=context)
         if context.get('type') == 'in_invoice':
             return ir_model_purchase_brw[0].res_id
@@ -150,6 +166,10 @@
     ] 
     
     def onchange_partner_id(self, cr, uid, ids, type, partner_id,context=None):
+        """ Return account depending of the invoice         
+        @param type: invoice type
+        @param partner_id: partner id
+        """
         if context is None: context = {}    
         acc_id = False
         res = {}
@@ -176,12 +196,16 @@
         
 
     def action_date_ret(self,cr,uid,ids,context=None):
+        """ if the retention date is empty, is filled with the current date
+        """
         for wh in self.browse(cr, uid, ids, context):
             wh.date_ret or self.write(cr, uid, [wh.id], {'date_ret':time.strftime('%Y-%m-%d')})
         return True
 
 
     def action_draft(self, cr, uid, ids, context={}):
+        """ Passes the document to draft status
+        """
         if context is None:
             context={}
         inv_obj = self.pool.get('account.invoice')
@@ -194,6 +218,8 @@
         return self.write(cr,uid,ids[0],{'state':'draft'})
 
     def action_confirm(self, cr, uid, ids, context={}):
+        """ Retention is valid to pass a status confirmed
+        """
         if context is None:
             context={}
         inv_obj = self.pool.get('account.invoice')
@@ -221,6 +247,8 @@
         return self.write(cr,uid,ids[0],{'state':'confirmed'})
         
     def action_done(self, cr, uid, ids, context=None):
+        """ Pass the document to state done
+        """
         if context is None:
             context = {}
 
@@ -231,10 +259,14 @@
         return self.write(cr,uid,ids,{'state':'done'})
         
     def action_cancel(self,cr,uid,ids,context={}):
+        """ Still not allowed to cancel these withholdings
+        """
         raise osv.except_osv(_('Invalid Procedure!'),_("For the moment, the systmen does not allow cancell these withholdings."))
         return True
         
     def copy(self,cr,uid,id,default,context=None):
+        """ Lines can not be duplicated in this model
+        """
         raise osv.except_osv('Invalid Procedure!',"You can not duplicate lines")
         return True
         
@@ -245,7 +277,8 @@
 
 
     def action_move_create(self, cr, uid, ids, context=None):
-        
+        """ Build account moves related to withholding invoice 
+        """
         inv_obj = self.pool.get('account.invoice')
         if context is None: context = {}
         
@@ -297,6 +330,8 @@
         return True
 
     def action_number(self, cr, uid, ids, *args):
+        """ Is responsible for generating a number for the document if it does not have one  
+        """
         obj_ret = self.browse(cr, uid, ids)[0]
         if obj_ret.type == 'in_invoice':
             cr.execute('SELECT id, number ' \
@@ -315,6 +350,8 @@
         
         
     def wh_src_confirmed(self, cr, uid, ids):
+        """ Confirm src document
+        """
         number = self.pool.get('account.wh.src.line')
         return True
         
@@ -343,6 +380,12 @@
     ] 
     
     def onchange_invoice_id(self, cr, uid, ids, type, invoice_id=False,base_amount=0.0,wh_src_rate=5.0,context=None):
+        """ Change src information to change the invoice 
+        @param type: invoice type
+        @param invoice_id: new invoice id
+        @param base_amount: new base amount
+        @param wh_src_rate: new rate of the withhold src
+        """
         if context is None: context = {}    
         res = {}
         inv_obj = self.pool.get('account.invoice')

=== modified file 'l10n_ve_withholding_src/report/wh_src_report.py'
--- l10n_ve_withholding_src/report/wh_src_report.py	2013-02-13 19:39:18 +0000
+++ l10n_ve_withholding_src/report/wh_src_report.py	2013-04-27 02:25:32 +0000
@@ -19,6 +19,8 @@
         
 
     def get_empresa(self, partner_id):
+        """ Get information company
+        """
         obj_addr = self.pool.get('res.partner')
         res = {}
         for row in obj_addr.browse(self.cr, self.uid, partner_id):

=== modified file 'l10n_ve_withholding_src/wizard/wizard_retention.py'
--- l10n_ve_withholding_src/wizard/wizard_retention.py	2013-02-28 16:24:39 +0000
+++ l10n_ve_withholding_src/wizard/wizard_retention.py	2013-04-27 02:25:32 +0000
@@ -32,6 +32,8 @@
     _description = "Wizard that changes the retention value"
 
     def set_retention(self, cr, uid, ids, context=None):
+        """ Change value of the retention
+        """
         if context is None:
             context={}
         data = self.pool.get('wiz.retention').read(cr, uid, ids)[0]