wrapMethodWithArguments and a table view

[steve1376656734]

Hi,

I am using the solution model to add a column to table view. I am assigning a method to the onAction event using the following code:

Code: Select all

var columnField = thisForm.newField(recCaseActionCategory.category_name.replace(" ", "_") + "_icon", JSField.IMAGE_MEDIA, currentX, colHeight, colWidth, colHeight);
columnField.name = recCaseActionCategory.category_name;
columnField.scrollbars = SM_SCROLLBAR.HORIZONTAL_SCROLLBAR_NEVER | SM_SCROLLBAR.VERTICAL_SCROLLBAR_NEVER;
columnField.toolTipText = recCaseActionCategory.category_name;
columnField.onAction = solutionModel.wrapMethodWithArguments(solutionModel.getGlobalMethod("myScopeName", "myMethodName"), [null, recCaseActionCategory.case_action_category_id]);


So far this has worked fine but the requirements have changed and I now need to add an additional argument to the function which is the primary key of the record that is selected when the column field is clicked. My problem is I cannot find a way to pass this as an argument. I have tried the following code:

Code: Select all

var columnField = thisForm.newField(recCaseActionCategory.category_name.replace(" ", "_") + "_icon", JSField.IMAGE_MEDIA, currentX, colHeight, colWidth, colHeight);
columnField.name = recCaseActionCategory.category_name;
columnField.scrollbars = SM_SCROLLBAR.HORIZONTAL_SCROLLBAR_NEVER | SM_SCROLLBAR.VERTICAL_SCROLLBAR_NEVER;
columnField.toolTipText = recCaseActionCategory.category_name;columnField.onAction = solutionModel.wrapMethodWithArguments(solutionModel.getGlobalMethod("myScopeName", "myMethodName"), [null, recCaseActionCategory.case_action_category_id, forms[formName].foundset.getSelectedRecord().getPKs()]);


but this just results in the error:
Stale form(s) detected, form(s) were altered by the solution model without destroying them first

Can anyone let me know how I need to specify my argument such that the called method receives the primary key of the currently selected record when the column field is clicked please?

Thanks
Steve

[patrick]

Looks like your problem is this

forms[formName].foundset.getSelectedRecord().getPKs()

There you already touch forms.xy, so a runtime instance. Once you have a runtime instance, you can no longer modify it. Apart from that, the idea doesn't work anyway from what I understand, because you are hard coding the ID of the first best record into your method call. So a click on any row in your table will always fire the method with the same ID. You should rather deal with the ID in your scope method. There you receive a JSEvent that allows you to figure out which form it was fired from and you can then ask that form for its selected record in your scope method.

[steve1376656734]

Thanks for the info Patrick - I hadn't realised that the call to getPKs() would instantiate the form.

My first thought was to use the event parameter but when I put a breakpoint in the scope method and inspect the arguments the event parameter always seems to be null and I can't figure out why.

[patrick]

It's not the getPKs(), but it's forms.xy that creates the runtime instance. You are asking a form (forms.xy) for something, meaning that form has to be instantiated. Whenever forms. is used, you are talking to a runtime instance.

[steve1376656734]

OK - thanks for clarifying.

Any idea why the event parameter is always Null though?

[patrick]

To make this a bit easier you could also simply create a form method for the onAction that will simply call your scope method. Something like

Code: Select all

var onActionMethod = thisForm.newMethod('function onAction_category_name(event) { myScopeName.myMethodName(event, recCaseActionCategory.case_action_category_id, foundset.getSelectedRecord().getPKs()); }');
columnField.onAction = onActionMethod

[patrick]

The arguments need to be quoted, so solutionModel.wrapMethodWithArguments(solutionModel.getGlobalMethod("myScopeName", "myMethodName"), [null, "'" + xx + "'"]

[steve1376656734]

Great idea - I'm relatively new to using the solutionModel in my application and it never occurred to me to create a form method on the fly.

Thanks.