| @@ -567,7 +567,11 class User < Principal | |||
|
|
567 | 567 | |
|
|
568 | 568 | # Is the user allowed to do the specified action on any project? |
|
|
569 | 569 | # See allowed_to? for the actions and valid options. |
|
|
570 | def allowed_to_globally?(action, options, &block) | |
|
|
570 | # | |
|
|
571 | # NB: this method is not used anywhere in the core codebase as of | |
|
|
572 | # 2.5.2, but it's used by many plugins so if we ever want to remove | |
|
|
573 | # it it has to be carefully deprecated for a version or two. | |
|
|
574 | def allowed_to_globally?(action, options={}, &block) | |
|
|
571 | 575 | allowed_to?(action, nil, options.reverse_merge(:global => true), &block) |
|
|
572 | 576 | end |
|
|
573 | 577 | |
| @@ -1019,6 +1019,19 class UserTest < ActiveSupport::TestCase | |||
|
|
1019 | 1019 | end |
|
|
1020 | 1020 | end |
|
|
1021 | 1021 | |
|
|
1022 | # this is just a proxy method, the test only calls it to ensure it doesn't break trivially | |
|
|
1023 | context "#allowed_to_globally?" do | |
|
|
1024 | should "proxy to #allowed_to? and reflect global permissions" do | |
|
|
1025 | @dlopper2 = User.find(5) #only Developper on a project, not Manager anywhere | |
|
|
1026 | @anonymous = User.find(6) | |
|
|
1027 | assert_equal true, @jsmith.allowed_to_globally?(:delete_issue_watchers) | |
|
|
1028 | assert_equal false, @dlopper2.allowed_to_globally?(:delete_issue_watchers) | |
|
|
1029 | assert_equal true, @dlopper2.allowed_to_globally?(:add_issues) | |
|
|
1030 | assert_equal false, @anonymous.allowed_to_globally?(:add_issues) | |
|
|
1031 | assert_equal true, @anonymous.allowed_to_globally?(:view_issues) | |
|
|
1032 | end | |
|
|
1033 | end | |
|
|
1034 | ||
|
|
1022 | 1035 | context "User#notify_about?" do |
|
|
1023 | 1036 | context "Issues" do |
|
|
1024 | 1037 | setup do |
General Comments 0
You need to be logged in to leave comments.
Login now