Changeset 9724

Timestamp:

04/15/2020 03:02:45 AM (6 years ago)

Author:

dd32

Message:

Translate: Only include a project in a users translated list if they hace contributed more than 5 strings to the project.

See #1664.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-custom-stats/stats/user.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-custom-stats/stats/user.php

@@ -60 +60 @@
                         }
 
-                        $this->user_project_stats[ "{$translation->user_id},{$project->id},{$translation_set->locale},{$translation_set->slug}" ] = true;
+                        $key = "{$translation->user_id},{$project->id},{$translation_set->locale},{$translation_set->slug}";
+                        if ( ! isset( $this->user_project_stats[ $key ] ) ) {
+                                $this->user_project_stats[ $key ] = 0;
+                        }
+                        $this->user_project_stats[ $key ]++;
                 }
         }
@@ -117 +121 @@
                 }
 
-                // Process the project stats too.
+                // Process the user project stats too.
                 $values = [];
-                foreach ( $this->user_project_stats as $key => $dummy ) {
+                foreach ( $this->user_project_stats as $key => $string_count ) {
                         list( $user_id, $project_id, $locale, $locale_slug ) = explode( ',', $key );
 
-                        $values[] = $wpdb->prepare( '(%d, %d, %s, %s, %s)',
-                                $user_id,
-                                $project_id,
-                                $locale,
-                                $locale_slug,
-                                $now
-                        );
-
-                        if ( count( $values ) > 50 ) {
-                                $wpdb->query(
-                                        "INSERT INTO {$wpdb->user_projects} (`user_id`, `project_id`, `locale`, `locale_slug`, `last_modified`)
-                                        VALUES " . implode( ', ', $values ) . "
-                                        ON DUPLICATE KEY UPDATE `last_modified` = GREATEST( `last_modified`, VALUES(`last_modified`) )"
-                                );
-                                $values = [];
-                        }
-                }
-                if ( $values ) {
-                        $wpdb->query(
-                                "INSERT INTO {$wpdb->user_projects} (`user_id`, `project_id`, `locale`, `locale_slug`, `last_modified`)
-                                VALUES " . implode( ', ', $values ) . "
-                                ON DUPLICATE KEY UPDATE `last_modified` = GREATEST( `last_modified`, VALUES(`last_modified`) )"
-                        );
-                        $values = [];
+                        // Step 1 - Does this user already have the project listed? Just Bump the date.
+                        if ( $id = $wpdb->get_var( $wpdb->prepare( 
+                                "SELECT id FROM {$wpdb->user_projects}
+                                WHERE user_id = %d AND project_id = %d AND locale = %s AND locale_slug = %s",
+                                $user_id, $project_id, $locale, $locale_slug
+                        ) ) ) {
+                                $wpdb->update(
+                                        $wpdb->user_projects,
+                                        [ 'last_modified' => $now ],
+                                        [ 'id' => $id ]
+                                );
+                                continue;
+                        }
+
+                        // Step 2 - More than 5 strings? Import Maybe? Just insert.
+                        if ( $string_count >= 5 ) {
+                                $wpdb->insert(
+                                        $wpdb->user_projects,
+                                        [
+                                                'user_id'       => $user_id,
+                                                'project_id'    => $project_id,
+                                                'locale'        => $locale,
+                                                'locale_slug'   => $locale_slug,
+                                                'last_modified' => $now
+                                        ]
+                                );
+                                continue;
+                        }
+
+                        // Step 3 - If not, find all the sub-project IDs, then all the translation sets, count strings by user.
+                        $this_project        = GP::$project->get( $project_id );
+                        $translation_set_ids = [];
+
+                        if ( ! $this_project->active ) {
+                                continue;
+                        }
+
+                        if ( $set = GP::$translation_set->by_project_id_slug_and_locale( $project->id, $locale_slug, $locale ) ) {
+                                $translation_set_ids[] = (int) $set->id;
+                        }
+
+                        // Fetch the strings from the sub projects too
+                        foreach ( $this_project->sub_projects() as $project ) {
+                                if ( ! $project->active ) {
+                                        continue;
+                                }
+                                if ( $set = GP::$translation_set->by_project_id_slug_and_locale( $project->id, $locale_slug, $locale ) ) {
+                                        $translation_set_ids[] = (int) $set->id;
+                                }
+                        }
+
+                        $user_translations = GP::$translation->find_many_no_map( [
+                                'user_id'            => $user_id,
+                                'translation_set_id' => $translation_set_ids
+                        ] );
+
+                        if ( $user_translations && count( $user_translations ) >= 5 ) {
+                                $wpdb->insert(
+                                        $wpdb->user_projects,
+                                        [
+                                                'user_id'       => $user_id,
+                                                'project_id'    => $project_id,
+                                                'locale'        => $locale,
+                                                'locale_slug'   => $locale_slug,
+                                                'last_modified' => $now
+                                        ]
+                                );
+                                continue;
+                        }
                 }
         }