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

Issue 4813050: modified contenttype filtering to handle contenttypes without the encoding and the unknown cases.

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 5 months ago by sinha
Modified:
13 years, 5 months ago
Reviewers:
Reimar Bauer
Visibility:
Public.

Description

please see diary for some questions.

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -9 lines) Patch
M MoinMoin/items/__init__.py View 2 chunks +24 lines, -9 lines 3 comments Download

Messages

Total messages: 2
Reimar Bauer
besides the comments, ok http://codereview.appspot.com/4813050/diff/1/MoinMoin/items/__init__.py File MoinMoin/items/__init__.py (right): http://codereview.appspot.com/4813050/diff/1/MoinMoin/items/__init__.py#newcode612 MoinMoin/items/__init__.py:612: # If a item's contenttype ...
13 years, 5 months ago (2011-07-26 06:10:41 UTC) #1
sinha
13 years, 5 months ago (2011-07-26 19:00:12 UTC) #2
replied to comment.

http://codereview.appspot.com/4813050/diff/1/MoinMoin/items/__init__.py
File MoinMoin/items/__init__.py (right):

http://codereview.appspot.com/4813050/diff/1/MoinMoin/items/__init__.py#newco...
MoinMoin/items/__init__.py:612: # If a item's contenttype not present in the
default contenttype list,
On 2011/07/26 06:10:41, Reimar Bauer wrote:
> apostrophe "'s" means "is" so that is wrong here.
> 
> some famous failures
> http://www.apostrophecatastrophes.com/
> 
> s/a/an/

But i guess apostrophe before the s is placed to show the possession. If you
think it is still wrong then, would this sentence also be wrong: "This is
Bauer's house" ?
http://www.grammarbook.com/punctuation/apostro.asp
Sign in to reply to this message.

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