[ http://jira.andromda.org/browse/BPM-256?page=comments#action_12129 ]
     
Igor Kuzmitshov commented on BPM-256:
-------------------------------------

OK, I will try in a few days (I have never tried to modify a cartridge before 
:) ).

> 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.




-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. 
Download it for free - -and be entered to win a 42" plasma tv or your very
own Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php

Reply via email to