Skip to content

Commit 21f2206

Browse files
craiggwilsoncraiggwilson
committed
CR Changes.
1 parent dd7dcab commit 21f2206

File tree

2 files changed

+27
-29
lines changed

2 files changed

+27
-29
lines changed

‎src/MongoDB.Driver/FieldDefinition.cs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,11 @@
1616
using System;
1717
using System.Linq;
1818
using System.Linq.Expressions;
19-
using System.Text;
2019
using MongoDB.Bson.Serialization;
2120
using MongoDB.Bson.Serialization.Serializers;
2221
using MongoDB.Driver.Core.Misc;
2322
using MongoDB.Driver.Linq.Expressions;
2423
using MongoDB.Driver.Linq.Processors;
25-
using MongoDB.Driver.Linq.Utils;
2624

2725
namespace MongoDB.Driver
2826
{
@@ -180,14 +178,14 @@ public ExpressionFieldDefinition(LambdaExpression expression)
180178
{
181179
_expression = Ensure.IsNotNull(expression, "expression");
182180

183-
if(expression.Parameters.Count != 1)
181+
if (expression.Parameters.Count != 1)
184182
{
185183
throw new ArgumentException("Only a single parameter lambda expression is allowed.", "expression");
186184
}
187-
if(expression.Parameters[0].Type != typeof(TDocument))
185+
if (expression.Parameters[0].Type != typeof(TDocument))
188186
{
189187
var message = string.Format("The lambda expression parameter must be of type {0}.", typeof(TDocument));
190-
throw new ArgumentException(message);
188+
throw new ArgumentException(message, "expression");
191189
}
192190
}
193191

@@ -336,7 +334,7 @@ public static void Resolve<TDocument>(string fieldName, IBsonSerializer<TDocumen
336334
}
337335

338336
// shortcut BsonDocumentSerializer since it is so common
339-
if(serializer.GetType() == typeof(BsonDocumentSerializer))
337+
if (serializer.GetType() == typeof(BsonDocumentSerializer))
340338
{
341339
return;
342340
}
@@ -364,7 +362,7 @@ public static void Resolve<TDocument>(string fieldName, IBsonSerializer<TDocumen
364362
resolvedFieldSerializer = documentSerializer;
365363
for (int i = 0; i < nameParts.Length; i++)
366364
{
367-
if(nameParts[i] == "$" || nameParts[i].All(char.IsDigit))
365+
if (nameParts[i] == "$" || nameParts[i].All(char.IsDigit))
368366
{
369367
arraySerializer = resolvedFieldSerializer as IBsonArraySerializer;
370368
if (arraySerializer != null)
@@ -387,7 +385,7 @@ public static void Resolve<TDocument>(string fieldName, IBsonSerializer<TDocumen
387385
{
388386
serializationInfo = arraySerializer.GetItemSerializationInfo();
389387
documentSerializer = serializationInfo.Serializer as IBsonDocumentSerializer;
390-
if(documentSerializer == null || !documentSerializer.TryGetMemberSerializationInfo(nameParts[i], out serializationInfo))
388+
if (documentSerializer == null || !documentSerializer.TryGetMemberSerializationInfo(nameParts[i], out serializationInfo))
391389
{
392390
resolvedFieldSerializer = null;
393391
break;

‎src/MongoDB.Driver/FilterDefinitionBuilder.cs

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ namespace MongoDB.Driver
3232
public sealed class FilterDefinitionBuilder<TDocument>
3333
{
3434
/// <summary>
35-
/// Creates an all filter.
35+
/// Creates an all filter for an array field.
3636
/// </summary>
3737
/// <typeparam name="TItem">The type of the item.</typeparam>
3838
/// <param name="field">The field.</param>
@@ -44,7 +44,7 @@ public FilterDefinition<TDocument> All<TItem>(FieldDefinition<TDocument> field,
4444
}
4545

4646
/// <summary>
47-
/// Creates an all filter.
47+
/// Creates an all filter for an array field.
4848
/// </summary>
4949
/// <typeparam name="TItem">The type of the item.</typeparam>
5050
/// <param name="field">The field.</param>
@@ -76,7 +76,7 @@ public FilterDefinition<TDocument> And(IEnumerable<FilterDefinition<TDocument>>
7676
}
7777

7878
/// <summary>
79-
/// Creates an equality filter.
79+
/// Creates an equality filter for an array field.
8080
/// </summary>
8181
/// <typeparam name="TItem">The type of the item.</typeparam>
8282
/// <param name="field">The field.</param>
@@ -88,7 +88,7 @@ public FilterDefinition<TDocument> AnyEq<TItem>(FieldDefinition<TDocument> field
8888
}
8989

9090
/// <summary>
91-
/// Creates an equality filter.
91+
/// Creates an equality filter for an array field.
9292
/// </summary>
9393
/// <typeparam name="TItem">The type of the item.</typeparam>
9494
/// <param name="field">The field.</param>
@@ -100,7 +100,7 @@ public FilterDefinition<TDocument> AnyEq<TItem>(Expression<Func<TDocument, IEnum
100100
}
101101

102102
/// <summary>
103-
/// Creates a greater than filter.
103+
/// Creates a greater than filter for an array field.
104104
/// </summary>
105105
/// <typeparam name="TItem">The type of the item.</typeparam>
106106
/// <param name="field">The field.</param>
@@ -115,7 +115,7 @@ public FilterDefinition<TDocument> AnyGt<TItem>(FieldDefinition<TDocument> field
115115
}
116116

117117
/// <summary>
118-
/// Creates a greater than filter.
118+
/// Creates a greater than filter for an array field.
119119
/// </summary>
120120
/// <typeparam name="TItem">The type of the item.</typeparam>
121121
/// <param name="field">The field.</param>
@@ -127,7 +127,7 @@ public FilterDefinition<TDocument> AnyGt<TItem>(Expression<Func<TDocument, IEnum
127127
}
128128

129129
/// <summary>
130-
/// Creates a greater than or equal filter.
130+
/// Creates a greater than or equal filter for an array field.
131131
/// </summary>
132132
/// <typeparam name="TItem">The type of the item.</typeparam>
133133
/// <param name="field">The field.</param>
@@ -142,7 +142,7 @@ public FilterDefinition<TDocument> AnyGte<TItem>(FieldDefinition<TDocument> fiel
142142
}
143143

144144
/// <summary>
145-
/// Creates a greater than or equal filter.
145+
/// Creates a greater than or equal filter for an array field.
146146
/// </summary>
147147
/// <typeparam name="TItem">The type of the item.</typeparam>
148148
/// <param name="field">The field.</param>
@@ -154,7 +154,7 @@ public FilterDefinition<TDocument> AnyGte<TItem>(Expression<Func<TDocument, IEnu
154154
}
155155

156156
/// <summary>
157-
/// Creates a less than filter.
157+
/// Creates a less than filter for an array field.
158158
/// </summary>
159159
/// <typeparam name="TItem">The type of the item.</typeparam>
160160
/// <param name="field">The field.</param>
@@ -169,7 +169,7 @@ public FilterDefinition<TDocument> AnyLt<TItem>(FieldDefinition<TDocument> field
169169
}
170170

171171
/// <summary>
172-
/// Creates a less than filter.
172+
/// Creates a less than filter for an array field.
173173
/// </summary>
174174
/// <typeparam name="TItem">The type of the item.</typeparam>
175175
/// <param name="field">The field.</param>
@@ -181,7 +181,7 @@ public FilterDefinition<TDocument> AnyLt<TItem>(Expression<Func<TDocument, IEnum
181181
}
182182

183183
/// <summary>
184-
/// Creates a less than or equal filter.
184+
/// Creates a less than or equal filter for an array field.
185185
/// </summary>
186186
/// <typeparam name="TItem">The type of the item.</typeparam>
187187
/// <param name="field">The field.</param>
@@ -196,7 +196,7 @@ public FilterDefinition<TDocument> AnyLte<TItem>(FieldDefinition<TDocument> fiel
196196
}
197197

198198
/// <summary>
199-
/// Creates a less than or equal filter.
199+
/// Creates a less than or equal filter for an array field.
200200
/// </summary>
201201
/// <typeparam name="TItem">The type of the item.</typeparam>
202202
/// <param name="field">The field.</param>
@@ -208,7 +208,7 @@ public FilterDefinition<TDocument> AnyLte<TItem>(Expression<Func<TDocument, IEnu
208208
}
209209

210210
/// <summary>
211-
/// Creates an in filter.
211+
/// Creates an in filter for an array field.
212212
/// </summary>
213213
/// <typeparam name="TItem">The type of the item.</typeparam>
214214
/// <param name="field">The field.</param>
@@ -220,7 +220,7 @@ public FilterDefinition<TDocument> AnyIn<TItem>(FieldDefinition<TDocument> field
220220
}
221221

222222
/// <summary>
223-
/// Creates an in filter.
223+
/// Creates an in filter for an array field.
224224
/// </summary>
225225
/// <typeparam name="TItem">The type of the item.</typeparam>
226226
/// <param name="field">The field.</param>
@@ -232,7 +232,7 @@ public FilterDefinition<TDocument> AnyIn<TItem>(Expression<Func<TDocument, IEnum
232232
}
233233

234234
/// <summary>
235-
/// Creates a not equal filter.
235+
/// Creates a not equal filter for an array field.
236236
/// </summary>
237237
/// <typeparam name="TItem">The type of the item.</typeparam>
238238
/// <param name="field">The field.</param>
@@ -247,7 +247,7 @@ public FilterDefinition<TDocument> AnyNe<TItem>(FieldDefinition<TDocument> field
247247
}
248248

249249
/// <summary>
250-
/// Creates a not equal filter.
250+
/// Creates a not equal filter for an array field.
251251
/// </summary>
252252
/// <typeparam name="TItem">The type of the item.</typeparam>
253253
/// <param name="field">The field.</param>
@@ -259,7 +259,7 @@ public FilterDefinition<TDocument> AnyNe<TItem>(Expression<Func<TDocument, IEnum
259259
}
260260

261261
/// <summary>
262-
/// Creates a not in filter.
262+
/// Creates a not in filter for an array field.
263263
/// </summary>
264264
/// <typeparam name="TItem">The type of the item.</typeparam>
265265
/// <param name="field">The field.</param>
@@ -271,7 +271,7 @@ public FilterDefinition<TDocument> AnyNin<TItem>(FieldDefinition<TDocument> fiel
271271
}
272272

273273
/// <summary>
274-
/// Creates a not in filter.
274+
/// Creates a not in filter for an array field.
275275
/// </summary>
276276
/// <typeparam name="TItem">The type of the item.</typeparam>
277277
/// <param name="field">The field.</param>
@@ -283,7 +283,7 @@ public FilterDefinition<TDocument> AnyNin<TItem>(Expression<Func<TDocument, IEnu
283283
}
284284

285285
/// <summary>
286-
/// Creates an element match filter.
286+
/// Creates an element match filter for an array field.
287287
/// </summary>
288288
/// <typeparam name="TItem">The type of the item.</typeparam>
289289
/// <param name="field">The field.</param>
@@ -295,7 +295,7 @@ public FilterDefinition<TDocument> ElemMatch<TItem>(FieldDefinition<TDocument> f
295295
}
296296

297297
/// <summary>
298-
/// Creates an element match filter.
298+
/// Creates an element match filter for an array field.
299299
/// </summary>
300300
/// <typeparam name="TItem">The type of the item.</typeparam>
301301
/// <param name="field">The field.</param>
@@ -307,7 +307,7 @@ public FilterDefinition<TDocument> ElemMatch<TItem>(Expression<Func<TDocument, I
307307
}
308308

309309
/// <summary>
310-
/// Creates an element match filter.
310+
/// Creates an element match filter for an array field.
311311
/// </summary>
312312
/// <typeparam name="TItem">The type of the item.</typeparam>
313313
/// <param name="field">The field.</param>

0 commit comments

Comments
 (0)