FrontAccounting forum — db_pager.inc bug, sql with no "where" keyword error

Re: db_pager.inc bug, sql with no "where" keyword error

2020-01-14T00:32:12Z

The inserted line in the proposed solution is:

$this->from = $parts[1];

There is no "else" in the current codebase nor thru from FA v2.2.1.
It is not there even in @FreeMan's cloned repo as of 4 years ago.

Wonder which version of FA is being used.

Re: db_pager.inc bug, sql with no "where" keyword error

2020-01-13T14:21:08Z

I will check, but @FreeMan, what is the difference between the current sentence and the fixed one?

Joe

Re: db_pager.inc bug, sql with no "where" keyword error

2020-01-13T09:06:10Z

@joe: is this necessary?

db_pager.inc bug, sql with no "where" keyword error

2020-01-12T04:28:26Z

File: includes/db_pager.inc

// parse the query

                $parts = preg_split('/\sFROM\s/si', $sql, 2);
                    if (count($parts) == 2) {
                        $this->select = $parts[0];
                        $sql = $parts[1];
                    } else {
                        if ($SysPrefs->go_debug)
                            display_error("Invalid sql input for db_pager");
                    }

// fixed

                               $parts = preg_split('/\sFROM\s/si', $sql, 2);
                    if (count($parts) == 2) {
                        $this->select = $parts[0];
                        $this->from = $parts[1];
                        $sql = $parts[1];
                        
                    } else {
                        if ($SysPrefs->go_debug)
                            display_error("Invalid sql input for db_pager");
                    }