Changeset View
Changeset View
Standalone View
Standalone View
src/infrastructure/customfield/field/PhabricatorCustomField.php
Show First 20 Lines • Show All 883 Lines • ▼ Show 20 Lines | public function getNewValueForApplicationTransactions() { | ||||
return $this->getValueForStorage(); | return $this->getValueForStorage(); | ||||
} | } | ||||
/** | /** | ||||
* @task appxaction | * @task appxaction | ||||
*/ | */ | ||||
public function setValueFromApplicationTransactions($value) { | public function setValueFromApplicationTransactions($value) { | ||||
if ($this->proxy) { | if ($this->proxy) { | ||||
return $this->proxy->setValueFromApplicationTransactions($value); | return $this->proxy->setValueFromApplicationTransactions($value); | ||||
} | } else if ($value) { | ||||
return $this->setValueFromStorage($value); | return $this->setValueFromStorage($value); | ||||
} else { | |||||
return $value; | |||||
} | |||||
speck: I think this would be a large breaking change - I think null is a valid value to be set here. | |||||
} | } | ||||
/** | /** | ||||
* @task appxaction | * @task appxaction | ||||
*/ | */ | ||||
public function getNewValueFromApplicationTransactions( | public function getNewValueFromApplicationTransactions( | ||||
PhabricatorApplicationTransaction $xaction) { | PhabricatorApplicationTransaction $xaction) { | ||||
▲ Show 20 Lines • Show All 810 Lines • Show Last 20 Lines |
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0
I think this would be a large breaking change - I think null is a valid value to be set here. The issue is likely further up the stack.