Edgewall Software

Ticket #2846 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

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

Reported by: athomas Owned by: cmlenz
Priority: low Milestone: 0.10
Component: general Version: devel
Severity: minor Keywords: webadmin
Cc:

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

Change History

Changed 3 years ago by cmlenz

  • owner changed from jonas to cmlenz
  • status changed from new to assigned

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

Changed 3 years ago by cmlenz

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in [2967].

Add/Change #2846 (WSGI merge removed `req.args.getlist()`, breaking WebAdmin)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from cmlenz. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.