parse_url(http://) [function.parse-url]: Unable to parse URL in /usr/home/hoslo/public_html/sites/all/modules/seo_friend/seo_friend.module on line 776.

Comments

kristen pol’s picture

Status: Active » Needs review

I just checked in some code that checks the url field before calling parse_url. I checked it into dev, so please test once the drupal cron has updated that today.

Thanks,
Kristen

Starminder’s picture

Status: Needs review » Needs work

Hi - for me it started displaying the report, but in the end it still hangs and locks up.

kristen pol’s picture

Does it give the same error or just hang?

Kristen

Starminder’s picture

It hangs and eventually crashes the browser (Firefox). Nothing in the log about it though.

kristen pol’s picture

Hmm... sounds like a memory problem. What's you php.ini setting?

Kristen

Starminder’s picture

Sorry - missed this. Looks like I'm at 256M. I tried to override it with Drupal Tweaks but no luck.

kristen pol’s picture

If you are comfortable changing the code, edit your seo_friend.module file and change line #784 from:

$limit = 200000;

to

$limit = 10000;

(or something "small") and see if it stops hanging. I imagine there is just too much data. I'm wondering how I should best deal with this type of situation.... just keep number pretty small, make it configurable, get rid of the report, ???

Kristen

kristen pol’s picture

Status: Needs work » Postponed (maintainer needs more info)
Starminder’s picture

Status: Postponed (maintainer needs more info) » Reviewed & tested by the community

I confirmed it was still hanging/crashing with 200000. Changed limit to 10000 and it works great. Thanks!!

kristen pol’s picture

Thanks for testing. I will add an admin setting so that number can be tweaked.

Kristen

kristen pol’s picture

I just checked in changes to dev so that you can set this max limit number in the admin settings:

[yourdomain/admin/content/seo_friend/default

at the bottom of the form.

If you want to try that out, you can play with the numbers.

Kristen

Starminder’s picture

That seems to work well, thanks!

kristen pol’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Awesome, I'll close this.

Kristen