Modify ↓
Opened 12 years ago
Closed 12 years ago
#10125 closed defect (duplicate)
UnboundLocalError: local variable 'data' referenced before assignment
Reported by: | Owned by: | Christopher Paredes | |
---|---|---|---|
Priority: | lowest | Component: | SimpleMultiProjectPlugin |
Severity: | trivial | Keywords: | |
Cc: | Trac Release: | 0.12 |
Description
Hi, I had this little issue with SimpleMultiProjectPlugin today, when after I added a new project, I had the following error in the logs :
2012-06-26 17:51:29,193 Trac[admin] ERROR: Add Project Error: not all arguments converted during string formatting 2012-06-26 17:51:29,194 Trac[admin] ERROR: SimpleMultiProject Error: Failed to added project 'test 1' 2012-06-26 17:51:29,197 Trac[main] ERROR: Internal Server Error: Traceback (most recent call last): File "build/bdist.freebsd-8.2-RELEASE-amd64/egg/trac/web/main.py", line 511, in _dispatch_request dispatcher.dispatch(req) File "build/bdist.freebsd-8.2-RELEASE-amd64/egg/trac/web/main.py", line 237, in dispatch resp = chosen_handler.process_request(req) File "build/bdist.freebsd-8.2-RELEASE-amd64/egg/trac/admin/web_ui.py", line 116, in process_request path_info) File "build/bdist.freebsd-8.2-RELEASE-amd64/egg/simplemultiproject/admin.py", line 118, in render_admin_panel return 'simplemultiproject_adminpanel.html', data UnboundLocalError: local variable 'data' referenced before assignment
So I tracked down the problem, and found that the insert_project method in model.py seemed to be wrong :
def insert_project(self, name, summary, description): cursor = self.__get_cursor() query = """INSERT INTO smp_project (name, summary, description) VALUES ('%s', '%s');""" % (name, summary, description) cursor.execute(query) self.__start_transacction()
At line 111 VALUES ('%s', '%s');""" % (name, summary, description)
, it should be 111 VALUES ('%s', '%s', '%s');""" % (name, summary, description)
if I'm not mistaken. I made the change, and I can now add a new project.
Attachments (0)
Note: See
TracTickets for help on using
tickets.
see #10124