Add the eslint space-infix-ops rule, and run npm run lint to fix the

issues it shows.

Author: drgrice1

.eslintrc.js

@@ -19,6 +19,7 @@ const baseRules = {
 	'one-var': 'off',
 	'no-void': 'off',
 	'multiline-ternary': 'off',
+	'space-infix-ops': ['error'],
 
 	// allow console and debugger during development only
 	'no-console': process.env.NODE_ENV === 'development' ? 'off' : 'error',

src/components/common/DateTimeInput.vue

@@ -40,7 +40,7 @@ export default defineComponent({
 	},
 	emits: ['update:modelValue'],
 	setup (props, { emit }) {
-		const date_string = ref(date.formatDate((props.modelValue || Date.now())*1000, 'YYYY-MM-DD HH:mm'));
+		const date_string = ref(date.formatDate((props.modelValue || Date.now()) * 1000, 'YYYY-MM-DD HH:mm'));
 
 		watch(
 			() => date_string.value,

src/components/instructor/ClasslistManagerComponents/AddUsersFromFile.vue

@@ -218,7 +218,7 @@ export default defineComponent({
 						parse_error = {
 							type: 'warn',
 							message: `The user with username '${merged_user.username ?? ''}'`
-								+' is already enrolled in the course.',
+								+ ' is already enrolled in the course.',
 							entire_row: true
 						};
 					} else {
@@ -238,7 +238,7 @@ export default defineComponent({
 					if (err.field === '_all') {
 						assign(parse_error, { entire_row: true });
 					} else if (err.field &&
-						(user_fields.indexOf(err.field)>=0 || course_user_fields.indexOf(err.field)>=0)) {
+						(user_fields.indexOf(err.field) >= 0 || course_user_fields.indexOf(err.field) >= 0)) {
 						assign(parse_error, {
 							col: user_param_map.value[err.field],
 							entire_row: user_param_map.value[err.field] == undefined
@@ -285,7 +285,7 @@ export default defineComponent({
 							};
 							d._row = index;
 							row.forEach((v: string, i: number) => {
-								d[`col${i+1}`] = v;
+								d[`col${i + 1}`] = v;
 							});
 							users.push(d);
 						});

src/components/instructor/ClasslistManagerComponents/AddUsersManually.vue

@@ -96,7 +96,7 @@ export default defineComponent({
 					(_setting: CourseSetting) => _setting.var === 'roles'
 				);
 				const r = clone(all_roles?.value as Array<string>);
-				remove(r, (v)=> v==='admin'); // don't allow to set admin level here.
+				remove(r, (v)=> v === 'admin'); // don't allow to set admin level here.
 				return r;
 			}),
 			addUser: async (close: boolean) => {

src/components/instructor/ClasslistManagerComponents/EditUsers.vue

@@ -83,7 +83,7 @@ export default defineComponent({
 					(_setting: CourseSetting) => _setting.var === 'roles'
 				);
 				const r = clone(all_roles?.value as Array<string>);
-				remove(r, v => v==='admin'); // don't allow to set admin level here.
+				remove(r, v => v === 'admin'); // don't allow to set admin level here.
 				return r;
 			}),
 		};

src/components/instructor/SetDetails/Quiz.vue

@@ -91,7 +91,7 @@ export default defineComponent({
 			checkDates: [
 				() => {
 					const d = set.value.set_dates;
-					return d.open <= d.due && d.due <=d.answer || 'The dates must be in order';
+					return d.open <= d.due && d.due <= d.answer || 'The dates must be in order';
 				}
 			],
 			quizDuration: [

src/layouts/MainLayout.vue

@@ -66,13 +66,13 @@ export default defineComponent({
 
 			if (! current_view) { // it may be a child component
 				current_view = views.find((view: ViewInfo) =>
-				 view.children ? view.children?.indexOf(route.name as string)> -1 : false);
+				 view.children ? view.children?.indexOf(route.name as string) > -1 : false);
 			}
 
 			logger.debug(`[MainLayout/updateViews] name: ${current_view?.name || 'no name!'}`);
 			if (current_view) {
 				sidebars.value = current_view.sidebars;
-				if (current_view.sidebars.length>0) {
+				if (current_view.sidebars.length > 0) {
 					logger.debug(`[MainLayout/updateViews] sidebar: ${current_view.sidebars.join(', ')}`);
 					right_sidebar_open.value = true;
 				} else {

src/store/models/index.ts

@@ -215,7 +215,7 @@ export const Model = <Bool extends string, Num extends string, Str extends strin
 		_dictionary_field_names: Array<Dic> = dictionary_fields;
 		_fields: F = fields;
 
-		constructor(params: Dictionary<generic | Dictionary<generic>>= {}) {
+		constructor(params: Dictionary<generic | Dictionary<generic>> = {}) {
 			// check that required fields are present
 
 			const common_fields = intersection(this.required_fields, Object.keys(params));

src/store/modules/session.ts

@@ -36,7 +36,7 @@ const getters: GetterTree<SessionState, StateInterface> & Getters = {
 	logged_in: (state) => state.logged_in,
 	user: (state) => state.user,
 	full_name: (state) => {
-		return `${state.user.first_name|| ''}  ${state.user.last_name || ''}`;
+		return `${state.user.first_name || ''}  ${state.user.last_name || ''}`;
 	},
 	course: (state) => state.course
 };

src/store/modules/users.ts

@@ -189,12 +189,12 @@ export default {
 			state. merged_users.push(_merged_user);
 		},
 		UPDATE_MERGED_USER(state: UserState, _merged_user: MergedUser): void {
-			const index = state. merged_users.findIndex((u) => u.course_user_id ===_merged_user.course_user_id);
+			const index = state. merged_users.findIndex((u) => u.course_user_id === _merged_user.course_user_id);
 			// splice is used so vue3 reacts to changes.
 			state. merged_users.splice(index, 1, _merged_user);
 		},
 		DELETE_COURSE_USER(state: UserState, _course_user: CourseUser): void {
-			remove(state.course_users, (u) => u.course_user_id ===_course_user.course_user_id);
+			remove(state.course_users, (u) => u.course_user_id === _course_user.course_user_id);
 		},
 		DELETE_MERGED_USER(state: UserState, _merged_user: MergedUser): void {
 			remove(state. merged_users, (u) => u.course_user_id === _merged_user.course_user_id);