Observation, problems, Ideas for improvements

Problems

quintagroup.captcha.core package has following code:

1. browser package which contains:
   • view with name captcha, which design for possibility to insert captcha in custom browser views/forms
   • Captcha zope.schema.ASCIILine based field
   • zope.app.form.browser.ASCIIWidget based widget for Captcha field
2. utils module, with following inconsistencies:
   • getWord bound to data.basic_english dictionary only - so can't be apply to static captcha
   • getIndex bound to data.basic_english dictionary only - so can't be apply to static captcha
   • getCaptchasCount bound to data.basic_english dictionary (for dynamic) and to CAPTCHAS_COUNT constant (for static captcha)

Ideas for improvements

• remove from browser package formlib related field and widget - it should present in quintagroup.formlib.captcha package. We can leave this code for a while, with deprication warning.
• update utility module functions to depends from set in config module dictionary, i.e.: define DICTIONARY in config module and use this DICTIONARY in utils in all cases.
• add tests:
  • for static captcha [3023]
  • for dynamic captcha [3031]
  • for configlet [3018]