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

Issue 13373047: check_vat_mx

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by zodman
Modified:
12 years, 1 month ago
Reviewers:
ced
Visibility:
Public.

Description

check_vat_mx

Patch Set 1 #

Patch Set 2 : More testing case #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+92 lines, -0 lines) Patch
M vatnumber/__init__.py View 2 chunks +83 lines, -0 lines 1 comment Download
M vatnumber/tests.py View 1 2 chunks +9 lines, -0 lines 1 comment Download

Messages

Total messages: 1
ced
12 years, 1 month ago (2013-09-12 17:20:25 UTC) #1
https://codereview.appspot.com/13373047/diff/3001/vatnumber/__init__.py File vatnumber/__init__.py (right): https://codereview.appspot.com/13373047/diff/3001/vatnumber/__init__.py#newco... vatnumber/__init__.py:1295: if char not in string.uppercase + "&Ñ": I think you should use a unicode https://codereview.appspot.com/13373047/diff/3001/vatnumber/tests.py File vatnumber/tests.py (right): https://codereview.appspot.com/13373047/diff/3001/vatnumber/tests.py#newcode186 vatnumber/tests.py:186: ('MX','XAXX010101000 ',False), space after ,
Sign in to reply to this message.

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