simplify the api
Bug #961340 reported by
fisadev
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
QR Tools |
In Progress
|
Undecided
|
Ramiro Algozino |
Bug Description
The current API requires you to instantiate a class, set attributes, call methods and then watch the side effect reading other attributes. That's a lot of innecesary complexity for a library that could simply be:
def create_qr(data, redundancy_level, pixel_size):
return an image object (so you can save it or do whatever you want)
def read_qr(
return a string with the readed data (or None if unable to read)
You should watch this video (a python core developer on the last pycon), and will understand why you shouldn't use classes and side effects in this case:
http://
I'll work on a patch if you agree.
Changed in qr-tools: | |
status: | New → In Progress |
milestone: | none → 2.1 |
assignee: | nobody → Ramiro Algozino (algozino) |
To post a comment you must log in.