[phenotips/patient-network] 7c2b16: PN-337: Optimize patients filtering for match find...

GitHub noreply at github.com
Mon Oct 29 15:27:50 EDT 2018


  Branch: refs/heads/PN-336-stable
  Home:   https://github.com/phenotips/patient-network
  Commit: 7c2b16b5f5e9311d2fe9eecf1d07dc55ead0c85f
      https://github.com/phenotips/patient-network/commit/7c2b16b5f5e9311d2fe9eecf1d07dc55ead0c85f
  Author: veronikaslc <veronika.koltunov at gmail.com>
  Date:   2018-10-29 (Mon, 29 Oct 2018)

  Changed paths:
    M similarity-search/src/main/java/org/phenotips/similarity/internal/SolrSimilarPatientsFinder.java

  Log Message:
  -----------
  PN-337: Optimize patients filtering for match finding.
Used PermissionsManager instead of PatientAccessHelper to compute AccessLevel, because they return different results (PermissionsManager instantiates DefaultPatientAccess that takes into considertion if user is owner or is Admin and resolved access level to 'owner' in those cases, while PatientAccessHelper does not consider those situations).


  Commit: daa157390b3243b075de69099991cbb288531bd5
      https://github.com/phenotips/patient-network/commit/daa157390b3243b075de69099991cbb288531bd5
  Author: veronikaslc <veronika.koltunov at gmail.com>
  Date:   2018-10-29 (Mon, 29 Oct 2018)

  Changed paths:
    M matching-notification-rest/pom.xml
    M matching-notification-rest/src/main/java/org/phenotips/matchingnotification/rest/internal/DefaultMatchingNotificationResource.java

  Log Message:
  -----------
  [misc] Code clean-up: removed unused injected view access level


Compare: https://github.com/phenotips/patient-network/compare/09a691d8a0ef...daa157390b32
      **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.


More information about the notifications mailing list