From e03af5e54c771344d0ceee7b00707a7198c8ebf0 Mon Sep 17 00:00:00 2001
From: "Saurang Suthar(OpenERP)" <>
Date: Wed, 25 Jul 2012 17:35:41 +0530
Subject: [PATCH] [IMP] Improved warning messages for caldav module

bzr revid: pso@tinyerp.com-20120725120541-31kvapmzwvafuk7e
---
 addons/caldav/caldav_node.py  | 12 ++++++------
 addons/caldav/calendar.py     |  6 +++---
 addons/caldav/i18n/caldav.pot |  6 +++---
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/addons/caldav/caldav_node.py b/addons/caldav/caldav_node.py
index da5c5f017a6..e32801f5cd5 100644
--- a/addons/caldav/caldav_node.py
+++ b/addons/caldav/caldav_node.py
@@ -252,20 +252,20 @@ class node_calendar(nodes.node_class):
                                                 elif cfe.getAttribute('end'):
                                                     _log.warning("Ignore end.. ")
                                             else:
-                                                _log.debug("Unknown comp-filter: %s", cfe.localName)
+                                                _log.debug("Unknown comp-filter: %s.", cfe.localName)
                                     else:
-                                        _log.debug("Unknown comp-filter: %s", vevent_filter.localName)
+                                        _log.debug("Unknown comp-filter: %s.", vevent_filter.localName)
                         else:
-                            _log.debug("Unknown filter element: %s", vcalendar_filter.localName)
+                            _log.debug("Unknown filter element: %s.", vcalendar_filter.localName)
                 else:
-                    _log.debug("Unknown calendar-query element: %s", filter_child.localName)
+                    _log.debug("Unknown calendar-query element: %s.", filter_child.localName)
             return res
         elif filters.localName == 'calendar-multiget':
             # this is not the place to process, as it wouldn't support multi-level
             # hrefs. So, the code is moved to document_webdav/dav_fs.py
             pass
         else:
-            _log.debug("Unknown element in REPORT: %s", filters.localName)
+            _log.debug("Unknown element in REPORT: %s.", filters.localName)
         return res
 
     def children(self, cr, domain=None):
@@ -341,7 +341,7 @@ class node_calendar(nodes.node_class):
                     if line.name == ourcal.type:
                         line_id = line.id
                         break
-                assert line_id, "Calendar #%d must have at least one %s line" % \
+                assert line_id, "Calendar #%d must have at least one %s line." % \
                                     (ourcal.id, ourcal.type)
                 if path.endswith('.ics'):
                     path = path[:-4]
diff --git a/addons/caldav/calendar.py b/addons/caldav/calendar.py
index 8678d617463..c8ee9553cb8 100644
--- a/addons/caldav/calendar.py
+++ b/addons/caldav/calendar.py
@@ -729,9 +729,9 @@ class Calendar(CalDAV, osv.osv):
             elif child.name.upper() in ('PRODID', 'VERSION'):
                 pass
             elif child.name.upper().startswith('X-'):
-                _logger.debug("skipping custom node %s", child.name)
+                _logger.debug("Skipping custom node %s.", child.name)
             else:
-                _logger.debug("skipping node %s", child.name)
+                _logger.debug("Skipping node %s.", child.name)
         
         res = []
         for obj_name in list(set(objs)):
@@ -856,7 +856,7 @@ class basic_calendar_fields(osv.osv):
     }
 
     _sql_constraints = [
-        ( 'name_type_uniq', 'UNIQUE(name, type_id)', 'Can not map a field more than once'),
+        ( 'name_type_uniq', 'UNIQUE(name, type_id)', 'Cannot map a field more than once.'),
     ]
 
     def check_line(self, cr, uid, vals, name, context=None):
diff --git a/addons/caldav/i18n/caldav.pot b/addons/caldav/i18n/caldav.pot
index 444a7e04bba..a6ab8304082 100644
--- a/addons/caldav/i18n/caldav.pot
+++ b/addons/caldav/i18n/caldav.pot
@@ -87,7 +87,7 @@ msgstr ""
 
 #. module: caldav
 #: sql_constraint:basic.calendar.fields:0
-msgid "Can not map a field more than once"
+msgid "Cannot map a field more than once."
 msgstr ""
 
 #. module: caldav
@@ -164,7 +164,7 @@ msgstr ""
 #. module: caldav
 #: code:addons/caldav/calendar.py:879
 #, python-format
-msgid "Please provide proper configuration of \"%s\" in Calendar Lines"
+msgid "Please provide proper configuration of \"%s\" in Calendar Lines."
 msgstr ""
 
 #. module: caldav
@@ -207,7 +207,7 @@ msgstr ""
 #. module: caldav
 #: code:addons/caldav/calendar.py:789
 #, python-format
-msgid "Can not create line \"%s\" more than once"
+msgid "Cannot create line \"%s\" more than once."
 msgstr ""
 
 #. module: caldav