[ http://jira.andromda.org/browse/BPM-256?page=comments#action_12251 ] Wouter Zoons commented on BPM-256: ----------------------------------
hi Igor, I'm afraid that detecting all possible cases of value objects would be a bit overkill here .. IMO it's okay to have the generate webapp code not compile in the implementation (you can always safely edit this file), the feature of having the fully deployable webapp is a nice extra for when you want to give a quick demo, but using it on applications that would go into production is bit much I think handling all possible cases of collections/arrays would overly complicate the template(s) for that reason I'm closing this issue .. please comment if you disagree or if you think there's a practical solution to this problem > Dummy list should use a factory, not "new" > ------------------------------------------ > > Key: BPM-256 > URL: http://jira.andromda.org/browse/BPM-256 > Project: Bpm4Struts Cartridge > Type: Bug > Versions: 3.1-RC1 > Reporter: Igor Kuzmitshov > Assignee: Wouter Zoons > > Currently generated dummy list uses "new" to create dummy data: > private static final com.my.data.Phone[] phoneListDummyList = > new com.my.data.Phone[] > { > new com.my.data.Phone(null, "number-1"), > new com.my.data.Phone(null, "number-2"), > new com.my.data.Phone(null, "number-3"), > new com.my.data.Phone(null, "number-4"), > new com.my.data.Phone(null, "number-5") > }; > resulting in compilation error (because entity representing class is > com.my.data.PhoneImpl): > .../PhoneControllerImpl.java:56: com.my.data.Phone is abstract; cannot be > instantiated > new com.my.data.Phone(null, "number-5") > ^ > A factory could be created to populate the list. ------------------------------------------------------- This SF.Net email is sponsored by the JBoss Inc. Get Certified Today * Register for a JBoss Training Course Free Certification Exam for All Training Attendees Through End of 2005 Visit http://www.jboss.com/services/certification for more information
