Skip to content

Commit 8e0985e

Browse files
committed
Stylecop warnings fixed
1 parent bd70950 commit 8e0985e

File tree

6 files changed

+33
-16
lines changed

6 files changed

+33
-16
lines changed

src/JQDT.MVC/ApplicationMvc.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ internal class ApplicationMvc<T> : ApplicationBase<T>
2020
/// </summary>
2121
/// <param name="filterContext">The filter context.</param>
2222
/// <param name="serviceLocator">The service locator.</param>
23+
/// <param name="modelBinder">The model binder.</param>
2324
public ApplicationMvc(ActionExecutedContext filterContext, DI.IServiceLocator serviceLocator, IFormModelBinder modelBinder)
2425
: base(serviceLocator, modelBinder)
2526
{

src/JQDT.WebAPI/ApplicationWebApi.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@ internal class ApplicationWebApi<T> : ApplicationBase<T>
2222
/// Initializes a new instance of the <see cref="ApplicationWebApi{T}"/> class.
2323
/// </summary>
2424
/// <param name="actionExecutedContext">The action executed context.</param>
25-
/// <param name="serviceLocator">The dependency resolver.</param>
25+
/// <param name="serviceLocator">The service locator.</param>
26+
/// <param name="formModelBinder">The form model binder.</param>
2627
public ApplicationWebApi(HttpActionExecutedContext actionExecutedContext, IServiceLocator serviceLocator, IFormModelBinder formModelBinder)
2728
: base(serviceLocator, formModelBinder)
2829
{

src/JQDT/Application/ApplicationBase.cs

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,37 +23,41 @@ public abstract class ApplicationBase<T> : IApplicationBase
2323
/// <summary>
2424
/// Initializes a new instance of the <see cref="ApplicationBase{T}"/> class.
2525
/// </summary>
26-
/// <param name="sreviceLocator">The service locator.</param>
27-
public ApplicationBase(IServiceLocator sreviceLocator, IFormModelBinder modelBinder)
26+
/// <param name="serviceLocator">The service locator.</param>
27+
/// <param name="modelBinder">The model binder.</param>
28+
public ApplicationBase(IServiceLocator serviceLocator, IFormModelBinder modelBinder)
2829
{
29-
this.serviceLocator = sreviceLocator;
30+
this.serviceLocator = serviceLocator;
3031
this.modelBinder = modelBinder;
3132
}
3233

34+
/// <summary>
35+
/// Occurs before the data is processed.
36+
/// </summary>
3337
public event DataProcessorEventHandler OnDataProcessingEvent = delegate { };
3438

3539
/// <summary>
36-
/// Occurs when [on data processed].
40+
/// Occurs when after the data is processed.
3741
/// </summary>
3842
public event DataProcessorEventHandler OnDataProcessedEvent = delegate { };
3943

4044
/// <summary>
41-
/// Occurs when [on search data processing event].
45+
/// Occurs before search data processing.
4246
/// </summary>
4347
public event DataProcessorEventHandler OnSearchDataProcessingEvent = delegate { };
4448

4549
/// <summary>
46-
/// Occurs when [on search data processed event].
50+
/// Occurs after search data processing.
4751
/// </summary>
4852
public event DataProcessorEventHandler OnSearchDataProcessedEvent = delegate { };
4953

5054
/// <summary>
51-
/// Occurs when [on custom filters data processing event].
55+
/// Occurs before custom filters data processing.
5256
/// </summary>
5357
public event DataProcessorEventHandler OnCustomFiltersDataProcessingEvent = delegate { };
5458

5559
/// <summary>
56-
/// Occurs when [on custom filters data processed event].
60+
/// Occurs after custom filters data processing.
5761
/// </summary>
5862
public event DataProcessorEventHandler OnCustomFiltersDataProcessedEvent = delegate { };
5963

@@ -100,7 +104,7 @@ public ResultModel Execute()
100104
{
101105
var ajaxForm = this.GetAjaxForm();
102106
var data = this.GetData();
103-
var requestModel = modelBinder.BindModel(ajaxForm, data);
107+
var requestModel = this.modelBinder.BindModel(ajaxForm, data);
104108

105109
var dataProcessChain = this.GetDataProcessChain(requestModel.Helpers.DataCollectionType);
106110

@@ -142,6 +146,7 @@ public ResultModel Execute()
142146
/// </summary>
143147
/// <param name="data">The data.</param>
144148
/// <param name="requestInfoModel">The request information model.</param>
149+
/// <param name="eventHandler">The event handler.</param>
145150
private void PerformDataProcessorEventHandler(ref IQueryable<T> data, RequestInfoModel requestInfoModel, DataProcessorEventHandler eventHandler)
146151
{
147152
var dataAsObj = (object)data;

src/JQDT/DI/IServiceLocator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public interface IServiceLocator
4747
/// <summary>
4848
/// Gets the form model binder.
4949
/// </summary>
50-
/// <returns></returns>
50+
/// <returns>A new instance of <see cref="IFormModelBinder"/></returns>
5151
IFormModelBinder GetFormModelBinder();
5252
}
5353
}

src/JQDT/DI/ServiceLocator.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ public IDataProcess<T> GetPagingDataProcessor<T>()
8888
/// <summary>
8989
/// Gets the form model binder.
9090
/// </summary>
91-
/// <returns></returns>
91+
/// <returns>A new instance of <see cref="IFormModelBinder"/></returns>
9292
public IFormModelBinder GetFormModelBinder()
9393
{
9494
return new FormModelBinder();
Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,20 @@
1-
using System.Collections.Specialized;
2-
using JQDT.Models;
3-
4-
namespace JQDT.ModelBinders
1+
namespace JQDT.ModelBinders
52
{
3+
using System.Collections.Specialized;
4+
using JQDT.Models;
5+
6+
/// <summary>
7+
/// Form data model binder
8+
/// </summary>
69
public interface IFormModelBinder
710
{
11+
/// <summary>
12+
/// Binds the model.
13+
/// </summary>
14+
/// <typeparam name="T">The type of the data collection.</typeparam>
15+
/// <param name="ajaxForm">The ajax form.</param>
16+
/// <param name="data">The data.</param>
17+
/// <returns><see cref="RequestInfoModel"/></returns>
818
RequestInfoModel BindModel<T>(NameValueCollection ajaxForm, T data);
919
}
1020
}

0 commit comments

Comments
 (0)