Merge lp:~danimaribeiro/openerp.pt-br-localiz/openerp.pt-br-localiz into lp:~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0

[Danimar Ribeiro]

Fixed problem converting prospects to sales order. Other problem with the type of a related field and a misspelled word.

[lp:~danimaribeiro/openerp.pt-br-localiz/openerp.pt-br-localiz]

546. By Danimar Ribeiro on 2013-06-06

Implemented new field 'justificative' to the class l10n_br_account_invoice_invalid_number.

[Renato Lima - http://www.akretion.com (renatonlima)]

Olá Danimar,

Você poderia fazer um revert apenas do l10n_br_sale/sale.py? porque a sua solução não esta correta, pois o ["id"] do retorno do método read é do objeto (sale.shop) e não do fiscal_category_id. O Carlos Almeida já corrigiu isso e pediu o merge. O restante parece que esta tudo ok! assim que você fizer o revert do l10n_br_sale/sale.py da sua branch eu faço o merge.

Obrigado pela contribuição,

Renato Lima - http://www.akretion.com/pt-BR

review: Needs Fixing

[lp:~danimaribeiro/openerp.pt-br-localiz/openerp.pt-br-localiz]

547. By Danimar Ribeiro on 2013-06-14

Reverted changes to l10n_br_sale/sale.py

[Danimar Ribeiro (danimaribeiro)]

Olá Renato.
Efetuei novo commit revertendo as alterações desta classe.

Apenas uma duvida: eu estava utilizando no read ["default_fc_id.id"] , ou seja a meu ver isso retorna o id já da relação? ou isso não funciona?

Abraços.

[Renato Lima - http://www.akretion.com (renatonlima)]

Se você fizer um:

self.pool.get("sale.shop").read(cr, uid, [shop_id], ["default_fc_id"])

Ele vai retornar isso:

[{'default_fc_id': (13, u'Venda'), 'id': 1}]

A chave id é o o campo sale.shop.id

[Renato Lima - http://www.akretion.com (renatonlima)]

review: Approve

=== modified file 'l10n_br_account/account_invoice.py'
--- l10n_br_account/account_invoice.py	2013-05-15 20:43:19 +0000
+++ l10n_br_account/account_invoice.py	2013-06-14 13:46:30 +0000
@@ -729,8 +729,8 @@
         fcategory.property_journal.id or False
         if not result['value'].get('journal_id', False):
             raise orm.except_orm(
-                _('Nenhuma Diário !'),
-                _("Categoria fisca: '%s', não tem um diário contábil para a \
+                _('Nenhum Diário !'),
+                _("Categoria fiscal: '%s', não tem um diário contábil para a \
                 empresa %s") % (fcategory.name, obj_company.name))
 
         obj_fp_rule = self.pool.get('account.fiscal.position.rule')

=== modified file 'l10n_br_account/l10n_br_account.py'
--- l10n_br_account/l10n_br_account.py	2013-04-16 13:39:09 +0000
+++ l10n_br_account/l10n_br_account.py	2013-06-14 13:46:30 +0000
@@ -230,6 +230,8 @@
         'state': fields.selection(
             [('draft', 'Rascunho'), ('cancel', 'Cancelado'),
             ('done', u'Concluído')], 'Status', required=True),
+        'justificative': fields.char('Justificativa', size=255,
+            readonly=True, states={'draft': [('readonly', False)]}, required=True),        
     }
     _defaults = {
         'state': 'draft',
@@ -243,6 +245,11 @@
          u'Sequência existente!'),
     ]
 
+    def _check_justificative(self, cr,uid, ids):
+        for invalid in self.browse(cr, uid, ids):
+            if len(invalid.justificative) < 15:return  False
+        return True
+
     def _check_range(self, cursor, user, ids, context=None):
         for invalid_number in self.browse(cursor, user, ids, context=context):
             where = []
@@ -264,7 +271,8 @@
 
     _constraints = [
         (_check_range, u'Não é permitido faixas sobrepostas!',
-            ['number_start', 'number_end'])
+            ['number_start', 'number_end']),
+        (_check_justificative,'Justificativa deve ter tamanho minimo de 15 caracteres.', ['justificative'])
     ]
 
     def action_draft_done(self, cr, uid, ids, *args):

=== modified file 'l10n_br_account/l10n_br_account_view.xml'
--- l10n_br_account/l10n_br_account_view.xml	2013-04-16 13:39:09 +0000
+++ l10n_br_account/l10n_br_account_view.xml	2013-06-14 13:46:30 +0000
@@ -185,6 +185,7 @@
 							<field name="document_serie_id"/>
 							<field name="number_start"/>
 							<field name="number_end"/>
+							<field name="justificative"/>
 						</group>
 					</form>
 				</field>

=== modified file 'l10n_br_account_payment_extension/account_move_line.py'
--- l10n_br_account_payment_extension/account_move_line.py	2013-01-29 03:08:16 +0000
+++ l10n_br_account_payment_extension/account_move_line.py	2013-06-14 13:46:30 +0000
@@ -24,7 +24,7 @@
     _inherit = 'account.move.line'
     _columns = {
         'revenue_expense': fields.related(
-            'journal_id', 'revenue_expense', type='many2one',
-            relation='account.journal', string='Revenue Expense',
+            'journal_id', 'revenue_expense', type='boolean',
+            string='Revenue Expense',
             store=True, readonly=True),
     }

=== modified file 'l10n_br_sale/sale.py'
--- l10n_br_sale/sale.py	2013-04-16 15:06:39 +0000
+++ l10n_br_sale/sale.py	2013-06-14 13:46:30 +0000
@@ -75,7 +75,7 @@
             cr, uid, ["shop_id"], context)["shop_id"])
         return  self.pool.get("sale.shop").read(
             cr, uid, [shop_id], ["default_fc_id"])[0]["default_fc_id"]
-
+    
     _defaults = {
         'fiscal_category_id': _default_fiscal_category,
     }