Forum Discussion

RaptureLazarus's avatar
RaptureLazarus
Ideator I
5 years ago
Solved

Client side Form function and script not evaluating correctly.

Good day all. TL;DR: I am having trouble getting an onLoad method to reliably print an evaluated value to a field. The form is as follows:  Stock message contains: [embed:dc8ab71f-3b9...
Forms
JavaScript
Methods
  • AngelaIp's avatar
    AngelaIp
    5 years ago

    Are your fields "string" or "integer" properties? According to your debugger, they look like strings! In this case greater and less may calculate based on wrong values.

    Try to parse the value  --> 

    parseInt(minStock, 10)

    And maybe use an onFormPopulated event instead of onLoad, when the fields shall also update when user edit the item.

RaptureLazarus's avatar
RaptureLazarus
Ideator I
5 years ago

Thanks for the quick reply.

The function only evaluates the two fields and displays a message when the item is opened. The only time when the values might change is when a user or future feature, changes it.

I added an ifmsg variable to show which is triggered more easily.

Here is an image of the debugger.


AngelaIp's avatar
AngelaIp
Ideator I
5 years ago

Are your fields "string" or "integer" properties? According to your debugger, they look like strings! In this case greater and less may calculate based on wrong values.

Try to parse the value  --> 

parseInt(minStock, 10)

And maybe use an onFormPopulated event instead of onLoad, when the fields shall also update when user edit the item.

  • RaptureLazarus's avatar
    RaptureLazarus
    Ideator I
    5 years ago

    Ahh, that was it, thanks!

    The properties for those values are set to integer, but it's fine now.

    If I set the method to trigger onFormPopulated, it just adds to the existing text with each render of the form. Is there a way around that?

    • AngelaIp's avatar
      AngelaIp
      Ideator I
      5 years ago

      When using onFormPopulated your Method will be triggered on any refresh, so your custom html will then be appended to the existing elements.

      Some rocket science helps: Just erase the existing content before applying the new one:

      E.g.

      var field = document.getElementById("stock_msg");
      field.innerHTML ="" ;

      More efficient would be, to just update the affected fields instead of rebuilding the complete div. But as you only show limited information, performance and loading time should be an issue.