Skip to content

Constructor Visibility #6885

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mhegazy merged 18 commits into from
Feb 17, 2016
Merged

Constructor Visibility #6885

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
f869b41
Added private and protected modifiers to constructors
AbubakerB Feb 3, 2016
c351ffc
Added and updated tests for constructor visibility
AbubakerB Feb 3, 2016
bbf92ce
Added constructor visibility in the declaration emitter
AbubakerB Feb 4, 2016
3d529a8
Update baselines
AbubakerB Feb 4, 2016
4200051
Addressed PR feedback.
AbubakerB Feb 4, 2016
4c93eb2
Accept baselines
AbubakerB Feb 4, 2016
5ce0202
Refactored visibility text on modifier grammar check
AbubakerB Feb 4, 2016
16b54e0
Addressed PR. Diagnostic messages
AbubakerB Feb 4, 2016
037b657
Accept baselines
AbubakerB Feb 4, 2016
a6a5a22
Addressed PR
AbubakerB Feb 5, 2016
d0843e1
Merge branch 'ts-master' into constructorAccessibility
AbubakerB Feb 12, 2016
8ef9599
Allow extending and instantiating a private or protected class within…
AbubakerB Feb 13, 2016
ec7e80e
Added tests and accept baselines
AbubakerB Feb 13, 2016
91ea83c
Addressed PR
AbubakerB Feb 13, 2016
208830e
Updated tests and accepted baseline
AbubakerB Feb 13, 2016
2d7a0f4
Merge branch 'master' into constructorAccessibility
AbubakerB Feb 14, 2016
ba8b168
Included previously ignored baseline .js file and slight refactoring
AbubakerB Feb 14, 2016
de23e2f
Merge branch 'master' into constructorAccessibility
AbubakerB Feb 16, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Addressed PR. Diagnostic messages
  • Loading branch information
@AbubakerB
AbubakerB committed Feb 4, 2016
commit 16b54e0d9fced4351fa8f2431150be29ddd5bfcc
6 changes: 3 additions & 3 deletions src/compiler/checker.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10182,10 +10182,10 @@ namespace ts {
// A private or protected constructor can only be instantiated within it's own class
if (declaringClass !== enclosingClass) {
Copy link
Copy Markdown
Contributor

@mhegazy mhegazy Feb 12, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as noted in , we #7059, if here is not sufficient, it should be a while loop walking up, looking at all enclosing classes, so that something like this should be legal:

class B {
    private constructor() { }

    method() {
        class C {
            method() {
                new B(); // should be fine
            }
        }
    }
}

Copy link
Copy Markdown
Contributor Author

@AbubakerB AbubakerB Feb 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

if (flags & NodeFlags.Private) {
error(node, Diagnostics.Constructor_of_type_0_is_private_and_only_accessible_within_class_1, signatureToString(signature), typeToString(declaringClass));
error(node, Diagnostics.Constructor_of_class_0_is_private_and_only_accessible_within_the_class_declaration, typeToString(declaringClass));
}
if (flags & NodeFlags.Protected) {
error(node, Diagnostics.Constructor_of_type_0_is_protected_and_only_accessible_within_class_1, signatureToString(signature), typeToString(declaringClass));
error(node, Diagnostics.Constructor_of_class_0_is_protected_and_only_accessible_within_the_class_declaration, typeToString(declaringClass));
}
return false;
}
Expand Down Expand Up @@ -14059,7 +14059,7 @@ namespace ts {
if (signatures.length) {
const declaration = signatures[0].declaration;
if (declaration && declaration.flags & NodeFlags.Private) {
error(node, Diagnostics.Cannot_extend_private_class_0, (<Identifier>node.expression).text);
error(node, Diagnostics.Cannot_extend_a_class_0_Class_constructor_is_marked_as_private, (<Identifier>node.expression).text);
}
}
}
Expand Down
6 changes: 3 additions & 3 deletions src/compiler/diagnosticMessages.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1827,15 +1827,15 @@
"category": "Error",
"code": 2672
},
"Constructor of type '{0}' is private and only accessible within class '{1}'.": {
"Constructor of class '{0}' is private and only accessible within the class declaration.": {
"category": "Error",
"code": 2673
},
"Constructor of type '{0}' is protected and only accessible within class '{1}'.": {
"Constructor of class '{0}' is protected and only accessible within the class declaration.": {
"category": "Error",
"code": 2674
},
"Cannot extend private class '{0}'.": {
"Cannot extend a class '{0}'. Class constructor is marked as private.": {
"category": "Error",
"code": 2675
},
Expand Down