knop-project / knop

Web application framework for Lasso 8 and 9
18 stars 46 forks source link

Knop_grid header and footer show incorrect maxrecords with -sql #98

Open phillipsIT opened 10 years ago

phillipsIT commented 10 years ago

Demo lib_customer_list.inc <?LassoScript

// find out the current skiprecords value based on the -page parameter and $maxrecords
$skiprecords = $list -> page_skiprecords($maxrecords);

// build search params
// first set some basic search parameters
var('searchparams'=array(-maxrecords=$maxrecords, -skiprecords=$skiprecords));

// use -uselimit only for non-filemaker data sources
!$d->'isfilemaker' ? $searchparams->insert(-uselimit);

// then add sort parameters, if any
$searchparams -> merge($list -> sortparams);

// then add search parameters, if any
$searchparams -> merge($list -> quicksearch);

// get list of records
//$d -> select($searchparams);

  local(sql = 'SELECT * FROM `knopdemo`.`customer` LIMIT ' + $skiprecords + ',' +   $maxrecords)
$d -> select(-sql=#sql)

/* Both the header and footer show the incorrect maxrecords of 50 when it should show 20. /*

// clear any record locks 
$s_user -> clearlocks;

?>

stevepiercy commented 10 years ago

This is an issue with knop_grid->renderfooter. @jolle-c do you have a fix available? If so, I'd like to avoid duplicating efforts to fix this bug. This works fine in Knop for Lasso 8.