diff options
| author | Lester Caine <lester@lsces.co.uk> | 2026-06-03 20:16:42 +0100 |
|---|---|---|
| committer | Lester Caine <lester@lsces.co.uk> | 2026-06-03 20:16:42 +0100 |
| commit | fcd97be1f1fd1587cc9954adfa84447c4b98d9c9 (patch) | |
| tree | fb0f7f84298c3e6d77515806926da9e9b6264ff7 /includes | |
| parent | 935b0df4284f5d78cdc3c0bccb4a18431d5449d2 (diff) | |
| download | stock-fcd97be1f1fd1587cc9954adfa84447c4b98d9c9.tar.gz stock-fcd97be1f1fd1587cc9954adfa84447c4b98d9c9.tar.bz2 stock-fcd97be1f1fd1587cc9954adfa84447c4b98d9c9.zip | |
stock: fix pagination on list_movements
StockMovement::getList() was missing a COUNT query so postGetList()
defaulted cant to max_records, giving total_pages=1 always. Also pass
filter params (ref_type, find, component_content_id, assembly_content_id)
through pagination links so filters survive page navigation.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'includes')
| -rw-r--r-- | includes/classes/StockMovement.php | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/includes/classes/StockMovement.php b/includes/classes/StockMovement.php index 5ee4aae..6e07cbf 100644 --- a/includes/classes/StockMovement.php +++ b/includes/classes/StockMovement.php @@ -246,6 +246,15 @@ class StockMovement extends LibertyContent { $whereSql = substr_replace( $whereSql, ' WHERE ', 0, 4 ); } + $pListHash['cant'] = (int)$this->mDb->getOne( + "SELECT COUNT(DISTINCT lc.`content_id`) + FROM `".BIT_DB_PREFIX."liberty_content` lc + INNER JOIN `".BIT_DB_PREFIX."users_users` uu ON uu.`user_id` = lc.`user_id` + $joinSql + $whereSql", + $bindVars + ); + $X = BIT_DB_PREFIX; $cmpQtySelect = $this->verifyId( $pListHash['component_content_id'] ?? 0 ) ? ", xcmp.`item` AS cmp_qty_type, CAST(xcmp.`xkey` AS DOUBLE PRECISION) AS cmp_qty" |