diff roundup/cgi/client.py @ 5073:d0aa596daca8

Remove 'import *' statement from cgi/client.py There are various 'import *' statements scattered through the code which are generally not a good thing. These should be fairly safe changes, but I'll commit them one file at a time to make it easier to track down issues with a bisect if they crop up later.
author John Kristensen <john@jerrykan.com>
date Fri, 10 Jun 2016 17:19:51 +1000
parents dce3cfe7ec61
children 65fef7858606
line wrap: on
line diff
--- a/roundup/cgi/client.py	Fri Jun 10 17:08:37 2016 +1000
+++ b/roundup/cgi/client.py	Fri Jun 10 17:19:51 2016 +1000
@@ -15,9 +15,11 @@
 
 from roundup import roundupdb, date, hyperdb, password
 from roundup.cgi import templating, cgitb, TranslationService
-from roundup.cgi.actions import *
+from roundup.cgi import actions
 from roundup.exceptions import LoginError, Reject, RejectRaw, Unauthorised
-from roundup.cgi.exceptions import *
+from roundup.cgi.exceptions import (
+    FormError, NotFound, NotModified, Redirect, SendFile, SendStaticFile,
+    SeriousError)
 from roundup.cgi.form_parser import FormParser
 from roundup.mailer import Mailer, MessageSendError, encode_quopri
 from roundup.cgi import accept_language
@@ -1222,18 +1224,18 @@
 
     # these are the actions that are available
     actions = (
-        ('edit',        EditItemAction),
-        ('editcsv',     EditCSVAction),
-        ('new',         NewItemAction),
-        ('register',    RegisterAction),
-        ('confrego',    ConfRegoAction),
-        ('passrst',     PassResetAction),
-        ('login',       LoginAction),
-        ('logout',      LogoutAction),
-        ('search',      SearchAction),
-        ('retire',      RetireAction),
-        ('show',        ShowAction),
-        ('export_csv',  ExportCSVAction),
+        ('edit',        actions.EditItemAction),
+        ('editcsv',     actions.EditCSVAction),
+        ('new',         actions.NewItemAction),
+        ('register',    actions.RegisterAction),
+        ('confrego',    actions.ConfRegoAction),
+        ('passrst',     actions.PassResetAction),
+        ('login',       actions.LoginAction),
+        ('logout',      actions.LogoutAction),
+        ('search',      actions.SearchAction),
+        ('retire',      actions.RetireAction),
+        ('show',        actions.ShowAction),
+        ('export_csv',  actions.ExportCSVAction),
     )
     def handle_action(self):
         """ Determine whether there should be an Action called.

Roundup Issue Tracker: http://roundup-tracker.org/