From 224870c869facc97899deee714e6a909b53b429c Mon Sep 17 00:00:00 2001
From: Nathan Heskew <nathan.heskew@microsoft.com>
Date: Wed, 24 Nov 2010 00:00:10 -0800
Subject: [PATCH 1/2] Updating the comment display(s) to use the Author
 property and changed the UserName to only be set if there is a CurrentUser in
 the WorkContext (i.e. the UserName is filled for authenticated comments)

work item: 16700

--HG--
branch : dev
---
 .../Orchard.Comments/Orchard.Comments.csproj   |  6 +++++-
 .../Services/CommentService.cs                 |  2 +-
 .../Modules/Orchard.Comments/Styles/Web.config | 18 ++++++++++++++++++
 .../Styles/orchard-comments-admin.css          | 10 ++++++++++
 .../Orchard.Comments/Views/Admin/Index.cshtml  | 13 +++++++++++--
 .../Views/ListOfComments.cshtml                |  2 +-
 6 files changed, 46 insertions(+), 5 deletions(-)
 create mode 100644 src/Orchard.Web/Modules/Orchard.Comments/Styles/Web.config

diff --git a/src/Orchard.Web/Modules/Orchard.Comments/Orchard.Comments.csproj b/src/Orchard.Web/Modules/Orchard.Comments/Orchard.Comments.csproj
index 5e580fe26..489c3cd2c 100644
--- a/src/Orchard.Web/Modules/Orchard.Comments/Orchard.Comments.csproj
+++ b/src/Orchard.Web/Modules/Orchard.Comments/Orchard.Comments.csproj
@@ -127,7 +127,11 @@
       <SubType>Designer</SubType>
     </Content>
   </ItemGroup>
-  <ItemGroup />
+  <ItemGroup>
+    <Content Include="Styles\Web.config">
+      <SubType>Designer</SubType>
+    </Content>
+  </ItemGroup>
   <Import Project="$(MSBuildBinPath)\Microsoft.CSharp.targets" />
   <Import Project="$(MSBuildExtensionsPath32)\Microsoft\VisualStudio\v10.0\WebApplications\Microsoft.WebApplication.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
diff --git a/src/Orchard.Web/Modules/Orchard.Comments/Services/CommentService.cs b/src/Orchard.Web/Modules/Orchard.Comments/Services/CommentService.cs
index cc0fa7127..9f44c5b7f 100644
--- a/src/Orchard.Web/Modules/Orchard.Comments/Services/CommentService.cs
+++ b/src/Orchard.Web/Modules/Orchard.Comments/Services/CommentService.cs
@@ -75,7 +75,7 @@ namespace Orchard.Comments.Services {
             comment.Record.CommentText = context.CommentText;
             comment.Record.Email = context.Email;
             comment.Record.SiteName = context.SiteName;
-            comment.Record.UserName = (_orchardServices.WorkContext.CurrentUser == null ? context.Author : _orchardServices.WorkContext.CurrentUser.UserName);
+            comment.Record.UserName = (_orchardServices.WorkContext.CurrentUser != null ? _orchardServices.WorkContext.CurrentUser.UserName : null);
             comment.Record.CommentedOn = context.CommentedOn;
 
             comment.Record.Status = _commentValidator.ValidateComment(comment)
diff --git a/src/Orchard.Web/Modules/Orchard.Comments/Styles/Web.config b/src/Orchard.Web/Modules/Orchard.Comments/Styles/Web.config
new file mode 100644
index 000000000..b4210f3e0
--- /dev/null
+++ b/src/Orchard.Web/Modules/Orchard.Comments/Styles/Web.config
@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<configuration>
+  <system.web>
+    <httpHandlers>
+      <!-- iis6 - for any request in this location, return via managed static file handler -->
+      <add path="*" verb="*" type="System.Web.StaticFileHandler" />
+    </httpHandlers>
+  </system.web>
+  <system.webServer>
+    <handlers accessPolicy="Script,Read">
+      <!--
+      iis7 - for any request to a file exists on disk, return it via native http module.
+      accessPolicy 'Script' is to allow for a managed 404 page.
+      -->
+      <add name="StaticFile" path="*" verb="*" modules="StaticFileModule" preCondition="integratedMode" resourceType="File" requireAccess="Read" />
+    </handlers>
+  </system.webServer>
+</configuration>
diff --git a/src/Orchard.Web/Modules/Orchard.Comments/Styles/orchard-comments-admin.css b/src/Orchard.Web/Modules/Orchard.Comments/Styles/orchard-comments-admin.css
index 2247cdf70..23bd8bd41 100644
--- a/src/Orchard.Web/Modules/Orchard.Comments/Styles/orchard-comments-admin.css
+++ b/src/Orchard.Web/Modules/Orchard.Comments/Styles/orchard-comments-admin.css
@@ -1,3 +1,13 @@
 table.items .actions {
     white-space:nowrap;   
+}
+table.items tr {
+    background:#f7f7f7;   
+}
+table.items tr.anonymous {
+    background:#fff;   
+}
+.anonymous-commenter-id,
+.authenticated-commenter-id {
+    font-style:italic;
 }
\ No newline at end of file
diff --git a/src/Orchard.Web/Modules/Orchard.Comments/Views/Admin/Index.cshtml b/src/Orchard.Web/Modules/Orchard.Comments/Views/Admin/Index.cshtml
index aecc00f7c..d7df77e17 100644
--- a/src/Orchard.Web/Modules/Orchard.Comments/Views/Admin/Index.cshtml
+++ b/src/Orchard.Web/Modules/Orchard.Comments/Views/Admin/Index.cshtml
@@ -53,7 +53,11 @@
 		    </thead>
             @{var commentIndex = 0;}
             @foreach (var commentEntry in Model.Comments) {
-            <tr itemscope="itemscope" itemid="@Model.Comments[commentIndex].Comment.Id" itemtype="http://orchardproject.net/data/Comment">
+                var commentClass = "";
+                if (!HasText(commentEntry.Comment.UserName)) {
+                    commentClass = "anonymous";
+                }
+            <tr itemscope="itemscope" itemid="@Model.Comments[commentIndex].Comment.Id" itemtype="http://orchardproject.net/data/Comment" class="@commentClass">
                 <td>
                     <input type="hidden" value="@Model.Comments[commentIndex].Comment.Id" name="@Html.NameOf(m => m.Comments[commentIndex].Comment.Id)"/>
                     <input type="checkbox" value="true" name="@Html.NameOf(m => m.Comments[commentIndex].IsChecked)"/>
@@ -63,7 +67,12 @@
                     else if (commentEntry.Comment.Status == CommentStatus.Pending) { @T("Pending") } 
                     else { @T("Approved") }
                 </td>
-                <td>@commentEntry.Comment.UserName</td>
+                <td>
+                    <div>@commentEntry.Comment.Author</div>
+                    @if (HasText(commentEntry.Comment.UserName) && commentEntry.Comment.Author != commentEntry.Comment.UserName) {
+                        <div class="authenticated-commenter-id">@commentEntry.Comment.UserName</div>
+                    }
+                </td>
                 <td>
                     @* would ideally have permalinks for individual comments *@
                     <p><a href="@Url.ItemDisplayUrl(commentEntry.CommentedOn)#comments"><time>@Html.DateTime(commentEntry.Comment.CommentDateUtc.GetValueOrDefault())</time></a></p>
diff --git a/src/Orchard.Web/Modules/Orchard.Comments/Views/ListOfComments.cshtml b/src/Orchard.Web/Modules/Orchard.Comments/Views/ListOfComments.cshtml
index d6cc3541e..24d5558df 100644
--- a/src/Orchard.Web/Modules/Orchard.Comments/Views/ListOfComments.cshtml
+++ b/src/Orchard.Web/Modules/Orchard.Comments/Views/ListOfComments.cshtml
@@ -7,7 +7,7 @@
         <article class="comment">
             <header>
                 <h4>
-                    <span class="who">@Html.LinkOrDefault(comment.Record.UserName, comment.Record.SiteName, new { rel = "nofollow" })
+                    <span class="who">@Html.LinkOrDefault(comment.Record.Author, comment.Record.SiteName, new { rel = "nofollow" })
                     </span>
                     <span class="when">said <time datetime="@comment.Record.CommentDateUtc.GetValueOrDefault()">@Html.Link((string)Display.DateTimeRelative(dateTimeUtc: comment.Record.CommentDateUtc.GetValueOrDefault()).ToString(), "#")</time>
                     </span>

From f8823e3831561ecacca5d790197d3a3e01288864 Mon Sep 17 00:00:00 2001
From: Nathan Heskew <nathan.heskew@microsoft.com>
Date: Wed, 24 Nov 2010 00:44:50 -0800
Subject: [PATCH 2/2] Updating the blogs admin to show unpublished blogs to fix
 the publish/unpublish workflow

work item: 16763

--HG--
branch : dev
---
 .../Orchard.Blogs/Controllers/BlogAdminController.cs        | 2 +-
 .../Modules/Orchard.Blogs/Services/BlogService.cs           | 6 +++++-
 .../Modules/Orchard.Blogs/Services/IBlogService.cs          | 1 +
 .../Views/Items/Content-Blog.SummaryAdmin.cshtml            | 1 -
 4 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/Orchard.Web/Modules/Orchard.Blogs/Controllers/BlogAdminController.cs b/src/Orchard.Web/Modules/Orchard.Blogs/Controllers/BlogAdminController.cs
index 88b276b5c..b62230233 100644
--- a/src/Orchard.Web/Modules/Orchard.Blogs/Controllers/BlogAdminController.cs
+++ b/src/Orchard.Web/Modules/Orchard.Blogs/Controllers/BlogAdminController.cs
@@ -135,7 +135,7 @@ namespace Orchard.Blogs.Controllers {
 
         public ActionResult List() {
             var list = Services.New.List();
-            list.AddRange(_blogService.Get()
+            list.AddRange(_blogService.Get(VersionOptions.Latest)
                               .Select(b => {
                                           var blog = Services.ContentManager.BuildDisplay(b, "SummaryAdmin");
                                           blog.TotalPostCount = _blogPostService.Get(b, VersionOptions.Latest).Count();
diff --git a/src/Orchard.Web/Modules/Orchard.Blogs/Services/BlogService.cs b/src/Orchard.Web/Modules/Orchard.Blogs/Services/BlogService.cs
index 5e127be94..c95182385 100644
--- a/src/Orchard.Web/Modules/Orchard.Blogs/Services/BlogService.cs
+++ b/src/Orchard.Web/Modules/Orchard.Blogs/Services/BlogService.cs
@@ -29,7 +29,11 @@ namespace Orchard.Blogs.Services {
         }
 
         public IEnumerable<BlogPart> Get() {
-            return _contentManager.Query<BlogPart, BlogPartRecord>()
+            return Get(VersionOptions.Published);
+        }
+
+        public IEnumerable<BlogPart> Get(VersionOptions versionOptions) {
+            return _contentManager.Query<BlogPart, BlogPartRecord>(versionOptions)
                 .Join<RoutePartRecord>()
                 .OrderBy(br => br.Title)
                 .List();
diff --git a/src/Orchard.Web/Modules/Orchard.Blogs/Services/IBlogService.cs b/src/Orchard.Web/Modules/Orchard.Blogs/Services/IBlogService.cs
index aa3242d15..21730ac3c 100644
--- a/src/Orchard.Web/Modules/Orchard.Blogs/Services/IBlogService.cs
+++ b/src/Orchard.Web/Modules/Orchard.Blogs/Services/IBlogService.cs
@@ -7,6 +7,7 @@ namespace Orchard.Blogs.Services {
         BlogPart Get(string slug);
         ContentItem Get(int id, VersionOptions versionOptions);
         IEnumerable<BlogPart> Get();
+        IEnumerable<BlogPart> Get(VersionOptions versionOptions);
         void Delete(ContentItem blog);
     }
 }
\ No newline at end of file
diff --git a/src/Orchard.Web/Modules/Orchard.Blogs/Views/Items/Content-Blog.SummaryAdmin.cshtml b/src/Orchard.Web/Modules/Orchard.Blogs/Views/Items/Content-Blog.SummaryAdmin.cshtml
index 16b4130bc..0a2d6e849 100644
--- a/src/Orchard.Web/Modules/Orchard.Blogs/Views/Items/Content-Blog.SummaryAdmin.cshtml
+++ b/src/Orchard.Web/Modules/Orchard.Blogs/Views/Items/Content-Blog.SummaryAdmin.cshtml
@@ -21,7 +21,6 @@
     </div>
     <div class="related">
         @Display(Model.Actions)
-        <a href="@Url.Blog(blog)" title="@T("View")">@T("View")</a>@T(" | ")
         <a href="@Url.BlogForAdmin(blog)" title="@T("List Posts")">@T("List Posts")</a>@T(" | ")
         <a href="@Url.BlogPostCreate(blog)" title="@T("New Post")">@T("New Post")</a>@T(" | ")
         <a href="@Url.BlogEdit(blog)" title="@T("Edit")">@T("Edit")</a>@T(" | ")