Problem/Motivation

use short array

Comments

thalles created an issue. See original summary.

thalles’s picture

Follow the path!

thalles’s picture

Status: Needs work » Needs review
manuel garcia’s picture

Title: Use short array » Use short array syntax
Status: Needs review » Needs work

Thanks, patch looks good, let's also update the code blocks on the README files:

                README.html  (4 usages found)
                    79 array(
                    80 mailsystem_default_id() => array(
                    91 array(
                    92 mailsystem_default_id() => array(
                README.txt  (4 usages found)
                    86   array(
                    87     mailsystem_default_id() => array(
                    99   array(
                    100     mailsystem_default_id() => array(
                README.markdown  (4 usages found)
                    91       array(
                    92         mailsystem_default_id() => array(
                    102       array(
                    103         mailsystem_default_id() => array(
thalles’s picture

thalles’s picture

Status: Needs work » Needs review
manuel garcia’s picture

Status: Needs review » Reviewed & tested by the community

Thanks @thalles for following up on this, I had a look locally and everything seems correct now.

manuel garcia’s picture

Status: Reviewed & tested by the community » Fixed

Verified with @Bedir over slack and committed - thanks!

thalles’s picture

Thanks @all!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.