Closed jo-elimu closed 1 month ago
This update renames various entities and methods from LetterSoundCorrespondence
to LetterSound
across multiple files for consistency and clarity. This involves changes in Java classes, SQL schema, controller methods, queries, and JSP files, ensuring the codebase uses the more concise term LetterSound
.
Files / Group of Files | Change Summary |
---|---|
.../LetterSoundCorrespondenceContributionEvent.java |
Renamed letterSoundCorrespondence to letterSound . |
.../DbContentImportHelper.java |
Updated method call from setLetterSoundCorrespondence to setLetterSound within LetterSoundContributionEvent class. |
.../LetterSoundCreateController.java , .../LetterSoundEditController.java |
Modified method setLetterSoundCorrespondence to setLetterSound . |
.../LetterSoundPeerReviewEventCreateController.java |
Updated various method calls and URL constructions from letterSoundCorrespondence to letterSound . |
.../META-INF/jpa-schema-export.sql , .../db/migration/2004008.sql |
Renamed column letterSoundCorrespondence_id to letterSound_id and updated related foreign key constraints. |
.../LetterSoundContributionEventDaoJpa.java , .../LetterSoundPeerReviewEventDaoJpa.java |
Modified query conditions and parameter settings to use letterSound instead of letterSoundCorrespondence . |
.../content/letter-sound/peer-reviews/pending.jsp |
Updated variable assignment from ${letterSoundContributionEvent.letterSoundCorrespondence} to ${letterSoundContributionEvent.letterSound} . |
LetterSoundCorrespondence
to LetterSound
, aligning directly with the changes made in this PR.Common learnings
``` Learnt from: venkatesh2k3 PR: elimu-ai/webapp#1682 File: src/main/java/ai/elimu/dao/LetterSoundContributionEventDao.java:13-13 Timestamp: 2024-07-06T17:37:44.413Z Learning: In the `LetterSoundContributionEventDao.java` file, the method signatures should use `LetterSoundContributionEvent` instead of `LetterSoundCorrespondenceContributionEvent` to maintain consistency with the renaming of entities. ```src/main/java/ai/elimu/dao/jpa/LetterSoundContributionEventDaoJpa.java (1)
``` Learnt from: venkatesh2k3 PR: elimu-ai/webapp#1682 File: src/main/java/ai/elimu/dao/LetterSoundContributionEventDao.java:13-13 Timestamp: 2024-07-06T17:37:44.413Z Learning: In the `LetterSoundContributionEventDao.java` file, the method signatures should use `LetterSoundContributionEvent` instead of `LetterSoundCorrespondenceContributionEvent` to maintain consistency with the renaming of entities. ```src/main/java/ai/elimu/dao/jpa/LetterSoundPeerReviewEventDaoJpa.java (1)
``` Learnt from: venkatesh2k3 PR: elimu-ai/webapp#1682 File: src/main/java/ai/elimu/dao/LetterSoundContributionEventDao.java:13-13 Timestamp: 2024-07-06T17:37:44.413Z Learning: In the `LetterSoundContributionEventDao.java` file, the method signatures should use `LetterSoundContributionEvent` instead of `LetterSoundCorrespondenceContributionEvent` to maintain consistency with the renaming of entities. ```src/main/webapp/WEB-INF/jsp/content/letter-sound/peer-reviews/pending.jsp (1)
``` Learnt from: venkatesh2k3 PR: elimu-ai/webapp#1682 File: src/main/java/ai/elimu/dao/LetterSoundContributionEventDao.java:13-13 Timestamp: 2024-07-06T17:37:44.413Z Learning: In the `LetterSoundContributionEventDao.java` file, the method signatures should use `LetterSoundContributionEvent` instead of `LetterSoundCorrespondenceContributionEvent` to maintain consistency with the renaming of entities. ```
src/main/java/ai/elimu/dao/jpa/LetterSoundContributionEventDaoJpa.java (3)
`26-28`: **LGTM!** The method `readAll` has been correctly updated to use `letterSound` instead of `letterSoundCorrespondence`. --- `48-48`: **LGTM!** The query in `readMostRecentPerLetterSound` has been correctly updated to use `letterSound_id`. --- Line range hint `31-33`: **LGTM!** The method `readAll` has been correctly updated to use `letterSound` instead of `letterSoundCorrespondence`.src/main/java/ai/elimu/dao/jpa/LetterSoundPeerReviewEventDaoJpa.java (1)
`31-33`: **LGTM!** The method `readAll` has been correctly updated to use `letterSound` instead of `letterSoundCorrespondence`.src/main/webapp/WEB-INF/jsp/content/letter-sound/peer-reviews/pending.jsp (1)
`28-28`: **LGTM!** The variable assignment has been correctly updated to use `letterSound` instead of `letterSoundCorrespondence`.
Attention: Patch coverage is 0%
with 11 lines
in your changes missing coverage. Please review.
Project coverage is 15.26%. Comparing base (
a0d4964
) to head (613bfc7
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
refs #1677
Current structure in the production database: