According to the coding standards we even enforce with a custom PHPCS sniff.
It currently does not pick these mistakes up because of the curly brackets.
I'm not sure if this is worth an update of the PHPCS sniff. I wanted to
suggest this fix anyway.
Change-Id: I9041ea7a00baf7f55e0ff0e56879a89fb74bb479
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public static function getType() {
return static::TYPE;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public static function getTypeId() {
return static::TYPE_ID;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function getBlockId() {
return $this->blockId;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function setBlockId( $blockId ) {
$this->blockId = (int)$blockId;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function getValue() {
return $this->value;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public static function newFromRow( \stdClass $row ) {
return new static( $row->ir_ipb_id, $row->ir_value );
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function toRow() {
return [
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function equals( Restriction $other ) {
return $this->getHash() === $other->getHash();
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function getHash() {
return $this->getType() . '-' . $this->getValue();
class NamespaceRestriction extends AbstractRestriction {
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
const TYPE = 'ns';
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
const TYPE_ID = 2;
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function matches( \Title $title ) {
return $this->getValue() === $title->getNamespace();
class PageRestriction extends AbstractRestriction {
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
const TYPE = 'page';
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
const TYPE_ID = 1;
protected $title;
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function matches( \Title $title ) {
if ( !$this->getTitle() ) {
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public static function newFromRow( \stdClass $row ) {
$restriction = parent::newFromRow( $row );
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*
* Use whatever the relative field is as the standard HTML input.
*/
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
protected function getOOUIModules() {
return array_merge(
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function getInputOOUI( $value ) {
return new ExpiryInputWidget(
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function loadDataFromRequest( $request ) {
return $this->relativeField->loadDataFromRequest( $request );
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function getInputHTML( $value ) {
$this->selected = $value;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
public function getConfig( &$config ) {
$config['required'] = $this->required;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
protected function getClass() {
return NamespaceRestriction::class;
}
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
protected function getClass() {
return PageRestriction::class;
*/
class SpecialBlockTest extends SpecialPageTestBase {
/**
- * {@inheritdoc}
+ * @inheritDoc
*/
protected function newSpecialPage() {
return new SpecialBlock();