[TAP5-1340] HibernateEntityValueEncoder shouldn't throw an exception when the entity being encoded isn't persisted yet

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Invalid
• Affects Version/s: 5.2.3
• Component/s: tapestry-hibernate
• Assignee: Robert Zeigler
• Reporter: Hugo Palma
• Created: 10/Nov/10 17:53
• Updated: 22/Aug/11 17:52
• Resolved: 22/Aug/11 17:52

Description

The code:

if (id == null)
throw new IllegalStateException(String.format(
"Entity %s has an %s property of null; this probably means that it has not been persisted yet.",
value, idPropertyName));

means that i can never encode an entity class before it's persisted. The correct behaviour should be:

1. HibernateEntityValueEncoder - if id is not null proceed with encoding.
2. HibernateEntityValueEncoder - if id is null then pass object to the next encoder on the chain.