From 484534acd09618ca81e7600cbf8017fa8df90d99 Mon Sep 17 00:00:00 2001
From: Ben Word <ben@benword.com>
Date: Sun, 23 Oct 2011 17:43:03 -0600
Subject: [PATCH] fixing undefined index error on widget and removing
 whitespace

---
 inc/roots-widgets.php | 45 ++++++++++++++++++++-----------------------
 1 file changed, 21 insertions(+), 24 deletions(-)

diff --git a/inc/roots-widgets.php b/inc/roots-widgets.php
index 1e511d9..84a5e08 100644
--- a/inc/roots-widgets.php
+++ b/inc/roots-widgets.php
@@ -4,19 +4,17 @@ class roots_vcard extends WP_Widget {
 
 	function roots_vcard() {
 		$widget_ops = array('description' => 'Display a vCard');
-		parent::WP_Widget(false, __('Roots: vCard', 'roots'), $widget_ops);      
+		parent::WP_Widget(false, __('Roots: vCard', 'roots'), $widget_ops);
 	}
-   
-	function widget($args, $instance) {  
-		extract($args);
-    extract($instance);
 
+	function widget($args, $instance) {
+		extract($args);
+		extract($instance);
 		echo $before_widget;
 		if ($title) {
-      echo $before_title, $title, $after_title;
-    }
+			echo $before_title, $title, $after_title;
+		}
   ?>
-
 		<p class="vcard">
 			<a class="fn org url" href="<?php echo home_url('/'); ?>"><?php bloginfo('name'); ?></a><br>
 			<span class="adr">
@@ -27,13 +25,12 @@ class roots_vcard extends WP_Widget {
 			</span>
 			<span class="tel"><span class="value"><span class="hidden">+1-</span><?php echo $tel; ?></span></span><br>
 			<a class="email" href="mailto:<?php echo $email; ?>"><?php echo $email; ?></a>
-		</p>        
-        
-    <?php echo $after_widget;
-        
+		</p>
+    <?php
+    	echo $after_widget;
 	}
-	
-	function update($new_instance, $old_instance) {                
+
+	function update($new_instance, $old_instance) {
 		return $new_instance;
 	}
 
@@ -41,35 +38,35 @@ class roots_vcard extends WP_Widget {
 	?>
 		<p>
 			<label for="<?php echo $this->get_field_id('title'); ?>"><?php _e('Title (optional):', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('title'); ?>" value="<?php echo esc_attr($instance['title']); ?>" class="widefat" id="<?php echo $this->get_field_id('title'); ?>" />
-		</p>       
+			<input type="text" name="<?php echo $this->get_field_name('title'); ?>" value="<?php if (isset($instance['title'])) { echo esc_attr($instance['title']); } ?>" class="widefat" id="<?php if (isset($instance['title'])) { echo $this->get_field_id('title'); } ?>" />
+		</p>
 		<p>
 			<label for="<?php echo $this->get_field_id('street_address'); ?>"><?php _e('Street Address:', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('street_address'); ?>" value="<?php echo esc_attr($instance['street_address']); ?>" class="widefat" id="<?php echo $this->get_field_id('street_address'); ?>" />
+			<input type="text" name="<?php echo $this->get_field_name('street_address'); ?>" value="<?php if (isset($instance['street_address'])) { echo esc_attr($instance['street_address']); } ?>" class="widefat" id="<?php if (isset($instance['street_address'])) { echo $this->get_field_id('street_address'); } ?>" />
 		</p>
 		<p>
 			<label for="<?php echo $this->get_field_id('locality'); ?>"><?php _e('City/Locality:', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('locality'); ?>" value="<?php echo esc_attr($instance['locality']); ?>" class="widefat" id="<?php echo $this->get_field_id('locality'); ?>" />
+			<input type="text" name="<?php echo $this->get_field_name('locality'); ?>" value="<?php if (isset($instance['locality'])) { echo esc_attr($instance['locality']); } ?>" class="widefat" id="<?php if (isset($instance['locality'])) { echo $this->get_field_id('locality'); } ?>" />
 		</p>
 		<p>
 			<label for="<?php echo $this->get_field_id('region'); ?>"><?php _e('State/Region:', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('region'); ?>" value="<?php echo esc_attr($instance['region']); ?>" class="widefat" id="<?php echo $this->get_field_id('region'); ?>" />
+			<input type="text" name="<?php echo $this->get_field_name('region'); ?>" value="<?php if (isset($instance['region'])) { echo esc_attr($instance['region']); } ?>" class="widefat" id="<?php if (isset($instance['region'])) { echo $this->get_field_id('region'); } ?>" />
 		</p>
 		<p>
 			<label for="<?php echo $this->get_field_id('postal_code'); ?>"><?php _e('Zipcode/Postal Code:', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('postal_code'); ?>" value="<?php echo esc_attr($instance['postal_code']); ?>" class="widefat" id="<?php echo $this->get_field_id('postal_code'); ?>" />
+			<input type="text" name="<?php echo $this->get_field_name('postal_code'); ?>" value="<?php if (isset($instance['postal_code'])) { echo esc_attr($instance['postal_code']); } ?>" class="widefat" id="<?php if (isset($instance['postal_code'])) { echo $this->get_field_id('postal_code'); } ?>" />
 		</p>
 		<p>
 			<label for="<?php echo $this->get_field_id('tel'); ?>"><?php _e('Telephone:', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('tel'); ?>" value="<?php echo esc_attr($instance['tel']); ?>" class="widefat" id="<?php echo $this->get_field_id('tel'); ?>" />
+			<input type="text" name="<?php echo $this->get_field_name('tel'); ?>" value="<?php if (isset($instance['tel'])) { echo esc_attr($instance['tel']); } ?>" class="widefat" id="<?php if (isset($instance['tel'])) { echo $this->get_field_id('tel'); } ?>" />
 		</p>
 		<p>
 			<label for="<?php echo $this->get_field_id('email'); ?>"><?php _e('Email:', 'roots'); ?></label>
-			<input type="text" name="<?php echo $this->get_field_name('email'); ?>" value="<?php echo esc_attr($instance['email']); ?>" class="widefat" id="<?php echo $this->get_field_id('email'); ?>" />
-		</p>                                   
+			<input type="text" name="<?php echo $this->get_field_name('email'); ?>" value="<?php if (isset($instance['email'])) { echo esc_attr($instance['email']); } ?>" class="widefat" id="<?php if (isset($instance['email'])) { echo $this->get_field_id('email'); } ?>" />
+		</p>
 	<?php
 	}
-} 
+}
 
 register_widget('roots_vcard');