[wp-meta] [Making WordPress.org] #7388: Github PR linking can get confused

Making WordPress.org noreply at wordpress.org
Fri Dec 22 06:18:11 UTC 2023


#7388: Github PR linking can get confused
--------------------------+--------------------
 Reporter:  dd32          |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:
Component:  Trac          |   Keywords:
--------------------------+--------------------
 As experienced with https://github.com/WordPress/wordpress.org/pull/186 +
 https://meta.trac.wordpress.org/ticket/7382 sometimes the GitHub PR
 linking gets confused as to the correct ticket to interact with.

 It appears that it treated a Themes.trac link as a meta.trac link, and
 subsequently threw an unhandled exception.

 The exception shouldn't have occurred, but it should've been handled
 gracefully as well.

 Exception generated by the above PR:
 {{{
 E_ERROR: Uncaught Exception: JSON Error: 404 Ticket 158677 does not exist.
 in dotorg/trac/pr/class-trac.php:145

 Stack trace:
 #0 dotorg/trac/pr/class-trac.php(104):
 WordPressdotorg\API\Trac\GithubPRs\Trac->_api('ticket.update', Array)
 #1 dotorg/trac/pr/class-trac.php(45):
 WordPressdotorg\API\Trac\GithubPRs\Trac->api('ticket.update', Array)
 #2 dotorg/trac/pr/webhook.php(149):
 WordPressdotorg\API\Trac\GithubPRs\Trac->update('158677', '''This ticket
 w...', Array, true)
 #3 {main}
  thrown
 Source: POST https://api.wordpress.org/dotorg/trac/pr/webhook.php
 }}}

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/7388>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list