Fixed issue where nested object literals were getting skipped with --…

…excludeNotExported

src/lib/converter/factories/declaration.ts

@@ -56,6 +56,10 @@ export function createDeclaration(context: Context, node: ts.Declaration, kind:
     let isExported: boolean;
     if (container.kindOf([ReflectionKind.Module, ReflectionKind.ExternalModule])) {
         isExported = false; // Don't inherit exported state in modules and namespaces
+    } else if (container.kindOf([ReflectionKind.TypeLiteral]) && context.converter.excludeNotExported) {
+        // If the container for this is a type literal, it never had the
+        // isExported flag set on it.  See https://github.com/TypeStrong/typedoc/issues/953
+        isExported = true;
     } else {
         isExported = container.flags.isExported;
     }
@@ -108,7 +112,7 @@ export function createDeclaration(context: Context, node: ts.Declaration, kind:
         child.setFlag(ReflectionFlag.Static, isStatic);
         child.setFlag(ReflectionFlag.Private, isPrivate);
         child.setFlag(ReflectionFlag.ConstructorProperty, isConstructorProperty);
-        child.setFlag(ReflectionFlag.Exported,  isExported);
+        child.setFlag(ReflectionFlag.Exported, isExported);
         child = setupDeclaration(context, child, node);
 
         if (child) {

src/test/converter.test.ts

@@ -156,6 +156,7 @@ describe('Converter with excludeNotExported=true', function() {
     const base = Path.join(__dirname, 'converter');
     const exportWithLocalDir = Path.join(base, 'export-with-local');
     const classDir = Path.join(base, 'class');
+    const literalsDir = Path.join(base, 'export-literals');
     let app: Application;
 
     before('constructs', function() {
@@ -204,4 +205,19 @@ describe('Converter with excludeNotExported=true', function() {
         });
     });
 
+    describe('export-literals', () => {
+        it('converts fixtures', function() {
+            resetReflectionID();
+            result = app.convert(app.expandInputFiles([literalsDir]));
+            Assert(result instanceof ProjectReflection, 'No reflection returned');
+        });
+
+        it('matches specs', function() {
+            const specs = JSON.parse(FS.readFileSync(Path.join(literalsDir, 'specs-without-exported.json')).toString());
+            let data = JSON.stringify(result!.toObject(), null, '  ');
+            data = data.split(normalizePath(base)).join('%BASE%');
+
+            compareReflections(JSON.parse(data), specs);
+        });
+    });
 });

src/test/converter/export-literals/export-literals.ts

@@ -0,0 +1,20 @@
+export interface INestedInterface
+{
+    nestedOptional?: {
+        innerMember: string;
+    };
+
+    nested: {
+        isIncluded: boolean;
+    };
+}
+
+export function func(param: {nested: string}): boolean {
+    return param.nested === 'yes';
+}
+
+export function createSomething() {
+    return {
+        foo: 'bar'
+    };
+}