Issue Details (XML | Word | Printable)

Key: APF-936
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Matt Raible
Reporter: Franz Allan Valencia See
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
AppFuse

BaseDaoTestCase#populate( Object ) does not work

Created: 11/Oct/07 12:41 PM   Updated: 12/Dec/08 02:31 AM   Resolved: 12/Dec/08 02:31 AM
Component/s: Tools - AppGen
Affects Version/s: 2.0 Final
Fix Version/s: 2.1.0-M1

File Attachments: 1. Text File org.appfuse.dao.BaseDaoTestCase.patch (0.5 kB)

Environment:
Maven version: 2.0.7
Java version: 1.5.0_11
OS name: "linux" version: "2.6.20-16-generic" arch: "i386"


 Description  « Hide
BaseDaoTestCase#populate( Object ) uses spring-2.0.6's BeanUtils.copyProperties( HashMap, Object ). However, I think the first parameter of copyProperties(...) should have the same properties of the 2nd parameter for it to work properly.

Matt Raible made changes - 11/Oct/07 01:03 PM
Field Original Value New Value
Fix Version/s 2.0.2 [ 10110 ]
3130 by  Matt Raible (5 files)
04/May/08 11:04 AM (27 months, 14 days ago)
Matt Raible made changes - 04/May/08 11:04 AM
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Anil Gangolli added a comment - 21/Sep/08 12:36 AM

I am seeing this in 2.0.2

The problem is that the call to BeanUtils.copyProperties is reversed. The destination is supposed to be the first parameter, the second is supposed to be the source.

The call at BaseDaoTestCase:76 should be

BeanUtils.copyProperties(obj, map);


Anil Gangolli added a comment - 21/Sep/08 10:04 AM
This is a patch based on the version of org.appfuse.dao.BaseDaoTestCase in appfuse-hibernate-2.0.2.jar


Anil Gangolli made changes - 21/Sep/08 10:04 AM
Anil Gangolli added a comment - 21/Sep/08 10:04 AM
Hey! There's no way to reopen issues !?

Matt Raible made changes - 30/Sep/08 02:38 PM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Matt Raible made changes - 30/Sep/08 02:38 PM
Fix Version/s 2.1 [ 10031 ]
Fix Version/s 2.0.2 [ 10110 ]
3192 by  Matt Raible (3 files)
12/Dec/08 02:30 AM (20 months, 9 days ago)
Matt Raible added a comment - 12/Dec/08 02:31 AM
Thanks for the patch!

Matt Raible made changes - 12/Dec/08 02:31 AM
Status Reopened [ 4 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]