DynamoDS
diff --git a/‎src/DynamoCore/Extensions/ExtensionLibraryLoader.cs
Copy file name to clipboard
+15-2Lines changed: 15 additions & 2 deletions b/‎src/DynamoCore/Extensions/ExtensionLibraryLoader.cs
Copy file name to clipboard
+15-2Lines changed: 15 additions & 2 deletions
diff --git a/‎src/DynamoCore/Library/ILibraryLoader.cs
Copy file name to clipboard
+5-1Lines changed: 5 additions & 1 deletion b/‎src/DynamoCore/Library/ILibraryLoader.cs
Copy file name to clipboard
+5-1Lines changed: 5 additions & 1 deletion
diff --git a/‎src/DynamoCore/Library/LibraryServices.cs
Copy file name to clipboard
+74-29Lines changed: 74 additions & 29 deletions b/‎src/DynamoCore/Library/LibraryServices.cs
Copy file name to clipboard
+74-29Lines changed: 74 additions & 29 deletions
diff --git a/‎src/DynamoCore/Logging/DynamoLogger.cs
Copy file name to clipboard
+35-1Lines changed: 35 additions & 1 deletion b/‎src/DynamoCore/Logging/DynamoLogger.cs
Copy file name to clipboard
+35-1Lines changed: 35 additions & 1 deletion
diff --git a/‎src/DynamoCore/Models/DynamoModel.cs
Copy file name to clipboard
+6-5Lines changed: 6 additions & 5 deletions b/‎src/DynamoCore/Models/DynamoModel.cs
Copy file name to clipboard
+6-5Lines changed: 6 additions & 5 deletions
diff --git a/‎src/DynamoCore/Properties/AssemblyInfo.cs
Copy file name to clipboard
+1Lines changed: 1 addition & 0 deletions b/‎src/DynamoCore/Properties/AssemblyInfo.cs
Copy file name to clipboard
+1Lines changed: 1 addition & 0 deletions
diff --git a/‎src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerClientViewModel.cs
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
@@ -1,4 +1,7 @@
-using System.Reflection;
+using System.Collections.Generic;
+using System.Linq;
+using System.Reflection;
+using Dynamo.Engine;
 using Dynamo.Library;
 using Dynamo.Models;
 
@@ -7,7 +10,7 @@ namespace Dynamo.Extensions
     /// <summary>
     /// Provides functionality for loading node's DLLs
     /// </summary>
-    public class ExtensionLibraryLoader : ILibraryLoader
+    public class ExtensionLibraryLoader : ILibraryLoader 
     {
         private readonly DynamoModel model;
 
@@ -28,5 +31,15 @@ public void LoadNodeLibrary(Assembly library)
         {
             model.LoadNodeLibrary(library);
         }
+
+        /// <summary>
+        /// Loads packages for import into VM and for node search.
+        /// </summary>
+        /// <param name="assemblies"></param>
+        public void LoadPackages(IEnumerable<Assembly> assemblies)
+        {
+            var libraryPaths = assemblies.Select(x => x.Location);
+            model.LibraryServices.OnLibrariesImported(new LibraryServices.LibraryLoadedEventArgs(libraryPaths));
+        }
     }
 }
@@ -1,4 +1,5 @@
-using System.Reflection;
+using System.Collections.Generic;
+using System.Reflection;
 
 namespace Dynamo.Library
 {
@@ -11,5 +12,8 @@ public interface ILibraryLoader
         /// Loads node's library
         /// </summary>
         void LoadNodeLibrary(Assembly library);
+
+        //TODO: Add LoadPackages to interface in 3.0
+        //void LoadPackages(IEnumerable<Assembly> assemblies);
     }
 }
@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
+using System.Reflection;
 using System.Xml;
 using Dynamo.Configuration;
 using Dynamo.Exceptions;
@@ -17,6 +18,7 @@
 using ProtoCore.Namespace;
 using ProtoCore.Utils;
 using ProtoFFI;
+using MethodAttributes = ProtoCore.AST.AssociativeAST.MethodAttributes;
 using Operator = ProtoCore.DSASM.Operator;
 
 namespace Dynamo.Engine
@@ -72,20 +74,6 @@ public UpgradeHint()
         private readonly Dictionary<string, UpgradeHint> priorNameHints =
             new Dictionary<string, UpgradeHint>();
 
-        /// <summary>
-        /// Copy properties from the liveCore
-        /// The properties to copy are only those used by the library core
-        /// </summary>
-        internal void UpdateLibraryCoreData()
-        {
-            // If a liverunner core is provided, sync the library core data
-            if (liveRunnerCore != null)
-            {
-                LibraryManagementCore.ProcTable = new ProtoCore.DSASM.ProcedureTable(liveRunnerCore.ProcTable);
-                LibraryManagementCore.ClassTable = new ProtoCore.DSASM.ClassTable(liveRunnerCore.ClassTable);
-            }
-        }
-
         /// <summary>
         /// Initializes a new instance of the <see cref="LibraryServices"/> class.
         /// </summary>
@@ -111,6 +99,34 @@ public LibraryServices(ProtoCore.Core libraryManagementCore, IPathManager pathMa
             PopulateOperators();
             PopulatePreloadLibraries();
             LibraryLoadFailed += new EventHandler<LibraryLoadFailedEventArgs>(LibraryLoadFailureHandler);
+
+            AppDomain.CurrentDomain.AssemblyResolve += ResolveAssembly;
+        }
+
+        private Assembly ResolveAssembly(object sender, ResolveEventArgs args)
+        {
+            var assemblyName = new AssemblyName(args.Name).Name + ".dll";
+            var assemblyPath = assemblyName;
+            try
+            {
+                if (pathManager.ResolveLibraryPath(ref assemblyPath))
+                {
+                    return Assembly.LoadFrom(assemblyPath);
+                }
+
+                foreach (var packagesDirectory in pathManager.PackagesDirectories)
+                {
+                    assemblyPath = Path.Combine(packagesDirectory, assemblyName);
+                    if (File.Exists(assemblyPath))
+                        return Assembly.LoadFrom(assemblyPath);
+                }
+
+                return null;
+            }
+            catch (Exception ex)
+            {
+                throw new Exception(string.Format("The location of the assembly, {0} could not be resolved for loading.", assemblyName), ex);
+            }
         }
 
         /// <summary>
@@ -121,6 +137,8 @@ public void Dispose()
             builtinFunctionGroups.Clear();
             importedFunctionGroups.Clear();
             importedLibraries.Clear();
+
+            AppDomain.CurrentDomain.AssemblyResolve -= ResolveAssembly;
         }
 
         /// <summary>
@@ -167,6 +185,7 @@ private void LibraryLoadFailureHandler(object sender, LibraryLoadFailedEventArgs
         {
             LibraryLoadFailedException ex = new LibraryLoadFailedException(args.LibraryPath, args.Reason);
             Log(ex.Message, WarningLevel.Moderate);
+            Log(ex);
 
             // NOTE: We do not want to throw an exception here if the failure was due
             // to a missing library that was explicitly (attempted to be) loaded
@@ -521,7 +540,7 @@ private static bool TryGetFunctionGroup(
         /// </summary>
         /// <param name="library">The library to be loaded</param>
         /// <param name="isExplicitlyImportedLib">Indicates if the library has been imported using the "File | ImportLibrary" command</param>
-        internal bool ImportLibrary(string library, bool isExplicitlyImportedLib = false)
+        internal bool LoadNodeLibrary(string library, bool isExplicitlyImportedLib)
         {
             if (null == library)
                 throw new ArgumentNullException();
@@ -563,14 +582,23 @@ internal bool ImportLibrary(string library, bool isExplicitlyImportedLib = false
             {
                 DLLFFIHandler.Register(FFILanguage.CSharp, new CSModuleHelper());
 
-                var functionTable = LibraryManagementCore.CodeBlockList[0].procedureTable;
+                ProcedureTable functionTable = null;
+                int functionNumber = 0;
+                if (LibraryManagementCore.CodeBlockList.Any())
+                {
+                    functionTable = LibraryManagementCore.CodeBlockList[0].procedureTable;
+                    functionNumber = functionTable.Procedures.Count;
+                }
                 var classTable = LibraryManagementCore.ClassTable;
-
-                int functionNumber = functionTable.Procedures.Count;
                 int classNumber = classTable.ClassNodes.Count;
 
                 CompilerUtils.TryLoadAssemblyIntoCore(LibraryManagementCore, library);
 
+                if(functionTable == null)
+                {
+                    functionTable = LibraryManagementCore.CodeBlockList[0].procedureTable;
+                    functionNumber = functionTable.Procedures.Count;
+                }
 
                 if (LibraryManagementCore.BuildStatus.ErrorCount > 0)
                 {
@@ -579,7 +607,7 @@ internal bool ImportLibrary(string library, bool isExplicitlyImportedLib = false
                     foreach (ErrorEntry error in LibraryManagementCore.BuildStatus.Errors)
                     {
                         Log(error.Message, WarningLevel.Moderate);
-                        errorMessage += error.Message + "\n";
+                        errorMessage += Environment.NewLine + error.Message;
                     }
 
                     throw new Exception(errorMessage);
@@ -605,14 +633,17 @@ internal bool ImportLibrary(string library, bool isExplicitlyImportedLib = false
                     throwOnFailure: !isExplicitlyImportedLib));
                 return false;
             }
+            importedLibraries.Add(library);
 
-            OnLibraryLoaded(new LibraryLoadedEventArgs(library));
-
-            // After a library is loaded, update the library core data with the liveRunner core data
-            UpdateLibraryCoreData();
             return true;
         }
 
+        internal void ImportLibrary(string library, bool isExplicitlyImportedLib = false)
+        {
+            LoadNodeLibrary(library, isExplicitlyImportedLib);
+            OnLibrariesImported(new LibraryLoadedEventArgs(new List<string> {library}));
+        }
+
 
         internal void LoadLibraryMigrations(string library)
         {
@@ -874,7 +905,7 @@ private void PopulateOperators()
         }
 
         /// <summary>
-        ///     Polulate preloaded libraries.
+        ///     Populate preloaded libraries.
         /// </summary>
         private void PopulatePreloadLibraries()
         {
@@ -1085,13 +1116,13 @@ private void OnLibraryLoadFailed(LibraryLoadFailedEventArgs e)
                 handler(this, e);
         }
 
-        private void OnLibraryLoaded(LibraryLoadedEventArgs e)
+        internal void OnLibrariesImported(LibraryLoadedEventArgs e)
         {
-            importedLibraries.Add(e.LibraryPath);
-
-            EventHandler<LibraryLoadedEventArgs> handler = LibraryLoaded;
+            var handler = LibraryLoaded;
             if (handler != null)
+            {
                 handler(this, e);
+            }
         }
 
         public static class Categories
@@ -1133,12 +1164,26 @@ public LibraryLoadFailedEventArgs(string libraryPath, string reason, bool throwO
 
         public class LibraryLoadedEventArgs : EventArgs
         {
+            // TODO: Remove in 3.0
+            [Obsolete("This constructor is obsolete. Use constructor that accepts collection of library paths instead.")]
             public LibraryLoadedEventArgs(string libraryPath)
             {
                 LibraryPath = libraryPath;
             }
 
-            public string LibraryPath { get; private set; }
+            public LibraryLoadedEventArgs(IEnumerable<string> libraryPaths)
+            {
+                LibraryPaths = libraryPaths;
+            }
+
+            // TODO: Remove in 3.0
+            [Obsolete("This property is obsolete. Use property LibraryPaths that returns collection of paths instead.")]
+            public string LibraryPath { get; }
+
+            /// <summary>
+            /// Paths to libraries that are loaded. 
+            /// </summary>
+            public IEnumerable<string> LibraryPaths { get; }
         }
 
         public class LibraryLoadingEventArgs : EventArgs
 
@@ -1,8 +1,10 @@
 using System;
+using System.Collections.Generic;
 using System.IO;
 using System.Text;
 using Dynamo.Configuration;
 using Dynamo.Core;
+using Dynamo.Exceptions;
 using Dynamo.Interfaces;
 
 namespace Dynamo.Logging
@@ -139,6 +141,16 @@ public string LogText
             }
         }
 
+        private List<NotificationMessage> notifications;
+        /// <summary>
+        /// Notifications logged during startup such as library load failures
+        /// that need to be displayed to user.
+        /// </summary>
+        public IEnumerable<NotificationMessage> StartupNotifications
+        {
+            get { return notifications; }
+        }
+
         /// <summary>
         /// Initializes a new instance of <see cref="DynamoLogger"/> class
         /// with specified debug settings and directory where to write logs
@@ -155,6 +167,8 @@ public DynamoLogger(DebugSettings debugSettings, string logDirectory)
                 WarningLevel = WarningLevel.Mild;
                 Warning = "";
 
+                notifications = new List<NotificationMessage>();
+
                 StartLogging(logDirectory);
             }
         }
@@ -239,7 +253,14 @@ public void LogNotification(string sender,string title, string shortMessage, str
         {
             var notificationMessage = string.Format("{0}:{3} {1}: {3} {2}", title, shortMessage, detailedMessage,Environment.NewLine);
             Log("notification",notificationMessage );
-            NotificationLogged(new NotificationMessage(sender, shortMessage, detailedMessage,title));
+            if (NotificationLogged != null)
+            {
+                NotificationLogged(new NotificationMessage(sender, shortMessage, detailedMessage, title));
+            }
+            else
+            {
+                notifications.Add(new NotificationMessage(sender, shortMessage, detailedMessage, title));
+            }
         }
 
 
@@ -301,6 +322,14 @@ internal void ResetWarning()
             }
         }
 
+        /// <summary>
+        /// Clear any notifications after displaying them.
+        /// </summary>
+        public void ClearStartupNotifications()
+        {
+            notifications.Clear();
+        }
+
         /// <summary>
         /// Log a message
         /// </summary>
@@ -316,6 +345,11 @@ public void Log(string message)
         /// <param name="e">Exception to log</param>
         public void Log(Exception e)
         {
+            var le = e as LibraryLoadFailedException;
+            if (le != null)
+            {
+                LogNotification(le.Source, le.ToString(), le.Message, le.Reason);
+            }
             Log(e.GetType() + ":", LogLevel.Console);
             Log(e.Message, LogLevel.Console);
             Log(e.StackTrace, LogLevel.Console);
 
@@ -966,10 +966,11 @@ void UpdateManager_Log(LogEventArgs args)
         /// <param name="e"></param>
         private void LibraryLoaded(object sender, LibraryServices.LibraryLoadedEventArgs e)
         {
-            string newLibrary = e.LibraryPath;
-
-            // Load all functions defined in that library.
-            AddZeroTouchNodesToSearch(LibraryServices.GetFunctionGroups(newLibrary));
+            foreach (var newLibrary in e.LibraryPaths)
+            {
+                // Load all functions defined in that library.
+                AddZeroTouchNodesToSearch(LibraryServices.GetFunctionGroups(newLibrary));
+            }
         }
 
         /// <summary>
@@ -1252,7 +1253,7 @@ internal void LoadNodeLibrary(Assembly assem)
         {
             if (!NodeModelAssemblyLoader.ContainsNodeModelSubType(assem))
             {
-                LibraryServices.ImportLibrary(assem.Location);
+                LibraryServices.LoadNodeLibrary(assem.Location, false);
                 return;
             }
 
 
@@ -29,3 +29,4 @@
 [assembly: InternalsVisibleTo("LibraryViewExtension")]
 [assembly: InternalsVisibleTo("ViewExtensionLibraryTests")]
 [assembly: InternalsVisibleTo("DynamoPerformanceTests")]
+[assembly:InternalsVisibleTo("PackageManagerTests")]
@@ -490,7 +490,7 @@ internal void DownloadAndInstall(PackageDownloadHandle packageDownloadHandle, st
                         if (packageDownloadHandle.Extract(DynamoViewModel.Model, downloadPath, out dynPkg))
                         {
                             var p = Package.FromDirectory(dynPkg.RootDirectory, DynamoViewModel.Model.Logger);
-                            pmExtension.PackageLoader.Load(p);
+                            pmExtension.PackageLoader.LoadPackages(new List<Package> {p});
 
                             packageDownloadHandle.DownloadState = PackageDownloadHandle.State.Installed;
                         }
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`		`-using System.Reflection;`
	`1`	`+using System.Collections.Generic;`
	`2`	`+using System.Reflection;`
`2`	`3`
`3`	`4`	`namespace Dynamo.Library`
`4`	`5`	`{`
`@@ -11,5 +12,8 @@ public interface ILibraryLoader`
`11`	`12`	`/// Loads node's library`
`12`	`13`	`/// </summary>`
`13`	`14`	`void LoadNodeLibrary(Assembly library);`
	`15`	`+`
	`16`	`+ //TODO: Add LoadPackages to interface in 3.0`
	`17`	`+ //void LoadPackages(IEnumerable<Assembly> assemblies);`
`14`	`18`	`}`
`15`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -966,10 +966,11 @@ void UpdateManager_Log(LogEventArgs args)`
`966`	`966`	`/// <param name="e"></param>`
`967`	`967`	`private void LibraryLoaded(object sender, LibraryServices.LibraryLoadedEventArgs e)`
`968`	`968`	`{`
`969`		`- string newLibrary = e.LibraryPath;`
`970`		`-`
`971`		`- // Load all functions defined in that library.`
`972`		`- AddZeroTouchNodesToSearch(LibraryServices.GetFunctionGroups(newLibrary));`
	`969`	`+ foreach (var newLibrary in e.LibraryPaths)`
	`970`	`+ {`
	`971`	`+ // Load all functions defined in that library.`
	`972`	`+ AddZeroTouchNodesToSearch(LibraryServices.GetFunctionGroups(newLibrary));`
	`973`	`+ }`
`973`	`974`	`}`
`974`	`975`
`975`	`976`	`/// <summary>`
`@@ -1252,7 +1253,7 @@ internal void LoadNodeLibrary(Assembly assem)`
`1252`	`1253`	`{`
`1253`	`1254`	`if (!NodeModelAssemblyLoader.ContainsNodeModelSubType(assem))`
`1254`	`1255`	`{`
`1255`		`- LibraryServices.ImportLibrary(assem.Location);`
	`1256`	`+ LibraryServices.LoadNodeLibrary(assem.Location, false);`
`1256`	`1257`	`return;`
`1257`	`1258`	`}`
`1258`	`1259`
Original file line number	Diff line number	Diff line change
`@@ -490,7 +490,7 @@ internal void DownloadAndInstall(PackageDownloadHandle packageDownloadHandle, st`
`490`	`490`	`if (packageDownloadHandle.Extract(DynamoViewModel.Model, downloadPath, out dynPkg))`
`491`	`491`	`{`
`492`	`492`	`var p = Package.FromDirectory(dynPkg.RootDirectory, DynamoViewModel.Model.Logger);`
`493`		`- pmExtension.PackageLoader.Load(p);`
	`493`	`+ pmExtension.PackageLoader.LoadPackages(new List<Package> {p});`
`494`	`494`
`495`	`495`	`packageDownloadHandle.DownloadState = PackageDownloadHandle.State.Installed;`
`496`	`496`	`}`