Improve handling of missing hit count and last access data

Resolves #10

Author: raamdev

wp-redirects/wp-redirects.inc.php

@@ -346,8 +346,6 @@ public static function redirect_stats_meta_box($post)
 					echo '<input type="hidden" id="wp-redirect-hits" name="wp_redirect_hits" value="'.((get_post_meta($post_id, 'wp_redirect_hits', TRUE)) ? esc_attr(get_post_meta($post_id, 'wp_redirect_hits', TRUE)) : '0').'" /><br />'."\n";
 					echo '<div style="display:none;color:red;" id="wp-redirect-hit-count-reset">'.__('The hit count for this redirect has been reset. To save these changes, click Update.', 'wp-redirects').'</div>';
 
-					echo '<input type="hidden" id="wp-redirect-last-access" name="wp_redirect_last_access" value="'.((get_post_meta($post_id, 'wp_redirect_last_access', TRUE)) ? esc_attr(get_post_meta($post_id, 'wp_redirect_hits', TRUE)) : '0').'" /><br />'."\n";
-
 					wp_nonce_field('wp-redirect-meta-boxes', 'wp_redirect_meta_boxes');
 				}
 			}
@@ -422,18 +420,31 @@ public static function show_admin_column_value($column, $post_id)
 				switch($column)
 				{
 					case 'hits':
-						echo get_post_meta($post_id, 'wp_redirect_hits', TRUE);
+						$_hits = get_post_meta($post_id, 'wp_redirect_hits', TRUE);
+
+						if($_hits == '')
+							update_post_meta($post_id, 'wp_redirect_hits', '0');
+
+						echo $_hits;
+
+						unset($_hits);
 						break;
 
 					case 'last_access':
-						if(get_post_meta($post_id, 'wp_redirect_last_access', TRUE) == '0')
+						$_last_access = get_post_meta($post_id, 'wp_redirect_last_access', TRUE);
+
+						if($_last_access == '')
 						{
-							echo __('Never', 'wp-redirects');
+							update_post_meta($post_id, 'wp_redirect_last_access', '0');
+							$_last_access = '0';
 						}
-						elseif(get_post_meta($post_id, 'wp_redirect_last_access', TRUE))
-						{
+
+						if($_last_access == '0')
+							echo __('Never', 'wp-redirects');
+						else
 							echo date(get_option('date_format'), get_post_meta($post_id, 'wp_redirect_last_access', TRUE));
-						}
+
+						unset($_last_access);
 						break;
 				}
 			}