Moving code 3.4.1 code to 3.6.0

[snehar-nd]

📋 Description

JIRA ID:
AMM-1963

✅ Type of Change

• 🐞 Bug fix (non-breaking change which resolves an issue)

[coderabbitai]

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.

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

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch release-3.4.1

---

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.}

[drtechie]

There seems to be typo with extra space between Before and Food. 🤔

[drtechie]

@snehar-nd didn't follow this fix.
The fix is to introduce a space?

[drtechie]

This is basically a dose and frequency multiplier.
Gave this to AI, got bacK:

public static double getQtyForOneDay(String form, String dose, String frequency) {
    if (form == null || dose == null || frequency == null) return 0d;

    String f = form.trim().toLowerCase();
    String d = dose.trim().toLowerCase();
    String freq = frequency.trim().toLowerCase();

    // dose multiplier for tablets (capsules treated as 1 per administration in original code)
    double doseMultiplier;
    if (f.contains("tablet")) {
        switch (d) {
            case "half tab":           doseMultiplier = 0.5; break;
            case "one tab":            doseMultiplier = 1.0; break;
            case "one & half tab":     doseMultiplier = 1.5; break;
            case "one & half tablet":  doseMultiplier = 1.5; break; // tolerate small wording variants
            case "two tabs":           doseMultiplier = 2.0; break;
            default:                   doseMultiplier = 0.0; // unknown dose
        }
    } else if (f.contains("capsule")) {
        // original code always used integer counts for capsule (1 per administration)
        doseMultiplier = 1.0;
    } else {
        return 0d; // unknown form
    }

    // frequency -> administrations per day
    double freqMultiplier;
    if (freq.contains("once in a week") || freq.contains("once a week") || freq.contains("weekly")) {
        freqMultiplier = 1.0 / 7.0;
    } else if (freq.contains("four") || freq.contains("qid") || freq.contains("four times")) {
        freqMultiplier = 4.0;
    } else if (freq.contains("thrice") || freq.contains("tid")) {
        freqMultiplier = 3.0;
    } else if (freq.contains("twice") || freq.contains("bd")) {
        freqMultiplier = 2.0;
    } else if (freq.contains("once daily") || freq.contains("od") || (freq.contains("once") && freq.contains("daily"))) {
        freqMultiplier = 1.0;
    } else if (freq.contains("single dose") || freq.contains("stat dose") || freq.contains("single dose before") || freq.contains("single dose after")) {
        freqMultiplier = 1.0;
    } else if (freq.contains("sos")) {
        freqMultiplier = 1.0;
    } else {
        // unknown frequency -> 0 (safer than making a guess)
        return 0d;
    }

    return doseMultiplier * freqMultiplier;
}

[snehar-nd]

That was a quick fix for the issue, I will implement quality code here.

[sonarqubecloud]

Quality Gate passed

Issues
2 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