diff --git a/mail_forward/static/src/js/mail_forward.js b/mail_forward/static/src/js/mail_forward.js
index b71f214a..eb2eb81c 100644
--- a/mail_forward/static/src/js/mail_forward.js
+++ b/mail_forward/static/src/js/mail_forward.js
@@ -2,6 +2,7 @@
  * License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
  */
 
+"use strict";
 openerp.mail_forward = function (instance) {
     var _t = instance.web._t;
     instance.mail.ThreadMessage.include({
@@ -26,7 +27,7 @@ openerp.mail_forward = function (instance) {
 
             // Get only ID from the attachments
             var attachment_ids = [];
-            for (n in this.attachment_ids) {
+            for (var n in this.attachment_ids) {
                 attachment_ids.push(this.attachment_ids[n].id);
             }