Skip to content

Commit

Permalink
fix: Missing namespace members when ns is created by re-exporting an …
Browse files Browse the repository at this point in the history
…entire module

Resolves #1499
  • Loading branch information
Gerrit0 committed Feb 6, 2021
1 parent ccd519a commit 7dfadcf
Showing 1 changed file with 6 additions and 14 deletions.
20 changes: 6 additions & 14 deletions src/lib/converter/symbols.ts
Expand Up @@ -22,18 +22,6 @@ import { convertIndexSignature } from "./factories/index-signature";
import { createSignature } from "./factories/signature";
import { convertJsDocAlias, convertJsDocCallback } from "./jsdoc";

function getSymbolExportsWithFlag(symbol: ts.Symbol, flag: ts.SymbolFlags) {
const childSymbols: ts.Symbol[] = [];

symbol.exports?.forEach((child) => {
if (child.flags & flag) {
childSymbols.push(child);
}
});

return childSymbols;
}

const symbolConverters: {
[K in ts.SymbolFlags]?: (
context: Context,
Expand Down Expand Up @@ -147,7 +135,9 @@ function convertEnum(

convertSymbols(
context.withScope(reflection),
getSymbolExportsWithFlag(symbol, ts.SymbolFlags.EnumMember)
context.checker
.getExportsOfModule(symbol)
.filter((s) => s.flags & ts.SymbolFlags.EnumMember)
);
}

Expand Down Expand Up @@ -198,7 +188,9 @@ function convertNamespace(

convertSymbols(
context.withScope(reflection),
getSymbolExportsWithFlag(symbol, exportFlags)
context.checker
.getExportsOfModule(symbol)
.filter((s) => s.flags & exportFlags)
);
}

Expand Down

0 comments on commit 7dfadcf

Please sign in to comment.