[zanata/zanata-server] 82ada4: rhbz978666 - add database table and liquibase chan...

GitHub noreply at github.com
Mon Jul 8 04:59:25 UTC 2013


  Branch: refs/heads/user-comments
  Home:   https://github.com/zanata/zanata-server
  Commit: 82ada447e15ed679b61095551142fd4769fa27d7
      https://github.com/zanata/zanata-server/commit/82ada447e15ed679b61095551142fd4769fa27d7
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-model/src/main/java/org/zanata/model/HTextFlowTarget.java
    A zanata-model/src/main/java/org/zanata/model/HTextFlowTargetReviewComment.java
    A zanata-war/src/main/resources/db/changelogs/db.changelog-3.1.xml
    M zanata-war/src/main/webapp-jboss/WEB-INF/classes/META-INF/persistence.xml
    M zanata-war/src/test/java/org/zanata/dao/TextFlowTargetDAOTest.java
    M zanata-war/src/test/resources/META-INF/persistence.xml
    M zanata-war/src/test/resources/arquillian/persistence.xml

  Log Message:
  -----------
  rhbz978666 - add database table and liquibase change log for review translation comment


  Commit: 2b78871e573eeb8b555d2ebb84fba575c14bef97
      https://github.com/zanata/zanata-server/commit/2b78871e573eeb8b555d2ebb84fba575c14bef97
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-model/src/main/java/org/zanata/model/HTextFlowTarget.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/TransUnitTransformer.java
    M zanata-war/src/main/java/org/zanata/webtrans/shared/model/TransUnit.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/model/UserComment.java
    M zanata-war/src/test/java/org/zanata/dao/TextFlowTargetDAOTest.java

  Log Message:
  -----------
  rhbz978666 - update TransUnit to indicate comment existance


  Commit: 995c7a5a96402cc328019864c8241f2db566a13b
      https://github.com/zanata/zanata-server/commit/995c7a5a96402cc328019864c8241f2db566a13b
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-model/src/main/java/org/zanata/model/HTextFlowTarget.java
    M zanata-model/src/main/java/org/zanata/model/HTextFlowTargetReviewComment.java
    A zanata-war/src/main/java/org/zanata/dao/TextFlowTargetReviewCommentsDAO.java
    A zanata-war/src/main/java/org/zanata/webtrans/client/presenter/ReviewCommentDataProvider.java
    A zanata-war/src/main/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/TargetContentsPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/resources/TableEditorMessages.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/resources/WebTransMessages.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/ui/EditorButtonsWidget.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/ui/EditorButtonsWidget.ui.xml
    A zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentDisplay.java
    A zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentView.java
    A zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentView.ui.xml
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsView.java
    A zanata-war/src/main/java/org/zanata/webtrans/server/rpc/AddReviewCommentHandler.java
    A zanata-war/src/main/java/org/zanata/webtrans/server/rpc/GetReviewCommentsHandler.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/GetTransUnitListHandler.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/TransUnitTransformer.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/model/ReviewComment.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/model/ReviewCommentId.java
    M zanata-war/src/main/java/org/zanata/webtrans/shared/model/TransUnit.java
    R zanata-war/src/main/java/org/zanata/webtrans/shared/model/UserComment.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/rpc/AddReviewCommentAction.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/rpc/AddReviewCommentResult.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/rpc/GetReviewCommentsAction.java
    A zanata-war/src/main/java/org/zanata/webtrans/shared/rpc/GetReviewCommentsResult.java
    M zanata-war/src/main/resources/db/changelogs/db.changelog-3.1.xml
    M zanata-war/src/main/resources/db/db.changelog.xml
    M zanata-war/src/test/java/org/zanata/dao/TextFlowTargetDAOTest.java
    A zanata-war/src/test/java/org/zanata/dao/TextFlowTargetReviewCommentsDAOJPATest.java
    A zanata-war/src/test/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenterTest.java
    M zanata-war/src/test/java/org/zanata/webtrans/client/presenter/TargetContentsPresenterTest.java
    A zanata-war/src/test/java/org/zanata/webtrans/server/rpc/AddReviewCommentHandlerTest.java
    A zanata-war/src/test/java/org/zanata/webtrans/server/rpc/GetReviewCommentsHandlerTest.java
    M zanata-war/src/test/resources/org/zanata/test/model/ClearAllTables.dbunit.xml
    M zanata-war/src/test/resources/performance/GetTransUnitListTest.dbunit.xml

  Log Message:
  -----------
  rhbz978666 - UI and tests

- add review comment rpc
- get review comments rpc
- review comment gwt MVP
- change to TransUnit
- change to editor buttons display
- annotate a few areas that have technical debt (hibernate performance related)


  Commit: 3359c149d675ee3b0c7c396193aac5842eaa548b
      https://github.com/zanata/zanata-server/commit/3359c149d675ee3b0c7c396193aac5842eaa548b
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/dao/TextFlowDAO.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/GetTransUnitListHandler.java
    M zanata-war/src/test/java/org/zanata/dao/TextFlowDAOTest.java
    M zanata-war/src/test/java/org/zanata/webtrans/client/service/MockHandlerFactory.java

  Log Message:
  -----------
  Tech Debt: eager load text flow targets when we intended to access them


  Commit: 28afbb90544dea76d66511bcedfde31ef5cb9aae
      https://github.com/zanata/zanata-server/commit/28afbb90544dea76d66511bcedfde31ef5cb9aae
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/dao/TextFlowDAO.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/GetTransUnitListHandler.java
    M zanata-war/src/test/java/org/zanata/dao/TextFlowDAOTest.java
    M zanata-war/src/test/java/org/zanata/webtrans/client/service/MockHandlerFactory.java

  Log Message:
  -----------
  rename methods after refactor


  Commit: 38fad4743f8af19306dd0ba9dc140e2b50f85dc1
      https://github.com/zanata/zanata-server/commit/38fad4743f8af19306dd0ba9dc140e2b50f85dc1
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/dao/TextFlowDAO.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/GetTransUnitListHandler.java
    M zanata-war/src/test/java/org/zanata/dao/TextFlowDAOTest.java
    M zanata-war/src/test/java/org/zanata/webtrans/client/service/MockHandlerFactory.java

  Log Message:
  -----------
  revert optimization change

this is too big a refactoring and should do in its own pull request


  Commit: 9692b5941629a4a67c7a0726c8d81878418b03af
      https://github.com/zanata/zanata-server/commit/9692b5941629a4a67c7a0726c8d81878418b03af
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsView.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsView.ui.xml

  Log Message:
  -----------
  rhbz978666 - add comment indicator


  Commit: f2da372834337208bab618addb16a0ebb5cbd239
      https://github.com/zanata/zanata-server/commit/f2da372834337208bab618addb16a0ebb5cbd239
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    A zanata-war/src/main/java/org/zanata/webtrans/client/events/ReviewCommentEvent.java
    A zanata-war/src/main/java/org/zanata/webtrans/client/events/ReviewCommentEventHandler.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/TargetContentsPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/ui/EditorButtonsWidget.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsView.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsView.ui.xml
    M zanata-war/src/test/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenterTest.java

  Log Message:
  -----------
  rhbz978666 - add comment indicator


  Commit: 1713421a115d050a7de9b0abb9f378cc45e63222
      https://github.com/zanata/zanata-server/commit/1713421a115d050a7de9b0abb9f378cc45e63222
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/DocumentListPresenter.java

  Log Message:
  -----------
  minor clean up


  Commit: 3fb2edb0ad0236db7d3eda06e15b222cefe91be8
      https://github.com/zanata/zanata-server/commit/3fb2edb0ad0236db7d3eda06e15b222cefe91be8
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/TargetContentsPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/TransUnitsTablePresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsView.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/AddReviewCommentHandler.java
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/TransUnitTransformer.java
    M zanata-war/src/main/java/org/zanata/webtrans/shared/rpc/AddReviewCommentAction.java
    M zanata-war/src/main/java/org/zanata/webtrans/shared/rpc/TransUnitUpdated.java
    M zanata-war/src/test/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenterTest.java
    M zanata-war/src/test/java/org/zanata/webtrans/server/rpc/AddReviewCommentHandlerTest.java

  Log Message:
  -----------
  rhbz978666 - comment change is broadcasted thru event service


  Commit: a645b8574ba2565e6845000dc6395a59a922e167
      https://github.com/zanata/zanata-server/commit/a645b8574ba2565e6845000dc6395a59a922e167
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/webtrans/server/rpc/TransUnitTransformer.java

  Log Message:
  -----------
  minor - reformat comment


  Commit: fe4a8219cc6ee46f2b0eb6d8c39609121e671a09
      https://github.com/zanata/zanata-server/commit/fe4a8219cc6ee46f2b0eb6d8c39609121e671a09
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenter.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentDisplay.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentView.java
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentView.ui.xml
    M zanata-war/src/test/java/org/zanata/webtrans/client/presenter/ReviewCommentPresenterTest.java

  Log Message:
  -----------
  rhbz978666 - style comments made against old translation


  Commit: f4a09400f8fc5f807a3120dc649e1200d78d9f81
      https://github.com/zanata/zanata-server/commit/f4a09400f8fc5f807a3120dc649e1200d78d9f81
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-model/src/main/java/org/zanata/model/HTextFlowTargetReviewComment.java
    M zanata-war/src/main/java/org/zanata/dao/TextFlowTargetReviewCommentsDAO.java
    M zanata-war/src/test/java/org/zanata/dao/TextFlowTargetReviewCommentsDAOJPATest.java
    M zanata-war/src/test/resources/org/zanata/test/model/TextFlowTestData.dbunit.xml

  Log Message:
  -----------
  rhbz978666 - minor query improvement


  Commit: 92a317ac768a90c1e9bf51a62725841d26bb32f5
      https://github.com/zanata/zanata-server/commit/92a317ac768a90c1e9bf51a62725841d26bb32f5
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-war/src/main/java/org/zanata/webtrans/client/view/ReviewCommentView.ui.xml

  Log Message:
  -----------
  minor ui change


  Commit: bf3cd60de10ce1964070db8482ecfd8acd520ec2
      https://github.com/zanata/zanata-server/commit/bf3cd60de10ce1964070db8482ecfd8acd520ec2
  Author: Patrick Huang <pahuang at redhat.com>
  Date:   2013-07-07 (Sun, 07 Jul 2013)

  Changed paths:
    M zanata-model/src/main/java/org/zanata/model/HTextFlowTargetReviewComment.java
    M zanata-war/src/test/java/org/zanata/dao/TextFlowTargetReviewCommentsDAOJPATest.java

  Log Message:
  -----------
  rhbz978666 - remove unused methods


Compare: https://github.com/zanata/zanata-server/compare/efe0a9f83e14...bf3cd60de10c


More information about the zanata-commits mailing list