Changeset 4728

Timestamp:

01/19/2017 06:48:03 AM (8 years ago)

Author:

dd32

Message:

Plugin Directory: Add stat recording for downloads to bump the plugin download counts.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory

Files:

• jobs/class-meta-sync.php (modified) (1 diff)
• zip/class-serve.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/jobs/class-meta-sync.php

@@ -34 +34 @@
 
         $download_count_table = PLUGINS_TABLE_PREFIX . 'download_counts';
-        $bbpress_topic_slug_table = PLUGINS_TABLE_PREFIX . 'topics';
 
         $changed_download_counts = $wpdb->get_results(
             "SELECT p.id as post_id, downloads
             FROM `{$wpdb->posts}` p
-                JOIN `{$bbpress_topic_slug_table}` t ON p.post_name = t.topic_slug
-                JOIN `{$download_count_table}` c on t.topic_id = c.topic_id
+                JOIN `{$download_count_table}` c on p.post_name = c.plugin_slug
                 LEFT JOIN `{$wpdb->postmeta}` pm ON p.id = pm.post_id AND pm.meta_key = 'downloads'
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/zip/class-serve.php

@@ -179 +179 @@
         global $wpdb;
 
-        $stats = array(
+        $stats_dedup_log_table = PLUGINS_TABLE_PREFIX . 'downloads';
+        $stats_download_table = PLUGINS_TABLE_PREFIX . 'download_counts';
+        $stats_download_daily_table = PLUGINS_TABLE_PREFIX . 'stats';
+
+        // Very basic de-duplication for downloads.
+        $recent = $wpdb->get_var( $wpdb->prepare(
+            "SELECT `client_ip`
+            FROM {$stats_dedup_log_table}
+            WHERE
+                plugin_slug = %s AND
+                client_ip = %s AND
+                user_agent = %s AND
+                stamp BETWEEN %s AND %s
+            LIMIT 1",
+            $request['slug'],
+            $_SERVER['REMOTE_ADDR'],
+            $_SERVER['HTTP_USER_AGENT'],
+            gmdate( 'Y-m-d 00:00:00' ),
+            gmdate( 'Y-m-d 23:59:59' )
+        ) );
+
+        if ( $recent ) {
+            return;
+        }
+
+        $wpdb->query( $wpdb->prepare(
+            "INSERT INTO {$stats_download_table} (`plugin_slug`, `downloads`) VALUES ( %s, 1 ) ON DUPLICATE KEY UPDATE `downloads` = `downloads` + 1",
+            $stats['plugin_slug']
+        ) );
+
+        $wpdb->query( $wpdb->prepare(
+            "INSERT INTO {$stats_download_daily_table} (`plugin_slug`, `stamp`, `downloads`) VALUES ( %s, %s, 1 ) ON DUPLICATE KEY UPDATE `downloads` = `downloads` + 1",
+            $stats['plugin_slug'],
+            gmdate( 'Y-m-d' )
+        ) );
+
+        $wpdb->insert( $stats_dedup_log_table, array(
             'plugin_slug' => $request['slug'],
             'client_ip'   => $_SERVER['REMOTE_ADDR'],
-            'client_host' => gethostbyaddr( $_SERVER['REMOTE_ADDR'] ),
-            'user_agent'  => $_SERVER['HTTP_USER_AGENT']
-        );
-        // $wpdb->insert( stats_table, $stats );
-        // TODO: This is stored in bbPress tables still.
-        // $post_id = $this->get_post_id( $request['slug'] );
-        // $wpdb->query( $wpdb->prepare( "UPDATE $wpdb->postmeta SET meta_value = meta_value + 1 WHERE post_id = %d AND meta_key = 'downloads'", $post_id ) );
+            'user_agent'  => $_SERVER['HTTP_USER_AGENT'],
+            'stamp'       => gmdate( 'Y-m-d H:i:s' )
+        ) );
+
     }