Closed itstanany closed 6 days ago
the code is duplicated not moved.
@itstanany will you be able to open another pr to move the api declaration of the search method to the actual search package? thanks for your contribution!
i will close this pr for now.
@itstanany will you be able to open another pr to move the api declaration of the search method to the actual search package? thanks for your contribution!
i will close this pr for now.
@jingtang10 , the api declaration is already in search package, so if you see it is better to be in search package, there is no more actions to do. just leave it as it is.
Thanks for your consideration
the search
and count
apis are defined in the com.google.android.fhir
package. I think it might make more sense for them to be moved to the com.google.android.fhir.search
package
@aditya-07 @MJ1998 @santosh-pingle fyi and feedback welcome
the search
and count
apis are defined in the
com.google.android.fhir
package. I think it might make more sense for them to be moved to thecom.google.android.fhir.search
package@aditya-07 @MJ1998 @santosh-pingle fyi and feedback welcome
@jingtang10
I noticed that search
and count
are defined as overload extension functions on FhirEngine
within the search
package (here: [link to search
Search.kt#L25]
My question is: Should these functions stay as part of the public API for the engine
package (like they're used in the demo app: [link to PatientListViewModel.kt#L90](https://github.com/itstanany/android-fhir/blob/da8e193f78cf75919d97809d6cafdf9738ac48a6/demo/src/main/java/com/google/android/fhir/demo/PatientListViewModel.kt#L90)) or be moved to the search
package as suggested? This would require client code to be aware of the search
package.
I'm a bit unsure of the best approach here. What are your thoughts?
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2540
Description Clear and concise code change description. I moved the search logic implementation from Search.execute to FhirEngine.search For better separation of concerns and single responsibility principles
Alternative(s) considered Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type Choose one: Code health
Screenshots (if applicable)
Checklist
./gradlew spotlessApply
and./gradlew spotlessCheck
to check my code follows the style guide of this project../gradlew check
and./gradlew connectedCheck
to test my changes locally.