Fixes, update readme

Author: geekwright

.scrutinizer.yml

@@ -1,53 +1,6 @@
 filter:
     excluded_paths:
         - 'tests/*'
-checks:
-    php:
-        code_rating: true
-        variable_existence: true
-        useless_calls: true
-        use_statement_alias_conflict: true
-        unused_variables: true
-        unused_properties: true
-        unused_parameters: true
-        unused_methods: true
-        unreachable_code: true
-        sql_injection_vulnerabilities: true
-        security_vulnerabilities: true
-        precedence_mistakes: true
-        precedence_in_conditions: true
-        parameter_non_unique: true
-        no_property_on_interface: true
-        no_non_implemented_abstract_methods: true
-        deprecated_code_usage: true
-        closure_use_not_conflicting: true
-        closure_use_modifiable: true
-        avoid_useless_overridden_methods: true
-        avoid_conflicting_incrementers: true
-        assignment_of_null_return: true
-        verify_property_names: true
-        verify_argument_usable_as_reference: true
-        verify_access_scope_valid: true
-        use_self_instead_of_fqcn: true
-        too_many_arguments: true
-        single_namespace_per_use: true
-        return_doc_comment_if_not_inferrable: true
-        return_doc_comments: true
-        require_scope_for_methods: true
-        require_scope_for_properties: true
-        require_php_tag_first: true
-        require_braces_around_control_structures: true
-        psr2_control_structure_declaration: true
-        psr2_switch_declaration: true
-        psr2_class_declaration: true
-        no_eval: true
-        no_else_if_statements: true
-        avoid_corrupting_byteorder_marks: true
-        argument_type_checks: true
-        php5_style_constructor: true
-        parameter_doc_comments: true
-        no_duplicate_arguments: true
-        missing_arguments: true
-        instanceof_class_exists: true
-        foreach_traversable: true
-        no_unnecessary_function_call_in_for_loop: true
+tools:
+    external_code_coverage:
+        timeout: 900    # Timeout in seconds.

README.md

@@ -1,4 +1,6 @@
 # Po
+[![Build Status](https://travis-ci.org/geekwright/Po.svg?branch=master)](https://travis-ci.org/geekwright/Po) [![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/geekwright/Po/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/geekwright/Po/?branch=master)
+
 __Po__ is a set of objects to assist in reading, manipulating and creating GNU gettext style PO files.
 
 ## Installing

src/PoEntry.php

@@ -137,7 +137,7 @@ public function getAsString($type)
      *
      * @param string $type PoToken constant
      *
-     * @return string|null
+     * @return string[]|null
      */
     public function getAsStringArray($type)
     {

src/PoFile.php

@@ -63,9 +63,9 @@ public function __construct($header = null, $entries = array(), $unkeyedEntries
     /**
      * Build the internal entries array key from id, context and plural id
      *
-     * @param string $msgid        the untranslated message of the entry
-     * @param string $msgctxt      the context of the entry, if any
-     * @param string $msgid_plural the untranslated plural message of the entry, if any
+     * @param string      $msgid        the untranslated message of the entry
+     * @param string|null $msgctxt      the context of the entry, if any
+     * @param string|null $msgid_plural the untranslated plural message of the entry, if any
      *
      * @return string
      */
@@ -221,9 +221,9 @@ public function mergeEntry(PoEntry $newEntry)
     /**
      * Get an entry based on key values - msgid, msgctxt and msgid_plural
      *
-     * @param string $msgid        the untranslated message of the entry
-     * @param string $msgctxt      the context of the entry, if any
-     * @param string $msgid_plural the untranslated plural message of the entry, if any
+     * @param string      $msgid        the untranslated message of the entry
+     * @param string|null $msgctxt      the context of the entry, if any
+     * @param string|null $msgid_plural the untranslated plural message of the entry, if any
      *
      * @return PoEntry|null matching entry, or null if not found
      */
@@ -335,8 +335,8 @@ public function dumpString()
     /**
      * Replace any current contents with entries from a file
      *
-     * @param string   $file    po file/stream to read
-     * @param resource $context context for stream if required
+     * @param string        $file    po file/stream to read
+     * @param resource|null $context context for stream if required
      *
      * @return void
      *

src/PoHeader.php

@@ -117,7 +117,7 @@ public function setHeader($key, $value)
     /**
      * Set the POT-Creation-Date header
      *
-     * @param integer $time unix timestamp, null to use current
+     * @param integer|null $time unix timestamp, null to use current
      *
      * @return void
      */
@@ -129,7 +129,7 @@ public function setCreateDate($time = null)
     /**
      * Set the PO-Revision-Date header
      *
-     * @param integer $time unix timestamp, null to use current
+     * @param integer|null $time unix timestamp, null to use current
      *
      * @return void
      */
@@ -141,7 +141,7 @@ public function setRevisionDate($time = null)
     /**
      * Format a timestamp following PO file conventions
      *
-     * @param integer $time unix timestamp, null to use current
+     * @param integer|null $time unix timestamp, null to use current
      *
      * @return string formatted timestamp
      */