From e8b8a8895fbfedc0c3ef650988a1b9db4ba4d1c6 Mon Sep 17 00:00:00 2001
From: Nathan Heskew <nathan.heskew@microsoft.com>
Date: Thu, 29 Apr 2010 14:03:24 -0700
Subject: [PATCH] Some work on the installed modules page.

- representing more of the module info and linking in to contained features

--HG--
branch : dev
---
 .../Orchard.Modules/Views/Admin/Features.ascx |  9 ++-----
 .../Orchard.Modules/Views/Admin/Index.ascx    | 24 ++++++++++++-------
 2 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Features.ascx b/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Features.ascx
index b604c24c6..d4f4474b0 100644
--- a/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Features.ascx
+++ b/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Features.ascx
@@ -23,7 +23,7 @@ using (Html.BeginFormAntiForgeryPost()) { %>
                 <h2><%=Html.Encode(featureGroup.First().Descriptor.Category ?? T("General")) %></h2>
                 <ul><%
                     foreach (var feature in featureGroup) {%>
-                    <li<%=feature == featureGroup.Last() ? " class=\"last\"" : "" %>>
+                    <li<%=feature == featureGroup.Last() ? " class=\"last\"" : "" %> id="<%=Html.Encode(feature.Descriptor.Name) %>">
                         <div class="summary">
                             <div class="properties">
                                 <input type="checkbox" name="selection" value="<%=Html.Encode(feature.Descriptor.Name) %>" />
@@ -40,12 +40,7 @@ using (Html.BeginFormAntiForgeryPost()) { %>
                                     </li><%
                                     //dependencies
                                     if (feature.Descriptor.Dependencies.Count() > 0) { %>
-                                    <li>&nbsp;&#124;&nbsp;<%=_Encoded("Depends on: ") %><%
-                                        foreach (var dependency in feature.Descriptor.Dependencies) {
-                                            %><% if (dependency != feature.Descriptor.Dependencies.First()) { %><%=_Encoded(", ") %><% }
-                                            %><%=Html.Encode(dependency) %><%
-                                        } %>
-                                    </li><%
+                                    <li>&nbsp;&#124;&nbsp;<%=T("Depends on: {0}", string.Join(", ", feature.Descriptor.Dependencies.Select(s => Html.Encode(s)).OrderBy(s => s).ToArray())) %></li><%
                                     } %>
                                 </ul>
                             </div>
diff --git a/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Index.ascx b/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Index.ascx
index ca0588562..c54567b93 100644
--- a/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Index.ascx
+++ b/src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Index.ascx
@@ -1,22 +1,28 @@
 <%@ Control Language="C#" Inherits="Orchard.Mvc.ViewUserControl<ModulesIndexViewModel>" %>
 <%@ Import Namespace="Orchard.Mvc.Html"%>
 <%@ Import Namespace="Orchard.Modules.ViewModels"%>
-<h1><%=Html.TitleForPage(T("Manage Modules").ToString()) %></h1>
-<div class="manage"><%=Html.ActionLink(T("Install a module").ToString(), "Features", new { }, new { @class = "button primaryAction" })%></div>
-<h2><%=T("Installed Modules") %></h2>
+<h1><%=Html.TitleForPage(T("Installed Modules").ToString()) %></h1>
+<%--<div class="manage"><%=Html.ActionLink(T("Install a module").ToString(), "Features", new { }, new { @class = "button primaryAction" })%></div>--%>
+<%--<h2><%=T("Installed Modules") %></h2>--%>
 <% if (Model.Modules.Count() > 0) { %>
-<fieldset class="pageList">
-<ul class="contentItems blogs"><%
+<fieldset class="">
+<ul class="contentItems"><%
     foreach (var module in Model.Modules.OrderBy(m => m.DisplayName)) { %>
     <li>
         <div class="summary">
             <div class="properties">
                 <h3><%=Html.Encode(module.DisplayName) %></h3>
-                <div class="related">
-                    <%=Html.ActionLink(T("Edit").ToString(), "edit", new {moduleName = module.ModuleName, area = "Orchard.Modules"}) %><%=_Encoded(" | ")%>
-                    <a href="#">Delete</a>
-                </div>
+                <ul class="pageStatus" style="color:#666">
+                    <li><%=T("Version: {0}", !string.IsNullOrEmpty(module.Version) ? Html.Encode(module.Version) : T("1.0")) %></li>
+                    <li>&nbsp;&#124;&nbsp;<%=T("Features: {0}", string.Join(", ", module.Features.Select(f => Html.Link(Html.Encode(f.Name), string.Format("{0}#{1}", Url.Action("features", new { area = "Orchard.Modules" }), Html.Encode(f.Name)) )).OrderBy(s => s).ToArray())) %></li>
+                    <li>&nbsp;&#124;&nbsp;<%=T("Author: {0}", !string.IsNullOrEmpty(module.Author) ? Html.Encode(module.Author) : (new []{"Bradley", "Bertrand", "Renaud", "Suha", "Sebastien", "Jon", "Nathan", "Erik"})[(module.DisplayName.Length + (new Random()).Next()) % 7]) %></li><%-- very efficient, I know --%>
+                    <li>&nbsp;&#124;&nbsp;<%=T("Website: {0}", !string.IsNullOrEmpty(module.HomePage) ? Html.Encode(module.HomePage) : T("<a href=\"http://orchardproject.net\">http://orchardproject.net</a>"))%></li>
+                </ul>
             </div>
+            <%--<div class="related">
+                <%=Html.ActionLink(T("Edit").ToString(), "edit", new {moduleName = module.ModuleName, area = "Orchard.Modules"}) %><%=_Encoded(" | ")%>
+                <a href="#">Delete</a>
+            </div>--%>
         </div><%
         if (!string.IsNullOrEmpty(module.Description)) { %>
         <p><%=Html.Encode(module.Description) %></p><%