Context Navigation

← Previous Changeset
Next Changeset →

Changeset 14431

Timestamp:

04/22/2025 03:27:10 AM (15 months ago)

Author:

dd32

Message:

Theme Directory: Add a retry into the communications with Trac to see if that helps with updates failing to be set live.

File:

: 1 edited

sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/lib/class-trac.php (modified) (5 diffs)

Legend:

: Unmodified
: Added
: Removed

sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/lib/class-trac.php

-              r14420
+              r14431
+                }
                 $ok = $this->rpc->query( 'ticket.create', $subj, $desc, $attr );
+                $ok = $this->rpc_query_retry( 5, 'ticket.create', $subj, $desc, $attr );
                 if ( ! $ok ) {
                         trigger_error( 'Trac: ticket.create: ' . $this->rpc->error->message, E_USER_WARNING );
 …
         public function ticket_update( $id, $comment, $attr = array(), $notify = false ) {
                 if ( empty( $attr['_ts'] ) ) {
+                        $get         = $this->ticket_get( $id );
+                        $get = $this->ticket_get( $id );
+                        if ( ! $get ) {
+                                trigger_error( 'Trac: ticket.update: Couldn\'t get: ' . $this->rpc->error->message, E_USER_WARNING );
+                                return false;
+                        }
                         $attr['_ts'] = $get['_ts'];
+                }
 …
+                }
                 $ok = $this->rpc->query( 'ticket.update', $id, $comment, $attr, $notify );
+                $ok = $this->rpc_query_retry( 5, 'ticket.update', $id, $comment, $attr, $notify );
                 if ( ! $ok ) {
                         trigger_error( 'Trac: ticket.update: ' . $this->rpc->error->message, E_USER_WARNING );
 …
          */
         public function ticket_get( $id ) {
+                $ok = $this->rpc->query( 'ticket.get', $id );
+                $ok = $this->rpc_query_retry( 5, 'ticket.get', $id );
                 if ( ! $ok ) {
+                        trigger_error( 'Trac: ticket.get: ' . $this->rpc->error->message, E_USER_WARNING );
                         return false;
+                }
 …
                 return $response;
+        }
+        /**
+         * Queries Trac RPC with a backoff retry.
+         *
+         * @param int $retries Number of retries.
+         * @param mixed ...$args Arguments to pass to the query.
+         */
+        public function rpc_query_retry( $retries, ...$args ) {
+                $ok    = false;
+                $retry = 0;
+                while ( ! $ok && $retry < $retries ) {
+                        if ( $retry ) {
+                                sleep( $retry );
+                        }
+                        $retry++;
+                        $ok = call_user_func_array( [ $this->rpc, 'query' ], $args );
+                }
+                if ( ! $ok ) {
+                        trigger_error( 'Trac: rpc query retry failed: ' . $args[0] . ' ' . $this->rpc->error->message, E_USER_WARNING );
+                }
+                return $ok;
+        }
+}

Note: See TracChangeset for help on using the changeset viewer.

Making WordPress.org

Context Navigation

Changeset 14431

Legend:

sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/lib/class-trac.php

Download in other formats: