Correct indentation for array arguments (tests/issues-14.php)

Commit 055d0810 began a transition to using more of cc-mode’s
machinery for customizing indentation.  In the process this broke the
indentation we expect from the ‘tests/issues-14.php’ script.  This
failure went unnoticed because we expected the test to fail, and so
every time we would run the test suite the test would successfully
fail.  After the code was in place to make indentation work properly
we failed to update the test accordingly so that it would no longer
expect a failure.

Due to this mismanagement of the test suite on my part, a regression
which re-broke indentation entered the master branch unnoticed.

This patch corrects the problem in a number of ways:

1. We no longer expect ‘php-mode-test-issue-14’ to fail.  When we run
the test suite it must pass.

2. We explicitly use the ‘php-lineup-arglist-intro’ and
‘php-lineup-arglist-close’ functions for the cc-mode offsets
‘arglist-intro’ and ‘arglist-close’, respectively.  We use these in
all three styles.

3. When loading php-mode we explicitly enable the user’s coding style
before running associated hooks.

With this patch in place we can start a fresh instance of ‘emacs -Q’,
load php-mode, and get the expected indentation in the issue-14 test
for all three coding styles we define: PEAR, Drupal, and WordPress.

Finally, this patch performs some reformatting of the Elisp itself for
readability and consistency.

Github-issue: 80
Do not expect failure for test issue-19

If php-mode fails to properly align arrows for chained method calls
then we want to know that.

Github-issue: 81
Do not trigger search error in php-mode-test-issue-19

We search for the string ‘$object->’ to determine if its position
matches the arrow on the next line.  Because we test this in different
contexts (e.g. in methods, in closures, at the top level) we search
for the string in a loop.  When we reach the end of the buffer,
however, search-forward returns an error because there are no more
‘$object->’ strings to find.  This patch changes the search so it does
not raise the error but only returns nil at the end, causing the loop
to end normally.

Github-issue: 81
Fix chained method alignment

Commit f933ec2d removed the cc-mode setting for ‘statement-cont’ but
we never re-introduced it when implementing the different PHP styles.
That breaks the alignment of chained methods across different lines.
This patch fixes this by re-introducing ‘statement-cont’ to each style
and by giving ‘brace-list-intro’ and ‘brace-list-entry’ proper values
so that methods are correctly aligned in the different contexts
presented in the issue-19 test.

This patch causes php-mode-test-issue-19 to pass successfully.

Github-issue: 81
Merge branch 'issue-81'

* issue-81:
  Fix chained method alignment
  Do not trigger search error in php-mode-test-issue-19
  Do not expect failure for test issue-19