ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

remove dead "highest_role" method 

refs CNVS-18529

found this while migrating the salesforce plugin
and digging into how roles work.  It's never called. removing it.

TEST PLAN:
 - nothing should change, this method is never used

Change-Id: Ibae6d4de5cc42e5c6dcfb2746d8241b2a71257a8
Reviewed-on: https://gerrit.instructure.com/50777
Reviewed-by: Cody Cutrer <cody@instructure.com>
Tested-by: Jenkins
QA-Review: August Thornton <august@instructure.com>
Product-Review: Ethan Vizitei <evizitei@instructure.com>
This commit is contained in:
Ethan Vizitei 2015-03-23 10:55:18 -06:00 committed by Ethan Vizitei
parent dff29fa987
commit a9512faebf
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/models/user.rb
View File

@ -2181,10 +2181,6 @@ class User < ActiveRecord::Base
TAB_EPORTFOLIOS = 3 TAB_EPORTFOLIOS = 3
TAB_HOME = 4 TAB_HOME = 4
def highest_role
roles.last
end
def roles(root_account) def roles(root_account)
return @roles if @roles return @roles if @roles
@roles = ['user'] @roles = ['user']