Rapid -- fix user-changer to work with login, not name

tslocke · Jul 4, 2008 · 33bb46b · 33bb46b
1 parent bca239a
commit 33bb46b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/hobo/lib/hobo/dev_controller.rb b/hobo/lib/hobo/dev_controller.rb
@@ -3,10 +3,14 @@ class Hobo::DevController < ActionController::Base
   hobo_controller
 
   before_filter :developer_modes_only
-
+  
   def set_current_user
     model = params[:model] || Hobo::User.default_user_model
-    self.current_user = params[:name] ? model.named(params[:name]) : model.find(params[:id])
+    self.current_user = if params[:login]
+                          model.find(:first, :conditions => {model.login_attribute => params[:login]})
+                        else
+                          model.find(params[:id])
+                        end
     redirect_to(request.env["HTTP_REFERER"] ? :back : home_page)
   end
 

diff --git a/hobo/taglibs/rapid_navigation.dryml b/hobo/taglibs/rapid_navigation.dryml
@@ -102,7 +102,7 @@ but rather write your own navigation using the `<navigation>` tag -->
   <set user="&Hobo::User.default_user_model"/>
   <select-menu if="&user && RAILS_ENV == 'development'"
                first-option="Guest" options="&user.all(:limit => 10).*.login"
-               onchange="location.href = '/dev/set_current_user?name=' + this.options[this.selectedIndex].value"
+               onchange="location.href = '/dev/set_current_user?login=' + this.options[this.selectedIndex].value"
                selected="#{current_user.login}"
                class="dev-user-changer"/>
 </def>