*imrpovements

bzr revid: qdp@tinyerp.com-20090108084926-n4dxtv1bczx9id1t

1 files changed, 9 insertions, 9 deletions

diff --git a/addons/base_module_quality/base_module_quality.py b/addons/base_module_quality/base_module_quality.py
index 1be9de1ec5b..48a4d6dc69a 100644
--- a/addons/base_module_quality/base_module_quality.py
+++ b/addons/base_module_quality/base_module_quality.py
@@ -21,6 +21,7 @@
 ##############################################################################
 import pooler
 import os
+import osv
 from tools import config
 
 class abstract_quality_check(object):
@@ -77,7 +78,8 @@ class abstract_quality_check(object):
         '''
         this method should do the test and fill the score, result and result_details var
         '''
-        raise 'Not Implemented'
+
+        raise osv.except_osv(_('Programming Error'), _('Test Is Not Implemented'))
 
     def get_objects(self, cr, uid, module):
         # This function returns all object of the given module..
@@ -103,14 +105,12 @@ class abstract_quality_check(object):
 
     def format_table(self, header=[], data_list=[]):
         detail = ""
-        header_list = []
-        final_list = []
-        for head in data_list[1]:
-            header_list.append(head)
-        detail += (header[0]) % tuple(header_list)
-        for res in data_list[0]:
-            data_list[0][res].append(res)
-            detail += (header[1]) % tuple(data_list[0][res])
+        detail += (header[0]) % tuple(header[1])
+        frow = '\n|-'
+        for i in header[1]:
+            frow += '\n| %s'
+        for key, value in data_list.items():
+            detail += (frow) % tuple(value)
         detail = detail + '\n|}'
         return detail