fix: Preserve non-PK conditions in entity find when full PK present

[hunterino]

Summary

Fixed a bug where EntityFindBase.oneInternal() would strip all non-PK conditions when a full primary key was provided, treating them as "over-constrained".

Root Cause

Lines 783-796 in EntityFindBase.groovy contained logic that:

1. Detected when a full PK was present along with additional conditions
2. Created a new map containing ONLY the PK fields
3. Replaced the original condition map, silently discarding all non-PK conditions

Example of the Bug

// Before fix: status condition was silently IGNORED
ec.entity.find("Order")
    .condition("orderId", "12345")      // PK
    .condition("orderStatus", "ACTIVE") // Non-PK - WAS DISCARDED!
    .one()
// Would return order 12345 regardless of its status!

The Fix

• Removed the problematic code block (14 lines deleted)
• Added explanatory comment documenting the fix
• Added comprehensive test case

Test Plan

• Added test "find with PK and non-PK conditions should honor all conditions"
• Verified fix compiles
• Run full test suite (./gradlew test)

Upstream Reference

• Issue: moqui/moqui-framework#606

Fixes #14

🤖 Generated with Claude Code