[zanata/zanata-server] ccd7b1: Move constructor logic to @Create methods

GitHub noreply at github.com
Mon Jul 27 04:22:22 UTC 2015


  Branch: refs/heads/move-constructor-logic
  Home:   https://github.com/zanata/zanata-server
  Commit: ccd7b1d20d828cdfd81588010c883e179afdf539
      https://github.com/zanata/zanata-server/commit/ccd7b1d20d828cdfd81588010c883e179afdf539
  Author: Sean Flanigan <sflaniga at redhat.com>
  Date:   2015-07-27 (Mon, 27 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