Hello,

I'm getting a lot of errors like this one:

Warning: parse_url(http://) [function.parse-url]: Unable to parse URL in video_embed_get_handler() (line 207 of /var/www/vhosts/****.org/httpdocs/sites/all/modules/video_embed_field/video_embed_field.module).

This is an URL, construction is always the same:
http://www.*****.org/?q=vef/load/76ad0b357bdfac5a5684f3b0b7b18779&width=...

Everything seems to be working fine.

Thumbnails are created using image_cache
Views handles the selection of video's to show on a page
Colorbox displays

When browsing through the site and clicking on video's I can't find anything going wrong.

CommentFileSizeAuthor
#3 video_embed_field-1910736-3-unable-to-parse-url.patch894 bytesmstrelan

Comments

wemmies’s picture

wemmies commented
Version: 7.x-2.0-beta4 » 7.x-2.0-beta5

Problem seems to be getting worse with warnings every 10 min. or so. Updated to beta5. Problem still exists.

wemmies’s picture

wemmies commented

Ok, so after a search on the hosts that are causing this error, it seems that GoogleBot is causing this. I now added a rel="nofollow" to the links as they don't have any significant meaning for my search results. This in the hope to reduce the errors.

Google doesn't seem to get the basePath or something, as the url that can't be parsed is shown as "http://"

***UPDATE 09-02-2013***

Only had one error logged since the nofollow implementation. It still was Google causing it. So either it works and Google sometimes slips through, or Google is currently not indexing my pages with links to videos. I Will keep monitoring this for the coming week.

***UPDATE 11-02-2013***

Haven't seen an error ever since 09-02. I think it's save to say that this prevents the errors from occurring. Now I just need to figure out why Googlebot isn't getting the right basePath and thus is gennerating this error.

***UPDATE 16-02-2013***
So that didn't work. Googlebot is back and generating plenty of warnings.

mstrelan’s picture

mstrelan commented
Version: 7.x-2.0-beta5 » 7.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new video_embed_field-1910736-3-unable-to-parse-url.patch894 bytes

I'm also getting lots of "unable to parse url" watchdog entries, but also getting broken links. Seems the module is printing links in the format http://www.example.com/?q=vef/load/b36de990d9e2968ac4a5e6a7f41214b0&widt... which includes the non-seo-friendly URL string (that is, q=internal/drupal/path).

On one server this correctly rewrites to http://www.example.com/?q=vef/load/b36de990d9e2968ac4a5e6a7f41214b0&widt... but on another running the exact same code it rewrites to http://www.example2.com/vef/load/b36de990d9e2968ac4a5e6a7f41214b0%26widt...

--- [EDIT] ---
Those URLs aren't formatted well to see the issue, but basically the query string is being encoded in to the URL and then also added on as a regular query string.
--- [/EDIT] ---

The latter example causes the parse_url error and just spits out the following.

<div class="embedded-video">
  <div class="player">
    <a href="http://">http://</a>  </div>
</div>

The attached patch rewrites the links in a much more standardised way, and resolves the broken links and parse_url errors.

  • No need to specify the base_path or q= when using the l() function.
  • Why is it writing an external URL?
  • Query strings should be passed in to the options array
  • Format the code so it is somewhat readable

@wemmies - please test this and see if it resolves your problem.

mrweiner’s picture

mrweiner commented

Patch in #3 fixed the issue for me. Waiting for other testers before changing the status.

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Issue summary: View changes
Status: Needs review » Fixed

This should be fixed once 3b68918 has been committed.

That patch provides some thumbnails and formatters generation improvements.

Please, feel free to reopen if your problems persist.

Thank you very much for your interest in this module.

Regards

Status: Fixed » Closed (fixed)

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

lmilani’s picture

lmilani commented

Hi to all.
I'm experiencing the same issue with Video Embed Field 7.x-2.0-beta10 and PHP 5.6.
When I click on my picture thumbnail with Colorbox, the modal window opens in the right mode but the content is empty, the YouTube IFRAME is not loaded.
The generated HTML code is:

<div class="embedded-video">
  <div class="player">
    <a href="http://">http://</a>  </div>
</div>

And the results is an empty modal window with only the text "http://" as a link.

Someone experienced the same issue?
I checked the patch http://cgit.drupalcode.org/video_embed_field/commit/?id=3b68918 but I'm yet using the last version of Video Embed Field.

Thanks & regards.
Luca