Changeset 11153

Timestamp:

07/27/2021 06:44:12 PM (23 months ago)

Author:

iandunn

Message:

Official WP Events: Clarify debugging output.

The initial assignment to $this->debug was removed because it's always overwritten in the constructor.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events

Files:

• meetup/class-meetup-client.php (modified) (2 diffs)
• official-wordpress-events.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/meetup/class-meetup-client.php

@@ -33 +33 @@
      * @var bool If true, the client will fetch fewer results, for faster debugging.
      */
-    protected $debug = false;
+    protected $debug;
 
     /**
@@ -165 +165 @@
             }
 
-            if ( $this->debug ) {
+            if ( $request_url && $this->debug ) {
+                if ( 'cli' === php_sapi_name() ) {
+                    echo "\nDebug mode: Skipping future paginated requests to $request_url";
+                }
+
                 break;
             }

sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php

@@ -304 +304 @@
         $events = $this->parse_wordcamp_events( $response );
 
-        $this->log( sprintf( 'returning %d events', count( $events ) ) );
+        $this->log( sprintf( 'returning %d WordCamp events', count( $events ) ) );
 
         return $events;
@@ -567 +567 @@
             return $events;
         }
+
+        $this->log( sprintf( 'received %d meetup groups', count( $groups ) ) );
 
         $yesterday    = date( 'c', strtotime( '-1 day' ) );
@@ -588 +590 @@
         $events = $this->parse_meetup_events( $meetups );
 
-        $this->log( sprintf( 'returning %d events', count( $events ) ) );
+        $this->log( sprintf( 'returning %d meetup events', count( $events ) ) );
 
         return $events;