Skip to content
This repository was archived by the owner on Apr 17, 2019. It is now read-only.

L5 fix #252

Open
wants to merge 6 commits into
base: version3
Choose a base branch
from
Open

L5 fix #252

wants to merge 6 commits into from

Conversation

bmartus
Copy link

@bmartus bmartus commented Mar 6, 2015

This is a temporary fix while you work through your rewrite that you've mentioned in some issue comments. It's based on a fork of bart/datatable l5 branch, with a couple more fixes to make it work out of the box, as far as I can tell.

@hillelcoren
Copy link

@Chumper

Any plans on merging this into the master?

Thanks

@yamilrevilla
Copy link

@hillelcoren @bmartus

Hi guys,
i'm getting a very weird issue when a follow this steps in my L5 project.

The views are returned encoded, like this:

 <table id="pEOW3vx2" class="table table-bordered">
    <colgroup>
                <col class="con0" />
                <col class="con1" />
                <col class="con2" />
            </colgroup>
    <thead>
    <tr>
                <th align="center" valign="middle" class="head1">id</th>
                <th align="center" valign="middle" class="head2">Nombre</th>
                <th align="center" valign="middle" class="head3">Slug</th>
            </tr>
    </thead>
        <tbody>
        </tbody>
</table>

    <script type="text/javascript">
    jQuery(document).ready(function(){

        // dynamic table
        oTable = jQuery('#pEOW3vx2').dataTable({
            "sPaginationType": "full_numbers",
            "bProcessing": false,
                "sPaginationType": "full_numbers",
                "bProcessing": false,
                "sAjaxSource": "http:\/\/localhost:8000\/test",
                "bServerSide": true,


        });

    oTable.addClass('table-striped ');

    });
</script>

I tried everything but i can't figure it out. Can you please help me?

@bmartus
Copy link
Author

bmartus commented Apr 2, 2015

Are you using the new blade tag syntax around the Datatable tags? {!! !!}

@minkbear
Copy link

@yamilrevilla I comfirmed as @bmartus mention above {!! !!}. It's work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants