Using the Table Library in Accessory views

So are you saying that you cannot insert a table without it being added to one already in existence in that view?

Not quite. I think it’s more accurate to say you cannot use the HTML Table Library on a CP page that also uses it. You can always build tables on your own. It also shouldn’t really matter on your own module or extension views because they’re most often called apart from CP pages with tables.

I’ve only encountered this while building views for Accessories. I imagine there may be other instances of this possibility but I have not looked for them.

Whoops, i was logged in on the mrs’ account for that reply!

What if you assign the table class to another name using the loader?

$this->EE->load->library('table', '', 'my_table');

What if you assign the table class to another name using the loader?

$this->EE->load->library('table', '', 'my_table');

It’s been so long since I’ve used CI alone that I forgot you could do that. I’ll give it a go and see if it works. Cheers 😊

No problem, good luck with it!

Looks like that just jacks up all tables in the CP (that are built with this library at least). Nice thought though!

That shouldn’t happen…probably.

You could get creative and do something like this:

$TEST =& get_instance();

$TEST->load->library(‘table’, ”, ‘test_table’);

Worth a shot perhaps…

Erik,

Can you show me the code from the accessory? I have an idea of where the problem’s coming from, but I want to make sure.

Wes

Sure thing Wes. Here’s a filtered down version:

The accessory contains:

function set_sections()
{
    $this->EE =& get_instance();
    $this->EE->load->library('table');    
    $this->sections['Recent Deployment Notes'] = $this->EE->load->view('test', '', TRUE);
}

The views/test.php file contents:

<?php

$this->table->clear();

$this->table->set_heading(
    'heading 1',
    'heading 2',
    'heading 3',
    'heading 4'
);

$this->table->add_row(
    'col 1',
    'col 2',
    'col 3',
    'col 4'
    );

$this->table->add_row(
    'col 1',
    'col 2',
    'col 3',
    'col 4'
    );

$this->table->add_row(
    'col 1',
    'col 2',
    'col 3',
    'col 4'
    );

$this->table->generate();

?>

The end result can be seen in the attached image.

Erik,

Can you please file this a bug? If $this->table->generate() is being used in views, and not calling $this->table->clear(), this behavior will happen. So we need to fix this up. 😊

-greg

Actually, changed my mind. This is a CodeIgniter bug, which is fixed up here (https://bitbucket.org/ellislab/codeigniter/changeset/ba3b278d52e1), and will be on the next EE release.

Thanks!

-greg

Ah okay. Thanks for clearing that up. 😊

I know CI is slow to add features. How is it for patching bugs? In other words, since this is a CI bug how long do you think it will be until it’s patched in an EE release?

Thanks again!

If $this->table->generate() is being used in views, and not calling $this->table->clear(), this behavior will happen.

Just saw this statement. I wanted to verify what you meant. I did, in fact, call $this->table->clear() in my view file. That doesn’t affect the CI bug you found though I presume.