From bb873a614f3f35c454e0f82426b17aff24aa6c29 Mon Sep 17 00:00:00 2001
From: resteve <resteve@zikzakmedia.com>
Date: Wed, 19 Aug 2015 23:29:04 +0200
Subject: [PATCH] Remove safe eval + pyflakes

---
 api.py                    | 13 +++++++++----
 view/carrier_api_form.xml |  2 +-
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/api.py b/api.py
index 1abfc30..6e24428 100644
--- a/api.py
+++ b/api.py
@@ -1,9 +1,8 @@
-# This file is part of the carrier_send_shipments module for Tryton.
+# This file is part of the carrier_send_shipments_seurvalencia module for Tryton.
 # The COPYRIGHT file at the top level of this repository contains the full
 # copyright notices and license terms.
-from trytond.model import fields
 from trytond.pool import PoolMeta
-from trytond.pyson import Eval
+from trytond.pyson import Eval, Not, Equal
 import logging
 
 try:
@@ -30,6 +29,13 @@ class CarrierApi:
         res.append(('seurvalencia', 'Seur Valencia'))
         return res
 
+    @classmethod
+    def view_attributes(cls):
+        return super(CarrierApi, cls).view_attributes() + [
+            ('//page[@id="seurvalencia"]', 'state', {
+                    'invisible': Not(Equal(Eval('method'), 'seurvalencia')),
+                    })]
+
     def test_seurvalencia(self, api):
         '''
         Test Seur Valencia connection
@@ -37,7 +43,6 @@ class CarrierApi:
         '''
         message = 'Connection unknown result'
         
-        seurvalencia_context = {}
         with API(api.username, api.password, api.debug) as seurvalencia_api:
             message = seurvalencia_api.test_connection()
         self.raise_user_error(message)
diff --git a/view/carrier_api_form.xml b/view/carrier_api_form.xml
index 6b43058..9c77b1f 100644
--- a/view/carrier_api_form.xml
+++ b/view/carrier_api_form.xml
@@ -4,7 +4,7 @@ The COPYRIGHT file at the top level of this repository contains the full
 copyright notices and license terms. -->
 <data>
     <xpath expr="/form/notebook/page[@id='api']" position="after">
-        <page string="Seur" id="seur" states="{'invisible': Not(Equal(Eval('method'), 'seurvalencia'))}">
+        <page string="Seur Valencia" id="seurvalencia">
         </page>
     </xpath>
 </data>