Notice: Undefined index: value in tokenauth_text_load() (line 187 of tokenauth/tokenauth.inc).

Notice: Undefined index: format in tokenauth_text_load() (line 190 of tokenauth/tokenauth.inc).

Patch to follow.

CommentFileSizeAuthor
#1 1524578-tokenauth-undefined_index_notices_on_user-1.patch1.9 KBsteve.elkins

Comments

steve.elkins’s picture

steve.elkins commented
StatusFileSize
new 1524578-tokenauth-undefined_index_notices_on_user-1.patch1.9 KB

Instead of $tokenauth_text['value'] and $tokenauth_text['filter']...

should be looking at $tokenauth_text['body']['value'] and $tokenauth_text['body']['filter'].

Proposed patch attached.

steve.elkins’s picture

steve.elkins commented
Status: Active » Closed (works as designed)

Closing, this is not an issue. I was using a sandbox version and still had old records in the variable table. Apologies for any confusion.

Grayside’s picture

Grayside commented
Status: Closed (works as designed) » Needs work

Hi steve,

Your patch should be generated from the root of the module, not the site. use git diff --relative from any directory to localize it within your repo. That said, I applied the patch successfully with a bit of manual work.

Your patch does not make sense to me. The form element that stores the text settings saves with 'value' and 'format', and those are consistently used.

Did you encounter this in a site upgrade from D6?

steve.elkins’s picture

steve.elkins commented

Hi, just wanted to make sure you saw my comment above (#2). The patch absolutely didn't make any sense after I looked in the variable table. I was initially using a roughly ported D7 version that I found in another post just to test a few things and did not properly uninstall before I enabled the official D7 dev release. So all problems were on my end. Also, yes, absolutely meant to create the patch from the module root.

Grayside’s picture

Grayside commented
Status: Needs work » Closed (works as designed)

Ah, I did see the email, but only after I cross-posted the above.

FYI, access control on the dev snapshot from earlier was broken, earlier this afternoon I pushed a fix that requires a cache clear.