Description: ------------ I guess I'm pushing the limits of the formatter functionality, or I do not see how it will be possible with the current code. I would like to be able to pass an (associative) array as parameter to the formatter. This would allow me to pick the correct text to be printed in the column based on record data which can be referenced to the array. Test script: --------------- $list=array('active','open','closed'); $column->setFormatter('printAssoc', $list); function printAssoc($params) { extract ($params); print $list[$record[$fieldName]]; }

Using array is indeed not possible yet. Wouldn't a "global $list;" in printAssoc() or defining $list in printAssoc() solve your problem?

Just for the records: The last comment came from me.

"...or defining $list in printAssoc() solve your problem?" That is about how I do it now, although I have to make specific formatters for each column that need to display an associative name (e.g. _formatYesNo or _formatStatus). 'printAssoc' would be a more generic library class function. p.s. I noticed you also include fieldName, columnName etc to the paramList. Great! This obviously is needed for creating generic formatters where the field name is not known.

Okay, maybe we can make the formatter more flexible in the future. For now, the work on the refactoring of the rendering layer is more important.

I agree with Chrisje about this. This second argument would make setFormatter() ressemble call_user_func().

This is fixed in CVS. You can now do : $data = array(...); $column->setFormatter($callback, $data); Or : $column = new Structures_DataGrid_Column(null, null, null, null, null, $callback, $data); Thanks for your report

Any confirmation on that this works? I cant get my test code to work. $dg->addColumn(new Structures_DataGrid_Column('Description', 'description', 'description', null, null, 'printTest', array(length=>150))); function printTest($params) { extract($params); echo "length: " . $length; }

Forgot to mension that $length paramether return some reason empty string.

rippe, at first: array(length=>150) should be array('length' => 150), but that's not the problem here, as PHP generates only a notice and is smart enough to guess that you mean the string 'length' instead of the undefined constant length. The problem is your formatter. This should work: function printTest($params, $args) { extract($args); echo "length: " . $length; } User defined arguments like your array with the length information are available via the new second parameter that is passed to the formatter callbacks.

Excellent wiesemann! I would newer have quesses how to fix it using my litle knowledge or the offical pear documentation. I found on Internet old example which I changed so it is compatiple with new datagrid version. It would be nice if the offical datagrid example could be reedited to include a extra $formatterArg example. That would help others a lot. http://pear.php.net/manual/en/package.structures.structures-datagrid.example-advanced.php This works for me, hopefully this helps some one struling with same task. $dg->addColumn(new Structures_DataGrid_Column('Description', 'description', 'description', null, null, 'printDesc', array('length'=>15))); function printDesc($params, $args) { extract ($args); extract ($params); if (strlen($record['description']) > $length) { return nl2br(substr($record['description'], 0, $length)) . '...'; } else { return nl2br($record['description']); } } Thanks again wiesemannn for the fast fix!

> Excellent wiesemann! You may call me Mark. ;-) > I would newer have quesses how to fix it using my litle > knowledge or the offical pear documentation. Good point, thanks for your example code. I've add an item on my ToDo list about updating the example, or adding another one. This might take some time, but I won't forget it. BTW: Although this was directly related to this request, please ask support questions on pear-general@lists.php.net. Thanks and regards, Mark

OK thanks mark ;-)