Changeset 2547

Timestamp:

02/02/2010 09:42:06 PM (15 years ago)

Author:

apeatling

Message:

Fixing checks for spaces in values.

Location:

trunk

Files:

• bp-activity.php (modified) (1 diff)
• bp-forums.php (modified) (1 diff)
• bp-groups.php (modified) (2 diffs)
• bp-xprofile.php (modified) (1 diff)
• bp-xprofile/bp-xprofile-classes.php (modified) (1 diff)

trunk/bp-activity.php

@@ -562 +562 @@
     extract( $r, EXTR_SKIP );
 
-    if ( empty( $content ) )
+    if ( empty( $content ) || !strlen( trim( $content ) ) )
         return false;
 

trunk/bp-forums.php

@@ -443 +443 @@
         $post_position = $post->post_position;
 
-    $post = bb_insert_post( array( 'post_id' => $post_id, 'topic_id' => $topic_id, 'post_text' => stripslashes( $post_text ), 'post_time' => $post_time, 'poster_id' => $poster_id, 'poster_ip' => $poster_ip, 'post_status' => $post_status, 'post_position' => $post_position ) );
+    $post = bb_insert_post( array( 'post_id' => $post_id, 'topic_id' => $topic_id, 'post_text' => stripslashes( trim( $post_text ) ), 'post_time' => $post_time, 'poster_id' => $poster_id, 'poster_ip' => $poster_ip, 'post_status' => $post_status, 'post_position' => $post_position ) );
 
     if ( $post )

trunk/bp-groups.php

@@ -1052 +1052 @@
 
         if ( 'group-details' == $bp->groups->current_create_step ) {
-            if ( empty( $_POST['group-name'] ) || empty( $_POST['group-desc'] ) ) {
+            if ( empty( $_POST['group-name'] ) || empty( $_POST['group-desc'] ) || !strlen( trim( $_POST['group-name'] ) ) || !strlen( trim( $_POST['group-desc'] ) ) ) {
                 bp_core_add_message( __( 'Please fill in all of the required fields', 'buddypress' ), 'error' );
                 bp_core_redirect( $bp->root_domain . '/' . $bp->groups->slug . '/create/step/' . $bp->groups->current_create_step . '/' );
@@ -1856 +1856 @@
     extract( $r, EXTR_SKIP );
 
-    if ( empty($content) || empty($user_id) || empty($group_id) )
+    if ( empty($content) || !strlen( trim( $content ) ) || empty($user_id) || empty($group_id) )
         return false;
 

trunk/bp-xprofile.php

@@ -719 +719 @@
         return false;
 
+    if ( empty( $value ) || !strlen( trim( $value ) ) )
+        return false;
+
     $field = new BP_XProfile_ProfileData();
     $field->field_id = $field_id;

trunk/bp-xprofile/bp-xprofile-classes.php

@@ -728 +728 @@
 
         if ( $this->is_valid_field() ) {
-            if ( $this->exists() && $this->value != '' ) {
+            if ( $this->exists() && !empty( $this->value ) && strlen( trim( $this->value ) ) ) {
                 $result = $wpdb->query( $wpdb->prepare( "UPDATE {$bp->profile->table_name_data} SET value = %s, last_updated = %s WHERE user_id = %d AND field_id = %d", $this->value, $this->last_updated, $this->user_id, $this->field_id ) );
             } else if ( $this->exists() && empty( $this->value ) ) {