remove most console.log & make rest console.error

hpi-schul-cloud · Nov 10, 2023 · 5a8f9cf · 5a8f9cf
1 parent 1b139c0
commit 5a8f9cf
Show file tree

Hide file tree

Showing 13 changed files with 4 additions and 62 deletions.
diff --git a/...nents/feature-board-file-element/content/display/file-description/FileDescription.unit.ts b/...nents/feature-board-file-element/content/display/file-description/FileDescription.unit.ts
@@ -116,7 +116,6 @@ describe("FileDescription", () => {
 						showTitle: true,
 						src,
 					});
-					console.log(wrapper.html());
 					const link = wrapper.find("a");
 
 					expect(link.attributes("href")).toBe(src);
@@ -206,7 +205,6 @@ describe("FileDescription", () => {
 						showTitle: true,
 						src,
 					});
-					console.log(wrapper.html());
 					const link = wrapper.find("a");
 
 					expect(link.attributes("href")).toBe(src);

diff --git a/src/main.ts b/src/main.ts
@@ -126,7 +126,7 @@ Vue.use(VueDOMPurifyHTML, {
 		try {
 			await authModule.login(jwt);
 		} catch (e) {
-			console.log("### JWT invalid: ", e);
+			console.error("### JWT invalid: ", e);
 		}
 	}
 

diff --git a/src/pages/ActivationCode.page.vue b/src/pages/ActivationCode.page.vue
@@ -89,7 +89,7 @@ export default {
 				this.keyword = this.data[0].keyword;
 				this.activated = this.data[0].success;
 			} catch (e) {
-				console.log(e);
+				console.error(e);
 			}
 			this.showModal = true;
 		},

diff --git a/src/pages/NewsCreate.page.vue b/src/pages/NewsCreate.page.vue
@@ -70,7 +70,6 @@ export default {
 					});
 				}
 			} catch (e) {
-				console.error(e);
 				notifierModule.show({
 					text: this.$t("components.organisms.FormNews.errors.create"),
 					status: "error",

diff --git a/src/pages/NewsEdit.page.vue b/src/pages/NewsEdit.page.vue
@@ -79,7 +79,6 @@ export default {
 					path: `/news/${this.news.id}`,
 				});
 			} catch (e) {
-				console.error(e);
 				notifierModule.show({
 					text: this.$t("components.organisms.FormNews.errors.patch"),
 					status: "error",
@@ -97,7 +96,6 @@ export default {
 				});
 				this.$router.push({ name: "news" });
 			} catch (e) {
-				console.error(e);
 				notifierModule.show({
 					text: this.$t("components.organisms.FormNews.errors.remove"),
 					status: "error",

diff --git a/src/pages/administration/StudentOverview.page.vue b/src/pages/administration/StudentOverview.page.vue
@@ -551,7 +551,6 @@ export default {
 					});
 				}
 			} catch (error) {
-				console.error(error);
 				notifierModule.show({
 					text: this.$tc("pages.administration.sendMail.error", rowIds.length),
 					status: "error",

diff --git a/src/plugins/polyfills.js b/src/plugins/polyfills.js
@@ -11,5 +11,3 @@ polyfill({
 	// full-type website, see https://github.com/timruffles/mobile-drag-drop/issues/167 for more
 	forceApply: true,
 });
-
-// console.log("mobileDragAndDrop loaded", polyfill);
diff --git a/src/store/collaborative-files.ts b/src/store/collaborative-files.ts
@@ -106,7 +106,7 @@ export default class CollaborativeFilesModule extends VuexModule {
 			await this.addFileMetaData(data);
 			this.setLoading(false);
 		} catch (error: any) {
-			console.log(error);
+			console.error(error);
 			this.setLoading(false);
 		}
 	}
@@ -141,7 +141,7 @@ export default class CollaborativeFilesModule extends VuexModule {
 			await this.addFileMetaData(data);
 			this.setLoading(false);
 		} catch (error: any) {
-			console.log(error);
+			console.error(error);
 			this.setLoading(false);
 		}
 	}

diff --git a/src/store/context-external-tools.ts b/src/store/context-external-tools.ts
@@ -136,8 +136,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -179,8 +177,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -215,8 +211,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -238,8 +232,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -274,8 +266,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -310,8 +300,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -344,8 +332,6 @@ export default class ContextExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,

diff --git a/src/store/group.ts b/src/store/group.ts
@@ -119,8 +119,6 @@ export default class GroupModule extends VuexModule {
 		} catch (error) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -163,8 +161,6 @@ export default class GroupModule extends VuexModule {
 		} catch (error) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,

diff --git a/src/store/rooms.ts b/src/store/rooms.ts
@@ -213,7 +213,6 @@ export default class RoomsModule extends VuexModule {
 			this.setRoomData(data.gridElements || []);
 			if (indicateLoading) this.setLoading(false);
 		} catch (error: any) {
-			console.log(error);
 			this.setError(error);
 			if (indicateLoading) this.setLoading(false);
 		}

diff --git a/src/store/school-external-tools.ts b/src/store/school-external-tools.ts
@@ -111,8 +111,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -145,8 +143,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -170,8 +166,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -202,8 +196,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -236,8 +228,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -267,8 +257,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -300,8 +288,6 @@ export default class SchoolExternalToolsModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,

diff --git a/src/store/user-login-migrations.ts b/src/store/user-login-migrations.ts
@@ -116,8 +116,6 @@ export default class UserLoginMigrationModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -158,8 +156,6 @@ export default class UserLoginMigrationModule extends VuexModule {
 			} catch (error: unknown) {
 				const apiError = mapAxiosErrorToResponseError(error);
 
-				console.log(apiError);
-
 				this.setBusinessError({
 					error: apiError,
 					statusCode: apiError.code,
@@ -200,8 +196,6 @@ export default class UserLoginMigrationModule extends VuexModule {
 					return;
 				}
 
-				console.log(apiError);
-
 				this.setBusinessError({
 					error: apiError,
 					statusCode: apiError.code,
@@ -232,8 +226,6 @@ export default class UserLoginMigrationModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -264,8 +256,6 @@ export default class UserLoginMigrationModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -294,8 +284,6 @@ export default class UserLoginMigrationModule extends VuexModule {
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,
@@ -316,21 +304,16 @@ export default class UserLoginMigrationModule extends VuexModule {
 		try {
 			const response: AxiosResponse<UserLoginMigrationResponse> =
 				await this.userLoginMigrationApi.userLoginMigrationControllerCloseMigration();
-			console.log(response.data);
 			if (response.data.closedAt) {
 				const userLoginMigration: UserLoginMigration =
 					UserLoginMigrationMapper.mapToUserLoginMigration(response.data);
-				console.log(userLoginMigration);
 				this.setUserLoginMigration(userLoginMigration);
 			} else {
-				console.log(this.userLoginMigration);
 				this.setUserLoginMigration(undefined);
 			}
 		} catch (error: unknown) {
 			const apiError = mapAxiosErrorToResponseError(error);
 
-			console.log(apiError);
-
 			this.setBusinessError({
 				error: apiError,
 				statusCode: apiError.code,