Changeset 12711

Timestamp:

07/05/2023 04:06:16 AM (17 months ago)

Author:

dd32

Message:

Trac: Add a custom API to fetch/set user preferences.

See #5055.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications

Files:

• trac-notifications-db.php (modified) (1 diff)
• trac-notifications-sqlite-driver.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-notifications-db.php

@@ -191 +191 @@
                 call_user_func( array( $this->db, $method ), '_notifications', (array) $where );
             }
+        }
+    }
+
+    /**
+     * Fetch user preferences from Trac.
+     *
+     * @param string $username The username.
+     * @return array The user preferences. [ key => value ].
+     */
+    function get_user_prefs( $username ) {
+        $data = $this->db->get_results( $this->db->prepare(
+            'SELECT name, value FROM session_attribute WHERE sid = %s',
+            $username
+        ), ARRAY_A );
+
+        return array_column( $data, 'value', 'name' );
+    }
+
+    /**
+     * Set a user preference in Trac.
+     *
+     * @param string $username   The username.
+     * @param string $name       The preference.
+     * @param string|null $value The value to set. If null, the preference will be deleted.
+     */
+    function set_user_pref( $username, $name, $value = null ) {
+        // The trac column for username is `sid`. All our users are authenticated.
+        // The session_attribute table has an index: UNIQUE (sid,authenticated,name)
+
+        if ( ! is_null( $value ) ) {
+            $result = $this->db->update(
+                'session_attribute',
+                [ 'value' => $value ],
+                [
+                    'sid'           => $username,
+                    'authenticated' => 1,
+                    'name'          => $name
+                ]
+            );
+
+            if ( ! $result ) {
+                // Insert the session if it doesn't exist.
+                $user_has_visited_trac = $this->db->get_var( $this->db->prepare(
+                    'SELECT sid FROM session WHERE sid = %s',
+                    $username
+                ) );
+                if ( ! $user_has_visited_trac ) {
+                    $this->db->insert( 'session', [
+                        'sid'           => $username,
+                        'authenticated' => 1,
+                        'last_visit'    => time(),
+                    ] );
+                }
+
+                // Set the pref.
+                $result = $this->db->insert( 'session_attribute', [
+                    'sid'           => $username,
+                    'authenticated' => 1,
+                    'name'          => $name,
+                    'value'         => $value,
+                ] );
+            }
+
+            return $result;
+        } else {
+            return $this->db->delete( 'session_attribute', [
+                'sid'           => $username,
+                'authenticated' => 1,
+                'name'          => $name
+            ] );
         }
     }

sites/trunk/wordpress.org/public_html/wp-content/plugins/trac-notifications/trac-notifications-sqlite-driver.php

@@ -69 +69 @@
         return (bool) $this->db->query( $query );
     }
+
+    public function update( $table, $data, $wheres ) {
+        $values     = [];
+        $sql_sets   = [];
+        $sql_wheres = [];
+
+        foreach ( $data as $field => $value ) {
+            $sql_sets[] = "$field = %s";
+            $values[]   = $value;
+        }
+        $sql_sets = implode( ', ', $sql_sets );
+
+        foreach ( $wheres as $field => $where ) {
+            $sql_wheres[] = "$field = %s";
+            $values[]     = $value;
+        }
+        $sql_wheres = implode( ' AND ', $sql_wheres );
+
+        if ( ! $values ) {
+            return false;
+        }
+
+        $query = $this->prepare(
+            "UPDATE $table SET $sql_sets WHERE $sql_wheres",
+            $values
+        );
+
+        return (bool) $this->db->query( $query );
+    }
 }