[#12779] Fix notifications update logic

TEAMMATES / teammates

This is the project website for the TEAMMATES feedback management tool for education

https://teammatesv4.appspot.com/

GNU General Public License v2.0

1.64k stars 3.26k forks source link

Open marquestye opened 2 months ago

marquestye commented 2 months ago

Part of #12779

Outline of Solution

Added flushAndEvict to HibernateUtil.java
During update, evict entity -> call setters -> check validity -> merge entity
Changed EntitiesDb#merge to print with returned entity to overcome invalid proxy issue for entities with no persistent instance
- Added required mocks for component tests calling merge
Added tests for invalid data persistence
Moved invalid params checks and tests from db to logic layer

nusoss-bot commented 1 week ago

Folks, This PR seems to be stalling (no activities for the past 58 days). :snail: :cry: Hope someone can get it to move forward again soon...

nusoss-bot commented 1 hour ago

Folks, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...