Fix most Coder Drupal standard violations

LGTM.

Updated patch for most recent code.

• Keep README lines within 80 chars.
• Remove leading \
• Document method parameter
• Change variable to camel case (and update all instances)
• Remove extraneous spaces
• Fix static variable definition typo

Updated for testing.

Same as #9 except adds missing newlines to some files, per Coder Drupal standard. For example, fixes:

--- a/password_policy_history/password_policy_history.info.yml
+++ b/password_policy_history/password_policy_history.info.yml
@@ -4,4 +4,4 @@ package: Security
 type: module
 core: 8.x
 dependencies:
-  - password_policy
\ No newline at end of file
+  - password_policy

Thanks, Zahor. The convention for patch file naming is that the comment # follow the issue #, by the way.

- Made issue title more specific.
- Changed patch to remove changes pertaining to $user_context. The tests pass NULL for $user_context, but it is type-hinted as array, hence the test failure. The fix to this seems less trivial so we can handle it as a separate issue. I think making the $user_context parameter optional may be the best solution, but not sure.

Uploaded incorrect patch file in previous comment.

Clarified issue title.

Going ahead with commit even though not all Coder Drupal standard violations are fixed.

(Doing so because some of these coding standard issues are flagged by the Drupal.org testing infrastructure, and it is distracting when looking through issues. This will clean things up a bit.)

Thanks, leopaccanaro and Zahor.