aboutsummaryrefslogtreecommitdiffstats
path: root/sonar-server
diff options
context:
space:
mode:
authorSimon Brandhof <simon.brandhof@gmail.com>2012-07-12 00:44:58 +0200
committerSimon Brandhof <simon.brandhof@gmail.com>2012-07-12 00:44:58 +0200
commita8b40677a72848ce94d437d4943674ad99f8b065 (patch)
tree8f879e549ce49990c09fd0083be3eb9bb1b2681a /sonar-server
parent2533b4baaa1af52aee74b73e14f8294637f12bcf (diff)
downloadsonarqube-a8b40677a72848ce94d437d4943674ad99f8b065.tar.gz
sonarqube-a8b40677a72848ce94d437d4943674ad99f8b065.zip
Fix creation of administrator
Diffstat (limited to 'sonar-server')
-rw-r--r--sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java14
-rw-r--r--sonar-server/src/main/webapp/WEB-INF/app/models/user.rb6
2 files changed, 10 insertions, 10 deletions
diff --git a/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java b/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java
index d2f4a2bb449..1c3f88c779b 100644
--- a/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java
+++ b/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java
@@ -520,10 +520,14 @@ public final class JRubyFacade {
// USERS
public void onNewUser(Map<String, String> fields) {
- getContainer().getComponentByType(NewUserNotifier.class).onNewUser(NewUserHandler.Context.builder()
- .setLogin(fields.get("login"))
- .setName(fields.get("name"))
- .setEmail(fields.get("email"))
- .build());
+ NewUserNotifier notifier = getContainer().getComponentByType(NewUserNotifier.class);
+ // notifier is null when creating the administrator in the migration script 011.
+ if (notifier != null) {
+ notifier.onNewUser(NewUserHandler.Context.builder()
+ .setLogin(fields.get("login"))
+ .setName(fields.get("name"))
+ .setEmail(fields.get("email"))
+ .build());
+ }
}
}
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/models/user.rb b/sonar-server/src/main/webapp/WEB-INF/app/models/user.rb
index 152900bede9..c70d33985a4 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/models/user.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/models/user.rb
@@ -207,10 +207,6 @@ class User < ActiveRecord::Base
private
def on_create
- facade = Java::OrgSonarServerUi::JRubyFacade.getInstance()
- if facade
- # the facade is null when creating the administration in migration 011
- facade.onNewUser({'login' => self.login, 'name' => self.name, 'email' => self.email})
- end
+ Java::OrgSonarServerUi::JRubyFacade.getInstance().onNewUser({'login' => self.login, 'name' => self.name, 'email' => self.email})
end
end