Topic: Add input data check for Gl Setup

Should add these checks for function can_process() gl_setup.php

if (!check_num('past_due_days', 0, 100))
    {
        display_error(_("The past due days interval allowance must be between 0 and 100."));
        set_focus('past_due_days');
        return false;
    }

    if (!check_num('default_quote_valid_days', 0))
    {
        display_error(_("Quote Valid Days is not valid number."));
        set_focus('default_quote_valid_days');
        return false;
    }

    if (!check_num('default_delivery_required', 0))
    {
        display_error(_("Delivery Required By is not valid number."));
        set_focus('default_delivery_required');
        return false;
    }

    if (!check_num('default_receival_required', 0))
    {
        display_error(_("Receival Required By is not valid number."));
        set_focus('default_receival_required');
        return false;
    }

    if (!check_num('default_workorder_required', 0))
    {
        display_error(_("Work Order Required By After is not valid number."));
        set_focus('default_workorder_required');
        return false;
    }
Phuong

Re: Add input data check for Gl Setup

Ok, willdo.

Joe

Re: Add input data check for Gl Setup

This has now been fixed and committed to stable repo. Thanks @notrinos.

The fixed file can be downloaded here.

Joe

Re: Add input data check for Gl Setup

A generic function encapsulating the error displayed and the checks may be in order. This may need to be done in a multitude of forms bloating the codebase otherwise.

@joe: There have been more than 50 commits since the last version release over 6 months ago - time for a another one with all translation strings as well?

Re: Add input data check for Gl Setup

We are planning a minor 2.4.5 release soon.

We just have to examine the Mantis Bug Tracker.

Joe

Re: Add input data check for Gl Setup

Should we add this modification to the next release also ?
https://frontaccounting.com/punbb/viewtopic.php?id=7764

Phuong

Re: Add input data check for Gl Setup

I will have a look.

Joe