Merge lp:~pokulo/qr-tools/bugfix_wifi_code_hidden_flag into lp:qr-tools

Proposed by Ramiro Algozino
Status: Merged
Approved by: Ramiro Algozino
Approved revision: 43
Merged at revision: 43
Proposed branch: lp:~pokulo/qr-tools/bugfix_wifi_code_hidden_flag
Merge into: lp:qr-tools
Diff against target: 12 lines (+1/-1)
1 file modified (+1/-1)
To merge this branch: bzr merge lp:~pokulo/qr-tools/bugfix_wifi_code_hidden_flag
Reviewer Review Type Date Requested Status
Ramiro Algozino Approve
Review via email:

Commit message

To post a comment you must log in.
Revision history for this message
Ramiro Algozino (algozino) wrote :

Thank you @pokulo for the patch :)

Revision history for this message
Ramiro Algozino (algozino) wrote :

Makes sense that the "Hidden" field shoudl be optional.

review: Approve
Revision history for this message
Pokulo (pokulo) wrote :

Thanks for merging so quickly. :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file ''
2--- 2020-10-24 11:53:42 +0000
3+++ 2020-12-24 08:38:49 +0000
4@@ -111,7 +111,7 @@
5 'geo': lambda data: re.findall(u"GEO:(.*),(.*)", data, re.IGNORECASE)[0],
6 'bookmark': lambda data: re.findall(u"MEBKM:TITLE:(.*);URL:(.*);;", data, re.IGNORECASE)[0],
7 'phonebook': lambda data: dict(re.findall("(.*?):(.*?);", data.replace("MECARD:",""), re.IGNORECASE)),
8- 'wifi' : lambda data: re.findall(u"WIFI:S:(.*);T:(.*);P:(.*);[H:]+(.*);", data, re.IGNORECASE)[0],
9+ 'wifi' : lambda data: re.findall(u"WIFI:S:(.*);T:(.*);P:(.*);[H:]*(.*);", data, re.IGNORECASE)[0],
10 'sepa' : lambda data: parse_qs(urlparse(data).query),
11 'bip21' : lambda data: {'address': urlparse(data).path, 'options': parse_qs(urlparse(data).query)},
12 }


People subscribed via source and target branches