Changeset 3347 for trunk/bp-groups.php

Timestamp:

11/03/2010 11:55:29 PM (14 years ago)

Author:

boonebgorges

Message:

Better handling of error messages during the creation of forum topics. Fixes #947

File:

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

trunk/bp-groups.php

@@ -477 +477 @@
                     groups_join_group( $bp->groups->current_group->id, $bp->loggedin_user->id );
 
-                if ( !$topic = groups_new_group_forum_topic( $_POST['topic_title'], $_POST['topic_text'], $_POST['topic_tags'], $forum_id ) )
-                    bp_core_add_message( __( 'There was an error when creating the topic', 'buddypress'), 'error' );
-                else
-                    bp_core_add_message( __( 'The topic was created successfully', 'buddypress') );
-
-                bp_core_redirect( bp_get_group_permalink( $bp->groups->current_group ) . 'forum/topic/' . $topic->topic_slug . '/' );
+                if ( empty( $_POST['topic_title'] ) )
+                    $error_message = __( 'Please provide a title for your forum topic.', 'buddypress' );
+                else if ( empty( $_POST['topic_text'] ) )
+                    $error_message = __( 'Forum posts cannot be empty. Please enter some text.', 'buddypress' );
+                
+                if ( $error_message ) {
+                    bp_core_add_message( $error_message, 'error' );
+                    $redirect = bp_get_group_permalink( $bp->groups->current_group ) . 'forum';
+                } else {
+                    if ( !$topic = groups_new_group_forum_topic( $_POST['topic_title'], $_POST['topic_text'], $_POST['topic_tags'], $forum_id ) ) {
+                        bp_core_add_message( __( 'There was an error when creating the topic', 'buddypress'), 'error' );
+                        $redirect = bp_get_group_permalink( $bp->groups->current_group ) . 'forum';
+                    } else {
+                        bp_core_add_message( __( 'The topic was created successfully', 'buddypress') );
+                        $redirect = bp_get_group_permalink( $bp->groups->current_group ) . 'forum/topic/' . $topic->topic_slug . '/';
+                    }
+                }
+                
+                bp_core_redirect( $redirect );
+                
             }