From 36fb5672128392e09ac6224b3b7b9107f4fe7079 Mon Sep 17 00:00:00 2001
From: Andre Rodrigues <andrerod@microsoft.com>
Date: Mon, 11 Oct 2010 15:51:44 -0700
Subject: [PATCH] Applied Louis comments.

--HG--
branch : dev
---
 .../Orchard.Blogs/Models/BlogPostPart.cs      |  5 -----
 .../Orchard.Roles/Models/UserSimulation.cs    |  1 -
 .../Modules/Orchard.Widgets/Migrations.cs     |  2 +-
 .../Orchard.Widgets/Models/LayerPart.cs       |  8 --------
 .../Orchard.Widgets/Models/WidgetPart.cs      |  8 --------
 .../Services/WidgetsService.cs                |  2 +-
 .../Modules/Orchard.Widgets/Styles/admin.css  | 20 +++++++++----------
 .../Orchard.Widgets/Views/Admin/Index.cshtml  | 20 +++++++++----------
 src/Orchard/ContentManagement/ContentPart.cs  |  9 +++++++++
 src/Orchard/ContentManagement/IContent.cs     |  5 +++++
 10 files changed, 36 insertions(+), 44 deletions(-)

diff --git a/src/Orchard.Web/Modules/Orchard.Blogs/Models/BlogPostPart.cs b/src/Orchard.Web/Modules/Orchard.Blogs/Models/BlogPostPart.cs
index 8e7678190..e0f2a2137 100644
--- a/src/Orchard.Web/Modules/Orchard.Blogs/Models/BlogPostPart.cs
+++ b/src/Orchard.Web/Modules/Orchard.Blogs/Models/BlogPostPart.cs
@@ -8,11 +8,6 @@ using Orchard.Security;
 
 namespace Orchard.Blogs.Models {
     public class BlogPostPart : ContentPart {
-        [HiddenInput(DisplayValue = false)]
-        public int Id {
-            get { return ContentItem.Id; }
-        }
-
         public string Title {
             get { return this.As<RoutePart>().Title; }
             set { this.As<RoutePart>().Title = value; }
diff --git a/src/Orchard.Web/Modules/Orchard.Roles/Models/UserSimulation.cs b/src/Orchard.Web/Modules/Orchard.Roles/Models/UserSimulation.cs
index 738dc4236..142f3a2df 100644
--- a/src/Orchard.Web/Modules/Orchard.Roles/Models/UserSimulation.cs
+++ b/src/Orchard.Web/Modules/Orchard.Roles/Models/UserSimulation.cs
@@ -17,7 +17,6 @@ namespace Orchard.Roles.Models {
         }
 
         class SimulatedUser : ContentPart, IUser {
-            public int Id { get { return ContentItem.Id; } }
             public string UserName { get { return null; } }
             public string Email { get { return null; } }
         }
diff --git a/src/Orchard.Web/Modules/Orchard.Widgets/Migrations.cs b/src/Orchard.Web/Modules/Orchard.Widgets/Migrations.cs
index 19ac7520b..b122b4633 100644
--- a/src/Orchard.Web/Modules/Orchard.Widgets/Migrations.cs
+++ b/src/Orchard.Web/Modules/Orchard.Widgets/Migrations.cs
@@ -70,7 +70,7 @@ namespace Orchard.Widgets {
                     .WithPart("WidgetPart")
                     .WithPart("BodyPart")
                     .WithPart("CommonPart")
-                    .WithSetting("stereotype", "widget")
+                    .WithSetting("Stereotype", "Widget")
                 );
 
             CreateDefaultLayers();
diff --git a/src/Orchard.Web/Modules/Orchard.Widgets/Models/LayerPart.cs b/src/Orchard.Web/Modules/Orchard.Widgets/Models/LayerPart.cs
index 6131fe34b..a1ceb042f 100644
--- a/src/Orchard.Web/Modules/Orchard.Widgets/Models/LayerPart.cs
+++ b/src/Orchard.Web/Modules/Orchard.Widgets/Models/LayerPart.cs
@@ -5,14 +5,6 @@ using Orchard.ContentManagement;
 namespace Orchard.Widgets.Models {
     public class LayerPart : ContentPart<LayerPartRecord> {
         
-        /// <summary>
-        /// The layer's identifier.
-        /// </summary>
-        [HiddenInput(DisplayValue = false)]
-        public int Id {
-            get { return ContentItem.Id; }
-        }
-        
         /// <summary>
         /// The layer's name.
         /// </summary>
diff --git a/src/Orchard.Web/Modules/Orchard.Widgets/Models/WidgetPart.cs b/src/Orchard.Web/Modules/Orchard.Widgets/Models/WidgetPart.cs
index 2f9a4b8e4..96f224fe6 100644
--- a/src/Orchard.Web/Modules/Orchard.Widgets/Models/WidgetPart.cs
+++ b/src/Orchard.Web/Modules/Orchard.Widgets/Models/WidgetPart.cs
@@ -7,14 +7,6 @@ using Orchard.ContentManagement.Aspects;
 namespace Orchard.Widgets.Models {
     public class WidgetPart : ContentPart<WidgetPartRecord> {
         
-        /// <summary>
-        /// The widget's identifier.
-        /// </summary>
-        [HiddenInput(DisplayValue = false)]
-        public int Id {
-            get { return ContentItem.Id; }
-        }
-
         /// <summary>
         /// The widget's title.
         /// </summary>
diff --git a/src/Orchard.Web/Modules/Orchard.Widgets/Services/WidgetsService.cs b/src/Orchard.Web/Modules/Orchard.Widgets/Services/WidgetsService.cs
index d1972b618..fd8bcb8b7 100644
--- a/src/Orchard.Web/Modules/Orchard.Widgets/Services/WidgetsService.cs
+++ b/src/Orchard.Web/Modules/Orchard.Widgets/Services/WidgetsService.cs
@@ -23,7 +23,7 @@ namespace Orchard.Widgets.Services {
 
         public IEnumerable<string> GetWidgetTypes() {
             return _contentManager.GetContentTypeDefinitions()
-                .Where(contentTypeDefinition => contentTypeDefinition.Settings.ContainsKey("stereotype") && contentTypeDefinition.Settings["stereotype"] == "widget")
+                .Where(contentTypeDefinition => contentTypeDefinition.Settings.ContainsKey("Stereotype") && contentTypeDefinition.Settings["Stereotype"] == "Widget")
                 .Select(contentTypeDefinition => contentTypeDefinition.Name);
         }
 
diff --git a/src/Orchard.Web/Modules/Orchard.Widgets/Styles/admin.css b/src/Orchard.Web/Modules/Orchard.Widgets/Styles/admin.css
index b373f4783..622a2354e 100644
--- a/src/Orchard.Web/Modules/Orchard.Widgets/Styles/admin.css
+++ b/src/Orchard.Web/Modules/Orchard.Widgets/Styles/admin.css
@@ -3,23 +3,23 @@
     font-weight:bold;
 }
 
-.availableWidgets {
+.widgets-availableWidgets {
     float: left;
     width: 35%;
 }
 
-.availableLayers {
+.widgets-availableLayers {
     float: left;
     margin: 0 0 0 2em;
     width: 45%;
 }
 
-.layerZones {
+.widgets-layerZones {
     float: left;
     width: 75%;
 }
 
-.layerZones .zone {
+.widgets-layerZones .widgets-zone {
     font-weight:700;
 	background:#f5f5f5;
 	border: 1px solid #f1f1f1;
@@ -27,7 +27,7 @@
 	margin: 0 0 1em 0;
 }
 
-.layerZones .zoneWidget {
+.widgets-layerZones .widgets-zoneWidget {
     margin: 0 0 1em 0;
     padding: 1em 2em 1em 3em;
 	background:#ffffff;
@@ -35,26 +35,26 @@
 	vertical-align: middle;
 }
 
-.layers {
+.widgets-layers {
     margin: 1em 0 0 0;
     float: left;
 }
 
-.layers ul {
+.widgets-layers ul {
 	background: #f5f5f5;
 	border: 1px solid #f1f1f1;
 }
 
-.layers .currentLayer {
+.widgets-layers .widgets-currentLayer {
 	background: #c3d9ff;
 }
 
-.layers .editLayer img {
+.widgets-layers .widgets-editLayer img {
     float: right;
     margin: 0 0 0 1em;
 }
 
-.layerZones ul li ul {
+.widgets-layerZones ul li ul {
 	margin: 0;
 	padding: 0;
 }
\ No newline at end of file
diff --git a/src/Orchard.Web/Modules/Orchard.Widgets/Views/Admin/Index.cshtml b/src/Orchard.Web/Modules/Orchard.Widgets/Views/Admin/Index.cshtml
index 7678e0e42..3f080ccea 100644
--- a/src/Orchard.Web/Modules/Orchard.Widgets/Views/Admin/Index.cshtml
+++ b/src/Orchard.Web/Modules/Orchard.Widgets/Views/Admin/Index.cshtml
@@ -10,9 +10,9 @@
 
 	<div class="manage">@Html.ActionLink(T("Add a layer").ToString(), "AddLayer", new { }, new { @class = "button primaryAction" })</div>
 
-	<div id="widgets" class="widgets">
+	<div id="widgets">
 		
-		<div class="availableWidgets">
+		<div class="widgets-availableWidgets">
 			<h2>Available Widgets</h2>
 
 			<fieldset>
@@ -38,18 +38,18 @@
 			</fieldset>
 		</div>
 
-		<div class="availableLayers">
+		<div class="widgets-availableLayers">
 				<h2>Widget Layers</h2>
 
 			<fieldset>
-				<div class="layerZonesWidgets">
-					<ul class="layerZones">
+				<div>
+					<ul class="widgets-layerZones">
 						@foreach (string zone in Model.Zones) {
 							<li>
-								<div class="zone">@zone</div>
+								<div class="widgets-zone">@zone</div>
 								<ul>
 									@foreach (WidgetPart widget in Model.CurrentLayerWidgets.Where(widgetPart => widgetPart.Zone == zone).OrderBy(widgetPart => widgetPart.Position)) {
-										<li class="zoneWidget">
+										<li class="widgets-zoneWidget">
 											@if (widget.Position != "1") {
 												<input type="image" name="submit.MoveUp.@widget.Id" src="@Url.Content("~/modules/orchard.widgets/Content/Admin/images/moveup.gif")" alt="Move up" value="@widget.Id" />
 											}
@@ -65,18 +65,18 @@
 					</ul>
 				</div>
 
-				<div class="layers">
+				<div class="widgets-layers">
 					<ul>
 						@foreach (var layer in Model.Layers) {
 							if (layer.Id == Model.CurrentLayer.Id) {
-								<li class="currentLayer editLayer">
+								<li class="widgets-currentLayer widgets-editLayer">
 									@Html.ActionLink(@layer.Name, "Index", new { @layer.Id })
 									<a href="@Url.Action("EditLayer", new { @layer.Id })">
 										<img width="15" height="15" src="@Url.Content("~/modules/orchard.widgets/Content/Admin/images/edit.gif")" />
 									</a>
 								</li>
 							} else {
-								<li class="editLayer">
+								<li class="widgets-editLayer">
 									@Html.ActionLink(@layer.Name, "Index", new { @layer.Id })
 									<a href="@Url.Action("EditLayer", new { @layer.Id })">
 										<img width="15" height="15" src="@Url.Content("~/modules/orchard.widgets/Content/Admin/images/edit.gif")" />
diff --git a/src/Orchard/ContentManagement/ContentPart.cs b/src/Orchard/ContentManagement/ContentPart.cs
index 5e071bfeb..1d7133754 100644
--- a/src/Orchard/ContentManagement/ContentPart.cs
+++ b/src/Orchard/ContentManagement/ContentPart.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Web.Mvc;
 using Orchard.ContentManagement.MetaData.Models;
 using Orchard.ContentManagement.Utilities;
 using Orchard.UI;
@@ -23,6 +24,14 @@ namespace Orchard.ContentManagement {
             }
         }
 
+        /// <summary>
+        /// The ContentItem's identifier.
+        /// </summary>
+        [HiddenInput(DisplayValue = false)]
+        public int Id {
+            get { return ContentItem.Id; }
+        }
+
         public ContentTypeDefinition TypeDefinition { get { return ContentItem.TypeDefinition; } }
         public ContentTypePartDefinition TypePartDefinition { get; set; }
         public ContentPartDefinition PartDefinition { get { return TypePartDefinition.PartDefinition; } }
diff --git a/src/Orchard/ContentManagement/IContent.cs b/src/Orchard/ContentManagement/IContent.cs
index 729335a95..8c0d3a812 100644
--- a/src/Orchard/ContentManagement/IContent.cs
+++ b/src/Orchard/ContentManagement/IContent.cs
@@ -3,5 +3,10 @@
 namespace Orchard.ContentManagement {
     public interface IContent {
         ContentItem ContentItem { get; }
+
+        /// <summary>
+        /// The ContentItem's identifier.
+        /// </summary>
+        int Id { get; }
     }
 }