From fbf3a7824c0d1d4b0a0eaf33b19073869d823c7d Mon Sep 17 00:00:00 2001
From: John Bintz <john@coswellproductions.com>
Date: Wed, 1 Jul 2009 21:21:28 -0400
Subject: [PATCH] allow for default ad to be selected on widget

---
 classes/PluginWonderful.php  |  1 +
 test/PluginWonderfulTest.php | 32 +++++++++++++++++---------------
 2 files changed, 18 insertions(+), 15 deletions(-)

diff --git a/classes/PluginWonderful.php b/classes/PluginWonderful.php
index 5c63a8d..3ec8354 100644
--- a/classes/PluginWonderful.php
+++ b/classes/PluginWonderful.php
@@ -365,6 +365,7 @@ class PluginWonderful {
       echo '<p>';
         echo 'Select an adbox:<br />';
         foreach ($this->publisher_info->adboxes as $box) {
+          if (empty($instance['adboxid'])) { $instance['adboxid'] = $box->adboxid; }
           echo '<label>';
             echo '<input type="radio" name="'
                  . $field_names['adboxid']
diff --git a/test/PluginWonderfulTest.php b/test/PluginWonderfulTest.php
index fcc2eb2..c919637 100644
--- a/test/PluginWonderfulTest.php
+++ b/test/PluginWonderfulTest.php
@@ -467,21 +467,23 @@ class PluginWonderfulTest extends PHPUnit_Framework_TestCase {
       (object)array('adboxid' => '234'),
       (object)array('adboxid' => '345'),
     );
-    
-    ob_start();
-    $this->pw->_render_adbox_admin(array('adboxid' => '123', 'center' => 1), array('adboxid' => 'adname', 'center' => 'centername'));
-    $source = ob_get_clean();
-    
-    $this->assertTrue(($xml = _to_xml($source)) !== false);
-    
-    foreach (array(
-      '//input[@type="radio" and @name="adname" and @value="123" and @checked="checked"]' => true,
-      '//input[@type="radio" and @name="adname" and @value="234" and not(@checked="checked")]' => true,
-      '//input[@type="radio" and @name="adname" and @value="345" and not(@checked="checked")]' => true,
-      '//input[@type="checkbox" and @name="centername" and @value="1" and @checked="checked"]' => true
-    ) as $xpath => $value) {
-      $this->assertTrue(_xpath_test($xml, $xpath, $value), $xpath);
-    }  
+
+    foreach (array('', '123') as $sample_adbox_id) {
+      ob_start();
+      $this->pw->_render_adbox_admin(array('adboxid' => $sample_adbox_id, 'center' => 1), array('adboxid' => 'adname', 'center' => 'centername'));
+      $source = ob_get_clean();
+      
+      $this->assertTrue(($xml = _to_xml($source)) !== false);
+      
+      foreach (array(
+        '//input[@type="radio" and @name="adname" and @value="123" and @checked="checked"]' => true,
+        '//input[@type="radio" and @name="adname" and @value="234" and not(@checked="checked")]' => true,
+        '//input[@type="radio" and @name="adname" and @value="345" and not(@checked="checked")]' => true,
+        '//input[@type="checkbox" and @name="centername" and @value="1" and @checked="checked"]' => true
+      ) as $xpath => $value) {
+        $this->assertTrue(_xpath_test($xml, $xpath, $value), $xpath);
+      }
+    }
   }
   
   function providerTestRenderPre28Widget() {