This patch fixes the Notice message displayed when generating content via the Devel module.
It simply checks to make sure the value is set, returns null if it doesn't.


Notice: Undefined index: extension in filefield_paths_tokens() (line 56 of /Volumes/Stuff/htdocs/php/undefeated.my/sites/all/modules/filefield_paths/modules/token.inc).

CommentFileSizeAuthor
#5 filefield_paths-token-1601104.patch501 bytesandypost
#1 filefield_patchs-undefined-index-extension-1601104.patch637 bytestseven

Comments

tseven’s picture

tseven commented
StatusFileSize
new filefield_patchs-undefined-index-extension-1601104.patch637 bytes
sinasalek’s picture

sinasalek commented
Status: Needs review » Reviewed & tested by the community

Just tried the patch and it fixed the issue

sw3b’s picture

sw3b commented

+1 on the patch also ! good work

andypost’s picture

andypost
he/him
Primary language Russian
commented

use patch -p5 < filefield_patchs-undefined-index-extension-1601104.patch to fix

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new filefield_paths-token-1601104.patch501 bytes

Re-roll with right format

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented

Just to confirm, are you uploading a file that has no extension? There's nothing wrong with the patch itself, but I want to confirm it's even necessary before committing it.

ColdSun’s picture

ColdSun commented

If someone uses the media module with the media_youtube submodule, there would be no extension.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Reviewed & tested by the community » Fixed

Fix committed, thanks for the work.

Status: Fixed » Closed (fixed)

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