WordPress.org

Make WordPress Core

Changeset 7336


Ignore:
Timestamp:
03/16/08 20:37:02 (6 years ago)
Author:
ryan
Message:

Don't allow duplicate category creation. Props westi and mdawaffe. fixes #3799

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/wp-admin/admin-ajax.php

    r7162 r7336  
    247247    } 
    248248 
     249    if ( category_exists( trim( $_POST['cat_name'] ) ) ) { 
     250        $x = new WP_Ajax_Response( array( 
     251            'what' => 'cat', 
     252            'id' => new WP_Error( 'cat_exists', __('The category you are trying to create already exists.'), array( 'form-field' => 'cat_name' ) ), 
     253        ) ); 
     254        $x->send(); 
     255    } 
     256     
    249257    $cat = wp_insert_category( $_POST, true ); 
    250258 
  • trunk/wp-includes/js/wp-ajax-response.js

    r7262 r7336  
    5151        return true; 
    5252    }, 
     53    invalidateForm: function ( selector ) { 
     54        return jQuery( selector ).addClass( 'form-invalid' ).change( function() { jQuery(this).removeClass( 'form-invalid' ); } ); 
     55    }, 
    5356    validateForm: function( selector ) { 
    5457        selector = jQuery( selector ); 
    55         return !selector.find('.form-required').andSelf().filter('.form-required:has(:input[value=""]), .form-required:input[value=""]').addClass( 'form-invalid' ).change( function() { jQuery(this).removeClass( 'form-invalid' ); } ).size(); 
     58        return !wpAjax.invalidateForm( selector.find('.form-required').andSelf().filter('.form-required:has(:input[value=""]), .form-required:input[value=""]') ).size(); 
    5659    } 
    5760}, wpAjax || { noPerm: 'You do not have permission to do that.', broken: 'AJAX is teh b0rked.' } ); 
  • trunk/wp-includes/script-loader.php

    r7316 r7336  
    4242        $this->add( 'prototype', '/wp-includes/js/prototype.js', false, '1.6'); 
    4343 
    44         $this->add( 'wp-ajax-response', '/wp-includes/js/wp-ajax-response.js', array('jquery'), '20080312' ); 
     44        $this->add( 'wp-ajax-response', '/wp-includes/js/wp-ajax-response.js', array('jquery'), '20080316' ); 
    4545        $this->localize( 'wp-ajax-response', 'wpAjax', array( 
    4646            'noPerm' => 'You do not have permission to do that.', 
Note: See TracChangeset for help on using the changeset viewer.