From ee7da268b073b1d1edda3295abc3ceef9141b7e5 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis@via.ecp.fr>
Date: Thu, 1 Jan 2015 21:12:04 +0100
Subject: [PATCH] [FIX] Bug #11 AMI test: success is now only reported when it
 really works !

---
 asterisk_click2dial/asterisk_click2dial.py | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/asterisk_click2dial/asterisk_click2dial.py b/asterisk_click2dial/asterisk_click2dial.py
index 4fd718b..5be18fe 100644
--- a/asterisk_click2dial/asterisk_click2dial.py
+++ b/asterisk_click2dial/asterisk_click2dial.py
@@ -248,6 +248,7 @@ class asterisk_server(orm.Model):
     def test_ami_connection(self, cr, uid, ids, context=None):
         assert len(ids) == 1, 'Only 1 ID'
         ast_server = self.browse(cr, uid, ids[0], context=context)
+        ast_manager = False
         try:
             ast_manager = Manager.Manager(
                 (ast_server.ip_address, ast_server.port),
@@ -258,15 +259,12 @@ class asterisk_server(orm.Model):
                 _("Connection Test Failed!"),
                 _("Here is the error message: %s" % e))
         finally:
-            try:
-                if ast_manager:
-                    ast_manager.Logoff()
-            except Exception:
-                pass
-            raise orm.except_orm(
-                _("Connection Test Successfull!"),
-                _("OpenERP can successfully login to the Asterisk Manager "
-                  "Interface."))
+            if ast_manager:
+                ast_manager.Logoff()
+        raise orm.except_orm(
+            _("Connection Test Successfull!"),
+            _("Odoo can successfully login to the Asterisk Manager "
+                "Interface."))
 
     def _get_calling_number(self, cr, uid, context=None):