Skip to content

(fix) Billing response - #192

Closed
senthil-athiban wants to merge 24 commits into
openmrs:mainfrom
indiemr:fix/billing-response
Closed

(fix) Billing response#192
senthil-athiban wants to merge 24 commits into
openmrs:mainfrom
indiemr:fix/billing-response

Conversation

@senthil-athiban

Copy link
Copy Markdown

No description provided.

Muta-Jonathan and others added 24 commits November 13, 2025 09:59
…tes-in-the-billable-services

(feat) Support for Location and Provider in BillableService API
…to-Payment-Mode-and-Cash-Point

(feat)Add is_default Column to Payment Mode and Cash Point Tables & Expose Field in REST Resource
* BillableService and CashierItemPrice should be metadata (openmrs#49)

* BillabeService and CashierItemPrice are metadata and should be treated as such

* Fixes from code review

* Remove unused imports across project

* Enhance unit tests for service implementations (openmrs#52)

* Revert "Allow bills to be viewed or download in the browser (openmrs#50)"

This reverts commit d0e9d3b.

* (feat) O3-5197: Enable custom REST representation support for Bill Resource (openmrs#54)

* (fix) O3-5122: Bill should not return voided line items (openmrs#46)

* (fix): Bill should not return voided line items

* Remove voided line items and payments from service level

* filter voided line items in get bill by receipt number

* Add filtering logic to the existing null remover function

* Add transaction annotation to implementation

* Review feedback

* rename include voided items param to include all

* Review feedback

* Review feedback

* O3-5187: Add server-side pagination (openmrs#53)

* (feat) O3-5057: Add server-side pagination

* Use alreadypaged with totalcount

* Correct rebase

* O3-5178: Exclude voided payments from bill status calculation (openmrs#48)

* (feat): Allow filtering for multiple statuses (openmrs#58)

* O3-5215: Enable Custom Representation for BillLineItemResource and PaymentResource (openmrs#62)

* O3-5211: Update Billing Status When a Bill Line Item is Deleted (openmrs#63)

* (fix) Generated bill should not get corrupted (openmrs#59)

* fix generating Bill Reciept

* fetch bill Reciept by uuid

* remove redundant logs and raduce reciept height

* Add patient name filtering support in Bill search (openmrs#68)

* O3-5200: Bill should allow modifications only in pending state (openmrs#64)

* O3-5156: Fix Biling itemList update functionality (openmrs#55)

* Fix for concurrent issue

* Addition of the test case

* TestAddition plus usage of Set

* O3-5156: Test Completion

* O3-5156: Test Completion

* Test changes

* Test changes

* Final changes

* Review comment changes

* Review comment changes

* Code review

---------

Co-authored-by: Ian <ian.c.bacher@gmail.com>

* Migrate to Platform 2.7.x and support Java 21 (openmrs#69)

* O3-5067: Replace JSON-based billing exemptions with database-backed service implementation (openmrs#57)

* O3-5246: Fixing the payments issue Post addition of Pending state check (openmrs#72)

* Migrate the BillService to OpenMRSService (openmrs#77)

---------

Co-authored-by: Ian <52504170+ibacher@users.noreply.github.com>
Co-authored-by: Wikum Weerakutti <wikum@linuxdeveloper.space>
Co-authored-by: Ian <ian.c.bacher@gmail.com>
Co-authored-by: Nethmi Rodrigo <nethmi.dilinika@gmail.com>
Co-authored-by: Raj Prakash <raj.24bcs10328@sst.scaler.com>
Co-authored-by: Mutesasira Moses <mozzymutesa@gmail.com>
Co-authored-by: JG <85500670+jayg2002@users.noreply.github.com>
* BillableService and CashierItemPrice should be metadata (openmrs#49)

* BillabeService and CashierItemPrice are metadata and should be treated as such

* Fixes from code review

* Remove unused imports across project

* Enhance unit tests for service implementations (openmrs#52)

* Revert "Allow bills to be viewed or download in the browser (openmrs#50)"

This reverts commit d0e9d3b.

* (feat) O3-5197: Enable custom REST representation support for Bill Resource (openmrs#54)

* (fix) O3-5122: Bill should not return voided line items (openmrs#46)

* (fix): Bill should not return voided line items

* Remove voided line items and payments from service level

* filter voided line items in get bill by receipt number

* Add filtering logic to the existing null remover function

* Add transaction annotation to implementation

* Review feedback

* rename include voided items param to include all

* Review feedback

* Review feedback

* O3-5187: Add server-side pagination (openmrs#53)

* (feat) O3-5057: Add server-side pagination

* Use alreadypaged with totalcount

* Correct rebase

* O3-5178: Exclude voided payments from bill status calculation (openmrs#48)

* (feat): Allow filtering for multiple statuses (openmrs#58)

* O3-5215: Enable Custom Representation for BillLineItemResource and PaymentResource (openmrs#62)

* O3-5211: Update Billing Status When a Bill Line Item is Deleted (openmrs#63)

* (fix) Generated bill should not get corrupted (openmrs#59)

* fix generating Bill Reciept

* fetch bill Reciept by uuid

* remove redundant logs and raduce reciept height

* Add patient name filtering support in Bill search (openmrs#68)

* O3-5200: Bill should allow modifications only in pending state (openmrs#64)

* O3-5156: Fix Biling itemList update functionality (openmrs#55)

* Fix for concurrent issue

* Addition of the test case

* TestAddition plus usage of Set

* O3-5156: Test Completion

* O3-5156: Test Completion

* Test changes

* Test changes

* Final changes

* Review comment changes

* Review comment changes

* Code review

---------

Co-authored-by: Ian <ian.c.bacher@gmail.com>

* Migrate to Platform 2.7.x and support Java 21 (openmrs#69)

* O3-5067: Replace JSON-based billing exemptions with database-backed service implementation (openmrs#57)

* O3-5246: Fixing the payments issue Post addition of Pending state check (openmrs#72)

* Migrate the BillService to OpenMRSService (openmrs#77)

* (feat) Add default value to Bill auto creation on Drug Orders

---------

Co-authored-by: Ian <52504170+ibacher@users.noreply.github.com>
Co-authored-by: Wikum Weerakutti <wikum@linuxdeveloper.space>
Co-authored-by: Ian <ian.c.bacher@gmail.com>
Co-authored-by: Nethmi Rodrigo <nethmi.dilinika@gmail.com>
Co-authored-by: Raj Prakash <raj.24bcs10328@sst.scaler.com>
Co-authored-by: Mutesasira Moses <mozzymutesa@gmail.com>
Co-authored-by: JG <85500670+jayg2002@users.noreply.github.com>
…21 / Platform 2.7 migration) (#10)

* BillableService and CashierItemPrice should be metadata (openmrs#49)

* BillabeService and CashierItemPrice are metadata and should be treated as such

* Fixes from code review

* Remove unused imports across project

* Enhance unit tests for service implementations (openmrs#52)

* Revert "Allow bills to be viewed or download in the browser (openmrs#50)"

This reverts commit d0e9d3b.

* (feat) O3-5197: Enable custom REST representation support for Bill Resource (openmrs#54)

* (fix) O3-5122: Bill should not return voided line items (openmrs#46)

* (fix): Bill should not return voided line items

* Remove voided line items and payments from service level

* filter voided line items in get bill by receipt number

* Add filtering logic to the existing null remover function

* Add transaction annotation to implementation

* Review feedback

* rename include voided items param to include all

* Review feedback

* Review feedback

* O3-5187: Add server-side pagination (openmrs#53)

* (feat) O3-5057: Add server-side pagination

* Use alreadypaged with totalcount

* Correct rebase

* O3-5178: Exclude voided payments from bill status calculation (openmrs#48)

* (feat): Allow filtering for multiple statuses (openmrs#58)

* O3-5215: Enable Custom Representation for BillLineItemResource and PaymentResource (openmrs#62)

* O3-5211: Update Billing Status When a Bill Line Item is Deleted (openmrs#63)

* (fix) Generated bill should not get corrupted (openmrs#59)

* fix generating Bill Reciept

* fetch bill Reciept by uuid

* remove redundant logs and raduce reciept height

* Add patient name filtering support in Bill search (openmrs#68)

* O3-5200: Bill should allow modifications only in pending state (openmrs#64)

* O3-5156: Fix Biling itemList update functionality (openmrs#55)

* Fix for concurrent issue

* Addition of the test case

* TestAddition plus usage of Set

* O3-5156: Test Completion

* O3-5156: Test Completion

* Test changes

* Test changes

* Final changes

* Review comment changes

* Review comment changes

* Code review

---------

Co-authored-by: Ian <ian.c.bacher@gmail.com>

---------

Co-authored-by: Ian <52504170+ibacher@users.noreply.github.com>
Co-authored-by: Wikum Weerakutti <wikum@linuxdeveloper.space>
Co-authored-by: Ian <ian.c.bacher@gmail.com>
Co-authored-by: Nethmi Rodrigo <nethmi.dilinika@gmail.com>
Co-authored-by: Raj Prakash <raj.24bcs10328@sst.scaler.com>
Co-authored-by: Mutesasira Moses <mozzymutesa@gmail.com>
Co-authored-by: JG <85500670+jayg2002@users.noreply.github.com>
* Add nested REST endpoints for managing bill line items

* remove unused imports
@sonarqubecloud

sonarqubecloud Bot commented Aug 3, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)
B Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants