Remove console log

Author: jahn96

src/CodeSnippetService.ts

@@ -34,7 +34,6 @@ export class CodeSnippetService {
       this.settingManager.get('snippets').user as JSONArray
     );
 
-    console.log(userSnippets.length);
     if (userSnippets.length !== 0) {
       this.codeSnippetList = userSnippets;
     } else {
@@ -46,8 +45,6 @@ export class CodeSnippetService {
       'snippets',
       (this.codeSnippetList as unknown) as PartialJSONValue
     );
-
-    console.log(this.codeSnippetList);
   }
 
   private convertToICodeSnippetList(snippets: JSONArray): ICodeSnippet[] {
@@ -66,7 +63,6 @@ export class CodeSnippetService {
   }
 
   static getCodeSnippetService(): CodeSnippetService {
-    console.log(this.codeSnippetService);
     return this.codeSnippetService;
   }
 
@@ -80,19 +76,10 @@ export class CodeSnippetService {
     );
   }
 
-  // isValidSnippet(): boolean {
-  //   // check duplicate name
-  //   // check required fields
-
-  // }
-
   async addSnippet(snippet: ICodeSnippet): Promise<boolean> {
     const id = snippet.id;
-    console.log(id);
     this.codeSnippetList.splice(id, 0, snippet);
 
-    console.log(this.codeSnippetList);
-
     const numSnippets = this.codeSnippetList.length;
 
     // update id's of snippets.
@@ -101,9 +88,6 @@ export class CodeSnippetService {
       this.codeSnippetList[i].id += 1;
     }
 
-    console.log(this.codeSnippetList);
-
-    console.log(this.codeSnippetList);
     await this.settingManager
       .set('snippets', (this.codeSnippetList as unknown) as PartialJSONValue)
       .catch((_) => {
@@ -142,13 +126,6 @@ export class CodeSnippetService {
   }
 
   async renameSnippet(oldName: string, newName: string): Promise<boolean> {
-    console.log('renaming');
-    try {
-      this.duplicateNameExists(newName);
-    } catch (e) {
-      return false;
-    }
-
     for (const snippet of this.codeSnippetList) {
       if (snippet.name === oldName) {
         snippet.name = newName;
@@ -163,28 +140,26 @@ export class CodeSnippetService {
     return true;
   }
 
-  duplicateNameExists(newName: string): void {
+  duplicateNameExists(newName: string): boolean {
     for (const snippet of this.codeSnippetList) {
       if (snippet.name.toLowerCase() === newName.toLowerCase()) {
-        throw Error('Duplicate Name of Code Snippet');
+        return true;
       }
     }
+    return false;
   }
 
   async modifyExistingSnippet(
     oldName: string,
     newSnippet: ICodeSnippet
   ): Promise<boolean> {
-    console.log(this.codeSnippetList);
     for (const snippet of this.codeSnippetList) {
       if (snippet.name.toLowerCase() === oldName.toLowerCase()) {
         this.codeSnippetList.splice(snippet.id, 1, newSnippet);
         break;
       }
     }
 
-    console.log(this.codeSnippetList);
-
     await this.settingManager
       .set('snippets', (this.codeSnippetList as unknown) as PartialJSONValue)
       .catch((_) => {
@@ -194,8 +169,6 @@ export class CodeSnippetService {
   }
 
   async moveSnippet(fromIdx: number, toIdx: number): Promise<boolean> {
-    console.log(fromIdx);
-    console.log(toIdx);
     if (toIdx > fromIdx) {
       toIdx = toIdx - 1;
     }

src/CodeSnippetWidget.tsx

@@ -84,17 +84,11 @@ export class CodeSnippetWidget extends ReactWidget {
   }
 
   updateCodeSnippetWidget(): void {
-    console.log('updating code snippets');
     const newSnippets = this.codeSnippetManager.snippets;
-    console.log(newSnippets);
-    console.log(this instanceof CodeSnippetWidget);
-    console.log(this);
-    console.log(this.renderCodeSnippetsSignal);
     this.renderCodeSnippetsSignal.emit(newSnippets);
   }
 
   onAfterShow(msg: Message): void {
-    console.log('onAfterShow');
     this.updateCodeSnippetWidget();
   }
 
@@ -320,7 +314,6 @@ export class CodeSnippetWidget extends ReactWidget {
 
   // move code snippet within code snippet explorer
   private moveCodeSnippet(srcIdx: number, targetIdx: number): void {
-    console.log('move snippet');
     this.codeSnippetManager
       .moveSnippet(srcIdx, targetIdx)
       .then((res: boolean) => {

src/index.ts

@@ -257,14 +257,7 @@ const codeSnippetExtensionSetting: JupyterFrontEndPlugin<void> = {
       .load(CODE_SNIPPET_SETTING_ID)
       .then((settings) => {
         CodeSnippetService.init(settings as Settings);
-
         console.log('JupyterLab extension code-snippets setting is activated!');
-
-        // Remove below after testing
-        const codeSnippetService = CodeSnippetService.getCodeSnippetService();
-        console.log(codeSnippetService);
-
-        // console.log(code)
       })
       .catch((e) => console.log(e));
   },