more temp

aozgaa · aozgaa · commit ecc029fd1c66 · 2016-10-26T14:27:51.000-07:00
diff --git a/src/services/codefixes/changeExtendsToImplementsFix.ts b/src/services/codefixes/changeExtendsToImplementsFix.ts
@@ -11,9 +11,13 @@ namespace ts.codefix {
             if (token.kind === SyntaxKind.Identifier && token.parent.parent.kind === SyntaxKind.HeritageClause) {
                 const children = (<HeritageClause>token.parent.parent).getChildren();
 
-                var hasImplements: boolean = ts.forEach(children, child => child.kind === SyntaxKind.ImplementsKeyword);
-
-                var childInterfaces = children.filter(child => child.kind === SyntaxKind.InterfaceDeclaration)
+                // If there is already an implements keyword, we currently have incorrect behavior.
+                // For now, we suppress the quickfix altogether.
+                /*
+                if(ts.forEach(children, child => child.kind === SyntaxKind.ImplementsKeyword)) {
+                    return undefined;
+                }
+                */
 
                 ts.forEach(children, child => {
                     if (child.kind === SyntaxKind.ExtendsKeyword) {
diff --git a/src/services/codefixes/interfaceFixes.ts b/src/services/codefixes/interfaceFixes.ts
@@ -136,6 +136,7 @@ namespace ts.codefix {
         return getNamedClassMemberDeclarations(classDeclaration).filter(member => getModifierFlags(member) & ModifierFlags.Abstract);
     }
 
+    /*
     function getMembersAndStartPosFromReference(variableDeclaration: VariableDeclaration): { startPos: number, members: string[] } {
         const children = variableDeclaration.getChildren();
         const variableMembers: string[] = [];
@@ -157,6 +158,7 @@ namespace ts.codefix {
 
         return { startPos: startPos, members: variableMembers };
     }
+    */
 
     function getDefaultValue(kind: SyntaxKind): string {
         switch (kind) {

Original file line number	Diff line number	Diff line change
`@@ -136,6 +136,7 @@ namespace ts.codefix {`
`136`	`136`	`return getNamedClassMemberDeclarations(classDeclaration).filter(member => getModifierFlags(member) & ModifierFlags.Abstract);`
`137`	`137`	`}`
`138`	`138`
	`139`	`+ /*`
`139`	`140`	`function getMembersAndStartPosFromReference(variableDeclaration: VariableDeclaration): { startPos: number, members: string[] } {`
`140`	`141`	`const children = variableDeclaration.getChildren();`
`141`	`142`	`const variableMembers: string[] = [];`
`@@ -157,6 +158,7 @@ namespace ts.codefix {`
`157`	`158`
`158`	`159`	`return { startPos: startPos, members: variableMembers };`
`159`	`160`	`}`
	`161`	`+ */`
`160`	`162`
`161`	`163`	`function getDefaultValue(kind: SyntaxKind): string {`
`162`	`164`	`switch (kind) {`