Skip to content

Commit a99c427

Browse files
authored
Standardize on Element.name everywhere (#170)
1 parent a83e3d7 commit a99c427

File tree

4 files changed

+7
-9
lines changed

4 files changed

+7
-9
lines changed

json_serializable/lib/src/generator_helper.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ class _GeneratorHelper {
135135

136136
Set<String> fieldsSetByFactory;
137137
if (_generator.checked) {
138-
var classLiteral = escapeDartString(_element.displayName);
138+
var classLiteral = escapeDartString(_element.name);
139139

140140
_buffer.writeln('\$checkedNew($classLiteral, json, ()');
141141

json_serializable/lib/src/json_key_helpers.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import 'package:source_gen/source_gen.dart';
1313
@alwaysThrows
1414
T _throwUnsupported<T>(FieldElement element, String message) =>
1515
throw new InvalidGenerationSourceError(
16-
'Error with `@JsonKey` on `${element.displayName}`. $message',
16+
'Error with `@JsonKey` on `${element.name}`. $message',
1717
element: element);
1818

1919
final _jsonKeyExpando = new Expando<JsonKeyWithConversion>();

json_serializable/lib/src/json_literal_generator.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class JsonLiteralGenerator extends GeneratorForAnnotation<JsonLiteral> {
3535
var thing = _jsonLiteralAsDart(content, asConst).toString();
3636
var marked = asConst ? 'const' : 'final';
3737

38-
return '$marked _\$${element.displayName}JsonLiteral = $thing;';
38+
return '$marked _\$${element.name}JsonLiteral = $thing;';
3939
}
4040
}
4141

json_serializable/lib/src/utils.dart

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,7 @@ Set<FieldElement> createSortedFieldSet(ClassElement element) {
127127

128128
var undefinedFields = fieldsList.where((fe) => fe.type.isUndefined).toList();
129129
if (undefinedFields.isNotEmpty) {
130-
var description =
131-
undefinedFields.map((fe) => '`${fe.displayName}`').join(', ');
130+
var description = undefinedFields.map((fe) => '`${fe.name}`').join(', ');
132131

133132
throw new InvalidGenerationSourceError(
134133
'At least one field has an invalid type: $description.',
@@ -194,7 +193,7 @@ CtorData writeConstructorInvocation(
194193
Map<String, String> unavailableReasons,
195194
String deserializeForField(String paramOrFieldName,
196195
{ParameterElement ctorParam})) {
197-
var className = classElement.displayName;
196+
var className = classElement.name;
198197

199198
var ctor = classElement.unnamedConstructor;
200199
if (ctor == null) {
@@ -212,7 +211,7 @@ CtorData writeConstructorInvocation(
212211
// ignore: deprecated_member_use
213212
if (arg.parameterKind == ParameterKind.REQUIRED) {
214213
var msg = 'Cannot populate the required constructor '
215-
'argument: ${arg.displayName}.';
214+
'argument: ${arg.name}.';
216215

217216
var additionalInfo = unavailableReasons[arg.name];
218217

@@ -277,8 +276,7 @@ void _validateConstructorArguments(
277276
var undefinedArgs =
278277
constructorArguments.where((pe) => pe.type.isUndefined).toList();
279278
if (undefinedArgs.isNotEmpty) {
280-
var description =
281-
undefinedArgs.map((fe) => '`${fe.displayName}`').join(', ');
279+
var description = undefinedArgs.map((fe) => '`${fe.name}`').join(', ');
282280

283281
throw new InvalidGenerationSourceError(
284282
'At least one constructor argument has an invalid type: $description.',

0 commit comments

Comments
 (0)