Ignited DataTables

[eluser]cryogenix[/eluser]
great posting yusuf. i looked at your code and i've got a question and some suggestion:

-> will the simple call:

Code:

$data['result'] = $this->datatables->generate($table, $columns, $index);

still work? or rather, what if the user doesn't want to make joins? based on your code, the user will always end up calling generate with 5 parameters and not with 3 anymore for the previous "non-join" call.

i've created an overloading mechanism starting at line 45 to make the generate call flexible so that it can be passed either with only the simple 3 arguments or 1 arg for sql (still working on this) and 5 for your additional columns and joins.

therefore you will end up with something like:

Code:

case 1:
  $result = $this->__call('generate_from_sql', $args);
  break;
case 3:
  $result = $this->__call('generate_from_tci', $args);
  break;
case 5:
  $result = $this->__call('generate_from_join', $args);
  break;

now this may be repetitive and of course will need refactoring to minimize code repetition but this gives the user the flexibility i am talking about. hope you get my point? i'm kinda sick today so i may be much of a blur to express my thoughts. do send me an email if you need help. i do love collaborative work and your help is much appreciated.

EDIT:
what i am telling is that if we could add your code for the missing joins feature, we would need it refactored in order to still enable the existing library to maintain the functions it currently has.