Skip to content

Commit 5e5cef4

Browse files
committed
added image_url return value to logs/images/add
1 parent 2c1b3a1 commit 5e5cef4

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

okapi/services/logs/images/add/WebService.php

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ private static function _call(OkapiRequest $request)
200200

201201
try
202202
{
203-
$position = self::db_insert_image(
203+
list($position, $image_url) = self::db_insert_image(
204204
$request->consumer->key, $request->token->user_id,
205205
$log_internal_id, $image_uuid, $position, $caption, $is_spoiler, $file_ext
206206
);
@@ -216,7 +216,7 @@ private static function _call(OkapiRequest $request)
216216
throw $e;
217217
}
218218

219-
return array($image_uuid, $position);
219+
return array($image_uuid, $position, $image_url);
220220
}
221221

222222
/**
@@ -333,6 +333,7 @@ private static function db_insert_image(
333333
"NOW(), NOW(), '', 0, NOW(), '".Db::escape_string($user_id)."'";
334334
}
335335

336+
$image_url = Settings::get('IMAGES_URL').$image_uuid.$file_ext;
336337
Db::execute("
337338
insert into pictures (
338339
uuid, node, local, title, spoiler, url, object_type, object_id,
@@ -345,7 +346,7 @@ private static function db_insert_image(
345346
1,
346347
'".Db::escape_string($caption)."',
347348
'".($is_spoiler == 'true' ? 1 : 0)."',
348-
'".Db::escape_string(Settings::get('IMAGES_URL').$image_uuid.$file_ext)."',
349+
'".Db::escape_string($image_url)."',
349350
1,
350351
'".Db::escape_string($log_internal_id)."',
351352
0,
@@ -386,7 +387,7 @@ private static function db_insert_image(
386387
Db::execute('commit');
387388
Db::execute('unlock tables');
388389

389-
return $position;
390+
return array($position, $image_url);
390391
}
391392

392393

@@ -401,11 +402,12 @@ public static function call(OkapiRequest $request)
401402

402403
try
403404
{
404-
list($image_uuid, $position) = self::_call($request);
405+
list($image_uuid, $position, $image_url) = self::_call($request);
405406
$result = array(
406407
'success' => true,
407408
'message' => _("Image has been successfully saved."),
408409
'image_uuid' => $image_uuid,
410+
'image_url' => $image_url,
409411
'position' => $position
410412
);
411413
Okapi::gettext_domain_restore();
@@ -417,6 +419,7 @@ public static function call(OkapiRequest $request)
417419
'success' => false,
418420
'message' => $e->getMessage(),
419421
'image_uuid' => null,
422+
'image_url' => null,
420423
'position' => null
421424
);
422425
}

okapi/services/logs/images/add/docs.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,10 @@
7373
<b>image_uuid</b> - ID of the appended image, or <b>null</b> in
7474
case of an error,
7575
</li>
76+
<li>
77+
<b>image_url</b> - URL of the appended image, or <b>null</b> in
78+
case of an error,
79+
</li>
7680
<li>
7781
<b>position</b> - 0-based position of the appended image
7882
on the log entry's list of images; <b>null</b> in case of

0 commit comments

Comments
 (0)