Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reset entity manager when it is closed #967

Closed
wants to merge 4 commits into from
Closed

Conversation

pierredup
Copy link
Member

No description provided.

@pierredup pierredup added this to the 2.2.6 milestone Oct 30, 2023
@what-the-diff
Copy link

what-the-diff bot commented Oct 30, 2023

PR Summary

  • Improvements in InvoiceManager.php
    • The invoice manager has been updated to handle errors better, improving the stability of the application
    • The precision of code in the Invoice Manager has been improved, contributing to a safer and faster application by ensuring that only the correct types of data are used.
    • A condition has been added to check if the 'EntityManager' is closed and it resets it if needed. This enhancement addresses any potential halt in operations due to the 'EntityManager' being closed.
  • Updates in ScheduleBuilder.php
    • The method to handle query results in the Schedule Builder has been changed. It now leaves out intermediary data structures leading to a more efficient and faster operation.
    • The system has been updated to only consider active invoices that have no end date or a future end date. This optimizes the system's focus and resources.
    • The way the system gets the current time has been improved to provide more accurate timing information.
    • Lastly, the description of the invoices has been updated, providing clarity and better understanding for users or clients interacting with these invoices.

Pierre du Plessis and others added 2 commits November 3, 2023 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant