Add more checks on comment submit

js/etersoft/timer.js

@@ -82,6 +82,9 @@ function getBugIdFromField() {
 //Является ли пользователь сотрудником Etersoft
 function isetersoft() {
 	var email = document.getElementById('useremail').className;
+	if (!email)
+		return false;
+
 	var domain = email.split("@")[1];
 	
 	return (domain === "etersoft.ru");
@@ -136,9 +139,29 @@ function premysubmit(realworktime) {
 		return;
 	}
 
-	// Запрет ставить более 12 часов отработанного времени
-	if ( wt > 12) {
-		alert('Вы указали отработанное время более 12 часов. Рекомендуется разбивать этапы своей работы на более мелкие отрезки времени.');
+	var comment = document.querySelector("#comment");
+
+	// Запрет ставить более 6 часов отработанного времени
+	if ( wt > 6) {
+		alert('Вы указали отработанное время более 6 часов. Нужно обязательно разбивать этапы своей работы на более мелкие отрезки времени.');
+		return false;
+	}
+
+	// Предупреждение
+	if ( wt > 3) {
+        // FIXME: тут нет возможности отмены
+		alert('Вы указали отработанное время более 3 часов. Правильным следованием рабочему процессу было бы выполнение работы по частям, о каждой из которых будет написано отдельно.');
+
+		if (comment.value.length < (wt * 60)) {
+			alert('Недопустимо коротко комментировать длительные работы.');
+			return false;
+		}
+		//comment.value = comment.value + '\n\nДа, я подтверждаю, что намеренно наплевал на предупреждение о том, что для нормального рабочего процесса необходимо вести более короткие записи в багзилле.';
+		//return false;
+	}
+
+	if (comment.value.length < 9) {
+		alert('Комментарий слишком короткий.');
 		return false;
 	}
 	
@@ -147,8 +170,8 @@ function premysubmit(realworktime) {
 		document.querySelector("#changeform #remaining_time").value =  document.querySelector("#timeQuestionDiv #realremaintime").value;
 	}
 
-	mysubmit();
 	timer.clear();
+	mysubmit();
 }
 
 function mysubmit() {
@@ -173,9 +196,13 @@ function mysubmit() {
 }
 
 function mysubmitnew() {
-	if (document.querySelector("#Create #work_time").value > 0 || !isetersoft()) {
+	if (!isetersoft()) {
+		document.Create.submit();
+		return;
+	}
+	if (document.querySelector("#Create #work_time").value > 0) {
+		timer.clear();
 		document.Create.submit();
-		timer.reset();
 		return;
 	}
 	etersoft_create = 1;