Changeset 4059

Timestamp:

02/21/2011 02:10:23 PM (16 years ago)

Author:

johnjamesjacoby

Message:

Check description fields in xprofile groups and fields, and prevent errors if they are left empty.

Location:

trunk/bp-xprofile

Files:

• bp-xprofile-admin.php (modified) (2 diffs)
• bp-xprofile-classes.php (modified) (1 diff)

trunk/bp-xprofile/bp-xprofile-admin.php

@@ -173 +173 @@
                 if ( BP_XProfile_Group::admin_validate( $_POST ) ) {
                         $group->name            = wp_filter_kses( $_POST['group_name'] );
-                        $group->description     = wp_filter_kses( $_POST['group_description'] );
+                        $group->description     = !empty( $_POST['group_description'] ) ? wp_filter_kses( $_POST['group_description'] ) : '';
 
                         if ( !$group->save() ) {
@@ -237 +237 @@
                 if ( BP_XProfile_Field::admin_validate() ) {
                         $field->name        = wp_filter_kses( $_POST['title'] );
-                        $field->description = wp_filter_kses( $_POST['description'] );
+                        $field->description = !empty( $_POST['description'] ) ? wp_filter_kses( $_POST['description'] ) : '';
                         $field->is_required = wp_filter_kses( $_POST['required'] );
                         $field->type        = wp_filter_kses( $_POST['fieldtype'] );

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

@@ -45 +45 @@
                         $sql = $wpdb->prepare( "INSERT INTO {$bp->profile->table_name_groups} (name, description, can_delete) VALUES (%s, %s, 1)", $this->name, $this->description );
 
-                if ( !$wpdb->query( $sql ) )
+                if ( is_wp_error( $wpdb->query( $sql ) ) )
                         return false;