Fix ITS: Revert "SONAR-3618 improve support of the Views plugin"

This reverts commit 4925839fd7c126f25c6352d6688b8fafd5cd81f0.

2 files changed, 4 insertions, 28 deletions

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/helpers/roles_helper.rb b/sonar-server/src/main/webapp/WEB-INF/app/helpers/roles_helper.rb
index 0516fc3a801..95bba72992a 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/helpers/roles_helper.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/helpers/roles_helper.rb
@@ -47,20 +47,13 @@ module RolesHelper
   end
 
   def default_project_group_names(role, qualifier)
-    group_names=(controller.java_facade.getConfigurationValue("sonar.role.#{role}.#{qualifier}.defaultGroups")||'').split(',')
-
-    # verify that groups still exist
-    result = []
-    if group_names.size>0
-      groups = Group.find(:all, :conditions => ['name in (?)', group_names])
-      result = Api::Utils.insensitive_sort(groups.map{|g| g.name})
-      result = ['Anyone'].concat(result) if group_names.include? 'Anyone'
-    end
-    result
+    property_value=(controller.java_facade.getConfigurationValue("sonar.role.#{role}.#{qualifier}.defaultGroups")||'')
+    Api::Utils.insensitive_sort(property_value.split(','))
   end
 
   def default_project_users(role, qualifier)
-    logins=(controller.java_facade.getConfigurationValue("sonar.role.#{role}.#{qualifier}.defaultUsers") || '').split(',')
+    property_value=(controller.java_facade.getConfigurationValue("sonar.role.#{role}.#{qualifier}.defaultUsers") || '')
+    logins=property_value.split(',')
     users = User.find(:all, :conditions => ['login in (?) and active=?', logins, true])
     Api::Utils.insensitive_sort(users) { |user| user.name }
   end
diff --git a/sonar-server/src/main/webapp/WEB-INF/db/migrate/320_move_default_roles.rb b/sonar-server/src/main/webapp/WEB-INF/db/migrate/320_move_default_roles.rb
index 75ee2d51658..df56495a5f4 100644
--- a/sonar-server/src/main/webapp/WEB-INF/db/migrate/320_move_default_roles.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/db/migrate/320_move_default_roles.rb
@@ -49,28 +49,11 @@ class MoveDefaultRoles < ActiveRecord::Migration
       # upgrade from version < 3.2.
       move_groups
       move_users
-    else
-      create_default_groups('admin', 'TRK', 'sonar-administrators')
-      create_default_groups('user', 'TRK', 'Anyone,sonar-users')
-      create_default_groups('codeviewer', 'TRK', 'Anyone,sonar-users')
-
-      # Support old versions of Views plugin
-      create_default_groups('admin', 'VW', 'sonar-administrators')
-      create_default_groups('user', 'VW', 'Anyone,sonar-users')
-      create_default_groups('codeviewer', 'VW', 'Anyone,sonar-users')
-      create_default_groups('admin', 'SVW', 'sonar-administrators')
-      create_default_groups('user', 'SVW', 'Anyone,sonar-users')
-      create_default_groups('codeviewer', 'SVW', 'Anyone,sonar-users')
     end
   end
 
   private
 
-  def self.create_default_groups(role, qualifier, groups)
-    Property.create(:prop_key => "sonar.role.#{role}.#{qualifier}.defaultGroups", :text_value => groups)
-    Property.create(:prop_key => "sonar.role.#{role}.#{qualifier}.defaultUsers", :text_value => '')
-  end
-
   def self.move_groups
     groups_per_role={}
     group_roles = GroupRole.find(:all, :conditions => ['resource_id is null and role like ?', 'default-%'])