Module to provide integration for Ingenico
Verdict: good quality code, well tested (phpunit), no major issues. Slight slowdown from global observers was my only worry.
+ no obfuscated code
+ no encryption
+ good quality code
+ monster "street splitter" regex (cool)
- global observers (adminhtml_block_html_before, core_block_abstract_to_html_after, core_block_abstract_prepare_layout_before, model_save_before)
- non-standard use of "use" (Netresearch_OPS_AliasController)
- blindly inserts new regions to Magento tables, but what if we have those regions already in?
- apropos, hardcoded region mappings in api
- some bad English (mostly on admin) "Payment information has been resend to customer"