Browse Source

mono fixes

Luke Pulverenti 11 years ago
parent
commit
54a5d63a61
22 changed files with 31 additions and 124 deletions
  1. 1 3
      MediaBrowser.Api/DefaultTheme/DefaultThemeService.cs
  2. 0 5
      MediaBrowser.Api/UserLibrary/YearsService.cs
  3. 0 5
      MediaBrowser.Common.Implementations/HttpClientManager/HttpClientManager.cs
  4. 0 5
      MediaBrowser.Common.Implementations/Updates/InstallationManager.cs
  5. 1 1
      MediaBrowser.Controller/Entities/Folder.cs
  6. 3 3
      MediaBrowser.Controller/Localization/LocalizedStrings.cs
  7. 22 16
      MediaBrowser.Server.Implementations/HttpServer/HttpServer.cs
  8. 0 4
      MediaBrowser.Server.Implementations/Library/UserDataManager.cs
  9. 1 7
      MediaBrowser.Server.Implementations/Library/Validators/GameGenresPostScanTask.cs
  10. 1 7
      MediaBrowser.Server.Implementations/Library/Validators/GenresPostScanTask.cs
  11. 1 7
      MediaBrowser.Server.Implementations/Library/Validators/MusicGenresPostScanTask.cs
  12. 1 7
      MediaBrowser.Server.Implementations/Library/Validators/StudiosPostScanTask.cs
  13. 0 5
      MediaBrowser.Server.Implementations/MediaEncoder/MediaEncoder.cs
  14. 0 5
      MediaBrowser.Server.Implementations/Persistence/SqliteChapterRepository.cs
  15. 0 4
      MediaBrowser.Server.Implementations/Persistence/SqliteDisplayPreferencesRepository.cs
  16. 0 10
      MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs
  17. 0 4
      MediaBrowser.Server.Implementations/Persistence/SqliteUserDataRepository.cs
  18. 0 10
      MediaBrowser.Server.Implementations/Persistence/SqliteUserRepository.cs
  19. 0 4
      MediaBrowser.Server.Implementations/Providers/ProviderManager.cs
  20. 0 5
      MediaBrowser.Server.Implementations/ServerManager/ServerManager.cs
  21. 0 5
      MediaBrowser.Server.Implementations/ServerManager/WebSocketConnection.cs
  22. 0 2
      MediaBrowser.Server.Mono/Program.cs

+ 1 - 3
MediaBrowser.Api/DefaultTheme/DefaultThemeService.cs

@@ -66,16 +66,14 @@ namespace MediaBrowser.Api.DefaultTheme
         private readonly ILogger _logger;
         private readonly ILogger _logger;
         private readonly ILibraryManager _libraryManager;
         private readonly ILibraryManager _libraryManager;
 
 
-        private readonly ILocalizationManager _localization;
         private readonly IImageProcessor _imageProcessor;
         private readonly IImageProcessor _imageProcessor;
 
 
-        public DefaultThemeService(IUserManager userManager, IDtoService dtoService, ILogger logger, ILibraryManager libraryManager, ILocalizationManager localization, IImageProcessor imageProcessor)
+        public DefaultThemeService(IUserManager userManager, IDtoService dtoService, ILogger logger, ILibraryManager libraryManager, IImageProcessor imageProcessor)
         {
         {
             _userManager = userManager;
             _userManager = userManager;
             _dtoService = dtoService;
             _dtoService = dtoService;
             _logger = logger;
             _logger = logger;
             _libraryManager = libraryManager;
             _libraryManager = libraryManager;
-            _localization = localization;
             _imageProcessor = imageProcessor;
             _imageProcessor = imageProcessor;
         }
         }
 
 

+ 0 - 5
MediaBrowser.Api/UserLibrary/YearsService.cs

@@ -48,11 +48,6 @@ namespace MediaBrowser.Api.UserLibrary
     /// </summary>
     /// </summary>
     public class YearsService : BaseItemsByNameService<Year>
     public class YearsService : BaseItemsByNameService<Year>
     {
     {
-        /// <summary>
-        /// The us culture
-        /// </summary>
-        private static readonly CultureInfo UsCulture = new CultureInfo("en-US");
-
         public YearsService(IUserManager userManager, ILibraryManager libraryManager, IUserDataManager userDataRepository, IItemRepository itemRepo, IDtoService dtoService)
         public YearsService(IUserManager userManager, ILibraryManager libraryManager, IUserDataManager userDataRepository, IItemRepository itemRepo, IDtoService dtoService)
             : base(userManager, libraryManager, userDataRepository, itemRepo, dtoService)
             : base(userManager, libraryManager, userDataRepository, itemRepo, dtoService)
         {
         {

+ 0 - 5
MediaBrowser.Common.Implementations/HttpClientManager/HttpClientManager.cs

@@ -555,11 +555,6 @@ namespace MediaBrowser.Common.Implementations.HttpClientManager
             {
             {
                 throw new ArgumentNullException("url");
                 throw new ArgumentNullException("url");
             }
             }
-
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
         }
         }
 
 
         /// <summary>
         /// <summary>

+ 0 - 5
MediaBrowser.Common.Implementations/Updates/InstallationManager.cs

@@ -361,11 +361,6 @@ namespace MediaBrowser.Common.Implementations.Updates
                 throw new ArgumentNullException("progress");
                 throw new ArgumentNullException("progress");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             var installationInfo = new InstallationInfo
             var installationInfo = new InstallationInfo
             {
             {
                 Id = Guid.NewGuid(),
                 Id = Guid.NewGuid(),

+ 1 - 1
MediaBrowser.Controller/Entities/Folder.cs

@@ -1066,7 +1066,7 @@ namespace MediaBrowser.Controller.Entities
         /// <returns>IList{BaseItem}.</returns>
         /// <returns>IList{BaseItem}.</returns>
         public IList<BaseItem> GetRecursiveChildren()
         public IList<BaseItem> GetRecursiveChildren()
         {
         {
-            return GetRecursiveChildren(null, null, true);
+            return GetRecursiveChildren(i => true);
         }
         }
 
 
         /// <summary>
         /// <summary>

+ 3 - 3
MediaBrowser.Controller/Localization/LocalizedStrings.cs

@@ -130,17 +130,17 @@ namespace MediaBrowser.Controller.Localization
                 {
                 {
                     value = field.GetValue(stringData) as string;
                     value = field.GetValue(stringData) as string;
                 }
                 }
-                catch (TargetException ex)
+                catch (TargetException)
                 {
                 {
                     //Logger.ErrorException("Error getting value for field: {0}", ex, field.Name);
                     //Logger.ErrorException("Error getting value for field: {0}", ex, field.Name);
                     continue;
                     continue;
                 }
                 }
-                catch (FieldAccessException ex)
+                catch (FieldAccessException)
                 {
                 {
                     //Logger.ErrorException("Error getting value for field: {0}", ex, field.Name);
                     //Logger.ErrorException("Error getting value for field: {0}", ex, field.Name);
                     continue;
                     continue;
                 }
                 }
-                catch (NotSupportedException ex)
+                catch (NotSupportedException)
                 {
                 {
                     //Logger.ErrorException("Error getting value for field: {0}", ex, field.Name);
                     //Logger.ErrorException("Error getting value for field: {0}", ex, field.Name);
                     continue;
                     continue;

+ 22 - 16
MediaBrowser.Server.Implementations/HttpServer/HttpServer.cs

@@ -382,22 +382,23 @@ namespace MediaBrowser.Server.Implementations.HttpServer
         /// <returns>Task.</returns>
         /// <returns>Task.</returns>
         private async Task ProcessWebSocketRequest(HttpListenerContext ctx)
         private async Task ProcessWebSocketRequest(HttpListenerContext ctx)
         {
         {
+            #if __MonoCS__
+            #else
             try
             try
-            {
-                var webSocketContext = await ctx.AcceptWebSocketAsync(null).ConfigureAwait(false);
-
-                if (WebSocketConnected != null)
-                {
-                    WebSocketConnected(this, new WebSocketConnectEventArgs { WebSocket = new NativeWebSocket(webSocketContext.WebSocket, _logger), Endpoint = ctx.Request.RemoteEndPoint.ToString() });
-                }
-            }
-            catch (Exception ex)
-            {
-                _logger.ErrorException("AcceptWebSocketAsync error", ex);
-
-                ctx.Response.StatusCode = 500;
-                ctx.Response.Close();
-            }
+              {
+               var webSocketContext = await ctx.AcceptWebSocketAsync(null).ConfigureAwait(false);
+               if (WebSocketConnected != null)
+               {
+                WebSocketConnected(this, new WebSocketConnectEventArgs { WebSocket = new NativeWebSocket(webSocketContext.WebSocket, _logger), Endpoint = ctx.Request.RemoteEndPoint.ToString() });
+               }
+              }
+              catch (Exception ex)
+              {
+               _logger.ErrorException("AcceptWebSocketAsync error", ex);
+               ctx.Response.StatusCode = 500;
+               ctx.Response.Close();
+              }
+            #endif
         }
         }
 
 
         /// <summary>
         /// <summary>
@@ -532,7 +533,12 @@ namespace MediaBrowser.Server.Implementations.HttpServer
         {
         {
             get
             get
             {
             {
-                if (!_supportsNativeWebSocket.HasValue)
+				#if __MonoCS__
+				return false;
+				#else
+				#endif
+
+				if (!_supportsNativeWebSocket.HasValue)
                 {
                 {
                     try
                     try
                     {
                     {

+ 0 - 4
MediaBrowser.Server.Implementations/Library/UserDataManager.cs

@@ -57,10 +57,6 @@ namespace MediaBrowser.Server.Implementations.Library
             {
             {
                 throw new ArgumentNullException("userData");
                 throw new ArgumentNullException("userData");
             }
             }
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
             if (userId == Guid.Empty)
             if (userId == Guid.Empty)
             {
             {
                 throw new ArgumentNullException("userId");
                 throw new ArgumentNullException("userId");

+ 1 - 7
MediaBrowser.Server.Implementations/Library/Validators/GameGenresPostScanTask.cs

@@ -15,20 +15,14 @@ namespace MediaBrowser.Server.Implementations.Library.Validators
         /// </summary>
         /// </summary>
         private readonly ILibraryManager _libraryManager;
         private readonly ILibraryManager _libraryManager;
 
 
-        /// <summary>
-        /// The _user manager
-        /// </summary>
-        private readonly IUserManager _userManager;
-
         /// <summary>
         /// <summary>
         /// Initializes a new instance of the <see cref="GameGenresPostScanTask"/> class.
         /// Initializes a new instance of the <see cref="GameGenresPostScanTask"/> class.
         /// </summary>
         /// </summary>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="userManager">The user manager.</param>
         /// <param name="userManager">The user manager.</param>
-        public GameGenresPostScanTask(ILibraryManager libraryManager, IUserManager userManager)
+        public GameGenresPostScanTask(ILibraryManager libraryManager)
         {
         {
             _libraryManager = libraryManager;
             _libraryManager = libraryManager;
-            _userManager = userManager;
         }
         }
 
 
         /// <summary>
         /// <summary>

+ 1 - 7
MediaBrowser.Server.Implementations/Library/Validators/GenresPostScanTask.cs

@@ -12,20 +12,14 @@ namespace MediaBrowser.Server.Implementations.Library.Validators
         /// </summary>
         /// </summary>
         private readonly ILibraryManager _libraryManager;
         private readonly ILibraryManager _libraryManager;
 
 
-        /// <summary>
-        /// The _user manager
-        /// </summary>
-        private readonly IUserManager _userManager;
-
         /// <summary>
         /// <summary>
         /// Initializes a new instance of the <see cref="ArtistsPostScanTask" /> class.
         /// Initializes a new instance of the <see cref="ArtistsPostScanTask" /> class.
         /// </summary>
         /// </summary>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="userManager">The user manager.</param>
         /// <param name="userManager">The user manager.</param>
-        public GenresPostScanTask(ILibraryManager libraryManager, IUserManager userManager)
+        public GenresPostScanTask(ILibraryManager libraryManager)
         {
         {
             _libraryManager = libraryManager;
             _libraryManager = libraryManager;
-            _userManager = userManager;
         }
         }
 
 
         /// <summary>
         /// <summary>

+ 1 - 7
MediaBrowser.Server.Implementations/Library/Validators/MusicGenresPostScanTask.cs

@@ -15,20 +15,14 @@ namespace MediaBrowser.Server.Implementations.Library.Validators
         /// </summary>
         /// </summary>
         private readonly ILibraryManager _libraryManager;
         private readonly ILibraryManager _libraryManager;
 
 
-        /// <summary>
-        /// The _user manager
-        /// </summary>
-        private readonly IUserManager _userManager;
-
         /// <summary>
         /// <summary>
         /// Initializes a new instance of the <see cref="ArtistsPostScanTask" /> class.
         /// Initializes a new instance of the <see cref="ArtistsPostScanTask" /> class.
         /// </summary>
         /// </summary>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="userManager">The user manager.</param>
         /// <param name="userManager">The user manager.</param>
-        public MusicGenresPostScanTask(ILibraryManager libraryManager, IUserManager userManager)
+        public MusicGenresPostScanTask(ILibraryManager libraryManager)
         {
         {
             _libraryManager = libraryManager;
             _libraryManager = libraryManager;
-            _userManager = userManager;
         }
         }
 
 
         /// <summary>
         /// <summary>

+ 1 - 7
MediaBrowser.Server.Implementations/Library/Validators/StudiosPostScanTask.cs

@@ -15,20 +15,14 @@ namespace MediaBrowser.Server.Implementations.Library.Validators
         /// </summary>
         /// </summary>
         private readonly ILibraryManager _libraryManager;
         private readonly ILibraryManager _libraryManager;
 
 
-        /// <summary>
-        /// The _user manager
-        /// </summary>
-        private readonly IUserManager _userManager;
-
         /// <summary>
         /// <summary>
         /// Initializes a new instance of the <see cref="ArtistsPostScanTask" /> class.
         /// Initializes a new instance of the <see cref="ArtistsPostScanTask" /> class.
         /// </summary>
         /// </summary>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="libraryManager">The library manager.</param>
         /// <param name="userManager">The user manager.</param>
         /// <param name="userManager">The user manager.</param>
-        public StudiosPostScanTask(ILibraryManager libraryManager, IUserManager userManager)
+        public StudiosPostScanTask(ILibraryManager libraryManager)
         {
         {
             _libraryManager = libraryManager;
             _libraryManager = libraryManager;
-            _userManager = userManager;
         }
         }
 
 
         /// <summary>
         /// <summary>

+ 0 - 5
MediaBrowser.Server.Implementations/MediaEncoder/MediaEncoder.cs

@@ -594,11 +594,6 @@ namespace MediaBrowser.Server.Implementations.MediaEncoder
                 throw new ArgumentNullException("outputPath");
                 throw new ArgumentNullException("outputPath");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             var slowSeekParam = offset.TotalSeconds > 0 ? " -ss " + offset.TotalSeconds.ToString(UsCulture) : string.Empty;
             var slowSeekParam = offset.TotalSeconds > 0 ? " -ss " + offset.TotalSeconds.ToString(UsCulture) : string.Empty;
 
 
             var process = new Process
             var process = new Process

+ 0 - 5
MediaBrowser.Server.Implementations/Persistence/SqliteChapterRepository.cs

@@ -188,11 +188,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
                 throw new ArgumentNullException("chapters");
                 throw new ArgumentNullException("chapters");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);

+ 0 - 4
MediaBrowser.Server.Implementations/Persistence/SqliteDisplayPreferencesRepository.cs

@@ -112,10 +112,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
             {
             {
                 throw new ArgumentNullException("displayPreferences.Id");
                 throw new ArgumentNullException("displayPreferences.Id");
             }
             }
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
 
 
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 

+ 0 - 10
MediaBrowser.Server.Implementations/Persistence/SqliteItemRepository.cs

@@ -186,11 +186,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
                 throw new ArgumentNullException("items");
                 throw new ArgumentNullException("items");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);
@@ -456,11 +451,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
                 throw new ArgumentNullException("children");
                 throw new ArgumentNullException("children");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);

+ 0 - 4
MediaBrowser.Server.Implementations/Persistence/SqliteUserDataRepository.cs

@@ -116,10 +116,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
             {
             {
                 throw new ArgumentNullException("userData");
                 throw new ArgumentNullException("userData");
             }
             }
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
             if (userId == Guid.Empty)
             if (userId == Guid.Empty)
             {
             {
                 throw new ArgumentNullException("userId");
                 throw new ArgumentNullException("userId");

+ 0 - 10
MediaBrowser.Server.Implementations/Persistence/SqliteUserRepository.cs

@@ -98,11 +98,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
                 throw new ArgumentNullException("user");
                 throw new ArgumentNullException("user");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             var serialized = _jsonSerializer.SerializeToBytes(user);
             var serialized = _jsonSerializer.SerializeToBytes(user);
@@ -199,11 +194,6 @@ namespace MediaBrowser.Server.Implementations.Persistence
                 throw new ArgumentNullException("user");
                 throw new ArgumentNullException("user");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);
             await _writeLock.WaitAsync(cancellationToken).ConfigureAwait(false);

+ 0 - 4
MediaBrowser.Server.Implementations/Providers/ProviderManager.cs

@@ -269,10 +269,6 @@ namespace MediaBrowser.Server.Implementations.Providers
             {
             {
                 throw new ArgumentNullException();
                 throw new ArgumentNullException();
             }
             }
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException();
-            }
 
 
             if (cancellationToken.IsCancellationRequested)
             if (cancellationToken.IsCancellationRequested)
             {
             {

+ 0 - 5
MediaBrowser.Server.Implementations/ServerManager/ServerManager.cs

@@ -285,11 +285,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager
                 throw new ArgumentNullException("dataFunction");
                 throw new ArgumentNullException("dataFunction");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             var connectionsList = connections.Where(s => s.State == WebSocketState.Open).ToList();
             var connectionsList = connections.Where(s => s.State == WebSocketState.Open).ToList();

+ 0 - 5
MediaBrowser.Server.Implementations/ServerManager/WebSocketConnection.cs

@@ -216,11 +216,6 @@ namespace MediaBrowser.Server.Implementations.ServerManager
                 throw new ArgumentNullException("buffer");
                 throw new ArgumentNullException("buffer");
             }
             }
 
 
-            if (cancellationToken == null)
-            {
-                throw new ArgumentNullException("cancellationToken");
-            }
-
             cancellationToken.ThrowIfCancellationRequested();
             cancellationToken.ThrowIfCancellationRequested();
 
 
             // Per msdn docs, attempting to send simultaneous messages will result in one failing.
             // Per msdn docs, attempting to send simultaneous messages will result in one failing.

+ 0 - 2
MediaBrowser.Server.Mono/Program.cs

@@ -47,8 +47,6 @@ namespace MediaBrowser.Server.Mono
 
 
 			bool createdNew;
 			bool createdNew;
 
 
-			var runningPath = Process.GetCurrentProcess().MainModule.FileName.Replace(Path.DirectorySeparatorChar.ToString(), string.Empty);
-
 			//_singleInstanceMutex = new Mutex(true, @"Local\" + runningPath, out createdNew);
 			//_singleInstanceMutex = new Mutex(true, @"Local\" + runningPath, out createdNew);
 			createdNew = true;
 			createdNew = true;