ajay-jain / php-oembed

Automatically exported from code.google.com/p/php-oembed
0 stars 0 forks source link

YouTubeProvider::getEmbed() returns invalid HTML #2

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Reference YouTube video, e.g. 
http://www.youtube.com/oembed?maxwidth=500&maxheight=600&url=http://www.youtube.
com/watch%3Fv%3DulcgFppkbLU

What is the expected output? What do you see instead?
Expect valid HTML in the "html" field, but it contains invalid </param> and 
</embed> tags.

What version of the product are you using? On what operating system?
N/A

Please provide any additional information below.

The code is clearly generated in YouTubeProvider.class.php in the function 
YouTubeProvider::getEmbed() at "$myEmbed->html=..."

(There might be additional places)

cf.
PARAM: http://www.w3.org/TR/html401/struct/objects.html#h-13.3.2
EMBED http://dev.w3.org/html5/spec/Overview.html#elements-0
“Void elements only have a start tag; end tags must not be specified for void 
elements.”

Reported to Google here:

http://code.google.com/p/gdata-issues/issues/detail?id=2346

Original issue reported on code.google.com by danor...@gmail.com on 6 Oct 2010 at 11:36