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

Timestamp:

03/25/2014 06:30:56 PM (11 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 ) ) {