[Piwik-trac] [Piwik] #173: Queued Filters should be possible to apply recursively
Piwik
trac at piwik.org
Sat Apr 5 20:49:09 CEST 2008
#173: Queued Filters should be possible to apply recursively
--------------------+-------------------------------------------------------
Reporter: matt | Owner:
Type: Bug | Status: new
Priority: major | Milestone: Future features
Component: Core | Version:
Resolution: | Keywords:
--------------------+-------------------------------------------------------
Changes (by matt):
* milestone: => Future features
Old description:
> For example we do:
> {{{
> $dataTable = $this->getDataTable('Actions_outlink', $idSite,
> $period, $date, $expanded, $idSubtable );
> $dataTable->queueFilter('Piwik_DataTable_Filter_ColumnCallbackAddDetail',
> array('label', 'url', create_function('$url', 'return $url;')));
>
> }}}
>
> But we may want to apply the filter recursively if the returned data
> table has many levels (the case for actions when searching recursively
> for example).
>
> I can see 2 implementations:
>
> * {{{queueRecursiveFilter( Filter )}}}
> * {{{dataTable->setFiltersRecursive() }}}
> both are not incompatible. The first is better in my opinion.
New description:
For example we do:
{{{
$dataTable = $this->getDataTable('Actions_outlink', $idSite,
$period, $date, $expanded, $idSubtable );
$dataTable->queueFilter('Piwik_DataTable_Filter_ColumnCallbackAddDetail',
array('label', 'url', create_function('$url', 'return $url;')));
}}}
But we may want to apply the filter recursively if the returned data table
has many levels (the case for actions when searching recursively for
example).
I can see 2 implementations:
* {{{queueRecursiveFilter( Filter )}}}
* {{{dataTable->setFiltersRecursive() }}}
both are not incompatible. The first is better in my opinion.
--
Ticket URL: <http://dev.piwik.org/trac/ticket/173#comment:1>
Piwik <http://piwik.org>
Piwik, open source web analytics software
More information about the Piwik-trac
mailing list