AshishJoshi-asj / zfdatagrid

Automatically exported from code.google.com/p/zfdatagrid
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

object factory patter implemented to generalize export actions #128

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:
I try to generalize the way the export actions could work with as little
code changes as possible.

One requirement for this functionality is that all deploy class
constructors need to accept exactly one parameter $options.

When reviewing my code changes, please focus on:
I had to touch Bvb_Grid_DataGrid.
* example exportAction() in JqgridController
* Bvb_Grid_DataGrid::factory()
* Bvb_Grid_DataGrid::getExports()
* Bvb_Grid_DataGrid::_runConfigCallbacks()
* Bvb_Grid_Deploy_JqGrid and Bvb_Grid_Deploy_Csv support this functionality

I need to know you opinion about this.

Original issue reported on code.google.com by martin.m...@gmail.com on 15 Jan 2010 at 2:47

GoogleCodeExporter commented 9 years ago
Set review issue status to: Invalid

Original comment by martin.m...@gmail.com on 15 Jan 2010 at 2:49

GoogleCodeExporter commented 9 years ago

Original comment by bento.vi...@gmail.com on 3 Mar 2010 at 4:56