Changeset 5358

Timestamp:

04/30/2007 08:02:02 PM (19 years ago)

Author:

ryan

Message:

Widget registration fixes from Andy. For 2.3. see #4169

File:

• trunk/wp-includes/widgets.php (modified) (5 diffs)

trunk/wp-includes/widgets.php

@@ -44 +44 @@
 
         $defaults = array(
-                'name' => sprintf(__('Sidebar %d'), count($wp_registered_sidebars) + 1 ),
+                'name' => sprintf(__('Sidebar %d'), $i ),
+                'id' => "sidebar-$i",
                 'before_widget' => '<li id="%1$s" class="widget %2$s">',
                 'after_widget' => "</li>\n",
@@ -53 +54 @@
         $sidebar = array_merge($defaults, $args);
 
-        if ( ! isset($sidebar['id']) ) $sidebar['id'] = sanitize_title($sidebar['name']);
         $wp_registered_sidebars[$sidebar['id']] = $sidebar;
 
@@ -221 +221 @@
 
 function wp_get_sidebars_widgets($update = true) {
-        global $wp_registered_widgets;
+        global $wp_registered_widgets, $wp_registered_sidebars;
 
         $sidebars_widgets = get_option('sidebars_widgets');
@@ -247 +247 @@
                         }
                         $_sidebars_widgets['array_version'] = 2;
+                        $sidebars_widgets = $_sidebars_widgets;
+                        unset($_sidebars_widgets);
+
+                case 2 :
+                        $sidebars = array_keys( $wp_registered_sidebars );
+                        if ( !empty( $sidebars ) ) {
+                                // Move the known-good ones first
+                                foreach ( $sidebars as $id ) {
+                                        if ( array_key_exists( $id, $sidebars_widgets ) ) {
+                                                $_sidebars_widgets[$id] = $sidebars_widgets[$id];
+                                                unset($sidebars_widgets[$id], $sidebars[$id]);
+                                        }
+                                }
+
+                                // Assign to each unmatched registered sidebar the first available orphan
+                                unset( $sidebars_widgets[ 'array_version' ] );
+                                while ( ( $sidebar = array_shift( $sidebars ) ) && $widgets = array_shift( $sidebars_widgets ) )
+                                        $_sidebars_widgets[ $sidebar ] = $widgets;
+
+                                $_sidebars_widgets['array_version'] = 3;
+                                $sidebars_widgets = $_sidebars_widgets;
+                                unset($_sidebars_widgets);
+                        }
+
                         if ( $update )
-                                update_option('sidebars_widgets', $_sidebars_widgets);
-                        break;
-                case 2 :
-                        $_sidebars_widgets = $sidebars_widgets;
-                        break;
-        }
-
-        unset($_sidebars_widgets['array_version']);
-
-        return $_sidebars_widgets;
+                                update_option('sidebars_widgets', $sidebars_widgets);
+        }
+
+        unset($sidebars_widgets['array_version']);
+
+        return $sidebars_widgets;
 }
 
@@ -544 +564 @@
     var dropdown = document.getElementById("cat");
     function onCatChange() {
-        location.href = "?cat="+dropdown.options[dropdown.selectedIndex].value;
+        location.href = "<?php echo get_option('siteurl'); ?>/?cat="+dropdown.options[dropdown.selectedIndex].value;
     }
     dropdown.onchange = onCatChange;