[Erp5-report] r17009 - /erp5/trunk/products/ERP5Form/Report.py

[nobody at svn.erp5.org]

Author: jerome
Date: Mon Oct 15 18:44:50 2007
New Revision: 17009

URL: http://svn.erp5.org?rev=17009&view=rev
Log:
ReportSection:
remove preferences= argument, it has been deprecated for long enough, and has no effect anyway.
deprecate translated_title=, simply pass a translated string or a message instance as title=


Modified:
    erp5/trunk/products/ERP5Form/Report.py

Modified: erp5/trunk/products/ERP5Form/Report.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/Report.py?rev=17009&r1=17008&r2=17009&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/Report.py (original)
+++ erp5/trunk/products/ERP5Form/Report.py Mon Oct 15 18:44:50 2007
@@ -34,6 +34,7 @@
 from zLOG import LOG, WARNING
 
 from urllib import quote
+from warnings import warn
 from Products.ERP5Type import PropertySheet
 
 from Form import ERP5Form
@@ -195,8 +196,7 @@
                      selection_name=None, selection_params=None,
                      listbox_display_mode=None, selection_columns=None,
                      selection_sort_order=None,
-                     selection_report_path=None, selection_report_list=None,
-                     preferences = None ) :
+                     selection_report_path=None, selection_report_list=None):
     """
       Initialize the line and set the default values
       Selected columns must be defined in parameter of listbox.render...
@@ -209,6 +209,8 @@
     self.path = path
     self.form_id = form_id
     self.title = title
+    if translated_title is not None:
+      warn("Don't use translated_title, but title directly", DeprecationWarning)
     self.translated_title = translated_title
     self.level = level
     self.saved_request = {}
@@ -221,9 +223,6 @@
     self.selection_report_path = selection_report_path
     self.selection_report_list = selection_report_list
     self.saved_request_form = {}
-    if preferences is not None :
-      LOG('ERP5Report', WARNING,
-       'using preferences in report is deprecated, please use selection only')
     
   security.declarePublic('getTitle')
   def getTitle(self):
@@ -231,6 +230,7 @@
 
   security.declarePublic('getTranslatedTitle')
   def getTranslatedTitle(self):
+    # deprecated: use title instead (with a translated string)
     return self.translated_title
 
   security.declarePublic('getLevel')