Issue 27116: Provide data model classes and helper methods to support messaging in Zend_Gdata_YouTube

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s :	save comment
<Esc> :	cancel edit

Issue 27116: Provide data model classes and helper methods to support messaging in Zend_Gdata_YouTube (Closed)

Can't Edit
Can't Publish+Mail
Start Review

Created:
3 years, 1 month ago by api.jhartmann-google

Modified:
2 years, 9 months ago

Reviewers:
Trevor Johns (Google), jfisher

CC:
rboyd

Base URL:
http://framework.zend.com/svn/framework/standard/trunk

Visibility:
Public.

Description

@see http://framework.zend.com/issues/browse/ZF-6042

Patch Set 1 #

Created: 3 years, 1 month ago

Download raw patch set

Unified diffs	Side-by-side diffs	Stats			Patch
library/Zend/Gdata/YouTube.php	View	12 chunks	+147 lines, -12 lines	0 comments	Download
library/Zend/Gdata/YouTube/InboxEntry.php	View	1 chunk	+280 lines, -0 lines	0 comments	Download
library/Zend/Gdata/YouTube/InboxFeed.php	View	1 chunk	+67 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTube/InboxEntryTest.php	View	1 chunk	+217 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTube/InboxFeedTest.php	View	1 chunk	+182 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTube/_files/InboxEntryDataSample1.xml	View	1 chunk	+82 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTube/_files/InboxEntryDataSampleV2.xml	View	1 chunk	+88 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTube/_files/InboxFeedDataSample1.xml	View	1 chunk	+107 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTube/_files/InboxFeedDataSampleV2.xml	View	1 chunk	+119 lines, -0 lines	0 comments	Download
tests/Zend/Gdata/YouTubeOnlineTest.php	View	8 chunks	+118 lines, -13 lines	0 comments	Download

Messages

Total messages: 2

Expand All Messages | Collapse All Messages

api.jhartmann-google

Trevor, Can you let me know if you will have time to review this. All ...

3 years, 1 month ago #1

LGTM

A slightly crazy amount of code for a simple feed. It was nice of you to add in
the support for the v1 feed too, though I'm not really sure there's value in it
since everyone should be using V2.

Expand All Messages | Collapse All Messages