Opened 10 years ago
Closed 8 years ago
#662 closed defect (bug) (fixed)
Component New Ticket link fails on multi-word components
Reported by: | kraftbj | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Trac | Keywords: | |
Cc: |
Description
If the component has multiple words, the link drops after the first space resulting in an invalid component.
Steps to duplicate:
- Visit https://core.trac.wordpress.org/component/Bundled+Theme
- Click the New Ticket button
- Notice the URL is lacking "+Theme" and the Component is set to Administrative (first in list).
Change History (4)
#2
@
9 years ago
https://core.trac.wordpress.org/component/Bundled+Theme is the same as https://core.trac.wordpress.org/report/31?COMPONENT=Bundled+Theme. The report itself is then using the dynamic variable in
[https://wordpress.org/support/bb-login.php?redirect_to=https://core.trac.wordpress.org/newticket?component=$COMPONENT Create a new ticket]
which outputs "Theme Create a new ticket" because $COMPONENT is not URL encoded.
I've changed the code to
[[https://wordpress.org/support/bb-login.php?redirect_to=https://core.trac.wordpress.org/newticket?component=$COMPONENT|Create a new ticket]]
which outputs
<a class="ext-link" href="https://wordpress.org/support/bb-login.php?redirect_to=https://core.trac.wordpress.org/newticket?component=Bundled Theme"><span class="icon">​</span>Create a new ticket</a>
But since it's still not URL encoded the space gets lost during the redirect.
Note: See
TracTickets for help on using
tickets.
Confirmed.