One of a few warnings/errors with a fresh install of Panopoly beta3

Tokens Problems detected

The following token types are not defined but have tokens:

$info['types']['fieldable_panels_pane']

Comments

populist’s picture

Project: Panopoly » Fieldable Panels Panes (FPP)
Version: 7.x-1.0-beta3 » 7.x-1.x-dev
Component: Install » Code

I am pretty sure this is an issue related to Fieldable Panels Panes. From my brief debugging, the UUID and VUUID is being provided as tokens but they are not also being defined as types.

merlinofchaos’s picture

It's possible we're somehow incorrectly turning on uuid support but I think uuid module handles all of this?

populist’s picture

Project: Fieldable Panels Panes (FPP) » Universally Unique IDentifier
Status: Active » Needs review

I did some more analysis and tracked this down to the UUID module. At issue here is the call in uuid_token_info() which defines the token types for all entities, but since FPP doesn't have any token it kicks out that error.

This could technically be fixed in FPP by adding a fieldable_panels_panes.tokens.inc file that defines some types, but think its easier to do it in the UUID module. I don't have a ton of UUID or token experience so this will likely take some review.

merlinofchaos’s picture

A token for the pane title would be sensible.

populist’s picture

StatusFileSize
new970 bytes

Here is my patch from #3, sorry about that!

I also opened an issue in FPP around token support #1798482: Improved(?) Token support and referenced merlinofchaos' suggestion for a title token.

populist’s picture

StatusFileSize
new752 bytes

I found a bit of trouble with the patch from #5, so I implemented a cleaner patch. This one adds a hook_token_info_alter() hook that removes any token types added by UUID that do *not* have tokens enabled. I tested this with Fieldable Panel Panes (which it removes since it doesn't have token), but it keeps the tokens from user / node and other entities.

gilgabar’s picture

Marked #1269918: token type not defined : $info['types']['field_collection_item'] as a duplicate of this issue as it affects other entities, e.g. field collections, as well.

The patch in #6 fixes the status message for me.

jaypark’s picture

have latest uuid enabled, 7.x-1.x branch commit:

commit 4bbf92d8c07485492c0f1065cb3647db492e1029
Author: Dave Hall
Date: Sun Feb 3 12:06:50 2013 +1100

Add URI hooks to hook_info()

applied patch at #6, still receive status message, but for field_collection_item:

The following token types are not defined but have tokens:
$info['types']['field_collection_item']
mgifford’s picture

Patch still applies nicely to the git repo. @populist have you bundled your patch into Panopoly? If so that's certainly had a lot of testing at this point.

How many of the uuid patches are you using in the Panopoly distro? Would be good to know what you're already using and what testing has already been done.

Ideally we'd just have a stable release so you don't have to patch uuid or use the dev release.

pangus’s picture

Patch #6 got rid of $info['types']['field_collection_item'] message for me - thanks!

populist’s picture

Status: Needs review » Reviewed & tested by the community

@mgifford - I am only using the patch in #6 above for UUID support, but may remove UUID entirely as per #1995066: Remove UUID Module from Panopoly?

@pangus - Good to hear. I am marking this RTBC since it does solve a lot of problems for Panopoly users and works for Field Collection also.

temkin’s picture

Just applied patch from #6 and the status message is gone. Thank you!

dixon_’s picture

Status: Reviewed & tested by the community » Fixed

Patch from #6 committed to 7.x-1.x-dev. Thanks all! I'll probably roll another release very soon.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.