Differential D17684 Diff 42532 src/applications/config/custom/PhabricatorConfigRegexOptionType.php

Changeset View

Standalone View

				<?php

				class PhabricatorConfigRegexOptionType
				joshuaspenceUnsubmitted Not Done Inline Actions Shouldn't this be `final` by preference? I thought there was a linter rule for this. joshuaspence: Shouldn't this be `final` by preference? I thought there was a linter rule for this.
				extends PhabricatorConfigJSONOptionType {

				public function validateOption(PhabricatorConfigOption $option, $value) {
				foreach ($value as $pattern => $spec) {
				joshuaspenceUnsubmitted Not Done Inline Actions What if `$value` isn't an array? joshuaspence: What if `$value` isn't an array?
				$ok = preg_match($pattern, '');
				if ($ok === false) {
				throw new Exception(
				pht(
				'The following regex is malformed and cannot be used: %s',
				$pattern));
				}
				}
				}

				}