Move moving classes around

src/Umbraco.Core/Packaging/ConflictingPackageContentFinder.cs

@@ -2,8 +2,9 @@
 using System.Linq;
 using System.Xml.Linq;
 using Umbraco.Core.Models;
+using Umbraco.Core.Services;
 
-namespace Umbraco.Core.Services
+namespace Umbraco.Core.Packaging
 {
     public class ConflictingPackageContentFinder : IConflictingPackageContentFinder
     {

src/Umbraco.Core/Packaging/IConflictingPackageContentFinder.cs

@@ -1,7 +1,7 @@
 using System.Xml.Linq;
 using Umbraco.Core.Models;
 
-namespace Umbraco.Core.Services
+namespace Umbraco.Core.Packaging
 {
     public interface IConflictingPackageContentFinder
     {

src/Umbraco.Core/Services/PackagingService.cs

@@ -1575,14 +1575,16 @@ namespace Umbraco.Core.Services
 
         internal IPackageInstallation PackageInstallation
         {
+            //NOTE The PackageInstallation class should be passed as IPackageInstallation through the 
+            //constructor (probably as an overload to avoid breaking stuff), so that its extendable.
+                // NOTE COMMENT: But is is not a service? and all other parced in constructor is services...
             private get { return _packageInstallation ?? new PackageInstallation(this, _macroService, _fileService, new PackageExtraction()); }
             set { _packageInstallation = value; }
-
-            
         }
 
         internal InstallationSummary InstallPackage(string packageFilePath, int userId = 0)
         {
+            //TODO Add events ?
             return PackageInstallation.InstallPackage(packageFilePath, userId);
         }
 

src/Umbraco.Core/Services/ServiceContext.cs

@@ -31,7 +31,7 @@ namespace Umbraco.Core.Services
         private Lazy<IMemberTypeService> _memberTypeService;
         private Lazy<IMemberGroupService> _memberGroupService;
         private Lazy<INotificationService> _notificationService;
-        
+
         /// <summary>
         /// public ctor - will generally just be used for unit testing
         /// </summary>

src/Umbraco.Core/Umbraco.Core.csproj

@@ -1045,7 +1045,7 @@
     <Compile Include="Services\IMembershipUserService.cs" />
     <Compile Include="Services\IMemberTypeService.cs" />
     <Compile Include="Services\INotificationService.cs" />
-    <Compile Include="Services\IConflictingPackageContentFinder.cs" />
+    <Compile Include="Packaging\IConflictingPackageContentFinder.cs" />
     <Compile Include="Services\IRelationService.cs" />
     <Compile Include="Services\ISectionService.cs" />
     <Compile Include="Services\IPackagingService.cs" />
@@ -1060,7 +1060,7 @@
     <Compile Include="Services\MemberService.cs" />
     <Compile Include="Services\MemberTypeService.cs" />
     <Compile Include="Services\NotificationService.cs" />
-    <Compile Include="Services\ConflictingPackageContentFinder.cs" />
+    <Compile Include="Packaging\ConflictingPackageContentFinder.cs" />
     <Compile Include="Services\RelationService.cs" />
     <Compile Include="Services\SectionService.cs" />
     <Compile Include="Services\ServerRegistrationService.cs" />