Added backward compatiblity

[ase-101]

Summary by CodeRabbit

• Bug Fixes
  • Corrected the initialization of facial biometric data handling to ensure proper system configuration.

[coderabbitai]

Walkthrough

A single change to the BiometricUtil class initializes an empty subtype list in the BDBInfo builder specifically for FACE biometric type processing. No other logic, flow, or behavioral modifications are introduced.

Changes

Cohort / File(s)	Change Summary
BiometricUtil initialization mosip-identity-plugin/src/main/java/io/mosip/signup/plugin/mosipid/util/BiometricUtil.java	Adds empty subtype list to BDBInfo builder for FACE biometric type

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Highly localized single-file change
• Simple initialization of an empty collection
• No behavioral logic modifications
• No cross-cutting concerns or dependencies introduced

Poem

🐰 A face needs no subtypes, clean and refined,
Empty lists bloom where clutter once pined,
One tiny change, yet purposeful and neat,
Making biometrics simple and sweet! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check	❓ Inconclusive	The title 'Added backward compatiblity' is vague and generic. While it references a real concept (backward compatibility), it lacks specificity about what feature or component is affected and doesn't clearly convey the actual change (adding empty subtype list to BDBInfo for FACE biometric).	Revise the title to be more specific, such as 'Initialize empty subtypes in BDBInfo for FACE biometric' or 'Add BDBInfo subtype initialization for backward compatibility', to better reflect the actual code change.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8569569 and 57f6e44.

📒 Files selected for processing (1)

• mosip-identity-plugin/src/main/java/io/mosip/signup/plugin/mosipid/util/BiometricUtil.java (1 hunks)

🔇 Additional comments (1)

mosip-identity-plugin/src/main/java/io/mosip/signup/plugin/mosipid/util/BiometricUtil.java (1)

52-60: Change is correct and resolves backward compatibility with library upgrade.

The git history confirms this is a necessary fix: the previous commit upgraded to kernel-cbeff-util, which likely changed how null values are handled for the BDBInfo subtype field. MOSIP's CBEFF specification confirms that Face biometrics have no finer subtype defined (full face capture), making the empty list the correct initialization. The one-line addition properly handles the library's new requirements while maintaining compliance with the biometric specification.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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