Differential D17517 Diff 42141 src/applications/differential/customfield/DifferentialCustomField.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/differential/customfield/DifferentialCustomField.php
| Show First 20 Lines • Show All 64 Lines • ▼ Show 20 Lines | abstract class DifferentialCustomField | ||||
| public function getWarningsForDetailView() { | public function getWarningsForDetailView() { | ||||
| if ($this->getProxy()) { | if ($this->getProxy()) { | ||||
| return $this->getProxy()->getWarningsForDetailView(); | return $this->getProxy()->getWarningsForDetailView(); | ||||
| } | } | ||||
| return array(); | return array(); | ||||
| } | } | ||||
| protected function getActiveDiff() { | |||||
| $object = $this->getObject(); | |||||
| try { | |||||
| return $object->getActiveDiff(); | |||||
| } catch (Exception $ex) { | |||||
| return null; | |||||
chad: ? | |||||
Not Done Inline ActionsIt's possible that getActiveDiff() isn't loaded in some cases so we'd get a "thing not attached" exception. Since all we need it for is to pretty-up the UI I'm just blarpin' out. epriestley: It's possible that `getActiveDiff()` isn't loaded in some cases so we'd get a "thing not… | |||||
| } | |||||
| } | |||||
| public function getRequiredHandlePHIDsForRevisionHeaderWarnings() { | public function getRequiredHandlePHIDsForRevisionHeaderWarnings() { | ||||
| return array(); | return array(); | ||||
| } | } | ||||
| public function getWarningsForRevisionHeader(array $handles) { | public function getWarningsForRevisionHeader(array $handles) { | ||||
| return array(); | return array(); | ||||
| } | } | ||||
| ▲ Show 20 Lines • Show All 51 Lines • Show Last 20 Lines | |||||
?