Fixed bug with user registration and changed hardcoded 'mandriva2011'

app/controllers/platforms_controller.rb

@@ -10,7 +10,7 @@ class PlatformsController < ApplicationController
   end
 
   def easy_urpmi
-    @platforms = Platform.where(:distrib_type => 'mandriva2011', :visibility => 'open', :platform_type => 'main')
+    @platforms = Platform.where(:distrib_type => APP_CONFIG['distr_type'].first, :visibility => 'open', :platform_type => 'main')
     respond_to do |format|
       format.json do
         render :json => {

app/models/personal_repository.rb

@@ -12,7 +12,7 @@ module PersonalRepository
       pl.name = "#{self.uname}_personal"
       pl.unixname = "#{self.uname}_personal"
       pl.platform_type = 'personal'
-      pl.distrib_type = 'mandriva2011'
+      pl.distrib_type = APP_CONFIG['distr_types'].first
       pl.visibility = 'open'
       pl.save!
     

app/models/platform.rb

@@ -34,6 +34,7 @@ class Platform < ActiveRecord::Base
 
   #attr_accessible :visibility
 
+
   def path
     build_path(unixname)
   end

app/models/repository.rb

@@ -20,6 +20,7 @@ class Repository < ActiveRecord::Base
   after_create :make_owner_rel
   before_save :check_owner_rel
   #before_save :create_directory
+  #before_save :add_owner_rel
   #after_destroy :remove_directory
   before_create :xml_rpc_create
   before_destroy :xml_rpc_destroy

app/models/role.rb

@@ -54,7 +54,7 @@ class Role < ActiveRecord::Base
     end
 
     def check_owner_default
-      self[:use_default_for_owner] = false if use_default_for_owner and (to.nil? || to == '')
+      self[:use_default_for_owner] = false if use_default_for_owner and (on.nil? || on == '')
       true
     end
 

app/models/user.rb

@@ -34,7 +34,7 @@ class User < ActiveRecord::Base
   attr_readonly :uname
   attr_accessor :login
 
-  after_create :add_default_role
+  before_create :add_default_role
 
   before_update {
     if ssh_key_was.blank? and ssh_key.present?

lib/ext/active_record/base.rb

@@ -106,14 +106,16 @@ class ActiveRecord::Base
     def add_owner owner
       return false unless ActiveRecord::Base.relation_acter? owner.class
       return false unless ActiveRecord::Base.relation_target? self.class
-      roles = Role.by_acter(owner.class).by_target(self.class).owner_default
+      roles = Role.by_acter(owner.class).by_target(self.class).owner_default +
+              Role.by_acter(:all).by_target(self.class).owner_default
       add_roles_to owner, roles
     end
 
     def remove_owner owner
       return false unless ActiveRecord::Base.relation_acter? owner.class
       return false unless ActiveRecord::Base.relation_target? self.class
-      roles = Role.by_acter(owner.class).by_target(self.class).owner_default
+      roles = Role.by_acter(owner.class).by_target(self.class).owner_default +
+              Role.by_acter(:all).by_target(self.class).owner_default
       remove_roles_form_acter owner, roles
     end