GitHub-Issue: 115 (Unit Test)

We expect this test to fail because of a current limitation in the
test suite.  In order to test for indentation we use ‘magic comments’
and describe the correct, expected offset for indentation that we want
to see when running the tests.  The test for issue 115 has this:

    // ###php-mode-test### ((indent c-lineup-cascaded-calls))

The test suite code will fail on this magic comment because it tries
to treat `c-lineup-cascaded-calls' like a variable that we can compare
to the indentation.  The problem is that `c-lineup-cascaded-calls' is
a function, and we need to apply it to the correct ‘language element’
in order to get an indentation we can compare against.

Until the test suite supports this behavior we will expect the test
for issue 115 to always fail.
Use the PEAR indentation style by default for unit tests

We can tell the `with-php-mode-test' macro to use an explicit
indentation style if necessary.  However, we rarely do so, meaning
that the temporary buffers we create for testing indentation take on
the user’s preferred default indentation style, e.g. `gnu' or `linux'.
We only ensure correct indentation for the styles that PHP Mode
explicitly provides to users.  So this patch changes the behavior of
the macro to use the PEAR style of indentation by default, that way we
can be sure we are testing indentation with a style we support.

This patch also modifies the macro so that it enables PHP Mode before
selecting any style, as the original order of trying to select the
style first can cause errors.

This change relates to the issue below, as its unit test would no
longer pass since I was not using one of PHP Mode’s specific styles.

GitHub-Issue: 29
Align chained method calls inside of arrays

This patch modifies the four PHP Mode specific indentation styles so
that method calls in arrays align like so:

    $x = ["x" => $this->foo()
                      ->bar()
                      ->baz()];

Admittedly this may introduce unexpected indentation in some
circumstances, particularly with the Drupal style, as it was the only
style that defined `arglist-cont-nonempty'.  We do not have any tests
which use that cc-mode syntactic symbol in other contexts.  So this
patch implements the indentation above but because our test suite does
not thoroughly examine `arglist-cont-nonempty' this patch should be
first on our list of ‘things to check’ and possibly revert if we
receive any bug reports related to `arglist-cont-nonempty'.

GitHub-Issue: 115
Update php-mode-modified
Merge branch 'ejmr/align-method-calls-in-arrays'

* ejmr/align-method-calls-in-arrays:
  Update php-mode-modified
  Align chained method calls inside of arrays
  Use the PEAR indentation style by default for unit tests
  GitHub-Issue: 115 (Unit Test)