fix: resolve ONLY_FULL_GROUP_BY violations and bulk registration username bug by vishwab1 · Pull Request #136 · PSMRI/Admin-API

vishwab1 · 2026-06-29T08:01:35Z

📋 Description

Please provide a summary of the change and the motivation behind it. Include relevant context and details.

Summary

Fix bulk employee registration setting username from contact number instead of the actual username field (BulkRegistrationServiceImpl.java)
Remove unnecessary GROUP BY clauses causing ONLY_FULL_GROUP_BY SQL mode violations in two service-lookup queries, replacing one with DISTINCT to preserve dedup behavior without grouping by a non-selected column

✅ Type of Change

ℹ️ Additional Information

Please describe how the changes were tested, and include any relevant screenshots, logs, or other information that provides additional context.

Bulk registration

# Conflicts: # src/main/java/com/iemr/admin/utils/JwtUserIdValidationFilter.java

3.3.0

remove unwanted line

fix code

* Httpheader content disposition changed * Coderabbitai comments adrressed

Feature/signaturerelease

Release 3.6.0

…igin

fix: amm-1927 send headers only if the request is from the allowed origin

Release 3.5.0

Release 3.6.1

# Conflicts: # src/main/java/com/iemr/admin/service/employeemaster/EmployeeSignatureServiceImpl.java

fix vulnerabilitie code

fix security hotspots

…#121) * fix:changed the pom xml * fix: added facilty type master change * feat: created facility creation * fix: rabiit review fix * fix: rabiit review fix * fix: rabiit review fix * fix: pom version

* fix:changed the pom xml * fix: added facilty type master change * feat: created facility creation * feat:added work location * feat:added work location * fix: rabiit review fix * fix: rabiit review fix * fix: rabiit review fix * fix: ui chnges * fix: pom version * fix: corrections * fix: facilty hierachy * fix: facility heirachy * fix: remove logs folder from repository Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: add logs/ to .gitignore Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix:changed the pom xml * fix: added facilty type master change * feat: created facility creation * feat:added work location * feat:added work location * fix: rabiit review fix * fix: rabiit review fix * fix: rabiit review fix * fix: ui chnges * fix: pom version * fix: corrections * fix: facilty hierachy * fix: facility heirachy * fix: item facility mapping and store updates Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: fixed inventory flow --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Cherry-pick health and version API enhancements to release-3.6.1 (#124) * feat(health,version): add health and version endponts * fix(health): add constant and remove duplicates * fix(health): avoid permanent DEGRADED from historical deadlocks * fix(health): Removed the unnecessary boolean literal * fix(health): Fixed the broken lock-wait detection * fix(health): avoid blocking DB I/O under write lock and restore interrupt flag * fix(health): add cancelFutures in healthservice * fix(health): close basic DB connection before advanced checks and remove shared-map race * feat: expose account lock state in SearchEmployee4 (#129) Co-authored-by: Varun Deep Saini <varun.23bcs10048@ms.sst.scaler.com> * Fix the reset password issue (#131) * fix: reset password * fix: build issue * fix: update the url * fix: update config properties * fix: initialize constructor --------- Co-authored-by: KOPPIREDDY DURGA PRASAD <144464542+DurgaPrasad-54@users.noreply.github.com> Co-authored-by: Varun Deep Saini <deep.sa@rippling.com> Co-authored-by: Varun Deep Saini <varun.23bcs10048@ms.sst.scaler.com> Co-authored-by: Vanitha S <116701245+vanitha1822@users.noreply.github.com>

GROUP BY sm.serviceName selected non-aggregated, non-functionally-dependent columns (serviceID, isNational, statusID), which MySQL rejects under ONLY_FULL_GROUP_BY. Use SELECT DISTINCT instead since no aggregation is actually needed.

…tead of username field mUser.setUserName(employee.getContactNo()) was using the uploaded contact number as the login username, ignoring the actual UserName column in the bulk upload sheet. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

# Conflicts: # src/main/java/com/iemr/admin/service/bulkRegistration/BulkRegistrationServiceImpl.java # src/main/java/com/iemr/admin/utils/JwtUserIdValidationFilter.java

coderabbitai · 2026-06-29T08:01:46Z

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a5309b1e-1156-4dc1-8897-fd2c4a681b58

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

sonarqubecloud · 2026-06-29T08:02:29Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

SauravBizbRolly and others added 30 commits March 4, 2025 10:37

Bulk registration

3c2135e

Bulk registration

23cccac

Bulk registration

83f693e

Merge pull request #69 from toarunmishra/bulk_registration

8ae4c09

Bulk registration

add userName and password in Bulk registration

cdd7b2a

add userName and password in Bulk registration

128c8f9

Merge pull request #88 from toarunmishra/bulk_registration

a069382

Bulk registration

Merge remote-tracking branch 'upstream/develop' into enhancement

3fd079a

# Conflicts: # src/main/java/com/iemr/admin/utils/JwtUserIdValidationFilter.java

Merge pull request #90 from toarunmishra/3.3.0

fcbc3da

3.3.0

remove unwanted line

5cd719d

Merge pull request #91 from toarunmishra/3.3.0

2a55c54

remove unwanted line

fix code

110fa59

Merge pull request #92 from toarunmishra/3.3.0

83d3387

fix code

fix code

b1a893f

fix code

b67332f

fix code

bdde896

fix code

5315e05

fix code

486d412

fix code

4e5dab6

fix code

cd7f567

fix code

f06d077

fix code

84b679a

fix code

7b123e9

fix code

a28294f

fix code

093ba26

Httpheader content disposition changed

8820fe2

Coderabbitai comments adrressed

2aab70e

Httpheader content disposition changed (#100)

20eaaed

* Httpheader content disposition changed * Coderabbitai comments adrressed

Compile error resolved

e04b9e5

Merge pull request #101 from PSMRI/feature/signaturerelease

bd11453

Feature/signaturerelease

SauravBizbRolly and others added 27 commits November 11, 2025 17:03

fix code

6f5a101

Merge branch 'release-3.6.1' into release-3.6.0

1d45257

Merge pull request #109 from PSMRI/release-3.6.0

0acb505

Release 3.6.0

fix: amm-1927 send headers only if the request is from the allowed or…

608773a

…igin

fix: amm-1927 coderabbit fixes

5a7c6bf

Update regex handling for localhost URLs

ad6ff5e

Enhance regex pattern for URL matching

eb98a5a

Merge pull request #110 from PSMRI/amm-1927

bffa971

fix: amm-1927 send headers only if the request is from the allowed origin

Merge branch 'release-3.6.0' into release-3.5.0

c6ded08

Merge pull request #111 from PSMRI/release-3.5.0

2a04bfb

Release 3.5.0

Merge pull request #112 from PSMRI/release-3.6.1

75d06ca

Release 3.6.1

fix code

b421d8b

Merge remote-tracking branch 'upstream/release-3.6.0' into release-3.6.0

9c206a4

# Conflicts: # src/main/java/com/iemr/admin/service/employeemaster/EmployeeSignatureServiceImpl.java

fix code

76db67c

Merge branch 'main' into release-3.6.0

d5b3e20

fix vulnerabilitie code

711128e

Merge pull request #115 from PSMRI/fix/vulnerabilitie

091bd75

fix vulnerabilitie code

fix security hotspots

f76f68c

Merge pull request #116 from PSMRI/fix/remove_log

a52081b

fix security hotspots

fixed conflicts

ea22db4

Add facility hierarchy creation with village and parent-child mapping (…

68e4130

…#121) * fix:changed the pom xml * fix: added facilty type master change * feat: created facility creation * fix: rabiit review fix * fix: rabiit review fix * fix: rabiit review fix * fix: pom version

Merge remote-tracking branch 'origin/release-3.8.2' into vb/fixservice

32e65ca

# Conflicts: # src/main/java/com/iemr/admin/service/bulkRegistration/BulkRegistrationServiceImpl.java # src/main/java/com/iemr/admin/utils/JwtUserIdValidationFilter.java

vanitha1822 approved these changes Jun 29, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: resolve ONLY_FULL_GROUP_BY violations and bulk registration username bug#136

fix: resolve ONLY_FULL_GROUP_BY violations and bulk registration username bug#136
vishwab1 wants to merge 82 commits into
release-3.8.2from
vb/fixservice

vishwab1 commented Jun 29, 2026

Uh oh!

coderabbitai Bot commented Jun 29, 2026

Review skipped

Uh oh!

sonarqubecloud Bot commented Jun 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Uh oh!

Conversation

vishwab1 commented Jun 29, 2026

📋 Description

Summary

✅ Type of Change

ℹ️ Additional Information

Uh oh!

coderabbitai Bot commented Jun 29, 2026

Review skipped

Uh oh!

sonarqubecloud Bot commented Jun 29, 2026

Quality Gate passed

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants