Mercurial > p > roundup > code
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.
