Reviewed & tested by the community
Project:
Persistent URL
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Apr 2012 at 01:32 UTC
Updated:
27 Jul 2016 at 16:32 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
benjaminkyta commentedHello. Am having the exact same error with "menu_token"
Could you be knowing how to go about with it?
Thanks
Comment #2
simeWe are in the position where we don't need purl's menu configuration options, and we are now maintaining a patch against purl that prevents it from altering the menu forms. If you look in purl.module you will see that I've modified the function below by adding
return;Comment #3
batje commentedit was more difficult than that. Attached is a patch
Comment #4
batje commentedthis patch also has the changes to purl.admin.inc and
Comment #5
batje commentedwas passing arguments as reference which was causing very funky behaviour.
Comment #6
simeOh nice @batje! I will try and test this next week.
Comment #7
simeIs working for me (applied patch by hand tho) and the only issue I see is:
Notice: Undefined index: menu_name in _purl_item_edit_submit()Comment #8
Yuri commentedYes, after applying the patch the error mentioned above in #7 appears, which refers to
line 353 of /home/testsite/public_html/sites/all/modules/purl/purl.admin.inc
which is:
@batje: can you have a look plz
Comment #9
socialnicheguru commentedmenu_name is probably not defined in the $form_stat['values']
should there be a check to see it is exists before it is called?
Comment #10
jramby commentedMaybe just by changing the :
to :
I'm not sure of my proposition, this function is not documented yet. And I think it just removes the symptom but not the source :-|
Comment #11
rickmanelius commentedI understand the concerns stated in #10 (not sure of symptom/source), however the patch in #5 at least gets us to the point of this functioning.
Marking RBTC and seeing if the maintainer wants to weigh in or just commit and spin off the remaining concerns into another ticket.
Comment #12
Gyver06 commentedApplying the patch in #5 solved my problem.
Why don't you include this patch in the last dev version ?
Comment #13
sigveio commentedPurl also seem to conflict with Token menu, ref. #1528442: Call to undefined function purl_item_edit_validate().
The patch in comment #5 fixes it.
It would be appreciated if the maintainer could include the patch in the latest dev version and/or a new release, so that people do not have to spend unnecessary time trying to work out what's gone wrong when things do not function properly.
(Bumping priority to Major, as this issue renders the entire functionality surrounding adding new menu entires non-functional when one of the conflicting modules are used. That should be a concern large enough to see some priority when there's been a working patch available since April of last year. :))
Comment #14
tom friedhof commentedThe patch in comment #5 fixes this error when using menuimage module as well.
Comment #15
drclaw commentedI also was having an issue with the redirect module's form_alter. To be honest, I'm using purl on a few sites with the redirect module and only one of them has this issue. I want to debug this one a little more, but don't have the time so it's Band-aid for now!
Comment #16
hatsch commentedthe patch in comment #5 seems to fix the error that come with menu_token #1528442: Call to undefined function purl_item_edit_validate()
Comment #17
recrit commentedThe patch in #15 created duplicate function declarations for purl_redirect_edit_validate. The attached patch #17 updates the admin.inc function to "_purl_redirect_edit_validate".
Comment #18
pianomansam commentedThis patch appears to be working for me. I think it's time to be RTBC.