IMP varios

Author: jjscarafia

l10n_ar_afipws_fe/models/invoice.py

@@ -101,7 +101,10 @@ def _get_barcode(self):
                     str(self.afip_cae), cae_due])
             barcode = barcode + self.verification_digit_modulo10(barcode)
         self.afip_barcode = barcode
+        self.afip_barcode_img = self._make_image_I25(barcode)
 
+    @api.model
+    def _make_image_I25(self, barcode):
         "Generate the required barcode Interleaved of 7 image using PIL"
         image = False
         if barcode:
@@ -117,7 +120,7 @@ def _get_barcode(self):
             image = output.getvalue()
             image = output.getvalue().encode("base64")
             output.close()
-        self.afip_barcode_img = image
+        return image
 
     @api.model
     def verification_digit_modulo10(self, code):

l10n_ar_invoice/__openerp__.py

@@ -20,7 +20,7 @@
 ##############################################################################
 {
     "name": "Argentina - Facturación y documentos AFIP",
-    'version': '8.0.0.20.1',
+    'version': '8.0.0.21.0',
     'category': 'Localization/Argentina',
     'sequence': 14,
     'author': 'ADHOC SA',

l10n_ar_invoice/models/invoice.py

@@ -144,7 +144,8 @@ class account_invoice(models.Model):
         'Document Type',
         readonly=True,
         ondelete='restrict',
-        states={'draft': [('readonly', False)]}
+        states={'draft': [('readonly', False)]},
+        copy=False
     )
     afip_incoterm_id = fields.Many2one(
         'afip.incoterm',
@@ -566,6 +567,18 @@ def _check_reference(self):
                     'Supplier Invoice Number must be unique per Supplier'
                     ' and Company!'))
 
+    # no lo usamos al final porque ya hay chequeo en chequeos de la loc
+    # @api.one
+    # def check_supplier_invoice_number(self):
+    #     # TODO podemos mejorar esto en la v9 con distintos chequeos del
+    #     # supplier number
+    #     if (
+    #             self.type in ('in_invoice', 'in_refund') and
+    #             self.use_documents and not self.supplier_invoice_number):
+    #         raise Warning(_(
+    #             'On supplier invoices with a "Use Document" Journal, supplier'
+    #             ' invoice number is mandatory, invoice %s' % self.id))
+
     @api.multi
     def check_use_documents(self):
         # check invoices has document class but journal require it (we check
@@ -703,6 +716,8 @@ def action_move_create(self):
         """
         self.check_use_documents()
         self.check_argentinian_invoice_taxes()
+        # no lo usamos al final porque ya hay chequeo en chequeos de la loc
+        # self.check_supplier_invoice_number()
         return super(account_invoice, self).action_move_create()
 
     @api.multi

l10n_ar_invoice/view/invoice_view.xml

@@ -144,11 +144,12 @@
                     <field name="internal_number" invisible="1"/>
                     <field name="use_argentinian_localization" invisible="1"/>
                 </form>
-
+                <!-- no lo hacemos obligatorio desde vista para que, por ejemplo, si queremos correr una operation no nos obligue a poner algo -->
+<!-- 
                 <field name="supplier_invoice_number" position="attributes">
                     <attribute name="attrs">{'required':[('use_documents','=',True)]}</attribute>
                 </field>
-
+ -->
                 <field name="number" position="attributes">
                     <attribute name="invisible">1</attribute>
                 </field>