Issues

ZF-12500: [CRITICAL BUG + FIX] Zend_Gdata 1.12.1 feed URL generation

Description

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'

Comments

Seems this issue tracker is dead. I've reported it here instead: https://github.com/zendframework/zf2/issues/3351

@Aitte No, this tracker is not dead! This is the tracker for version 1 and the tracker on Github is for version 2.

Patch with fix and unit tests added.

@Frank Excellent work including unit tests in the patch! Looks good to me.

Fixed on trunk (25184) and release-1.12 (25185)