From 273cbdf62449e024c138a4f12c9acc928a7c78df Mon Sep 17 00:00:00 2001 From: Donna Wu Date: Tue, 17 Apr 2012 16:30:06 +0800 Subject: [PATCH] [General] Fix issue: same file selection can not trigger Import --- src/js/views/project.js | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/js/views/project.js b/src/js/views/project.js index 858f77e5..dd3d5084 100644 --- a/src/js/views/project.js +++ b/src/js/views/project.js @@ -48,7 +48,8 @@ .attr({id:'importFile'}) .addClass('hidden-accessible') .change(widget._importChangedCallback) - .appendTo(this.element[0].ownerDocument.body); + .appendTo(this.element[0].ownerDocument.body) + .wrap("
"); // Add a project setting dialog element, used to trigger to configure // project setting when user want to create or modify project setting @@ -63,20 +64,23 @@ }, _importChangedCallback: function (e) { - var file; - if (e.currentTarget.files.length === 1) { - file = e.currentTarget.files[0]; + var selected, file; + selected = e.currentTarget.files; + if (selected.length === 1) { + file = selected[0]; $.rib.pmUtils.importProject(file, function () { // show the layout tab $(document.body).tabs('select', 1); }); } else { - if (e.currentTarget.files.length <= 1) { + if (selected.length <= 1) { console.warn("No files specified to import"); } else { console.warn("Multiple file import not supported"); } } + // clear the selected file + e.currentTarget.form.reset(); }, _setOption: function(key, value) { switch (key) {