warning: Missing argument 1 for _token_custom_delete_page() in token_custom\token_custom.module on line 312
on admin/build/tokens/delete page.

Comments

hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Active » Postponed (maintainer needs more info)

How did you access 'admin/build/tokens/delete'? I don't see any link to it from within the module, have you typed it manually? That URL alone should not be linked from anywhere, there should always be an integer appended:
admin/buid/tokens/delete/, that works OK.

kenorb’s picture

kenorb commented

I found it by mistake, when I wanted to remove manually one custom token, when I couldn't remove it in normal way because it was behind my right block.
It's not linked anywhere, but in my opinion all warnings should be fixed for security reasons.
If not, that's fine.

hanoii’s picture

hanoii
Primary language Spanish
Location 🇦🇷UTC-3
commented
Status: Postponed (maintainer needs more info) » Fixed

After further thoughts, I improved the menu callbacks with better paths and the use of the menu loaders. Some functions were removed and the warning would not happen again, committed to D6 branch, not sure if I will release a new version with this though.

Status: Fixed » Closed (fixed)

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