Create form based on existing row : advice from specialists

Question

hello,
i've coded a "create form" based on a row selected by the user.
this create form displays the same information as the selected row (except for the primary key which is a sequence trigger-calculated) the user can then modify any field and then commit the creation. It's quicker for tables with many columns since the user will override only several fields instead of setting every field one by one.
I've done it this way :
1) creation of a createCustomMethod() in my ViewObject implementation class and published it in the client interface.
2) drag and drop the method as a command button on the selection list page
3) set the action of the command button to the navigation case value to open the create form when the button is clicked.
the code of the createCustomMethod() is the following
public void createCustomMethod() {       
Row currentRow = this.getCurrentRow();
Row newRow = this.createRow();
newRow.setAttribute("Attribute1",currentRow.getAttribute("Attribute1"));
newRow.setAttribute("Attribute2",currentRow.getAttribute("Attribute2"));
/* more initialization code here */
this.insertRow(newRow);
this.setCurrentRow(newRow);
it works fine but as i'm knew to ADF BC is this a good solution ? are there any drawbacks to it ?
thanks
-regards

Answer 1

what's the advantage of exposing it in the application module rather than in the view object ?
Message was edited by:
user562278
i've tried implementing the method in the application module (i had to call the findViewObject() now) but the "Row currentRow = vo.getCurrentRow();" instruction throws a NPE.