Changeset 8157 for trunk/bp-groups/bp-groups-classes.php

Timestamp:

03/25/2014 06:30:56 PM (12 years ago)

Author:

boonebgorges

Message:

Improve validation and documentation for groups_create_group() and BP_Groups_Group::save()

Fixes #5214

Props r-a-y

File:

• trunk/bp-groups/bp-groups-classes.php (modified) (1 diff)

trunk/bp-groups/bp-groups-classes.php

@@ -265 +265 @@
 
                 do_action_ref_array( 'groups_group_before_save', array( &$this ) );
+
+                // Groups need at least a name
+                if ( empty( $this->name ) ) {
+                        return false;
+                }
+
+                // Set slug with group title if not passed
+                if ( empty( $this->slug ) ) {
+                        $this->slug = sanitize_title( $this->name );
+                }
+
+                // Sanity check
+                if ( empty( $this->slug ) ) {
+                        return false;
+                }
+
+                // Check for slug conflicts if creating new group
+                if ( empty( $this->id ) ) {
+                        $this->slug = groups_check_slug( $this->slug );
+                }
 
                 if ( !empty( $this->id ) ) {