Ticket #31206: 31206.diff

src/wp-admin/includes/ajax-actions.php

@@ -597 +597 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'delete-post'.
  */
-function wp_ajax_delete_post( $action ) {
-        if ( empty( $action ) )
-                $action = 'delete-post';
+function wp_ajax_delete_post( $action = 'delete-post' ) {
         $id = isset( $_POST['id'] ) ? (int) $_POST['id'] : 0;
 
         check_ajax_referer( "{$action}_$id" );
@@ -622 +620 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'trash-post'.
  */
-function wp_ajax_trash_post( $action ) {
-        if ( empty( $action ) )
-                $action = 'trash-post';
+function wp_ajax_trash_post( $action = 'trash-post' ) {
         $id = isset( $_POST['id'] ) ? (int) $_POST['id'] : 0;
 
         check_ajax_referer( "{$action}_$id" );
@@ -652 +648 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'untrash-post'.
  */
-function wp_ajax_untrash_post( $action ) {
-        if ( empty( $action ) )
-                $action = 'untrash-post';
+function wp_ajax_untrash_post( $action = 'untrash-post' ) {
         wp_ajax_trash_post( $action );
 }
 
-function wp_ajax_delete_page( $action ) {
-        if ( empty( $action ) )
-                $action = 'delete-page';
+/**
+ * Ajax handler to delete a page.
+ * 
+ * @param string $action Optional. Action to perform; expects 'delete-page'.
+ */
+function wp_ajax_delete_page( $action = 'delete-page' ) {
         $id = isset( $_POST['id'] ) ? (int) $_POST['id'] : 0;
 
         check_ajax_referer( "{$action}_$id" );
@@ -725 +722 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'add-link-category'.
  */
-function wp_ajax_add_link_category( $action ) {
-        if ( empty( $action ) )
-                $action = 'add-link-category';
+function wp_ajax_add_link_category( $action = 'add-link-category' ) {
         check_ajax_referer( $action );
         if ( !current_user_can( 'manage_categories' ) )
                 wp_die( -1 );
@@ -864 +859 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'get-comments'.
  */
-function wp_ajax_get_comments( $action ) {
+function wp_ajax_get_comments( $action = 'get-comments' ) {
         global $wp_list_table, $post_id;
-        if ( empty( $action ) )
-                $action = 'get-comments';
 
         check_ajax_referer( $action );
 
@@ -915 +908 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'replyto-comment'.
  */
-function wp_ajax_replyto_comment( $action ) {
+function wp_ajax_replyto_comment( $action = 'replyto-comment') {
         global $wp_list_table;
-        if ( empty( $action ) )
-                $action = 'replyto-comment';
 
         check_ajax_referer( $action, '_ajax_nonce-replyto-comment' );
 
@@ -1229 +1220 @@
  *
  * @since 3.1.0
  *
- * @param string $action Action to perform.
+ * @param string $action Optional. Action to perform; expects 'add-user'.
  */
-function wp_ajax_add_user( $action ) {
+function wp_ajax_add_user( $action = 'add-user' ) {
         global $wp_list_table;
-        if ( empty( $action ) )
-                $action = 'add-user';
-
+        
         check_ajax_referer( $action );
         if ( ! current_user_can('create_users') )
                 wp_die( -1 );
@@ -2733 +2722 @@
         ) );
 }
 
+/**
+ * Enables sandboxed previewing of media shortcodes in the visual editor.
+ * 
+ * @since 4.0.0
+ */
 function wp_ajax_parse_media_shortcode() {
         global $post, $wp_scripts;