diff --git a/app/views/attachments/upload_attachment_version.js.erb b/app/views/attachments/upload_attachment_version.js.erb
index 5c74d70cf..7803d73cd 100644
--- a/app/views/attachments/upload_attachment_version.js.erb
+++ b/app/views/attachments/upload_attachment_version.js.erb
@@ -1,7 +1,7 @@
 <% if @flag %>
     hideModal();
-    alert('更新成功')
+    alert('更新成功');
     $(".re_search").submit(); // 为了刷新
 <%else%>
-    $("#upload_file_count").html('(更新失败)')
+    $("#upload_file_count").html('(更新失败)');
 <%end %>
\ No newline at end of file
diff --git a/public/javascripts/application.js b/public/javascripts/application.js
index bc9f86880..05e6457cc 100644
--- a/public/javascripts/application.js
+++ b/public/javascripts/application.js
@@ -426,13 +426,15 @@ function hideModal(el) {
   if($("#popupWrap").length > 0){
       $("#popupWrap").hide();
   }
-  var modal;
-  if (el) {
-    modal = $(el).parents('.ui-dialog-content');
-  } else {
-    modal = $('#ajax-modal');
+  else{
+      var modal;
+      if (el) {
+          modal = $(el).parents('.ui-dialog-content');
+      } else {
+          modal = $('#ajax-modal');
+      }
+      modal.dialog("close");
   }
-  modal.dialog("close");
 }
 
 function submitPreview(url, form, target) {