[wp-meta] [Making WordPress.org] #662: Component New Ticket link fails on multi-word components

Making WordPress.org noreply at wordpress.org
Sun Feb 28 13:34:33 UTC 2016


#662: Component New Ticket link fails on multi-word components
---------------------+------------------
 Reporter:  kraftbj  |       Owner:
     Type:  defect   |      Status:  new
 Priority:  normal   |   Milestone:
Component:  Trac     |  Resolution:
 Keywords:           |
---------------------+------------------

Comment (by ocean90):

 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
 [https://trac.edgewall.org/wiki/TracReports#AdvancedReports:DynamicVariables
 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

 {{{#!htm
 <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.

--
Ticket URL: <https://meta.trac.wordpress.org/ticket/662#comment:2>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list