analizators/s3442

ONLYOFFICE · Jan 24, 2022 · fb3a61d · fb3a61d
1 parent 9292ba4
commit fb3a61d
Show file tree

Hide file tree

Showing 28 changed files with 34 additions and 34 deletions.
diff --git a/common/ASC.Api.Core/Routing/FormatRoute.cs b/common/ASC.Api.Core/Routing/FormatRoute.cs
@@ -10,7 +10,7 @@ public abstract class CustomHttpMethodAttribute : HttpMethodAttribute
         public bool Check { get; set; }
         public bool DisableFormat { get; set; }
 
-        public CustomHttpMethodAttribute(string method, string template = null, bool check = true, int order = 1)
+        protected CustomHttpMethodAttribute(string method, string template = null, bool check = true, int order = 1)
             : base(new List<string>() { method }, $"[controller]{(template != null ? $"/{template}" : "")}")
         {
             Check = check;

diff --git a/common/ASC.Common/Collections/HttpRequestDictionary.cs b/common/ASC.Common/Collections/HttpRequestDictionary.cs
@@ -29,7 +29,7 @@ namespace ASC.Collections
 {
     public sealed class HttpRequestDictionary<T> : CachedDictionaryBase<T>
     {
-        private class CachedItem
+        private sealed class CachedItem
         {
             internal T Value { get; set; }
 

diff --git a/common/ASC.Common/DIHelper.cs b/common/ASC.Common/DIHelper.cs
@@ -97,14 +97,14 @@ public abstract class DIAttribute : Attribute
         public Type Service { get; }
         public Type Additional { get; set; }
 
-        public DIAttribute() { }
+        protected DIAttribute() { }
 
-        public DIAttribute(Type service)
+        protected DIAttribute(Type service)
         {
             Service = service;
         }
 
-        public DIAttribute(Type service, Type implementation)
+        protected DIAttribute(Type service, Type implementation)
         {
             Implementation = implementation;
             Service = service;

diff --git a/common/ASC.Core.Common/Notify/Engine/NotifyEngine.cs b/common/ASC.Core.Common/Notify/Engine/NotifyEngine.cs
@@ -575,7 +575,7 @@ private void PrepareRequestFillTags(NotifyRequest request, IServiceScope service
         }
 
 
-        private class SendMethodWrapper
+        private sealed class SendMethodWrapper
         {
             private readonly object locker = new object();
             private readonly CronExpression cronExpression;

diff --git a/common/ASC.Core.Common/Notify/NotifySource.cs b/common/ASC.Core.Common/Notify/NotifySource.cs
@@ -67,7 +67,7 @@ public string ID
         private UserManager UserManager { get; }
         private SubscriptionManager SubscriptionManager { get; }
 
-        public NotifySource(string id, UserManager userManager, IRecipientProvider recipientsProvider, SubscriptionManager subscriptionManager)
+        protected NotifySource(string id, UserManager userManager, IRecipientProvider recipientsProvider, SubscriptionManager subscriptionManager)
         {
             if (string.IsNullOrEmpty(id)) throw new ArgumentNullException("id");
 
@@ -77,7 +77,7 @@ public NotifySource(string id, UserManager userManager, IRecipientProvider recip
             SubscriptionManager = subscriptionManager;
         }
 
-        public NotifySource(Guid id, UserManager userManager, IRecipientProvider recipientsProvider, SubscriptionManager subscriptionManager)
+        protected NotifySource(Guid id, UserManager userManager, IRecipientProvider recipientsProvider, SubscriptionManager subscriptionManager)
             : this(id.ToString(), userManager, recipientsProvider, subscriptionManager)
         {
         }

diff --git a/common/ASC.Core.Common/Notify/Patterns/PatternFormatter.cs b/common/ASC.Core.Common/Notify/Patterns/PatternFormatter.cs
@@ -45,16 +45,16 @@ protected Regex RegEx
         }
 
 
-        public PatternFormatter()
+        protected PatternFormatter()
         {
         }
 
-        public PatternFormatter(string tagSearchRegExp)
+        protected PatternFormatter(string tagSearchRegExp)
             : this(tagSearchRegExp, false)
         {
         }
 
-        internal PatternFormatter(string tagSearchRegExp, bool formatMessage)
+        protected PatternFormatter(string tagSearchRegExp, bool formatMessage)
         {
             if (string.IsNullOrEmpty(tagSearchRegExp)) throw new ArgumentException("tagSearchRegExp");
 

diff --git a/common/ASC.Data.Backup.Core/Service/BackupWorker.cs b/common/ASC.Data.Backup.Core/Service/BackupWorker.cs
@@ -334,7 +334,7 @@ public int TenantId
 
         protected IServiceProvider ServiceProvider { get; set; }
 
-        public BaseBackupProgressItem(IOptionsMonitor<ILog> options, IServiceProvider serviceProvider)
+        protected BaseBackupProgressItem(IOptionsMonitor<ILog> options, IServiceProvider serviceProvider)
         {
             Log = options.CurrentValue;
             ServiceProvider = serviceProvider;

diff --git a/common/ASC.Data.Backup.Core/Tasks/Modules/ModuleSpecificsBase.cs b/common/ASC.Data.Backup.Core/Tasks/Modules/ModuleSpecificsBase.cs
@@ -49,7 +49,7 @@ public virtual string ConnectionStringName
         public abstract IEnumerable<TableInfo> Tables { get; }
         public abstract IEnumerable<RelationInfo> TableRelations { get; }
         private readonly Helpers helpers;
-        public ModuleSpecificsBase(Helpers helpers)
+        protected ModuleSpecificsBase(Helpers helpers)
         {
             this.helpers = helpers;
         }

diff --git a/common/ASC.Data.Storage/BaseStorage.cs b/common/ASC.Data.Storage/BaseStorage.cs
@@ -55,8 +55,8 @@ public abstract class BaseStorage : IDataStore
         protected IHttpContextAccessor HttpContextAccessor { get; }
         protected IOptionsMonitor<ILog> Options { get; }
         protected IHttpClientFactory ClientFactory { get; }
-            
-        public BaseStorage(
+
+        protected BaseStorage(
             TempStream tempStream,
             TenantManager tenantManager,
             PathUtils pathUtils,

diff --git a/common/ASC.Textile/Blocks/PhraseBlockModifier.cs b/common/ASC.Textile/Blocks/PhraseBlockModifier.cs
@@ -61,7 +61,7 @@ protected string PhraseModifierFormat(string input, string modifier, string tag)
             return res;
         }
 
-        private class PhraseModifierMatchEvaluator
+        private sealed class PhraseModifierMatchEvaluator
         {
             readonly string m_tag;
 

diff --git a/common/ASC.Textile/FormatterState.cs b/common/ASC.Textile/FormatterState.cs
@@ -36,7 +36,7 @@ public abstract class FormatterState
         /// Public constructor.
         /// </summary>
         /// <param name="f">The parent formatter.</param>
-        public FormatterState(TextileFormatter formatter)
+        protected FormatterState(TextileFormatter formatter)
         {
             Formatter = formatter;
         }

diff --git a/common/ASC.Textile/States/ListFormatterState.cs b/common/ASC.Textile/States/ListFormatterState.cs
@@ -36,7 +36,7 @@ protected int NestingDepth
             get { return m_tag.Length; }
         }
 
-        public ListFormatterState(TextileFormatter formatter)
+        protected ListFormatterState(TextileFormatter formatter)
             : base(formatter)
         {
         }

diff --git a/common/services/ASC.AuditTrail/AuditEventsRepository.cs b/common/services/ASC.AuditTrail/AuditEventsRepository.cs
@@ -69,7 +69,7 @@ public IEnumerable<AuditEvent> Get(int tenant, DateTime from, DateTime to)
             return Get(tenant, from, to, null);
         }
 
-        private class Query
+        private sealed class Query
         {
             public Core.Common.EF.Model.AuditEvent AuditEvent { get; set; }
             public User User { get; set; }

diff --git a/common/services/ASC.AuditTrail/LoginEventsRepository.cs b/common/services/ASC.AuditTrail/LoginEventsRepository.cs
@@ -61,7 +61,7 @@ public LoginEventsRepository(UserFormatter userFormatter, DbContextManager<Audit
             LazyMessagesContext = new Lazy<MessagesContext>(() => dbMessagesContext.Value);
         }
 
-        private class Query
+        private sealed class Query
         {
             public LoginEvents LoginEvents { get; set; }
             public User User { get; set; }

diff --git a/common/services/ASC.Feed.Aggregator/Modules/FeedModule.cs b/common/services/ASC.Feed.Aggregator/Modules/FeedModule.cs
@@ -49,7 +49,7 @@ protected int Tenant
         protected TenantManager TenantManager { get; }
         protected WebItemSecurity WebItemSecurity { get; }
 
-        public FeedModule(TenantManager tenantManager, WebItemSecurity webItemSecurity)
+        protected FeedModule(TenantManager tenantManager, WebItemSecurity webItemSecurity)
         {
             TenantManager = tenantManager;
             WebItemSecurity = webItemSecurity;

diff --git a/common/services/ASC.TelegramService/Core/Core.cs b/common/services/ASC.TelegramService/Core/Core.cs
@@ -186,7 +186,7 @@ protected ParamParser(Type type)
 
     public abstract class ParamParser<T> : ParamParser
     {
-        public ParamParser() : base(typeof(T)) { }
+        protected ParamParser() : base(typeof(T)) { }
 
         public override abstract object FromString(string arg);
         public override abstract string ToString(object arg);

diff --git a/products/ASC.Files/Core/Core/Entries/FileEntry.cs b/products/ASC.Files/Core/Core/Entries/FileEntry.cs
@@ -46,7 +46,7 @@ protected FileEntry()
 
         }
 
-        public FileEntry(FileHelper fileHelper, Global global)
+        protected FileEntry(FileHelper fileHelper, Global global)
         {
             FileHelper = fileHelper;
             Global = global;

diff --git a/products/ASC.Files/Core/Core/Security/FileSecurity.cs b/products/ASC.Files/Core/Core/Security/FileSecurity.cs
@@ -942,7 +942,7 @@ public List<Guid> GetUserSubjects(Guid userId)
             return result;
         }
 
-        private class SubjectComparer : IComparer<FileShareRecord>
+        private sealed class SubjectComparer : IComparer<FileShareRecord>
         {
             private readonly List<Guid> _subjects;
 

diff --git a/products/ASC.Files/Core/Core/Thirdparty/Box/BoxDaoBase.cs b/products/ASC.Files/Core/Core/Thirdparty/Box/BoxDaoBase.cs
@@ -51,7 +51,7 @@ internal abstract class BoxDaoBase : ThirdPartyProviderDao<BoxProviderInfo>
     {
         protected override string Id { get => "box"; }
 
-        public BoxDaoBase(
+        protected BoxDaoBase(
             IServiceProvider serviceProvider,
             UserManager userManager,
             TenantManager tenantManager,

diff --git a/products/ASC.Files/Core/Core/Thirdparty/Dropbox/DropboxDaoBase.cs b/products/ASC.Files/Core/Core/Thirdparty/Dropbox/DropboxDaoBase.cs
@@ -51,7 +51,7 @@ internal abstract class DropboxDaoBase : ThirdPartyProviderDao<DropboxProviderIn
     {
         protected override string Id { get => "dropbox"; }
 
-        public DropboxDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath) 
+        protected DropboxDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath) 
             : base(serviceProvider, userManager, tenantManager, tenantUtil, dbContextManager, setupInfo, monitor, fileUtility, tempPath)
         {
         }

diff --git a/products/ASC.Files/Core/Core/Thirdparty/GoogleDrive/GoogleDriveDaoBase.cs b/products/ASC.Files/Core/Core/Thirdparty/GoogleDrive/GoogleDriveDaoBase.cs
@@ -52,7 +52,7 @@ internal abstract class GoogleDriveDaoBase : ThirdPartyProviderDao<GoogleDrivePr
     {
         protected override string Id { get => "drive"; }
 
-        public GoogleDriveDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath)
+        protected GoogleDriveDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath)
             : base(serviceProvider, userManager, tenantManager, tenantUtil, dbContextManager, setupInfo, monitor, fileUtility, tempPath)
         {
         }

diff --git a/products/ASC.Files/Core/Core/Thirdparty/IThirdPartyProviderDao.cs b/products/ASC.Files/Core/Core/Thirdparty/IThirdPartyProviderDao.cs
@@ -229,7 +229,7 @@ internal abstract class ThirdPartyProviderDao<T> : ThirdPartyProviderDao, IDispo
 
         protected abstract string Id { get; }
 
-        public ThirdPartyProviderDao(
+        protected ThirdPartyProviderDao(
             IServiceProvider serviceProvider,
             UserManager userManager,
             TenantManager tenantManager,

diff --git a/products/ASC.Files/Core/Core/Thirdparty/OneDrive/OneDriveDaoBase.cs b/products/ASC.Files/Core/Core/Thirdparty/OneDrive/OneDriveDaoBase.cs
@@ -49,7 +49,7 @@ internal abstract class OneDriveDaoBase : ThirdPartyProviderDao<OneDriveProvider
     {
         protected override string Id { get => "onedrive"; }
 
-        public OneDriveDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath) 
+        protected OneDriveDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath) 
             : base(serviceProvider, userManager, tenantManager, tenantUtil, dbContextManager, setupInfo, monitor, fileUtility, tempPath)
         {
         }

diff --git a/products/ASC.Files/Core/Core/Thirdparty/Sharpbox/SharpBoxDaoBase.cs b/products/ASC.Files/Core/Core/Thirdparty/Sharpbox/SharpBoxDaoBase.cs
@@ -53,7 +53,7 @@ internal abstract class SharpBoxDaoBase : ThirdPartyProviderDao<SharpBoxProvider
     {
         protected override string Id { get => "sbox"; }
 
-        public SharpBoxDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath)
+        protected SharpBoxDaoBase(IServiceProvider serviceProvider, UserManager userManager, TenantManager tenantManager, TenantUtil tenantUtil, DbContextManager<FilesDbContext> dbContextManager, SetupInfo setupInfo, IOptionsMonitor<ILog> monitor, FileUtility fileUtility, TempPath tempPath)
             : base(serviceProvider, userManager, tenantManager, tenantUtil, dbContextManager, setupInfo, monitor, fileUtility, tempPath)
         {
         }

diff --git a/products/ASC.Files/Core/Services/WCFService/FileOperations/FileOperation.cs b/products/ASC.Files/Core/Services/WCFService/FileOperations/FileOperation.cs
@@ -79,7 +79,7 @@ public abstract class FileOperation : DistributedTask
 
         protected DistributedTask TaskInfo { get; set; }
 
-        public FileOperation(IServiceProvider serviceProvider)
+        protected FileOperation(IServiceProvider serviceProvider)
         {
             principal = serviceProvider.GetService<Microsoft.AspNetCore.Http.IHttpContextAccessor>()?.HttpContext?.User ?? Thread.CurrentPrincipal;
             culture = Thread.CurrentThread.CurrentCulture.Name;

diff --git a/web/ASC.Web.Core/Calendars/BaseCalendar.cs b/web/ASC.Web.Core/Calendars/BaseCalendar.cs
@@ -35,7 +35,7 @@ namespace ASC.Web.Core.Calendars
 {
     public abstract class BaseCalendar : ICalendar, ICloneable
     {
-        public BaseCalendar(AuthContext authContext, TimeZoneConverter timeZoneConverter)
+        protected BaseCalendar(AuthContext authContext, TimeZoneConverter timeZoneConverter)
         {
             this.Context = new CalendarContext();
             this.SharingOptions = new SharingOptions();

diff --git a/web/ASC.Web.Core/Calendars/BaseEvent.cs b/web/ASC.Web.Core/Calendars/BaseEvent.cs
@@ -33,7 +33,7 @@ public abstract class BaseEvent : IEvent, ICloneable
     {
         public virtual TimeZoneInfo TimeZone { get; set; }
 
-        public BaseEvent()
+        protected BaseEvent()
         {
             this.Context = new EventContext();
             this.AlertType = EventAlertType.Never;

diff --git a/web/ASC.Web.Core/Calendars/BaseTodo.cs b/web/ASC.Web.Core/Calendars/BaseTodo.cs
@@ -33,7 +33,7 @@ public abstract class BaseTodo : ITodo, ICloneable
     {
         internal TimeZoneInfo TimeZone { get; set; }
 
-        public BaseTodo()
+        protected BaseTodo()
         {
             this.Context = new TodoContext();
         }