GNUsocial/gnu-social
4
0
Fork 1
Code Issues Releases Wiki Activity

Get uploaded media_ids via File::getByID()

Browse Source 
Filters out failed ones...
This commit is contained in:
Mikael Nordfeldth
2016-01-14 22:05:03 +01:00
parent 16088d9439
commit 8df38df5cb
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
actions/apistatusesupdate.php
View File

@@ -173,10 +173,11 @@ class ApiStatusesUpdateAction extends ApiAuthAction
if (preg_match_all('/\d+/', $this->trimmed('media_ids'), $matches) !== false) { if (preg_match_all('/\d+/', $this->trimmed('media_ids'), $matches) !== false) {
foreach (array_unique($matches[0]) as $match) { foreach (array_unique($matches[0]) as $match) {
try { try {
$this->media_ids[$match] = true; // = File::getByID($match); $this->media_ids[$match] = File::getByID($match);
} catch (Exception $e) { } catch (EmptyIdException $e) {
// Either $match was 0 (EmptyIdException) or File was not found (NoResultException) // got a zero from the client, at least Twidere does this on occasion
// Do we abort and report to the client? } catch (NoResultException $e) {
// File ID was not found. Do we abort and report to the client?
} }
} }
} }
@@ -261,6 +262,7 @@ class ApiStatusesUpdateAction extends ApiAuthAction
foreach(array_keys($this->media_ids) as $media_id) { foreach(array_keys($this->media_ids) as $media_id) {
// FIXME: Validation on this... Worst case is that if someone sends bad media_ids then // FIXME: Validation on this... Worst case is that if someone sends bad media_ids then
// we'll fill the notice with non-working links, so no real harm, done, but let's fix. // we'll fill the notice with non-working links, so no real harm, done, but let's fix.
// The File objects are in the array, so we could get URLs from them directly.
$this->status .= ' ' . common_local_url('attachment', array('attachment' => $media_id)); $this->status .= ' ' . common_local_url('attachment', array('attachment' => $media_id));
} }