Code Sniffer sweep against Coder 8.x-2.x-dev. Any remaining departures from Coder rules are either due to bugs reported under Coder issues, or they are deliberate departures due mainly to disagreements with the Drupal Coding Standards as applied to OO coding, as explained under http://drupal7demo.webel.com.au/node/1759, or due to disagreements on comment standard recommendations. There are also some deliberate minor departues from Coder rules preserved for sake of demonstration for Coder and Drupal Coding Standards issue reporting and tracking.

Minor fixes for compatibility with PHP > 5.3. OOE demo page info links are now absolute to http://drupal7demo.webel.com.au

A major coding standards sweep was performed through every PHP file, as well as .info and .install. This sweep reduces the errors reported by PAReview.sh by about 80%, but because of the specific UML-friendlier PHP coding style used here, and because of disagreements with the Drupal Coding Standards for OO, an disagreements with inline comment rules, this module does not and will likely never 100% correspond with the Drupal Coding Standards, and this is deliberate. Darren Kelly (Webel IT Australia).

Improvements (only) in meeting Coder 8.x rules and PAReview.sh report. Can't meet all rules because of UML-friendly OOE style, and very strongly disagree with some comment standards.

OOE: After Paraview review. Removed run-time call-by-references (tolerated in PHP5.3, but now deprecated in PHP5.4). Also some minor changes, mostly to docs and spelling. But it's wildly different from Coder 7.x-2.0 review, so can't get close to passing. 

OOE: removed call-time pass-by-reference, for pre PHP5.3, deprecated in PHP5.4. Visit http://stackoverflow.com/questions/12322811/call-time-pass-by-reference-has-been-removed, http://stackoverflow.com/questions/4665782/call-time-pass-by-reference-has-been-deprecated

OOE: improvements to compatibility with Drupal translation/localization registry, especially in the form controller demo. One can now compare a WET-Drupal style submit handler (compatible with t()) and a DRY-OO style submit handler (not yet compatible with t()). The translation compatibility is improved but not complete. See also: https://www.drupal.org/node/2320343 (Support » Translations: Concerning the difficulty in achieving Don't Repeat Yourself (DRY) coding when using placeholder strings compatible with t()) and https://www.drupal.org/node/2318319 (Concerning recommendations for internal links using placeholders).

OOE: improvements to compatibility with Drupal translation/localization registry. No longer use automatic internal translation applied as policy to class string variables or encapsulated strings in Render classes (except for counter-examples). Visit also Coder issue with detailed examples: https://www.drupal.org/node/2304947#comment-9039951 (This requirement is at odds with OO encapsulation: WARNING | Only string literals should be passed to t() where possible)

OOE: reverted Render translation policy handler to not use placeholders, as it escapes encapsulated markup. The matter of encapsulating translatable text in a fashion compatible with Drupal's translation registry is unresolved work-in-progress.

OOE: MAJOR SANDBOX COMMIT: after very extensive Coder run on every file. Complete compatibility with Coder not possible with the Webel coding recipe for UML-friendly PHP, and because of some non-OO aspects of Coder and the current Drupal coding and doc standards (see Coder issue reports by webel). Also includes completely new README.txt and some minor UML diagram updates. LICENCE.txt excluded in preparation for move from sandbox.

OOE ooehub update (after @sync to lunarpages): MAJOR COMMIT: contains mostly consistent UML diagrams for every package and element, and is known to run ok on http://drupal7demo.webel.com.au, but some documentation tags may still be incomplete.