sop / x501

lib/X501/ASN1/AttributeValue/Feature/PrintableStringValue.php

Indentation +61 added lines, -61 removed lines

@@ -17,72 +17,72 @@
  */
 abstract class PrintableStringValue extends AttributeValue
 {
-    /**
-     * String value.
-     *
-     * @var string
-     */
-    protected $_string;
+	/**
+	 * String value.
+	 *
+	 * @var string
+	 */
+	protected $_string;
 
-    /**
-     * Constructor.
-     *
-     * @param string $value String value
-     */
-    public function __construct(string $value)
-    {
-        $this->_string = $value;
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param string $value String value
+	 */
+	public function __construct(string $value)
+	{
+		$this->_string = $value;
+	}
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return self
-     */
-    public static function fromASN1(UnspecifiedType $el): AttributeValue
-    {
-        return new static($el->asPrintableString()->string());
-    }
+	/**
+	 * {@inheritdoc}
+	 *
+	 * @return self
+	 */
+	public static function fromASN1(UnspecifiedType $el): AttributeValue
+	{
+		return new static($el->asPrintableString()->string());
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function toASN1(): Element
-    {
-        return new PrintableString($this->_string);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function toASN1(): Element
+	{
+		return new PrintableString($this->_string);
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function stringValue(): string
-    {
-        return $this->_string;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function stringValue(): string
+	{
+		return $this->_string;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function equalityMatchingRule(): MatchingRule
-    {
-        // default to caseIgnoreMatch
-        return new CaseIgnoreMatch(Element::TYPE_PRINTABLE_STRING);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function equalityMatchingRule(): MatchingRule
+	{
+		// default to caseIgnoreMatch
+		return new CaseIgnoreMatch(Element::TYPE_PRINTABLE_STRING);
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function rfc2253String(): string
-    {
-        return DNParser::escapeString($this->_transcodedString());
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function rfc2253String(): string
+	{
+		return DNParser::escapeString($this->_transcodedString());
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    protected function _transcodedString(): string
-    {
-        // PrintableString maps directly to UTF-8
-        return $this->_string;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	protected function _transcodedString(): string
+	{
+		// PrintableString maps directly to UTF-8
+		return $this->_string;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\ASN1\AttributeValue\Feature;
 

lib/X501/ASN1/AttributeValue/SurnameValue.php

Indentation +12 added lines, -12 removed lines

@@ -14,16 +14,16 @@
  */
 class SurnameValue extends DirectoryString
 {
-    /**
-     * Constructor.
-     *
-     * @param string $value      String value
-     * @param int    $string_tag Syntax choice
-     */
-    public function __construct(string $value,
-        int $string_tag = DirectoryString::UTF8)
-    {
-        $this->_oid = AttributeType::OID_SURNAME;
-        parent::__construct($value, $string_tag);
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param string $value      String value
+	 * @param int    $string_tag Syntax choice
+	 */
+	public function __construct(string $value,
+		int $string_tag = DirectoryString::UTF8)
+	{
+		$this->_oid = AttributeType::OID_SURNAME;
+		parent::__construct($value, $string_tag);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\ASN1\AttributeValue;
 

lib/X501/ASN1/AttributeValue/UnknownAttributeValue.php

Indentation +53 added lines, -53 removed lines

@@ -13,63 +13,63 @@
  */
 class UnknownAttributeValue extends AttributeValue
 {
-    /**
-     * ASN.1 element.
-     *
-     * @var Element
-     */
-    protected $_element;
+	/**
+	 * ASN.1 element.
+	 *
+	 * @var Element
+	 */
+	protected $_element;
 
-    /**
-     * Constructor.
-     *
-     * @param string  $oid
-     * @param Element $el
-     */
-    public function __construct(string $oid, Element $el)
-    {
-        $this->_oid = $oid;
-        $this->_element = $el;
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param string  $oid
+	 * @param Element $el
+	 */
+	public function __construct(string $oid, Element $el)
+	{
+		$this->_oid = $oid;
+		$this->_element = $el;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function toASN1(): Element
-    {
-        return $this->_element;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function toASN1(): Element
+	{
+		return $this->_element;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function stringValue(): string
-    {
-        // return DER encoding as a hexstring
-        return '#' . bin2hex($this->_element->toDER());
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function stringValue(): string
+	{
+		// return DER encoding as a hexstring
+		return '#' . bin2hex($this->_element->toDER());
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function equalityMatchingRule(): MatchingRule
-    {
-        return new BinaryMatch();
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function equalityMatchingRule(): MatchingRule
+	{
+		return new BinaryMatch();
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function rfc2253String(): string
-    {
-        return $this->stringValue();
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function rfc2253String(): string
+	{
+		return $this->stringValue();
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    protected function _transcodedString(): string
-    {
-        return $this->stringValue();
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	protected function _transcodedString(): string
+	{
+		return $this->stringValue();
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\ASN1\AttributeValue;
 

lib/X501/ASN1/AttributeValue/PseudonymValue.php

Indentation +12 added lines, -12 removed lines

@@ -14,16 +14,16 @@
  */
 class PseudonymValue extends DirectoryString
 {
-    /**
-     * Constructor.
-     *
-     * @param string $value      String value
-     * @param int    $string_tag Syntax choice
-     */
-    public function __construct(string $value,
-        int $string_tag = DirectoryString::UTF8)
-    {
-        $this->_oid = AttributeType::OID_PSEUDONYM;
-        parent::__construct($value, $string_tag);
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param string $value      String value
+	 * @param int    $string_tag Syntax choice
+	 */
+	public function __construct(string $value,
+		int $string_tag = DirectoryString::UTF8)
+	{
+		$this->_oid = AttributeType::OID_PSEUDONYM;
+		parent::__construct($value, $string_tag);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\ASN1\AttributeValue;
 

lib/X501/ASN1/AttributeValue/StateOrProvinceNameValue.php

Indentation +12 added lines, -12 removed lines

@@ -14,16 +14,16 @@
  */
 class StateOrProvinceNameValue extends DirectoryString
 {
-    /**
-     * Constructor.
-     *
-     * @param string $value      String value
-     * @param int    $string_tag Syntax choice
-     */
-    public function __construct(string $value,
-        int $string_tag = DirectoryString::UTF8)
-    {
-        $this->_oid = AttributeType::OID_STATE_OR_PROVINCE_NAME;
-        parent::__construct($value, $string_tag);
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param string $value      String value
+	 * @param int    $string_tag Syntax choice
+	 */
+	public function __construct(string $value,
+		int $string_tag = DirectoryString::UTF8)
+	{
+		$this->_oid = AttributeType::OID_STATE_OR_PROVINCE_NAME;
+		parent::__construct($value, $string_tag);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\ASN1\AttributeValue;
 

lib/X501/ASN1/Feature/TypedAttribute.php

Indentation +24 added lines, -24 removed lines

@@ -11,30 +11,30 @@
  */
 trait TypedAttribute
 {
-    /**
-     * Attribute type.
-     *
-     * @var AttributeType
-     */
-    protected $_type;
+	/**
+	 * Attribute type.
+	 *
+	 * @var AttributeType
+	 */
+	protected $_type;
 
-    /**
-     * Get attribute type.
-     *
-     * @return AttributeType
-     */
-    public function type(): AttributeType
-    {
-        return $this->_type;
-    }
+	/**
+	 * Get attribute type.
+	 *
+	 * @return AttributeType
+	 */
+	public function type(): AttributeType
+	{
+		return $this->_type;
+	}
 
-    /**
-     * Get OID of the attribute.
-     *
-     * @return string
-     */
-    public function oid(): string
-    {
-        return $this->_type->oid();
-    }
+	/**
+	 * Get OID of the attribute.
+	 *
+	 * @return string
+	 */
+	public function oid(): string
+	{
+		return $this->_type->oid();
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\ASN1\Feature;
 

lib/X501/MatchingRule/CaseIgnoreMatch.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\MatchingRule;
 

lib/X501/MatchingRule/MatchingRule.php

Indentation +10 added lines, -10 removed lines

@@ -11,14 +11,14 @@
  */
 abstract class MatchingRule
 {
-    /**
-     * Compare attribute value to assertion.
-     *
-     * @param mixed $assertion Value to assert
-     * @param mixed $value     Attribute value
-     *
-     * @return null|bool True if value matches. Null shall be returned if match
-     *                   evaluates to Undefined.
-     */
-    abstract public function compare($assertion, $value): ?bool;
+	/**
+	 * Compare attribute value to assertion.
+	 *
+	 * @param mixed $assertion Value to assert
+	 * @param mixed $value     Attribute value
+	 *
+	 * @return null|bool True if value matches. Null shall be returned if match
+	 *                   evaluates to Undefined.
+	 */
+	abstract public function compare($assertion, $value): ?bool;
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\MatchingRule;
 

lib/X501/DN/DNParser.php

Indentation +393 added lines, -393 removed lines

@@ -15,414 +15,414 @@
  */
 class DNParser
 {
-    /**
-     * RFC 2253 special characters.
-     *
-     * @var string
-     */
-    const SPECIAL_CHARS = ',=+<>#;';
+	/**
+	 * RFC 2253 special characters.
+	 *
+	 * @var string
+	 */
+	const SPECIAL_CHARS = ',=+<>#;';
 
-    /**
-     * DN string.
-     *
-     * @var string
-     */
-    private $_dn;
+	/**
+	 * DN string.
+	 *
+	 * @var string
+	 */
+	private $_dn;
 
-    /**
-     * DN string length.
-     *
-     * @var int
-     */
-    private $_len;
+	/**
+	 * DN string length.
+	 *
+	 * @var int
+	 */
+	private $_len;
 
-    /**
-     * Constructor.
-     *
-     * @param string $dn Distinguised name
-     */
-    protected function __construct(string $dn)
-    {
-        $this->_dn = $dn;
-        $this->_len = strlen($dn);
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param string $dn Distinguised name
+	 */
+	protected function __construct(string $dn)
+	{
+		$this->_dn = $dn;
+		$this->_len = strlen($dn);
+	}
 
-    /**
-     * Parse distinguished name string to name-components.
-     *
-     * @param string $dn
-     *
-     * @return array
-     */
-    public static function parseString(string $dn): array
-    {
-        $parser = new self($dn);
-        return $parser->parse();
-    }
+	/**
+	 * Parse distinguished name string to name-components.
+	 *
+	 * @param string $dn
+	 *
+	 * @return array
+	 */
+	public static function parseString(string $dn): array
+	{
+		$parser = new self($dn);
+		return $parser->parse();
+	}
 
-    /**
-     * Escape a AttributeValue string conforming to RFC 2253.
-     *
-     * @see https://tools.ietf.org/html/rfc2253#section-2.4
-     *
-     * @param string $str
-     *
-     * @return string
-     */
-    public static function escapeString(string $str): string
-    {
-        // one of the characters ",", "+", """, "\", "<", ">" or ";"
-        $str = preg_replace('/([,\+"\\\<\>;])/u', '\\\\$1', $str);
-        // a space character occurring at the end of the string
-        $str = preg_replace('/( )$/u', '\\\\$1', $str);
-        // a space or "#" character occurring at the beginning of the string
-        $str = preg_replace('/^([ #])/u', '\\\\$1', $str);
-        // implementation specific special characters
-        $str = preg_replace_callback('/([\pC])/u',
-            function ($m) {
-                $octets = str_split(bin2hex($m[1]), 2);
-                return implode('',
-                    array_map(
-                        function ($octet) {
-                            return '\\' . strtoupper($octet);
-                        }, $octets));
-            }, $str);
-        return $str;
-    }
+	/**
+	 * Escape a AttributeValue string conforming to RFC 2253.
+	 *
+	 * @see https://tools.ietf.org/html/rfc2253#section-2.4
+	 *
+	 * @param string $str
+	 *
+	 * @return string
+	 */
+	public static function escapeString(string $str): string
+	{
+		// one of the characters ",", "+", """, "\", "<", ">" or ";"
+		$str = preg_replace('/([,\+"\\\<\>;])/u', '\\\\$1', $str);
+		// a space character occurring at the end of the string
+		$str = preg_replace('/( )$/u', '\\\\$1', $str);
+		// a space or "#" character occurring at the beginning of the string
+		$str = preg_replace('/^([ #])/u', '\\\\$1', $str);
+		// implementation specific special characters
+		$str = preg_replace_callback('/([\pC])/u',
+			function ($m) {
+				$octets = str_split(bin2hex($m[1]), 2);
+				return implode('',
+					array_map(
+						function ($octet) {
+							return '\\' . strtoupper($octet);
+						}, $octets));
+			}, $str);
+		return $str;
+	}
 
-    /**
-     * Parse DN to name-components.
-     *
-     * @throws \RuntimeException
-     *
-     * @return array
-     */
-    protected function parse(): array
-    {
-        $offset = 0;
-        $name = $this->_parseName($offset);
-        if ($offset < $this->_len) {
-            $remains = substr($this->_dn, $offset);
-            throw new \UnexpectedValueException(
-                'Parser finished before the end of string' .
-                     ", remaining: '{$remains}'.");
-        }
-        return $name;
-    }
+	/**
+	 * Parse DN to name-components.
+	 *
+	 * @throws \RuntimeException
+	 *
+	 * @return array
+	 */
+	protected function parse(): array
+	{
+		$offset = 0;
+		$name = $this->_parseName($offset);
+		if ($offset < $this->_len) {
+			$remains = substr($this->_dn, $offset);
+			throw new \UnexpectedValueException(
+				'Parser finished before the end of string' .
+					 ", remaining: '{$remains}'.");
+		}
+		return $name;
+	}
 
-    /**
-     * Parse 'name'.
-     *
-     * name-component *("," name-component)
-     *
-     * @param int $offset
-     *
-     * @return array Array of name-components
-     */
-    private function _parseName(int &$offset): array
-    {
-        $idx = $offset;
-        $names = [];
-        while ($idx < $this->_len) {
-            $names[] = $this->_parseNameComponent($idx);
-            if ($idx >= $this->_len) {
-                break;
-            }
-            $this->_skipWs($idx);
-            if (',' != $this->_dn[$idx] && ';' != $this->_dn[$idx]) {
-                break;
-            }
-            ++$idx;
-            $this->_skipWs($idx);
-        }
-        $offset = $idx;
-        return array_reverse($names);
-    }
+	/**
+	 * Parse 'name'.
+	 *
+	 * name-component *("," name-component)
+	 *
+	 * @param int $offset
+	 *
+	 * @return array Array of name-components
+	 */
+	private function _parseName(int &$offset): array
+	{
+		$idx = $offset;
+		$names = [];
+		while ($idx < $this->_len) {
+			$names[] = $this->_parseNameComponent($idx);
+			if ($idx >= $this->_len) {
+				break;
+			}
+			$this->_skipWs($idx);
+			if (',' != $this->_dn[$idx] && ';' != $this->_dn[$idx]) {
+				break;
+			}
+			++$idx;
+			$this->_skipWs($idx);
+		}
+		$offset = $idx;
+		return array_reverse($names);
+	}
 
-    /**
-     * Parse 'name-component'.
-     *
-     * attributeTypeAndValue *("+" attributeTypeAndValue)
-     *
-     * @param int $offset
-     *
-     * @return array Array of [type, value] tuples
-     */
-    private function _parseNameComponent(int &$offset): array
-    {
-        $idx = $offset;
-        $tvpairs = [];
-        while ($idx < $this->_len) {
-            $tvpairs[] = $this->_parseAttrTypeAndValue($idx);
-            $this->_skipWs($idx);
-            if ($idx >= $this->_len || '+' != $this->_dn[$idx]) {
-                break;
-            }
-            ++$idx;
-            $this->_skipWs($idx);
-        }
-        $offset = $idx;
-        return $tvpairs;
-    }
+	/**
+	 * Parse 'name-component'.
+	 *
+	 * attributeTypeAndValue *("+" attributeTypeAndValue)
+	 *
+	 * @param int $offset
+	 *
+	 * @return array Array of [type, value] tuples
+	 */
+	private function _parseNameComponent(int &$offset): array
+	{
+		$idx = $offset;
+		$tvpairs = [];
+		while ($idx < $this->_len) {
+			$tvpairs[] = $this->_parseAttrTypeAndValue($idx);
+			$this->_skipWs($idx);
+			if ($idx >= $this->_len || '+' != $this->_dn[$idx]) {
+				break;
+			}
+			++$idx;
+			$this->_skipWs($idx);
+		}
+		$offset = $idx;
+		return $tvpairs;
+	}
 
-    /**
-     * Parse 'attributeTypeAndValue'.
-     *
-     * attributeType "=" attributeValue
-     *
-     * @param int $offset
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return array A tuple of [type, value]. Value may be either a string or
-     *               an Element, if it's encoded as hexstring.
-     */
-    private function _parseAttrTypeAndValue(int &$offset): array
-    {
-        $idx = $offset;
-        $type = $this->_parseAttrType($idx);
-        $this->_skipWs($idx);
-        if ($idx >= $this->_len || '=' != $this->_dn[$idx++]) {
-            throw new \UnexpectedValueException('Invalid type and value pair.');
-        }
-        $this->_skipWs($idx);
-        // hexstring
-        if ($idx < $this->_len && '#' == $this->_dn[$idx]) {
-            ++$idx;
-            $data = $this->_parseAttrHexValue($idx);
-            try {
-                $value = Element::fromDER($data);
-            } catch (DecodeException $e) {
-                throw new \UnexpectedValueException(
-                    'Invalid DER encoding from hexstring.', 0, $e);
-            }
-        } else {
-            $value = $this->_parseAttrStringValue($idx);
-        }
-        $offset = $idx;
-        return [$type, $value];
-    }
+	/**
+	 * Parse 'attributeTypeAndValue'.
+	 *
+	 * attributeType "=" attributeValue
+	 *
+	 * @param int $offset
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return array A tuple of [type, value]. Value may be either a string or
+	 *               an Element, if it's encoded as hexstring.
+	 */
+	private function _parseAttrTypeAndValue(int &$offset): array
+	{
+		$idx = $offset;
+		$type = $this->_parseAttrType($idx);
+		$this->_skipWs($idx);
+		if ($idx >= $this->_len || '=' != $this->_dn[$idx++]) {
+			throw new \UnexpectedValueException('Invalid type and value pair.');
+		}
+		$this->_skipWs($idx);
+		// hexstring
+		if ($idx < $this->_len && '#' == $this->_dn[$idx]) {
+			++$idx;
+			$data = $this->_parseAttrHexValue($idx);
+			try {
+				$value = Element::fromDER($data);
+			} catch (DecodeException $e) {
+				throw new \UnexpectedValueException(
+					'Invalid DER encoding from hexstring.', 0, $e);
+			}
+		} else {
+			$value = $this->_parseAttrStringValue($idx);
+		}
+		$offset = $idx;
+		return [$type, $value];
+	}
 
-    /**
-     * Parse 'attributeType'.
-     *
-     * (ALPHA 1*keychar) / oid
-     *
-     * @param int $offset
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return string
-     */
-    private function _parseAttrType(int &$offset): string
-    {
-        $idx = $offset;
-        // dotted OID
-        $type = $this->_regexMatch('/^(?:oid\.)?([0-9]+(?:\.[0-9]+)*)/i', $idx);
-        if (null === $type) {
-            // name
-            $type = $this->_regexMatch('/^[a-z][a-z0-9\-]*/i', $idx);
-            if (null === $type) {
-                throw new \UnexpectedValueException('Invalid attribute type.');
-            }
-        }
-        $offset = $idx;
-        return $type;
-    }
+	/**
+	 * Parse 'attributeType'.
+	 *
+	 * (ALPHA 1*keychar) / oid
+	 *
+	 * @param int $offset
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return string
+	 */
+	private function _parseAttrType(int &$offset): string
+	{
+		$idx = $offset;
+		// dotted OID
+		$type = $this->_regexMatch('/^(?:oid\.)?([0-9]+(?:\.[0-9]+)*)/i', $idx);
+		if (null === $type) {
+			// name
+			$type = $this->_regexMatch('/^[a-z][a-z0-9\-]*/i', $idx);
+			if (null === $type) {
+				throw new \UnexpectedValueException('Invalid attribute type.');
+			}
+		}
+		$offset = $idx;
+		return $type;
+	}
 
-    /**
-     * Parse 'attributeValue' of string type.
-     *
-     * @param int $offset
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return string
-     */
-    private function _parseAttrStringValue(int &$offset): string
-    {
-        $idx = $offset;
-        if ($idx >= $this->_len) {
-            return '';
-        }
-        if ('"' == $this->_dn[$idx]) { // quoted string
-            $val = $this->_parseQuotedAttrString($idx);
-        } else { // string
-            $val = $this->_parseAttrString($idx);
-        }
-        $offset = $idx;
-        return $val;
-    }
+	/**
+	 * Parse 'attributeValue' of string type.
+	 *
+	 * @param int $offset
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return string
+	 */
+	private function _parseAttrStringValue(int &$offset): string
+	{
+		$idx = $offset;
+		if ($idx >= $this->_len) {
+			return '';
+		}
+		if ('"' == $this->_dn[$idx]) { // quoted string
+			$val = $this->_parseQuotedAttrString($idx);
+		} else { // string
+			$val = $this->_parseAttrString($idx);
+		}
+		$offset = $idx;
+		return $val;
+	}
 
-    /**
-     * Parse plain 'attributeValue' string.
-     *
-     * @param int $offset
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return string
-     */
-    private function _parseAttrString(int &$offset): string
-    {
-        $idx = $offset;
-        $val = '';
-        $wsidx = null;
-        while ($idx < $this->_len) {
-            $c = $this->_dn[$idx];
-            // pair (escape sequence)
-            if ('\\' == $c) {
-                ++$idx;
-                $val .= $this->_parsePairAfterSlash($idx);
-                $wsidx = null;
-                continue;
-            }
-            if ('"' == $c) {
-                throw new \UnexpectedValueException('Unexpected quotation.');
-            }
-            if (false !== strpos(self::SPECIAL_CHARS, $c)) {
-                break;
-            }
-            // keep track of the first consecutive whitespace
-            if (' ' == $c) {
-                if (null === $wsidx) {
-                    $wsidx = $idx;
-                }
-            } else {
-                $wsidx = null;
-            }
-            // stringchar
-            $val .= $c;
-            ++$idx;
-        }
-        // if there was non-escaped whitespace in the end of the value
-        if (null !== $wsidx) {
-            $val = substr($val, 0, -($idx - $wsidx));
-        }
-        $offset = $idx;
-        return $val;
-    }
+	/**
+	 * Parse plain 'attributeValue' string.
+	 *
+	 * @param int $offset
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return string
+	 */
+	private function _parseAttrString(int &$offset): string
+	{
+		$idx = $offset;
+		$val = '';
+		$wsidx = null;
+		while ($idx < $this->_len) {
+			$c = $this->_dn[$idx];
+			// pair (escape sequence)
+			if ('\\' == $c) {
+				++$idx;
+				$val .= $this->_parsePairAfterSlash($idx);
+				$wsidx = null;
+				continue;
+			}
+			if ('"' == $c) {
+				throw new \UnexpectedValueException('Unexpected quotation.');
+			}
+			if (false !== strpos(self::SPECIAL_CHARS, $c)) {
+				break;
+			}
+			// keep track of the first consecutive whitespace
+			if (' ' == $c) {
+				if (null === $wsidx) {
+					$wsidx = $idx;
+				}
+			} else {
+				$wsidx = null;
+			}
+			// stringchar
+			$val .= $c;
+			++$idx;
+		}
+		// if there was non-escaped whitespace in the end of the value
+		if (null !== $wsidx) {
+			$val = substr($val, 0, -($idx - $wsidx));
+		}
+		$offset = $idx;
+		return $val;
+	}
 
-    /**
-     * Parse quoted 'attributeValue' string.
-     *
-     * @param int $offset Offset to starting quote
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return string
-     */
-    private function _parseQuotedAttrString(int &$offset): string
-    {
-        $idx = $offset + 1;
-        $val = '';
-        while ($idx < $this->_len) {
-            $c = $this->_dn[$idx];
-            if ('\\' == $c) { // pair
-                ++$idx;
-                $val .= $this->_parsePairAfterSlash($idx);
-                continue;
-            }
-            if ('"' == $c) {
-                ++$idx;
-                break;
-            }
-            $val .= $c;
-            ++$idx;
-        }
-        $offset = $idx;
-        return $val;
-    }
+	/**
+	 * Parse quoted 'attributeValue' string.
+	 *
+	 * @param int $offset Offset to starting quote
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return string
+	 */
+	private function _parseQuotedAttrString(int &$offset): string
+	{
+		$idx = $offset + 1;
+		$val = '';
+		while ($idx < $this->_len) {
+			$c = $this->_dn[$idx];
+			if ('\\' == $c) { // pair
+				++$idx;
+				$val .= $this->_parsePairAfterSlash($idx);
+				continue;
+			}
+			if ('"' == $c) {
+				++$idx;
+				break;
+			}
+			$val .= $c;
+			++$idx;
+		}
+		$offset = $idx;
+		return $val;
+	}
 
-    /**
-     * Parse 'attributeValue' of binary type.
-     *
-     * @param int $offset
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return string
-     */
-    private function _parseAttrHexValue(int &$offset): string
-    {
-        $idx = $offset;
-        $hexstr = $this->_regexMatch('/^(?:[0-9a-f]{2})+/i', $idx);
-        if (null === $hexstr) {
-            throw new \UnexpectedValueException('Invalid hexstring.');
-        }
-        $data = hex2bin($hexstr);
-        $offset = $idx;
-        return $data;
-    }
+	/**
+	 * Parse 'attributeValue' of binary type.
+	 *
+	 * @param int $offset
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return string
+	 */
+	private function _parseAttrHexValue(int &$offset): string
+	{
+		$idx = $offset;
+		$hexstr = $this->_regexMatch('/^(?:[0-9a-f]{2})+/i', $idx);
+		if (null === $hexstr) {
+			throw new \UnexpectedValueException('Invalid hexstring.');
+		}
+		$data = hex2bin($hexstr);
+		$offset = $idx;
+		return $data;
+	}
 
-    /**
-     * Parse 'pair' after leading slash.
-     *
-     * @param int $offset
-     *
-     * @throws \UnexpectedValueException
-     *
-     * @return string
-     */
-    private function _parsePairAfterSlash(int &$offset): string
-    {
-        $idx = $offset;
-        if ($idx >= $this->_len) {
-            throw new \UnexpectedValueException(
-                'Unexpected end of escape sequence.');
-        }
-        $c = $this->_dn[$idx++];
-        // special | \ | " | SPACE
-        if (false !== strpos(self::SPECIAL_CHARS . '\\" ', $c)) {
-            $val = $c;
-        } else { // hexpair
-            if ($idx >= $this->_len) {
-                throw new \UnexpectedValueException('Unexpected end of hexpair.');
-            }
-            $val = @hex2bin($c . $this->_dn[$idx++]);
-            if (false === $val) {
-                throw new \UnexpectedValueException('Invalid hexpair.');
-            }
-        }
-        $offset = $idx;
-        return $val;
-    }
+	/**
+	 * Parse 'pair' after leading slash.
+	 *
+	 * @param int $offset
+	 *
+	 * @throws \UnexpectedValueException
+	 *
+	 * @return string
+	 */
+	private function _parsePairAfterSlash(int &$offset): string
+	{
+		$idx = $offset;
+		if ($idx >= $this->_len) {
+			throw new \UnexpectedValueException(
+				'Unexpected end of escape sequence.');
+		}
+		$c = $this->_dn[$idx++];
+		// special | \ | " | SPACE
+		if (false !== strpos(self::SPECIAL_CHARS . '\\" ', $c)) {
+			$val = $c;
+		} else { // hexpair
+			if ($idx >= $this->_len) {
+				throw new \UnexpectedValueException('Unexpected end of hexpair.');
+			}
+			$val = @hex2bin($c . $this->_dn[$idx++]);
+			if (false === $val) {
+				throw new \UnexpectedValueException('Invalid hexpair.');
+			}
+		}
+		$offset = $idx;
+		return $val;
+	}
 
-    /**
-     * Match DN to pattern and extract the last capture group.
-     *
-     * Updates offset to fully matched pattern.
-     *
-     * @param string $pattern
-     * @param int    $offset
-     *
-     * @return null|string Null if pattern doesn't match
-     */
-    private function _regexMatch(string $pattern, int &$offset): ?string
-    {
-        $idx = $offset;
-        if (!preg_match($pattern, substr($this->_dn, $idx), $match)) {
-            return null;
-        }
-        $idx += strlen($match[0]);
-        $offset = $idx;
-        return end($match);
-    }
+	/**
+	 * Match DN to pattern and extract the last capture group.
+	 *
+	 * Updates offset to fully matched pattern.
+	 *
+	 * @param string $pattern
+	 * @param int    $offset
+	 *
+	 * @return null|string Null if pattern doesn't match
+	 */
+	private function _regexMatch(string $pattern, int &$offset): ?string
+	{
+		$idx = $offset;
+		if (!preg_match($pattern, substr($this->_dn, $idx), $match)) {
+			return null;
+		}
+		$idx += strlen($match[0]);
+		$offset = $idx;
+		return end($match);
+	}
 
-    /**
-     * Skip consecutive spaces.
-     *
-     * @param int $offset
-     */
-    private function _skipWs(int &$offset): void
-    {
-        $idx = $offset;
-        while ($idx < $this->_len) {
-            if (' ' != $this->_dn[$idx]) {
-                break;
-            }
-            ++$idx;
-        }
-        $offset = $idx;
-    }
+	/**
+	 * Skip consecutive spaces.
+	 *
+	 * @param int $offset
+	 */
+	private function _skipWs(int &$offset): void
+	{
+		$idx = $offset;
+		while ($idx < $this->_len) {
+			if (' ' != $this->_dn[$idx]) {
+				break;
+			}
+			++$idx;
+		}
+		$offset = $idx;
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Sop\X501\DN;
 
@@ -79,11 +79,11 @@ 
         $str = preg_replace('/^([ #])/u', '\\\\$1', $str);
         // implementation specific special characters
         $str = preg_replace_callback('/([\pC])/u',
-            function ($m) {
+            function($m) {
                 $octets = str_split(bin2hex($m[1]), 2);
                 return implode('',
                     array_map(
-                        function ($octet) {
+                        function($octet) {
                             return '\\' . strtoupper($octet);
                         }, $octets));
             }, $str);