Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I installed 7.x-1.2 and also DataTables 1.9.4. When I tried to access the table, i get the error:
Undefined index: table_tools in template_preprocess_datatables_view() (line 348 of /home5/retamavi/public_html/sites/all/modules/datatables/datatables.module).
I insured I had placed the media folder where it needed to be.
I found DataTables 7.x-1.1 and it worked fine. It appears to be a compatibility issue.
Comment | File | Size | Author |
---|---|---|---|
#8 | fix-undefined-index-table-tools-1951760-8.patch | 534 bytes | Elin Yordanov |
#5 | screenshot-2014-10-07 22:19:32.png | 158.98 KB | formatC'vt |
#3 | fix-undefined-index-table_tools-1951760.patch | 572 bytes | formatC'vt |
Comments
Comment #1
brussam CreditAttribution: brussam commentedIncorrectly assigned to myself
Comment #2
Ranieri Machado CreditAttribution: Ranieri Machado commentedFor get rid of the annoying message just check if the array element exists:
Comment #3
formatC'vt CreditAttribution: formatC'vt commentedComment #4
Elin Yordanov CreditAttribution: Elin Yordanov commentedUsing 7.x-1.x-dev cannot reproduce it. Please try it with dev version and re-open the issue if you still have this problem.
Comment #5
formatC'vt CreditAttribution: formatC'vt commentedThis is easy to reproduce, use clean drupal install.
Comment #6
Elin Yordanov CreditAttribution: Elin Yordanov commented@formatC'vt
You should know that all new code will be made into dev-versions of modules, i.e. 7.x-1.x-dev. Please install the dev version, not 7.x-1.2, and you'll see that the issue is not there.If you still see the issue, then provide a new patch, since your patch doesn't apply to the dev version.Comment #7
Elin Yordanov CreditAttribution: Elin Yordanov commented@formatC'vt
I'm sorry, you're right. The issue is still there. I was on the master branch, not on dev, that's why I couldn't reproduce it at the first time.
I'll check your patch as soon as I got more time (give me 2 days) and I'll commit it if it works.
Comment #8
Elin Yordanov CreditAttribution: Elin Yordanov commentedI've a little improved the patch, reducing the redundant check.
Comment #9
Elin Yordanov CreditAttribution: Elin Yordanov commentedSince this is a trivial check, I've just pushed the patch to the dev-Version.
Thanks @Ranieri Machado and @formatC'vt.
Comment #12
ayushmanshrestha CreditAttribution: ayushmanshrestha commented#8 worked for me.
Thanks!