Merge lp:~stefanor/ibid/iso4217-table-location into lp:ibid

Proposed by Stefano Rivera on 2013-01-30
Status: Merged
Approved by: Stefano Rivera on 2013-01-30
Approved revision: 1057
Merged at revision: 1057
Proposed branch: lp:~stefanor/ibid/iso4217-table-location
Merge into: lp:ibid
Diff against target: 13 lines (+2/-1)
1 file modified
ibid/plugins/conversions.py (+2/-1)
To merge this branch: bzr merge lp:~stefanor/ibid/iso4217-table-location
Reviewer Review Type Date Requested Status
Jonathan Hitchcock 2013-01-30 Approve on 2013-01-30
Review via email: mp+145581@code.launchpad.net

Commit message

Update URL for ISO 4217 table, they've moved it

To post a comment you must log in.
Jonathan Hitchcock (vhata) :
review: Approve
Max Rabkin (max-rabkin) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ibid/plugins/conversions.py'
2--- ibid/plugins/conversions.py 2012-07-29 19:47:18 +0000
3+++ ibid/plugins/conversions.py 2013-01-30 11:26:26 +0000
4@@ -313,7 +313,8 @@
5
6 def _load_currencies(self):
7 iso4127_file = cacheable_download(
8- 'http://www.currency-iso.org/dl_iso_table_a1.xml',
9+ 'http://www.currency-iso.org/dam/isocy/downloads'
10+ '/dl_iso_table_a1.xml',
11 'conversions/iso4217.xml')
12 document = ElementTree.parse(iso4127_file)
13 # Code -> [Countries..., Currency Name, post-decimal digits]

Subscribers

People subscribed via source and target branches

to all changes: