wp/wp-admin/includes/bookmark.php
changeset 5 5e2f62d02dcd
parent 0 d970ebf37754
child 7 cf61fcea0001
--- a/wp/wp-admin/includes/bookmark.php	Mon Jun 08 16:11:51 2015 +0000
+++ b/wp/wp-admin/includes/bookmark.php	Tue Jun 09 03:35:32 2015 +0200
@@ -27,7 +27,7 @@
  */
 function edit_link( $link_id = 0 ) {
 	if ( !current_user_can( 'manage_links' ) )
-		wp_die( __( 'Cheatin’ uh?' ) );
+		wp_die( __( 'Cheatin’ uh?' ), 403 );
 
 	$_POST['link_url'] = esc_html( $_POST['link_url'] );
 	$_POST['link_url'] = esc_url($_POST['link_url']);
@@ -50,7 +50,7 @@
  *
  * @since 2.0.0
  *
- * @return object Default link
+ * @return stdClass Default link
  */
 function get_default_link_to_edit() {
 	$link = new stdClass;
@@ -146,15 +146,17 @@
 
 	$defaults = array( 'link_id' => 0, 'link_name' => '', 'link_url' => '', 'link_rating' => 0 );
 
-	$linkdata = wp_parse_args( $linkdata, $defaults );
-	$linkdata = sanitize_bookmark( $linkdata, 'db' );
+	$args = wp_parse_args( $linkdata, $defaults );
+	$r = wp_unslash( sanitize_bookmark( $args, 'db' ) );
 
-	extract( wp_unslash( $linkdata ), EXTR_SKIP );
+	$link_id   = $r['link_id'];
+	$link_name = $r['link_name'];
+	$link_url  = $r['link_url'];
 
 	$update = false;
-
-	if ( !empty( $link_id ) )
+	if ( ! empty( $link_id ) ) {
 		$update = true;
+	}
 
 	if ( trim( $link_name ) == '' ) {
 		if ( trim( $link_url ) != '' ) {
@@ -164,54 +166,41 @@
 		}
 	}
 
-	if ( trim( $link_url ) == '' )
+	if ( trim( $link_url ) == '' ) {
 		return 0;
-
-	if ( empty( $link_rating ) )
-		$link_rating = 0;
-
-	if ( empty( $link_image ) )
-		$link_image = '';
-
-	if ( empty( $link_target ) )
-		$link_target = '';
-
-	if ( empty( $link_visible ) )
-		$link_visible = 'Y';
+	}
 
-	if ( empty( $link_owner ) )
-		$link_owner = get_current_user_id();
-
-	if ( empty( $link_notes ) )
-		$link_notes = '';
-
-	if ( empty( $link_description ) )
-		$link_description = '';
-
-	if ( empty( $link_rss ) )
-		$link_rss = '';
-
-	if ( empty( $link_rel ) )
-		$link_rel = '';
+	$link_rating      = ( ! empty( $r['link_rating'] ) ) ? $r['link_rating'] : 0;
+	$link_image       = ( ! empty( $r['link_image'] ) ) ? $r['link_image'] : '';
+	$link_target      = ( ! empty( $r['link_target'] ) ) ? $r['link_target'] : '';
+	$link_visible     = ( ! empty( $r['link_visible'] ) ) ? $r['link_visible'] : 'Y';
+	$link_owner       = ( ! empty( $r['link_owner'] ) ) ? $r['link_owner'] : get_current_user_id();
+	$link_notes       = ( ! empty( $r['link_notes'] ) ) ? $r['link_notes'] : '';
+	$link_description = ( ! empty( $r['link_description'] ) ) ? $r['link_description'] : '';
+	$link_rss         = ( ! empty( $r['link_rss'] ) ) ? $r['link_rss'] : '';
+	$link_rel         = ( ! empty( $r['link_rel'] ) ) ? $r['link_rel'] : '';
+	$link_category    = ( ! empty( $r['link_category'] ) ) ? $r['link_category'] : array();
 
 	// Make sure we set a valid category
-	if ( ! isset( $link_category ) || 0 == count( $link_category ) || !is_array( $link_category ) ) {
+	if ( ! is_array( $link_category ) || 0 == count( $link_category ) ) {
 		$link_category = array( get_option( 'default_link_category' ) );
 	}
 
 	if ( $update ) {
-		if ( false === $wpdb->update( $wpdb->links, compact('link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_rating', 'link_rel', 'link_notes', 'link_rss'), compact('link_id') ) ) {
-			if ( $wp_error )
+		if ( false === $wpdb->update( $wpdb->links, compact( 'link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_rating', 'link_rel', 'link_notes', 'link_rss' ), compact( 'link_id' ) ) ) {
+			if ( $wp_error ) {
 				return new WP_Error( 'db_update_error', __( 'Could not update link in the database' ), $wpdb->last_error );
-			else
+			} else {
 				return 0;
+			}
 		}
 	} else {
-		if ( false === $wpdb->insert( $wpdb->links, compact('link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_owner', 'link_rating', 'link_rel', 'link_notes', 'link_rss') ) ) {
-			if ( $wp_error )
+		if ( false === $wpdb->insert( $wpdb->links, compact( 'link_url', 'link_name', 'link_image', 'link_target', 'link_description', 'link_visible', 'link_owner', 'link_rating', 'link_rel', 'link_notes', 'link_rss' ) ) ) {
+			if ( $wp_error ) {
 				return new WP_Error( 'db_insert_error', __( 'Could not insert link into the database' ), $wpdb->last_error );
-			else
+			} else {
 				return 0;
+			}
 		}
 		$link_id = (int) $wpdb->insert_id;
 	}