Differential D25186 Diff 666 src/applications/files/storage/PhabricatorFile.php

Changeset View

Standalone View

Show First 20 Lines • Show All 1,272 Lines • ▼ Show 20 Lines	if (!$this->isViewableImage()) {
return null;		return null;
}		}
return idx($this->metadata, self::METADATA_IMAGE_WIDTH);		return idx($this->metadata, self::METADATA_IMAGE_WIDTH);
}		}

public function getAltText() {		public function getAltText() {
$alt = $this->getCustomAltText();		$alt = $this->getCustomAltText();

if (strlen($alt)) {		if (phutil_nonempty_string($alt)) {
		valerio.bozzolanUnsubmitted Done Inline Actions ✅ I verified the above line The `$alt` variable is the alternative image text that assumes null as default, or a string. The function `phutil_nonempty_string()` will report alien types, and this is good. valerio.bozzolan: ✅ I verified the above line The `$alt` variable is the alternative image text that assumes…
return $alt;		return $alt;
}		}

return $this->getDefaultAltText();		return $this->getDefaultAltText();
}		}

public function getCustomAltText() {		public function getCustomAltText() {
return idx($this->metadata, self::METADATA_ALT_TEXT);		return idx($this->metadata, self::METADATA_ALT_TEXT);
▲ Show 20 Lines • Show All 475 Lines • Show Last 20 Lines