Ticket #17698: 17698.diff

wp-admin/js/nav-menu.dev.js

@@ -41 +41 @@
                         this.attachTabsPanelListeners();
 
                         this.attachUnsavedChangesListener();
+                        
+                        this.attachValidators();
 
                         if( api.menuList.length ) // If no menu, we're in the + tab.
                                 this.initSortables();
@@ -108 +110 @@
                                                 });
                                         });
                                 },
+
+                                validateSelections : function() {
+                                        return !! this.find('.tabs-panel-active .categorychecklist li input:checked').length;
+                                },
                                 /**
                                  * Adds selected menu items to the menu.
                                  *
@@ -516 +522 @@
                         });
                 },
 
+                validateCustomLink : function() {
+                        var url = $('#custom-menu-item-url').val();
+                        return ! ( '' == url || 'http://' == url );
+                },
+                
+                attachValidators : function() {
+                        var onCustomLinkChange = function() {
+                                $('#submit-customlinkdiv').prop('disabled', !api.validateCustomLink());
+                        },
+                        onChecklistChange = function() {
+                                var t = $(this).closest('.posttypediv, .taxonomydiv');
+                                t.find('.submit-add-to-menu').prop('disabled', !t.validateSelections());
+                        };
+                        
+                        $('#custom-menu-item-url').change(onCustomLinkChange);
+                        onCustomLinkChange();
+                        
+                        $('.posttypediv, .taxonomydiv').each(function() {
+                                onChecklistChange.apply(this);
+                                $(this).delegate( '.tabs-panel .categorychecklist li input', 'change', onChecklistChange );
+                        });
+                },
+
                 addCustomLink : function( processMethod ) {
                         var url = $('#custom-menu-item-url').val(),
                                 label = $('#custom-menu-item-name').val();
@@ -621 +650 @@
                                         wrapper = target.parents('.inside').first();
 
                                         // upon changing tabs, we want to uncheck all checkboxes
-                                        $('input', wrapper).removeAttr('checked');
+                                        $('input', wrapper).removeAttr('checked').change();
 
                                         $('.tabs-panel-active', wrapper).removeClass('tabs-panel-active').addClass('tabs-panel-inactive');
                                         $('#' + panelId, wrapper).removeClass('tabs-panel-inactive').addClass('tabs-panel-active');