From b124fad4175bb28465d2c4090bf3eb052071b124 Mon Sep 17 00:00:00 2001
From: Marcus Wendt <marcus@wendt.dk>
Date: Tue, 24 Apr 2012 15:49:06 +0200
Subject: [PATCH] Fixing #3926 - crach boom if use do "Add Page" and
 positioning = select position and Next.

---
 .../PageElementProvider/AddNewPageWorkflow.cs       | 13 ++++++++-----
 .../forms/Administrative/AddNewPageStep2.xml        |  4 ++--
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/Composite.Workflows/Plugins/Elements/ElementProviders/PageElementProvider/AddNewPageWorkflow.cs b/Composite.Workflows/Plugins/Elements/ElementProviders/PageElementProvider/AddNewPageWorkflow.cs
index 60a9add2d4..9ee726897f 100644
--- a/Composite.Workflows/Plugins/Elements/ElementProviders/PageElementProvider/AddNewPageWorkflow.cs
+++ b/Composite.Workflows/Plugins/Elements/ElementProviders/PageElementProvider/AddNewPageWorkflow.cs
@@ -436,9 +436,12 @@ private void PrepareStep2_ExecuteCode(object sender, EventArgs e)
 
             if (this.GetBinding<string>("SelectedSortOrder") == "Relative")
             {
-                var existingPages =
-                    (from page in PageServices.GetChildren(GetParentId())
-                     select new { Title = page.Title, Id = page.Id }).ToList();
+                var existingPages = new Dictionary<Guid, string>();
+
+                foreach (IPage page in PageServices.GetChildren(GetParentId()))
+	            {
+		            existingPages.Add(page.Id, page.Title);
+            	}
 
                 if (this.BindingExist("ExistingPages") == false)
                 {
@@ -451,11 +454,11 @@ private void PrepareStep2_ExecuteCode(object sender, EventArgs e)
 
                 if (this.BindingExist("RelativeSelectedPageId") == false)
                 {
-                    this.Bindings.Add("RelativeSelectedPageId", existingPages[0].Id);
+                    this.Bindings.Add("RelativeSelectedPageId", existingPages.First().Key);
                 }
                 else
                 {
-                    this.Bindings["RelativeSelectedPageId"] = existingPages[0].Id;
+                    this.Bindings["RelativeSelectedPageId"] = existingPages.First().Key;
                 }
             }
             else
diff --git a/Website/Composite/content/forms/Administrative/AddNewPageStep2.xml b/Website/Composite/content/forms/Administrative/AddNewPageStep2.xml
index 26cb4d2884..06fb1442e3 100644
--- a/Website/Composite/content/forms/Administrative/AddNewPageStep2.xml
+++ b/Website/Composite/content/forms/Administrative/AddNewPageStep2.xml
@@ -2,7 +2,7 @@
 <cms:formdefinition xmlns="http://www.composite.net/ns/management/bindingforms/std.ui.controls.lib/1.0" xmlns:f="http://www.composite.net/ns/management/bindingforms/std.function.lib/1.0" xmlns:cms="http://www.composite.net/ns/management/bindingforms/1.0">
   <cms:bindings>
     <cms:binding name="NewPage" type="Composite.Data.Types.IPage" />
-    <cms:binding name="ExistingPages" type="System.Collections.IList" optional="true" />
+    <cms:binding name="ExistingPages" type="System.Collections.IEnumerable" optional="true" />
     <cms:binding name="RelativeSelectedPageId" type="System.Guid" optional="true" />
     <cms:binding name="UrlTitleIsRequired" type="System.Boolean"/>
   </cms:bindings>
@@ -29,7 +29,7 @@
         </f:NullCheck.CheckValue>
         <f:NullCheck.WhenNotNull>
           <FieldGroup Label="${Composite.Plugins.PageElementProvider, AddNewPageStep2.LabelPositionSelectorPanel}">
-            <KeySelector Label="${Composite.Plugins.PageElementProvider, AddNewPageStep2.LabelPositionSelector}" OptionsKeyField="Id" OptionsLabelField="Title" Help="${Composite.Plugins.PageElementProvider, AddNewPageStep2.HelpPositionSelector}">
+            <KeySelector Label="${Composite.Plugins.PageElementProvider, AddNewPageStep2.LabelPositionSelector}" OptionsKeyField="Key" OptionsLabelField="Value" Help="${Composite.Plugins.PageElementProvider, AddNewPageStep2.HelpPositionSelector}">
               <KeySelector.Options>
                 <cms:read source="ExistingPages" />
               </KeySelector.Options>