Merge lp:~bellini666/stoq/boleto_corrections into lp:~stoq-dev/stoq/master

[Thiago Bellini]

Colocando strings para tradução. Algumas estavam mal escritas, é uma oportunidade para traduzir certo inclusive.

Arrumado a validação de campos custom no editor de contas do financial.

Remoção do 'len_convenio' do BB.

[Ronaldo Maia (romaia)]

131 - u'Tamanho do numero maior que o permitido')
132 + _('Number length must be less than %s') % tamanho)

Lower em vez de less.

61 if len(list_sacado) > 3:
62 - raise BoletoException(u'Número de linhas do sacado maior que 3')
65 for line in list_sacado:
66 if len(line) > 80:
67 raise BoletoException(

Em vez de levantar os errors, trunca os valores: pega só list_sacado[:3] e para cada linha, só line[:80]

Dessa forma não vai dar erro se o nome do cliente for muito grande.

Aprovado com essas mudanças.

review: Approve

=== modified file 'stoqlib/gui/editors/accounteditor.py'
--- stoqlib/gui/editors/accounteditor.py	2012-01-25 11:33:24 +0000
+++ stoqlib/gui/editors/accounteditor.py	2012-02-02 18:10:27 +0000
@@ -261,10 +261,13 @@
                 entry = ProxyEntry()
                 entry.model_attribute = name
                 setattr(self, name, entry)
+                # Set the model attr too so it can be validated
+                setattr(self.bank_model, name, '')
                 entry.props.data_type = 'str'
                 entry.connect('validate', self._on_bank_option__validate,
                               bank_info, option)
-                self._add_widget("<i>%s</i>:" % (option, ), entry, options=True)
+                self._add_widget("<i>%s</i>:" % (option.capitalize(), ),
+                                 entry, options=True)
                 entry.show()
                 self._option_fields[option] = entry
                 attributes.append(entry.model_attribute)
@@ -407,10 +410,12 @@
                 return ValidationError(str(e))
 
     def _on_bank_option__validate(self, entry, value, bank_info, option):
-        try:
-            bank_info.validate_option(option, value)
-        except BoletoException, e:
-            return ValidationError(str(e))
+        # Only validate if there's a value. Maybe it's not mandatory
+        if value:
+            try:
+                bank_info.validate_option(option, value)
+            except BoletoException, e:
+                return ValidationError(str(e))
         self.bank_model.set_option(option, value)
 
     def _on_test_button__clicked(self, button):

=== modified file 'stoqlib/lib/boleto.py'
--- stoqlib/lib/boleto.py	2012-01-23 16:52:45 +0000
+++ stoqlib/lib/boleto.py	2012-02-02 18:10:27 +0000
@@ -64,7 +64,7 @@
             setattr(self, internal_attr, '-'.join(val))
 
         else:
-            raise BoletoException('Wrong value format')
+            raise BoletoException(_('Wrong value format'))
 
     return property(
         lambda self: getattr(self, internal_attr),
@@ -147,7 +147,7 @@
         num = num.replace('X', str(dv), 1)
         if len(num) != 44:
             raise BoletoException(
-                'Código com %d caracteres' % len(num))
+                _('The barcode must have 44 caracteres, found %d') % len(num))
         return num
 
     @property
@@ -243,11 +243,12 @@
 
     def _sacado_set(self, list_sacado):
         if len(list_sacado) > 3:
-            raise BoletoException(u'Número de linhas do sacado maior que 3')
+            raise BoletoException(
+                _('The drawee number of lines must be less than 3'))
         for line in list_sacado:
             if len(line) > 80:
                 raise BoletoException(
-                    u'Linha de sacado possui mais que 80 caracteres')
+                    _('The drawee lines must have less than 80 characters'))
         self._sacado = list_sacado
     sacado = property(_sacado_get, _sacado_set)
 
@@ -306,25 +307,20 @@
     @classmethod
     def validate_field(cls, field, dv_10=None, func=None):
         if ' ' in field:
-            raise BoletoException(
-                u'Campo não pode ter espaços')
+            raise BoletoException(_('The field cannot have spaces'))
         if '.' in field or ',' in field:
-            raise BoletoException(
-                u'Campo não pode ter pontos ou virgulas')
+            raise BoletoException(_('The field cannot have dots of commas'))
         dv = None
         if '-' in field:
             if field.count('-') != 1:
-                raise BoletoException(
-                    u'Só pode ter um hífen')
+                raise BoletoException(_('More than one hyphen found'))
             field, dv = field.split('-', 1)
             if not dv:
-                raise BoletoException(
-                    u'Digito verificador não pode ser vazio')
+                raise BoletoException(_('Verifier digit cannot be empty'))
         try:
             int(field)
         except ValueError:
-            raise BoletoException(
-                u'Conta precisa ser um número')
+            raise BoletoException(_('Account needs to be a number'))
 
         if dv and dv_10 is not None:
             func = cls.validate_field_func
@@ -343,19 +339,16 @@
                 elif (ret is not None and
                     str(ret) != dv.lower() and
                     ret < 10):
-                    raise BoletoException(
-                        u'Dígito verificador invalido')
+                    raise BoletoException(_('Invalid verifier digit'))
             else:
                 try:
                     dv = int(dv)
                 except ValueError:
                     raise BoletoException(
-                        u'Dígito verificador tem que ser um número ou %s' % (
-                        dv_10))
+                        _('Verifier digit must be a number or %s') % dv_10)
 
                 if ret is not None and ret != dv:
-                    raise BoletoException(
-                        u'Dígito verificador invalido')
+                    raise BoletoException(_('Invalid verifier digit'))
 
     @classmethod
     def validate_option(cls, option, value):
@@ -365,14 +358,14 @@
     def formata_numero(numero, tamanho):
         if len(numero) > tamanho:
             raise BoletoException(
-                u'Tamanho do numero maior que o permitido')
+                _('Number length must be less than %s') % tamanho)
         return numero.zfill(tamanho)
 
     @staticmethod
     def formata_texto(texto, tamanho):
         if len(texto) > tamanho:
             raise BoletoException(
-                u'Tamanho do texto maior que o permitido')
+                _('Text length must be less than %s') % tamanho)
         return texto.ljust(tamanho)
 
     @staticmethod
@@ -568,11 +561,16 @@
             try:
                 value = int(value)
             except ValueError:
-                raise BoletoException("carteira tem que ser um número")
+                # TRANSLATORS: Do not translate 'Carteira'
+                raise BoletoException(_("Carteira must be a number"))
             if 0 > value:
-                raise BoletoException("carteira tem que ser entre 0 e 99")
+                raise BoletoException(
+                    # TRANSLATORS: Do not translate 'Carteira'
+                    _("Carteira must be a number between 0 and 99"))
             if value > 99:
-                raise BoletoException("carteira tem que ser entre 0 e 99")
+                raise BoletoException(
+                    # TRANSLATORS: Do not translate 'Carteira'
+                    _("Carteira must be a number between 0 and 99"))
 
 
 @register_bank
@@ -581,7 +579,6 @@
     bank_number = 1
     logo = 'logo_bb.gif'
     options = {'convenio': BILL_OPTION_CUSTOM,
-               'len_convenio': BILL_OPTION_CUSTOM,
                'agencia': BILL_OPTION_BANK_BRANCH,
                'conta': BILL_OPTION_BANK_BRANCH}
 
@@ -592,10 +589,13 @@
         if not 'carteira' in kwargs:
             kwargs['carteira'] = '18'
 
-        self.len_convenio = 7
-        if 'len_convenio' in kwargs:
-            self.len_convenio = int(kwargs.pop('len_convenio'))
-        self.convenio = ''
+        convenio = kwargs.pop('convenio', None)
+        if convenio:
+            self.len_convenio = len(convenio)
+            self.convenio = convenio
+        else:
+            self.len_convenio = 7
+            self.convenio = ''
 
         self.format_nnumero = 1
         if 'format_nnumero' in kwargs:
@@ -674,17 +674,18 @@
     def validate_option(cls, option, value):
         if option == 'convenio':
             if value == '':
-                raise BoletoException('Convenio não pode ser vazio')
+                # TRANSLATORS: Do not translate 'Convenio'
+                raise BoletoException(_('Convenio cannot be empty'))
             try:
                 int(value)
             except ValueError:
-                raise BoletoException(_("Must be a number"))
-            if len(value) > 8:
-                raise BoletoException(_("Cannot be longer than %d") % (8,))
-
-        if option == 'len_convenio':
-            if value not in ['6', '7', '8']:
-                raise BoletoException('Tem que ser 6, 7 ou 8')
+                # TRANSLATORS: Do not translate 'Convenio'
+                raise BoletoException(_("Convenio must be a number"))
+            if len(value) not in [6, 7, 8]:
+                raise BoletoException(
+                    # TRANSLATORS: Do not translate 'Convenio'
+                    _("Convenio length must be 6, 7 or 8. Try filing it with "
+                      "'0's at the left."))
 
 
 @register_bank