[phenotips/patient-network] 98f72b: PN-252: Run local matches search only for patients...

GitHub noreply at github.com
Mon Jan 22 16:03:09 EST 2018


  Branch: refs/heads/PN-253
  Home:   https://github.com/phenotips/patient-network
  Commit: 98f72b510470b239d0c9008894c18e2bb522fe32
      https://github.com/phenotips/patient-network/commit/98f72b510470b239d0c9008894c18e2bb522fe32
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinder.java
    A matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/AbstractMatchFinder.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/LocalMatchFinder.java

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search

Refactored code
 - separated common matcher functionality into AbstractMatchFinder
 - extended MatchFinder API to provide more control over matching process


  Commit: d71316d37b3fd0ddf6137a7ef6823068c463179d
      https://github.com/phenotips/patient-network/commit/d71316d37b3fd0ddf6137a7ef6823068c463179d
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/MatchingNotificationManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/MatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/finder/internal/DefaultMatchFinderManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/internal/DefaultMatchingNotificationManager.java
    M matching-notification-api/src/main/java/org/phenotips/matchingnotification/script/MatchingNotificationScriptService.java

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search
PN-243: Finding local and remote matches should have separate buttons

Back-end changes in various MatchingNotification components to allow better control over what matches are found
(local or remote, all or updated since last matcher run)


  Commit: 63ee144b931cd544922f522b10fc2631153232e5
      https://github.com/phenotips/patient-network/commit/63ee144b931cd544922f522b10fc2631153232e5
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M pom.xml

  Log Message:
  -----------
  PN-252: Run local matches search only for patients updated/created since last search
PN-243: Finding local and remote matches should have separate buttons

pom.xml update


  Commit: 44c118c7c87c5c6de59084ffaf8aaed3e1b6f577
      https://github.com/phenotips/patient-network/commit/44c118c7c87c5c6de59084ffaf8aaed3e1b6f577
  Author: Andrew Misyura <asm at phenotips.org>
  Date:   2018-01-22 (Mon, 22 Jan 2018)

  Changed paths:
    M matching-notification-api/src/main/resources/ApplicationResources.properties
    A matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/findMatches.css
    A matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/findMatches.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/matchesTable.css
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/notifiedMatchesTable.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/unnotifiedMatchesTable.js
    M matching-notification-resources/src/main/resources/resources/uicomponents/matchingNotification/utils.js
    A matching-notification-ui/src/main/resources/MatchingNotification/MatchingRunInfoClass.xml
    A matching-notification-ui/src/main/resources/MatchingNotification/MatchingUpdateAndInfo.xml
    M matching-notification-ui/src/main/resources/MatchingNotification/RequestHandler.xml
    M matching-notification-ui/src/main/resources/MatchingNotification/UnnotifiedTable.xml
    R ui/src/main/resources/PhenoTips/MatchingRunInfoClass.xml
    R ui/src/main/resources/PhenoTips/MatchingUpdateAndInfo.xml

  Log Message:
  -----------
  PN-243: Finding local and remote matches should have separate buttons
PN-252: Run local matches search only for patients updated/created since last search

UI changes:
  - moved "find matches" button from the matching notification page to the "refresh matches" page
  - added separate buttons to find local matches and remore matches, as well as find all matches or only matches for updated patients
  - moved some .xml files around to have them in the same place with other code using common .js files


Compare: https://github.com/phenotips/patient-network/compare/08b778a226e8...44c118c7c87c


More information about the notifications mailing list