JPASupport.edit() method problem

Bug #481304 reported by true_cp
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
play framework
Status tracked in 1.0
1.0
Fix Released
Undecided
Unassigned
1.1
Fix Committed
Undecided
Unassigned

Bug Description

line 121 in JPASupport.edit() method:
                        } else {
                            bw.set(field.getName(), o, null);
                        }

This code seems useless and in certain situation it would flush the value to null which has already setup by line 66:
        bw.bind(name, o.getClass(), params, "", o);

I suggest remove this else clause.

Tags: binding
Revision history for this message
Fırat Küçük (firatkucuk) wrote :

I confirm this bug.
this lines overrides the binded values.

acoording to following goole groups thread (nested binding problem) :
http://groups.google.com/group/play-framework/browse_thread/thread/299de0b2d5e58d65

i prepare a branch for it.

tags: added: binding
Changed in play:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.