@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Fix specification of `ttl.relative` via LFS
Summary: Fixes T12828. This API was tightened up D17616, but I missed this callsite.
Test Plan: I've just got a good feeling in my bones.
Reviewers: chad
Reviewed By: chad
Maniphest Tasks: T12828
Differential Revision: https://secure.phabricator.com/D18119
+8
-2
+8
-2
src/applications/files/uploadsource/PhabricatorFileUploadSource.php
+8
-2
src/applications/files/uploadsource/PhabricatorFileUploadSource.php
···
208
208
}
209
209
210
210
private function getNewFileParameters() {
211
-
return array(
211
+
$parameters = array(
212
212
'name' => $this->getName(),
213
-
'ttl.relative' => $this->getRelativeTTL(),
214
213
'viewPolicy' => $this->getViewPolicy(),
215
214
);
215
+
216
+
$ttl = $this->getRelativeTTL();
217
+
if ($ttl !== null) {
218
+
$parameters['ttl.relative'] = $ttl;
219
+
}
220
+
221
+
return $parameters;
216
222
}
217
223
218
224
private function getChunkEngine() {