Edgewall Software

Ticket #5898 (new defect)

Opened 13 months ago

Last modified 11 months ago

Wrong req.redirect in process_admin_request

Reported by: airadier@… Owned by: cmlenz
Priority: normal Milestone: not applicable
Component: admin/web Version: devel
Severity: normal Keywords: verify
Cc:

Description

When using req.redirect method in process_admin_request, like:

req.redirect(self.env.href.admin(cat, page))

the browser is redirected to:

http://host/admin/cat/page

instead of

http://host/trac/project/admin/cat/page

(so the "trac/project" part is missing).

Attachments

Change History

Changed 11 months ago by cboos

  • keywords verify added
  • milestone changed from 0.11 to not applicable

This is related to the [trac] base_url setting which is missing, see #5064.

Anyway, the redirect should do req.redirect(req.href.admin(cat, page)) (req.href, not self.env) and that's what the admin module does in the trunk.

The WebAdmin plugin for 0.10 used to (still has?) that problem, though. Needs to be verified and eventually fixed.

Add/Change #5898 (Wrong req.redirect in process_admin_request)

Author



Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will change. Next status will be 'new'
The owner will change to anonymous. Next status will be 'assigned'
 
Note: See TracTickets for help on using tickets.