Merge lp:~vauxoo/openerp-venezuela-localization/ovl70-invoice-cancel-kty into lp:openerp-venezuela-localization

[Katherine Zaoral (Vauxoo)]

[IMP] cancel invoice process now take in count the fiscal book management (add exception)

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

review: Approve

=== modified file 'l10n_ve_fiscal_book/model/invoice.py'
--- l10n_ve_fiscal_book/model/invoice.py	2013-04-25 16:22:59 +0000
+++ l10n_ve_fiscal_book/model/invoice.py	2013-06-04 19:08:29 +0000
@@ -24,9 +24,10 @@
 #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 ###############################################################################
 from openerp.osv import osv, fields
-
-
-class inherited_invoice(osv.osv):
+from openerp.tools.translate import _
+
+
+class account_invoice(osv.osv):
     _inherit = "account.invoice"
 
     _columns = {
@@ -36,12 +37,19 @@
         'issue_fb_id': fields.many2one('fiscal.book', 'Fiscal Book',
                                        help='Fiscal Book where this invoice \
                                        needs to be add'),
-        # TODO: THIS FIELD TO BE CHANGED TO A STORABLE FUNCTIONAL FIELD
-        # CHANGE EVEN FROM boolean to selection
-        'fb_submitted': fields.boolean('Fiscal Book Submitted?',
-                                       help='Indicates if this invoice is in \
-                                       a Fiscal Book which has  being already \
-                                       submitted to the statutory institute'),
     }
 
-inherited_invoice()
+    def action_cancel(self, cr, uid, ids, context=None):
+        """ Verify first in the invoice have a fiscal book associated and if
+        the sate of the book is in cancel. """
+        context = context or {}
+        for inv_brw in self.browse(cr, uid, ids, context=context):
+            if not inv_brw.fb_id or (inv_brw.fb_id and inv_brw.fb_id.state == 'cancel'):
+                super(account_invoice, self).action_cancel(cr, uid, ids,
+                                                           context=context)
+            else:
+                raise osv.except_osv(_("Error!"),
+                _("You can't cancel an invoice that is loaded in a processed"
+                  "Fiscal Book. You need to go to Fiscal Book and set the book"
+                  " to Draft. Then you could be able to cancel the invoice."))
+        return True