Closed (fixed)
Project:
Display Suite
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Mar 2016 at 03:47 UTC
Updated:
27 Apr 2016 at 12:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
swentel commentedNot sure if that is our problem since we wrap it in a module exists check and token_tree theme function seems to be defined (at least in my version of token, should maybe update)
Comment #3
spacereactor commentedI just test with https://simplytest.me/ drupal 8.1.0 rc with token, Display Suite, Layout plugin and Admin Toolbar. After install, go to extend and on the following modules. Admin Toolbar, Admin Toolbar Extra Tools, Display Suite, Layout Plugin, Token.
Flush all caches and run cron, go to structure -> content types -> article -> manage display.
Change layout for article in default to any ds layout. Check recent log messages, and refresh articles manage display. you have another Theme hook token_tree not found. in your log.
simplytest.me using php 5.5 and apache so i don't think got anything to do with server settings
Comment #4
spacereactor commentedi also try ds 8.x-2.3 same problem. Maybe need to change token_tree to token_tree_link
https://www.drupal.org/node/2640138
Comment #5
spacereactor commentedAt ds/src/Plugin/DsLayout.php
to
To remove the "Theme hook token_tree not found." error each time ds manage display is use.
I also find ds/src/Form/TokenFieldForm.php using token_tree.
Comment #6
4aficiona2 commentedAfter enabling a Display Suite layout (default, One column layout) on a node bundle (content type) I can also see the warning message in the Watchdog
Theme hook token_tree not found..What I also observed is when I enable a Display Suite layout that
HOOK_preprocess_node()is not anymore triggered for this node. Might those issues be related? I filed an issue on Drupal Exchange for that already a couple of days ago.I use Drupal with version 8.0.6, DS 8.x-2.3, Layout Plugin 8.x-1.0-alpha22, Token 8.x-1.0-alpha2 and some more modules.
Comment #7
swentel commentedSo yeah, we need to change the link. Patch attached, will commit in a few.
Comment #9
swentel commentedcomitted and pushed.