Skip to content

Commit

Permalink
fixed: delete rows - would load all records in the current list view,…
Browse files Browse the repository at this point in the history
… rather than just the selected ones. Meant that all fileupload images would be deleted rather than just those in the records selected by the user
  • Loading branch information
pollen8 committed Aug 16, 2013
1 parent 5ff754e commit dd9aa91
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion components/com_fabrik/models/list.php
Original file line number Diff line number Diff line change
Expand Up @@ -7745,7 +7745,8 @@ public function deleteRows(&$ids, $key = '')
{
$nav = $this->getPagination($c, 0, $c);
}
$this->_whereSQL['string'][true] = ' WHERE ' . $key . ' IN (' . $val . ')';
$this->setPluginQueryWhere('list.deleteRows', $key . ' IN (' . $val . ')');

/* $$$ hugh - need to clear cached data, 'cos we called getTotalRecords from the controller, which now
* calls getData(), and will have cached all rows on this page, not just the ones being deleted, which means
* things like form and element onDelete plugins will get handed a whole page of rows, not just the ones
Expand Down

0 comments on commit dd9aa91

Please sign in to comment.