[CRITICAL BUG + FIX] Zend_Gdata 1.12.1 feed URL generation #3351

Closed
aitte opened this Issue Jan 5, 2013 · 2 comments

Projects

None yet

2 participants

@aitte
aitte commented Jan 5, 2013

Name of API affected:
VideoQuery.php: Zend_Gdata_YouTube_VideoQuery()->setFeedType()

Issue summary:
The "responses" and "comments" types construct invalid URLs by concatenating as follows:

  • responses: Zend_Gdata_YouTube::VIDEO_URI . '/' . $videoId . 'responses'
  • comments: Zend_Gdata_YouTube::VIDEO_URI . '/' . $videoId . 'comments'

This generates illegal URLs such as "https://gdata.youtube.com/feeds/api/videos/SZj6rAYkYOgcomments"
The "related" type constructs a valid URL in contrast:

  • related: Zend_Gdata_YouTube::VIDEO_URI . '/' . $videoId . '/related';

Solution:
To fix this bug for the responses and comments types, simply change their concatenation lines to:

  • responses: Zend_Gdata_YouTube::VIDEO_URI . '/' . $videoId . '/responses'
  • comments: Zend_Gdata_YouTube::VIDEO_URI . '/' . $videoId . '/comments'
@froschdesign
Member

This is the tracker for version 2!
Your entry on framework.zend.com/issues is correct. (ZF-12500)

Please close this report here.

@aitte
aitte commented Jan 5, 2013

Sorry about that. It got really confusing since people on the other tracker kept being told to post here instead. I'll close this issue now!

@aitte aitte closed this Jan 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment