Show error details to the user

Implemented the feature
#2

Author: cvrajeesh

Web/Content/Site.css

@@ -126,7 +126,7 @@ body {
     display: none;
 }
 
-#findresult
+.otherwindow
 {
     border: 1px solid #ddd;
     border-bottom: none;
@@ -135,19 +135,24 @@ body {
     font-size: 0.9em;    
 }
 
-.findresultheader
+.otherwindowheader
 {
     padding: 7px 8px;
     background-color: #ddd;    
     background-repeat: repeat-x;
     font-weight: bold;	
 }
 
-.findresultpane
+.otherwindowpane
 {
     height: 150px;  
 }
 
+.otherwindowpane .message {
+    padding:7px 10px;
+}
+
+
 a.versionInfo,  a.versionInfo:hover, a.versionInfo:active{
     color: #fff;
 }

Web/Models/ProjectClientStatus.cs

@@ -15,6 +15,7 @@ public enum ProjectStatus
 
     public class ProjectClientStatus
     {
+        public string DetailedMessage { get; set; }
 
         public string Message { get; set; }
 

Web/Scripts/App/Application.js

@@ -163,6 +163,32 @@ function projectInfoViewModel() {
 
 };
 
+/// Output window view model
+function outPutWindowViewModel(title, content) {
+    var self = this;
+
+    self.show = ko.observable(true);
+    self.title = title;
+    self.content = content;
+
+    self.close = function () {
+        self.show(false);
+    }
+}
+
+/// Find result window view model
+function findResultWindowViewModel(title, items) {
+    var self = this;
+
+    self.show = ko.observable(true);
+    self.title = title;
+    self.items = items;
+
+    self.close = function () {
+        self.show(false);
+    }
+}
+
 /// Application level view model class
 function appViewModel() {
     var self = this;
@@ -173,17 +199,19 @@ function appViewModel() {
     self.projectIsActive = ko.observable();
     self.projectStatus = ko.observable();
     self.findResult = ko.observable();
-    self.showFindResultWindow = ko.observable(false);
-    self.findResultWindowTitle = ko.observable();
+    self.output = ko.observable();
 
     self.projectHub = $.connection.projectHub;
 
     self.projectHub.projectStatus = function (data) {
-        if (data) {
-            // If completed
-            if (data.Status == 2) {
+        if (data) {            
+            if (data.Status == 2) { // If completed
                 self.projectStatus('');
-            } else {
+            } else if (data.Status == 3) { // Error     
+                self.projectStatus(data.Message);
+                self.output(new outPutWindowViewModel(data.Message, data.DetailedMessage));
+            }
+            else {
                 self.projectStatus(data.Message);
             }
         }
@@ -195,15 +223,11 @@ function appViewModel() {
         }
     };
 
-    self.closeFindReferenceWindow = function () {
-        self.showFindResultWindow(false);
-    };
-
     self.findReferences = function (kind, text, position) {
         var project = self.project();
         var currentFilePath = project.file().path;
-        var findReferencesUrl = '/api/solution/findreferences';
-        self.findResultWindowTitle('Find result for "' + text + '"');
+        var findReferencesUrl = '/api/solution/findreferences';        
+        
 
         $.post(findReferencesUrl,
             {
@@ -215,8 +239,7 @@ function appViewModel() {
                 kind: kind
             },
             function (result) {
-                self.findResult({ items: result });
-                self.showFindResultWindow(true);
+                self.findResult(new findResultWindowViewModel('Find result for "' + text + '"', result));
             }
         );
     };

Web/Services/DefaultSourceCodeOpeningProgressListener.cs

@@ -50,9 +50,9 @@ public void OnProjectLoaded()
         }
 
 
-        public void OnProjectLoadingError()
+        public void OnProjectLoadingError(string message)
         {
-            this.Caller.projectStatus(new ProjectClientStatus { Status = ProjectStatus.Error, Message = "An error has occured while loading the project." });
+            this.Caller.projectStatus(new ProjectClientStatus { Status = ProjectStatus.Error, Message = "An error has occured while loading the project.", DetailedMessage = message });
         }
 
 

Web/Services/GitHub/GitHubSourceCodeProviderService.cs

@@ -105,12 +105,12 @@ public ProjectItem GetContent(string username, string project, string path, ISou
                     }
                 });
 
-                openingProgressListener.OnProjectLoadingError();
+                openingProgressListener.OnProjectLoadingError(ex.Message + Environment.NewLine + ex.StackTrace);
                 this.logger.Error(ex, "Path too long exception {0} from project {1}/{2} ", path, username, project);
             }
             catch (Exception ex)
             {
-                openingProgressListener.OnProjectLoadingError();
+                openingProgressListener.OnProjectLoadingError(ex.Message + Environment.NewLine + ex.StackTrace);
                 this.logger.Error(ex, "An error has occured while getting the content for path {0} from project {1}/{2} ", path, username, project);
             }            
 

Web/Services/ISourceCodeOpeningProgress.cs

@@ -25,7 +25,7 @@ public interface ISourceCodeOpeningProgress
 
         void OnProjectPreparing();
 
-        void OnProjectLoadingError();
+        void OnProjectLoadingError(string message);
 
         void OnBuildingWorkspace();
     }

Web/Views/Home/Index.cshtml

@@ -81,10 +81,11 @@
                 </table>
             </div>
         </div>
-        <div data-bind="with: $root.findResult, visible: $root.showFindResultWindow" id="findresult" class="dockrow">
-            <button class="close" data-bind="click: $root.closeFindReferenceWindow">&times;</button>
-            <div class="findresultheader"><span data-bind="text: $root.findResultWindowTitle"></span></div>
-            <div class="findresultpane container stylishscrollbar scroll-x scroll-y">
+         <!-- ko with: $root.findResult -->
+        <div data-bind="visible: show" class="otherwindow dockrow">
+            <button class="close" data-bind="click: close">&times;</button>
+            <div class="otherwindowheader"><span data-bind="text: title"></span></div>
+            <div class="otherwindowpane container stylishscrollbar scroll-x scroll-y">
                 <table class="table">
                     <thead>
 	                    <tr>		             
@@ -105,6 +106,19 @@
                 </table>
             </div>
         </div>
+        <!-- /ko -->
+        <!-- ko with: $root.output -->
+        <div data-bind="visible: show" class="otherwindow dockrow">
+            <button class="close" data-bind="click: close">&times;</button>
+            <div class="otherwindowheader"><span data-bind="text: title"></span></div>
+            <div class="otherwindowpane container stylishscrollbar scroll-x scroll-y">
+                <div data-bind="text: content" class="message">
+
+                </div>
+            </div>
+        </div>
+        <!-- /ko -->
+        
         <div class="statusbar dockrow" data-bind="text: $root.projectStatus">
         </div>
     </div>   

Web/Web.Debug.config

@@ -17,4 +17,19 @@
   <system.web>
     <customErrors mode="Off" xdt:Transform="Replace" />
   </system.web>
+  <log4net xdt:Transform="Replace">
+    <appender name="LeAppender" type="log4net.Appender.LeAppender, LeLog4net">
+      <Key value="LOGENTRIES_ACCOUNT_KEY" />
+      <Location value="LOGENTRIES_LOCATION" />
+      <Debug value="true" />
+      <Ssl value="false" />
+      <layout type="log4net.Layout.PatternLayout">
+        <param name="ConversionPattern" value="%d{ddd MMM dd HH:mm:ss zzz yyyy} %logger %: %level%, %m" />
+      </layout>
+    </appender>
+    <root>
+      <level value="ALL" />
+      <appender-ref ref="LeAppender" />
+    </root>
+  </log4net>
 </configuration>