Some slight improvements in AJAX errors handling.

git-svn-id: http://redmine.rubyforge.org/svn/trunk@874 e93f8b46-1217-0410-a6f0-8f06a7374b81

Some slight improvements in AJAX errors handling.
git-svn-id: http://redmine.rubyforge.org/svn/trunk@874 e93f8b46-1217-0410-a6f0-8f06a7374b81
9a3d743a · Jean-Philippe Lang · 0af6f347 · 9a3d743a · 9a3d743a · 9a3d743a
Commit 9a3d743a authored Oct 28, 2007 by Jean-Philippe Lang
Showing with 4 additions and 4 deletions

application.rb app/controllers/application.rb +2 -2

_dir_list_content.rhtml app/views/repositories/_dir_list_content.rhtml +1 -1

context_menu.js public/javascripts/context_menu.js +1 -1

No files found.
--- a/app/controllers/application.rb
+++ b/app/controllers/application.rb
@@ -124,12 +124,12 @@ class ApplicationController < ActionController::Base
  
  def render_403
    @project = nil
-    render :template => "common/403", :layout => true, :status => 403
+    render :template => "common/403", :layout => !request.xhr?, :status => 403
    return false
  end
    
  def render_404
-    render :template => "common/404", :layout => true, :status => 404
+    render :template => "common/404", :layout => !request.xhr?, :status => 404
    return false
  end
  

--- a/app/views/repositories/_dir_list_content.rhtml
+++ b/app/views/repositories/_dir_list_content.rhtml
@@ -6,7 +6,7 @@
 <%= if entry.is_dir?
  link_to_remote h(entry.name),
                 {:url => {:action => 'browse', :id => @project, :path => entry.path, :rev => @rev, :depth => (depth + 1), :parent_id => tr_id},
-                  :update => tr_id,
+                  :update => { :success => tr_id },
                  :position => :after,
                  :success => "scmEntryLoaded('#{tr_id}')",
                  :condition => "scmEntryClick('#{tr_id}')"

--- a/public/javascripts/context_menu.js
+++ b/public/javascripts/context_menu.js
@@ -39,6 +39,6 @@ ContextMenu.prototype = {
 		this.selection = tr;
 		var id = tr.id.substring(6, tr.id.length);
 		/* TODO: do not hard code path */
-		new Ajax.Updater('context-menu', '../../issues/context_menu/' + id, {asynchronous:true, evalScripts:true, onComplete:function(request){Effect.Appear('context-menu', {duration: 0.20})}})		
+		new Ajax.Updater({success:'context-menu'}, '../../issues/context_menu/' + id, {asynchronous:true, evalScripts:true, onComplete:function(request){Effect.Appear('context-menu', {duration: 0.20})}})		
 	}
 }