From ffb6961fe0fd2eb8281a075a3a635624028e1f24 Mon Sep 17 00:00:00 2001
From: James Kolpack <james.kolpack@gmail.com>
Date: Thu, 17 Jan 2013 20:14:04 -0500
Subject: [PATCH] Refactoring

---
 Web/ViewModels/Account/LoginViewModel.cs |  1 +
 Web/Views/User/Details.cshtml            | 47 +++++-------------------
 Web/Views/User/Index.cshtml              |  2 +-
 Web/Views/User/_DateTimeHistoric.cshtml  | 12 ++++++
 Web/Views/User/_LastActivity.cshtml      | 12 ------
 Web/Web.config                           |  4 +-
 Web/Web.csproj                           |  2 +-
 7 files changed, 26 insertions(+), 54 deletions(-)
 create mode 100644 Web/Views/User/_DateTimeHistoric.cshtml
 delete mode 100644 Web/Views/User/_LastActivity.cshtml

diff --git a/Web/ViewModels/Account/LoginViewModel.cs b/Web/ViewModels/Account/LoginViewModel.cs
index ecf53e3..2ac450c 100644
--- a/Web/ViewModels/Account/LoginViewModel.cs
+++ b/Web/ViewModels/Account/LoginViewModel.cs
@@ -6,6 +6,7 @@ namespace MileageTraker.Web.ViewModels.Account
 	public class LoginViewModel
 	{
 		[Required]
+		[RegularExpression("[^@]*", ErrorMessage = "Enter just your username, not your email")]
 		public string Username { get; set; }
 
 		[Required]
diff --git a/Web/Views/User/Details.cshtml b/Web/Views/User/Details.cshtml
index 5b1fc2d..8a37597 100644
--- a/Web/Views/User/Details.cshtml
+++ b/Web/Views/User/Details.cshtml
@@ -1,5 +1,4 @@
-@using MileageTraker.Web.Utility
-@model MileageTraker.Web.Models.User
+@model MileageTraker.Web.Models.User
 
 @{
     ViewBag.Title = "User Details";
@@ -49,15 +48,8 @@
             @Html.DisplayNameFor(m => m.LastActivityDate)
         </dt>
         <dd>
-            @if (!Model.LastActivityDate.IsSqlMinValue())
-            {
-                @Html.Encode(Model.LastActivityDate)
-                <span class="muted">(@Html.Encode((DateTime.Now - Model.LastActivityDate).ToVerboseStringHistoric()))</span>
-            }
-            else
-            {
-                <span class='label label-info'>No Activity</span>
-            }
+            @Html.Partial("_DateTimeHistoric",
+                new Tuple<DateTime,string>(Model.LastActivityDate, "No Activity"))
         </dd>
     </dl>
     
@@ -66,15 +58,8 @@
             @Html.DisplayNameFor(m => m.LastLoginDate)
         </dt>
         <dd>
-            @if (!Model.LastLoginDate.IsSqlMinValue())
-            {
-                @Html.Encode(Model.LastLoginDate)
-                <span class="muted">(@Html.Encode((DateTime.Now - Model.LastLoginDate).ToVerboseStringHistoric()))</span>
-            }
-            else
-            {
-                <span class='label label-info'>Never Logged In</span>
-            }
+            @Html.Partial("_DateTimeHistoric",
+                new Tuple<DateTime,string>(Model.LastLoginDate, "Never Logged In"))
         </dd>
     </dl>
 
@@ -83,15 +68,8 @@
             @Html.DisplayNameFor(m => m.LastPasswordChangedDate)
         </dt>
         <dd>
-            @if (!Model.LastPasswordChangedDate.IsSqlMinValue())
-            {
-                @Html.Encode(Model.LastPasswordChangedDate)
-                <span class="muted">(@Html.Encode((DateTime.Now - Model.LastPasswordChangedDate).ToVerboseStringHistoric()))</span>
-            }
-            else
-            {
-                <span class='label label-info'>Never Changed</span>
-            }
+            @Html.Partial("_DateTimeHistoric",
+                new Tuple<DateTime,string>(Model.LastPasswordChangedDate, "Never Changed"))
         </dd>
     </dl>
     
@@ -101,15 +79,8 @@
                 Lockout Password Date
             </dt>
             <dd>
-                @if (!Model.LastLockoutDate.IsSqlMinValue())
-                {
-                    @Html.Encode(Model.LastLockoutDate)
-                    <span class="muted">(@Html.Encode((DateTime.Now - Model.LastLockoutDate).ToVerboseStringHistoric()))</span>
-                }
-                else
-                {
-                    <span class='label label-info'>Never Locked (?)</span>
-                }
+                @Html.Partial("_DateTimeHistoric",
+                    new Tuple<DateTime,string>(Model.LastLockoutDate, "Never Locked (?)"))
             </dd>
         </dl>
     }
diff --git a/Web/Views/User/Index.cshtml b/Web/Views/User/Index.cshtml
index b3d8420..33dd369 100644
--- a/Web/Views/User/Index.cshtml
+++ b/Web/Views/User/Index.cshtml
@@ -57,7 +57,7 @@
                                     </text>),
                 grid.Column("LastActivityDate", "Last Activity",
                             @<text>
-                                 @Html.Partial("_LastActivity", (DateTime)item.LastActivityDate)
+                                 @Html.Partial("_DateTimeHistoric", new Tuple<DateTime,string>((DateTime)item.LastActivityDate, "No Activity"))
                             </text>),
                 grid.Column(format:
                             @<div class='btn-group'>
diff --git a/Web/Views/User/_DateTimeHistoric.cshtml b/Web/Views/User/_DateTimeHistoric.cshtml
new file mode 100644
index 0000000..b4ba8ba
--- /dev/null
+++ b/Web/Views/User/_DateTimeHistoric.cshtml
@@ -0,0 +1,12 @@
+@using MileageTraker.Web.Utility
+@model Tuple<DateTime,string>
+
+@if (!Model.Item1.IsSqlMinValue())
+{
+    @Html.Encode((DateTime.Now - Model.Item1).ToVerboseStringHistoric())
+    <span class="muted">(@Html.Encode(Model.Item1))</span>
+}
+else
+{
+    <span class='label label-info'>@Model.Item2</span>
+}
\ No newline at end of file
diff --git a/Web/Views/User/_LastActivity.cshtml b/Web/Views/User/_LastActivity.cshtml
deleted file mode 100644
index 1024a69..0000000
--- a/Web/Views/User/_LastActivity.cshtml
+++ /dev/null
@@ -1,12 +0,0 @@
-@using MileageTraker.Web.Utility
-@model DateTime
-
-@if (!Model.IsSqlMinValue())
-{
-    @Html.Encode(Model)
-    <span class="muted">(@Html.Encode((DateTime.Now - Model).ToVerboseStringHistoric()))</span>
-}
-else
-{
-    <span class='label label-info'>No Activity</span>
-}
\ No newline at end of file
diff --git a/Web/Web.config b/Web/Web.config
index 108ebf5..329d71d 100644
--- a/Web/Web.config
+++ b/Web/Web.config
@@ -14,8 +14,8 @@
     <add key="UnobtrusiveJavaScriptEnabled" value="true" />
     <add key="enableSimpleMembership" value="false" />
     <add key="EmailFromAddress" value="Mileage Traker &lt;noreply@ethra.org&gt;" />
-    <add key="ResetPasswordSubject" value="New Password Request" />
-    <add key="ResetPasswordBody" value="{1}, please open this link to set a new password for your Mileage Traker account: {0}" />
+    <add key="ResetPasswordSubject" value="New Mileage Traker Password Request" />
+    <add key="ResetPasswordBody" value="Hello {1}, please open this link to set a new password for your Mileage Traker account: {0}" />
     <add key="InitializePasswordSubject" value="Initialize Mileage Traker Account" />
     <add key="InitializetPasswordBody" value="Hello {1}, welcome to Mileage Traker.  Your username is {2}. Please open this link to initialize your password: {0}" />
   </appSettings>
diff --git a/Web/Web.csproj b/Web/Web.csproj
index 111c092..d0ace1d 100644
--- a/Web/Web.csproj
+++ b/Web/Web.csproj
@@ -300,9 +300,9 @@
     <Content Include="Views\User\_UserStatusLabels.cshtml" />
     <Content Include="Views\Account\ResetPassword.cshtml" />
     <Content Include="Views\Account\NewPassword.cshtml" />
-    <Content Include="Views\User\_LastActivity.cshtml" />
     <Content Include="Views\Shared\EditorTemplates\CheckBoxViewModel.cshtml" />
     <Content Include="Views\User\InviteUninitialized.cshtml" />
+    <Content Include="Views\User\_DateTimeHistoric.cshtml" />
   </ItemGroup>
   <ItemGroup>
     <Content Include="packages.config">