Skip to content

Commit 68fcd2d

Browse files
Merge branch 'WordPress:trunk' into feature/prefetch-lightbox-images
2 parents 48be441 + 41c80e8 commit 68fcd2d

File tree

4 files changed

+39
-15
lines changed

4 files changed

+39
-15
lines changed

docs/reference-guides/data/data-core.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ _Parameters_
150150

151151
_Returns_
152152

153-
- `undefined< 'edit' >`: Current user object.
153+
- `ET.User< 'edit' >`: Current user object.
154154

155155
### getDefaultTemplateId
156156

@@ -178,7 +178,7 @@ _Parameters_
178178

179179
_Returns_
180180

181-
- `undefined< EntityRecord > | false`: The entity record, merged with its edits.
181+
- `ET.Updatable< EntityRecord > | false`: The entity record, merged with its edits.
182182

183183
### getEmbedPreview
184184

@@ -504,7 +504,7 @@ _Parameters_
504504

505505
_Returns_
506506

507-
- `undefined< 'edit' >[]`: Users list.
507+
- `ET.User< 'edit' >[]`: Users list.
508508

509509
### hasEditsForEntityRecord
510510

packages/core-data/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -471,7 +471,7 @@ _Parameters_
471471

472472
_Returns_
473473

474-
- `undefined< 'edit' >`: Current user object.
474+
- `ET.User< 'edit' >`: Current user object.
475475

476476
### getDefaultTemplateId
477477

@@ -499,7 +499,7 @@ _Parameters_
499499

500500
_Returns_
501501

502-
- `undefined< EntityRecord > | false`: The entity record, merged with its edits.
502+
- `ET.Updatable< EntityRecord > | false`: The entity record, merged with its edits.
503503

504504
### getEmbedPreview
505505

@@ -825,7 +825,7 @@ _Parameters_
825825

826826
_Returns_
827827

828-
- `undefined< 'edit' >[]`: Users list.
828+
- `ET.User< 'edit' >[]`: Users list.
829829

830830
### hasEditsForEntityRecord
831831

packages/docgen/lib/get-type-annotation.js

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -237,16 +237,20 @@ function getMappedTypeAnnotation( typeAnnotation ) {
237237
* @param {babelTypes.TSTypeReference} typeAnnotation
238238
*/
239239
function getTypeReferenceTypeAnnotation( typeAnnotation ) {
240-
if ( ! typeAnnotation.typeParameters ) {
241-
if ( babelTypes.isTSQualifiedName( typeAnnotation.typeName ) ) {
242-
return unifyQualifiedName( typeAnnotation.typeName );
243-
}
244-
return typeAnnotation.typeName.name;
240+
let typeName;
241+
if ( babelTypes.isTSQualifiedName( typeAnnotation.typeName ) ) {
242+
typeName = unifyQualifiedName( typeAnnotation.typeName );
243+
} else {
244+
typeName = typeAnnotation.typeName.name;
245245
}
246-
const typeParams = typeAnnotation.typeParameters.params
247-
.map( getTypeAnnotation )
248-
.join( ', ' );
249-
return `${ typeAnnotation.typeName.name }< ${ typeParams } >`;
246+
247+
if ( typeAnnotation.typeParameters ) {
248+
const typeParams = typeAnnotation.typeParameters.params
249+
.map( getTypeAnnotation )
250+
.join( ', ' );
251+
typeName = `${ typeName }< ${ typeParams } >`;
252+
}
253+
return typeName;
250254
}
251255

252256
/**

packages/docgen/test/get-type-annotation.js

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,26 @@ describe( 'Type annotations', () => {
100100
} );
101101
} );
102102

103+
describe( 'qualified types', () => {
104+
const node = parse( `
105+
function fn( foo: My.Foo< string >, bar: My.Bar ) {
106+
return 0;
107+
}
108+
` );
109+
110+
it( 'should get the qualified param type with type parameters', () => {
111+
expect(
112+
getTypeAnnotation( { tag: 'param', name: 'foo' }, node, 0 )
113+
).toBe( 'My.Foo< string >' );
114+
} );
115+
116+
it( 'should get the qualified param type without type parameters', () => {
117+
expect(
118+
getTypeAnnotation( { tag: 'param', name: 'bar' }, node, 1 )
119+
).toBe( 'My.Bar' );
120+
} );
121+
} );
122+
103123
describe( 'literal values', () => {
104124
it.each( [ "'a-string-literal'", '1000n', 'true', '1000' ] )(
105125
'should handle %s',

0 commit comments

Comments
 (0)