Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 CreditAttribution: Mad Maks commentedthis error is in my log when somebody tries to viit a link in the list
Comment #2
Mad Maks CreditAttribution: 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 CreditAttribution: Bèr Kessels commentedThanks for reporting. PHP5 is very strict about arrays ans objects . will look into this later.
Comment #4
Mad Maks CreditAttribution: Mad Maks commentedwhat is the status?
Comment #5
Bèr Kessels CreditAttribution: Bèr Kessels commentedno change yet. but this week the links bundle will be released. look out for that :)
Comment #6
ron_mahon CreditAttribution: ron_mahon commentedWhat is the name of the links bundel? I cant find it.
Thanks
Comment #7
Bèr Kessels CreditAttribution: Bèr Kessels commentedIt does not yet exist.
Comment #8
mahcsig CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: Bèr Kessels commentedthe problem is almost certainly in weblink module.
Comment #11
19th CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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