From 3e8a03a151b43e4a58bbcc1ea0f295ca5d9ff9c1 Mon Sep 17 00:00:00 2001
From: Peter Forstmeier <peter.forstmeier@t-online.de>
Date: Mon, 21 Apr 2014 12:32:36 +0200
Subject: [PATCH] Cleanup

---
 .../Src/Wpf/Visitor/WpfVisitor.cs             | 51 +++++++++----------
 1 file changed, 25 insertions(+), 26 deletions(-)

diff --git a/src/AddIns/Misc/Reporting/ICSharpCode.Reporting/Src/Wpf/Visitor/WpfVisitor.cs b/src/AddIns/Misc/Reporting/ICSharpCode.Reporting/Src/Wpf/Visitor/WpfVisitor.cs
index 167d923a61..9d4428e1bd 100644
--- a/src/AddIns/Misc/Reporting/ICSharpCode.Reporting/Src/Wpf/Visitor/WpfVisitor.cs
+++ b/src/AddIns/Misc/Reporting/ICSharpCode.Reporting/Src/Wpf/Visitor/WpfVisitor.cs
@@ -17,6 +17,7 @@
 // DEALINGS IN THE SOFTWARE.
 
 using System;
+using System.Collections.Generic;
 using System.Windows;
 using System.Windows.Controls;
 using System.Windows.Documents;
@@ -117,7 +118,6 @@ namespace ICSharpCode.Reporting.WpfReportViewer.Visitor
 			}
 			var dragingElement = new DrawingElement(visual);
 			UIElement = dragingElement;
-			
 		}
 
 		
@@ -140,34 +140,32 @@ namespace ICSharpCode.Reporting.WpfReportViewer.Visitor
 			var border = CreateBorder(exportRectangle);
 			border.CornerRadius = new CornerRadius(Convert.ToDouble(exportRectangle.CornerRadius));
 			CanvasHelper.SetPosition(border, new Point(0,0));
-			var panel = new StackPanel();
-			panel.Orientation = Orientation.Horizontal;
-			foreach (var element in exportRectangle.ExportedItems) {
-				var acceptor = element as IAcceptor;
-				acceptor.Accept(this);
-				panel.Children.Add(UIElement);
-			}
-			border.Child = panel;
+			var containerCanvas = CreateItemsInContainer(exportRectangle.ExportedItems);
+			border.Child = containerCanvas;
 			UIElement = border;
 		}
 		
 		
 		public override void Visit(ExportCircle exportCircle)
 		{
-			var containerCanvas = new Canvas();
-		
 			var drawingElement = CircleVisual(exportCircle);
-			
-			containerCanvas.Children.Add(drawingElement);
-			foreach (var element in exportCircle.ExportedItems) {
-					var acceptor = element as IAcceptor;
-					acceptor.Accept(this);
-					containerCanvas.Children.Add(UIElement);
-				}
+			var containerCanvas =  CreateItemsInContainer(exportCircle.ExportedItems);
+			containerCanvas.Children.Insert(0,drawingElement);
 			UIElement = containerCanvas;
 		}
 		
 		
+		Canvas CreateItemsInContainer (List<IExportColumn> items) {
+			var canvas = new Canvas();
+			foreach (var element in items) {
+				var acceptor = element as IAcceptor;
+				acceptor.Accept(this);
+				canvas.Children.Add(UIElement);
+			}
+			return canvas;
+		}
+		
+		
 		bool IsGraphicsContainer (IExportColumn column) {
 			return column is GraphicsContainer;
 		}
@@ -196,21 +194,22 @@ namespace ICSharpCode.Reporting.WpfReportViewer.Visitor
 		
 		Border CreateBorder(IExportColumn exportColumn)
 		{
-			double bT;
-			var gc = IsGraphicsContainer(exportColumn);
-			if (!IsGraphicsContainer(exportColumn)) {
-				bT = 1;
-			} else {
-				bT = Convert.ToDouble(((GraphicsContainer)exportColumn).Thickness);
-			}
 			var border = new Border();
-			border.BorderThickness = new Thickness(bT);
+			border.BorderThickness =  Thickness(exportColumn);
 			border.BorderBrush = FixedDocumentCreator.ConvertBrush(exportColumn.ForeColor);
 			border.Background = FixedDocumentCreator.ConvertBrush(exportColumn.BackColor);
 			border.Width = exportColumn.Size.Width;
 			border.Height = exportColumn.Size.Height;
 			return border;
 		}
+
+		
+		Thickness Thickness(IExportColumn exportColumn)
+		{
+			double bT;
+			bT = !IsGraphicsContainer(exportColumn) ? 1 : Convert.ToDouble(((GraphicsContainer)exportColumn).Thickness);
+			return new Thickness(bT);
+		}
 		
 		
 		static Point CalcRadius(System.Drawing.Size size) {