Closed (won't fix)
Project:
Weblink
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 May 2005 at 16:17 UTC
Updated:
30 Jan 2006 at 18:42 UTC
Jump to comment: Most recent file
With the new weblink module the visit link does not work anymore.
greetings
MM
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | weblink.module_1.patch | 397 bytes | mgifford |
Comments
Comment #1
Mad Maks commentedthis error is in my log when somebody tries to viit a link in the list
Comment #2
Mad Maks commentedi only have this problem at a site that uses php 5, an othet site that is running on a older php version has no problem
Comment #3
Bèr Kessels commentedThanks for reporting. PHP5 is very strict about arrays ans objects . will look into this later.
Comment #4
Mad Maks commentedwhat is the status?
Comment #5
Bèr Kessels commentedno change yet. but this week the links bundle will be released. look out for that :)
Comment #6
ron_mahon commentedWhat is the name of the links bundel? I cant find it.
Thanks
Comment #7
Bèr Kessels commentedIt does not yet exist.
Comment #8
mahcsig commentedI got mine to work by making the 'callback arguments' in weblink_menu an array like this:
'callback arguments' => array(arg(2)),
don't know if this breaks anything else, but it seems to work for me
Comment #9
ron_mahon commentedBer
I am getting the same error at the top of the screen.
Warning: array_merge() [function.array-merge]: Argument #1 is not an array in W:\www\inmrc\includes\menu.inc on line 351.
Using PHP5, Drupal 4.6.3, and weblink from Head
Do you think the error is in the weblink module?
Or the menu.inc file?
Or Both?
Thanks
Ron
Comment #10
Bèr Kessels commentedthe problem is almost certainly in weblink module.
Comment #11
19th commentedI have replaced
$arguments = array_merge($arguments, explode('/', $arg));with
$arguments = array_merge((array) $arguments, explode('/', $arg));in menu.inc and it works ok.
Comment #12
Timotheos commentedAny chance some of you are using the article module to create a directory of web links like I am?
I found that if I change the article.module line 102 to
Then the menu.inc on line 351 error goes away.
Comment #13
dumell commentedI followed the advice by 19th (in #11) and changed line 351 in menu.inc and now weblinks work again. Weblinks stoped working when we moved Drupal 4.6.4 from one hosting company to another and it turns out the old one used PHP4 and the new one PHP5. I had not thoguht about this difference before as Drupal seemed to work in general, this is the only problem so far that we have run into because of PHP5.
Comment #14
mgiffordThe weblink patch worked for me.
Comment #15
mgiffordComment #16
Bèr Kessels commentedThanks you for your work. But CVS is no longer maintained. Everythnig has moved over to links.http://drupal.org/node/24719