[zanata/zanata-server] 033e81: Move constructor logic to @Create methods
GitHub
noreply at github.com
Tue Jul 7 08:04:24 UTC 2015
Branch: refs/heads/move-constructor-logic
Home: https://github.com/zanata/zanata-server
Commit: 033e81db1a182b9fe4b0bc7c2990d531f1defd91
https://github.com/zanata/zanata-server/commit/033e81db1a182b9fe4b0bc7c2990d531f1defd91
Author: Sean Flanigan <sflaniga at redhat.com>
Date: 2015-07-07 (Tue, 07 Jul 2015)
Changed paths:
M zanata-war/src/main/java/org/zanata/WebAssetsConfiguration.java
M zanata-war/src/main/java/org/zanata/email/EmailBuilder.java
M zanata-war/src/main/java/org/zanata/i18n/Messages.java
M zanata-war/src/main/java/org/zanata/rest/service/VersionService.java
M zanata-war/src/main/java/org/zanata/service/impl/TranslationStateCacheImpl.java
M zanata-war/src/main/java/org/zanata/service/impl/VersionStateCacheImpl.java
M zanata-war/src/main/java/org/zanata/util/CommonMarkRenderer.java
M zanata-war/src/main/java/org/zanata/webtrans/server/SeamDispatch.java
M zanata-war/src/main/java/org/zanata/webtrans/server/TranslationWorkspaceManagerImpl.java
M zanata-war/src/test/java/org/zanata/email/EmailStrategyTest.java
M zanata-war/src/test/java/org/zanata/notification/LanguageTeamPermissionChangeJmsMessagePayloadHandlerTest.java
M zanata-war/src/test/java/org/zanata/service/impl/TranslationStateCacheImplTest.java
Log Message:
-----------
Move constructor logic to @Create methods
In both Seam and CDI, the bean's constructor will be invoked twice:
once for the proxy object, and once for the actual instance, so any
logic should be in @Create/@PostConstruct methods, not the constructor.
In Seam, @javax.annotation.PostConstruct methods appear to cause Seam to
treat the bean as an EJB, which means some injections don't happen before
the @PostConstruct method (or at all, possibly). (These methods will be
migrated to @PostConstruct for CDI.)
More information about the zanata-commits
mailing list