Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1682)

Issue 6458084: Content.create (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by xiaq
Modified:
11 years, 9 months ago
Reviewers:
thomas.j.waldmann, Reimar Bauer
Visibility:
Public.

Description

Content.create

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -23 lines) Patch
M MoinMoin/items/__init__.py View 3 chunks +2 lines, -4 lines 1 comment Download
M MoinMoin/items/_tests/test_Content.py View 1 chunk +12 lines, -7 lines 1 comment Download
M MoinMoin/items/_tests/test_Item.py View 1 chunk +0 lines, -12 lines 0 comments Download
M MoinMoin/items/content.py View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 3
xiaq
This patch set adds a classmethod create to Content. http://codereview.appspot.com/6458084/diff/1/MoinMoin/items/_tests/test_Content.py File MoinMoin/items/_tests/test_Content.py (left): http://codereview.appspot.com/6458084/diff/1/MoinMoin/items/_tests/test_Content.py#oldcode30 MoinMoin/items/_tests/test_Content.py:30: ...
11 years, 9 months ago (2012-08-06 03:08:25 UTC) #1
Reimar Bauer
ok http://codereview.appspot.com/6458084/diff/1/MoinMoin/items/__init__.py File MoinMoin/items/__init__.py (left): http://codereview.appspot.com/6458084/diff/1/MoinMoin/items/__init__.py#oldcode193 MoinMoin/items/__init__.py:193: logging.debug("Content class {0!r} handles {1!r}".format(content.__class__, contenttype)) hmm, someone ...
11 years, 9 months ago (2012-08-08 07:28:32 UTC) #2
ThomasJWaldmann
11 years, 9 months ago (2012-08-09 11:42:41 UTC) #3
looks ok
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b