minor fixes

iliakan · iliakan · commit b9ba80d145e4 · 2015-05-02T16:20:06.000+03:00
diff --git a/client/polyfill/hidden.js b/client/polyfill/hidden.js
@@ -1,5 +1,5 @@
 if (document.documentElement.hidden === undefined) {
-  document.head.insertAdjacentHTML('<style> [hidden] { display: none } </style>');
+  document.head.insertAdjacentHTML('beforeEnd', '<style> [hidden] { display: none } </style>');
   Object.defineProperty(Element.prototype, "hidden", {
     set: function(value) {
       this.setAttribute('hidden', value);
@@ -8,4 +8,4 @@ if (document.documentElement.hidden === undefined) {
       return this.getAttribute('hidden');
     }
   });
-}
+}
diff --git a/fixture/init/user.js b/fixture/init/user.js
@@ -8,5 +8,11 @@ exports.User = [{
   profileName:   'tester',
   password:      "123456",
   verifiedEmail: true
+}, {
+  email:         "iliakan@javascript.ru",
+  displayName:   "Ilya Kantor",
+  profileName:   'iliakan',
+  password:      "123456",
+  verifiedEmail: true
 }];
 
diff --git a/handlers/auth/client/authModal.js b/handlers/auth/client/authModal.js
@@ -259,7 +259,7 @@ AuthModal.prototype.submitForgotForm = function(form) {
   var request = this.request({
     method:         'POST',
     url:            '/auth/forgot',
-    normalStatuses: [200, 404],
+    normalStatuses: [200, 404, 403],
     body:           payload
   });
 
@@ -271,6 +271,8 @@ AuthModal.prototype.submitForgotForm = function(form) {
       self.showFormMessage(event.result, 'success');
     } else if (this.status == 404) {
       self.showFormMessage(event.result, 'error');
+    } else if (this.status == 403) {
+      self.showFormMessage(event.result.message || "Действие запрещено.", 'error');
     }
   });
 
diff --git a/handlers/auth/controller/forgotRecover.js b/handlers/auth/controller/forgotRecover.js
@@ -39,11 +39,18 @@ exports.post = function* (next) {
     this.throw(404, 'Ваша ссылка на восстановление недействительна или устарела.');
   }
 
+  var error = "";
   if (!this.request.body.password) {
+    error = "Пароль не должен быть пустым.";
+  }
+  if (this.request.body.password.length < 4) {
+    error = "Пароль должен содержать минимум 4 символа.";
+  }
 
+  if (error) {
     this.body = this.render('forgot-recover', {
       passwordResetToken: passwordResetToken,
-      error: "Пароль не должен быть пустым."
+      error: error
     });
 
     return;
diff --git a/handlers/auth/lib/mustNotBeAuthenticated.js b/handlers/auth/lib/mustNotBeAuthenticated.js
@@ -3,6 +3,6 @@ module.exports = function*(next) {
   if (!this.isAuthenticated()) {
     yield* next;
   } else {
-    this.throw(403, 'Это действие доступно только для неавторизованных посетителей');
+    this.throw(403, 'Это действие доступно только для неавторизованных посетителей.');
   }
 };
diff --git a/handlers/courses/lib/sendOrderInvites.js b/handlers/courses/lib/sendOrderInvites.js
@@ -61,11 +61,10 @@ function* createInvites(order) {
 
 function* sendInvites(invites) {
 
-  for (var i = 0; i < invites.length; i++) {
-    var invite = invites[i];
-    log.debug("send invite", invite);
-    yield* sendInvite(invite);
-  }
+  // send invites in parallel, for speed
+  yield invites.map(function(invite) {
+    return sendInvite(invite);
+  });
 
 }
 
diff --git a/handlers/tutorial/templates/frontpage.jade b/handlers/tutorial/templates/frontpage.jade
@@ -2,7 +2,7 @@ extends /layouts/main
 
 block append variables
   - var layout_main_class = "main_width-limit-wide"
-  - var comments = env.NODE_ENV == 'development'
+  - var comments = (env.NODE_ENV == 'development')
   - var layout_header_class = "main__header_center"
 
 block content

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,6 @@ module.exports = function*(next) {`
`3`	`3`	`if (!this.isAuthenticated()) {`
`4`	`4`	`yield* next;`
`5`	`5`	`} else {`
`6`		`- this.throw(403, 'Это действие доступно только для неавторизованных посетителей');`
	`6`	`+ this.throw(403, 'Это действие доступно только для неавторизованных посетителей.');`
`7`	`7`	`}`
`8`	`8`	`};`