I noticed on my sites status reports, I got the following warning

TOKENS OR TOKEN TYPES MISSING NAME PROPERTY - $info['tokens']['node-field_editor_notes']['value']

Comments

anthonyroundtree created an issue. See original summary.

Rostislav Sergeenkov’s picture

Title: Missing Tokens or Token error on status report page » Implement token support, fix "Missing Tokens or Token error" on status report page

Bug report was transformed to feature request "Implement token support".

Rostislav Sergeenkov’s picture

Category: Bug report » Feature request
promo-il’s picture

+1 PHP:7.1 D:8.8.5

codesmith’s picture

I wa able to remove the warning by changing the following code in src/Plugin/FieldType/EditorNoteItem.php in the propertyDefinitions() function

Change:
$properties['value'] = DataDefinition::create('string');

To:
$properties['value'] = DataDefinition::create('string')->setLabel(t('Editor Note'));

I can make a patch if requested.

driverok’s picture

Status: Active » Postponed (maintainer needs more info)

Hi.

Thanks for using this module.

Unfortunately, I wasn't able to reproduce the issue.
For now, the module is not supporting any tokens.
If the tokens are required, please provide more information about the use case.

Until that, postponing the issue.

duckydan’s picture

StatusFileSize
new94.33 KB

I am seeing this as well.

System Lord’s picture

Same issue here.

D939

Might be an issue with the themes. I'm using core bartik and the form doesn't appear on the content type, but it does appear when on Seven (for admin). The table of notes appears on bartik after the Admin enters a note, but again, no form for the editor/author to post.

Permission are set for authenticated users.
No custom field settings (ie other module preferences).

delty’s picture

Having the same issue here. I know it's not hurting the site in any way, I just like to see a clean status report. Maybe @codesmith can roll a patch?

katekuratsu’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new644 bytes

Got the same error on 9.4.3. This patch works for me.

driverok’s picture

Status: Needs review » Fixed

Thanks everyone, the fix is merged

Status: Fixed » Closed (fixed)

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