= 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
 1. `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]