Changeset 1017 for trunk/bp-xprofile/bp-xprofile-admin.php

Timestamp:

02/04/2009 12:29:21 AM (16 years ago)

Author:

matt

Message:

Minor cleanups, mods, and todos.

File:

• trunk/bp-xprofile/bp-xprofile-admin.php (modified) (6 diffs)

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

@@ -11 +11 @@
 function xprofile_admin( $message = '', $type = 'error' ) {
     global $bp;
-    
+
+    $type = preg_replace( '|[^a-z]|i', '', $type );
+
     $groups = BP_XProfile_Group::get_all();
-    
+
     if ( isset($_GET['mode']) && isset($_GET['group_id']) && $_GET['mode'] == "add_field" ) {
         xprofile_admin_manage_field($_GET['group_id']);
@@ -45 +47 @@
         ?>
             <div id="message" class="<?php echo $type; ?> fade">
-                <p><?php echo $message; ?></p>
+                <p><?php echo wp_specialchars( $message ); ?></p>
             </div>
         <?php }
@@ -63 +65 @@
             </script>
             
-            <?php if ( function_exists('wp_nonce_field') )
-                wp_nonce_field('xprofile_reorder_fields');
-            
-            for ( $i = 0; $i < count($groups); $i++ ) {
+            <?php 
+            wp_nonce_field('xprofile_reorder_fields');
+            
+            for ( $i = 0; $i < count($groups); $i++ ) { // TODO: foreach
             ?>
                 <p>
@@ -121 +123 @@
                 
         <?php } else { ?>
-            <div id="message" class="error"><p>You have no groups.</p></div>
+            <div id="message" class="error"><p><?php _e('You have no groups.', 'buddypress' ); ?></p></div>
             <p><a href="admin.php?page=xprofile_settings&amp;mode=add_group"><?php _e( 'Add New Group', 'buddypress' ) ?></a></p>
         <?php } ?>
@@ -184 +186 @@
     }
     
-    unset($_GET['mode']);
+    unset($_GET['mode']); // TODO: wtf?
     xprofile_admin( $message, $type );
 }
@@ -261 +263 @@
     xprofile_admin($message, $type);
 }
-
-?>