Edgewall Software
Modify

Opened 19 years ago

Closed 19 years ago

#2846 closed defect (fixed)

WSGI merge removed `req.args.getlist()`, breaking WebAdmin

Reported by: Alec Thomas Owned by: Christopher Lenz
Priority: low Milestone: 0.10
Component: general Version: devel
Severity: minor Keywords: webadmin
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

This patch seems to fix it:

  • webadmin/plugin.py

     
    125125
    126126    def _do_update(self, req):
    127127        """Update component enablement."""
    128         components = req.args.getlist('component')
    129         enabled = req.args.getlist('enable')
     128        components = req.args['component']
     129        enabled = req.args['enable']
    130130        changes = False
    131131
    132132        # FIXME: this needs to be more intelligent and minimize multiple

Attachments (0)

Change History (2)

comment:1 by Christopher Lenz, 19 years ago

Owner: changed from Jonas Borgström to Christopher Lenz
Status: newassigned

I think it'd be better to reintroduce support for getlist() and getvalue() as described here.

comment:2 by Christopher Lenz, 19 years ago

Resolution: fixed
Status: assignedclosed

Fixed in [2967].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christopher Lenz.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christopher Lenz to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.