Enhancement（機能追加・改善） #3203: activity/post.json の images パラメータを複数枚の画像アップロードに対応させる - OpenPNE 3 - OpenPNE Issue Tracking System

操作

リンクをコピー

Enhancement（機能追加・改善） #3203

完了

activity/post.json の images パラメータを複数枚の画像アップロードに対応させる

Youichi Kimura さんが約12年前に追加. 7年以上前に更新.

ステータス:

Won't fix（対応せず）

優先度:

Normal（通常）

担当者:

Youichi Kimura

対象バージョン:

OpenPNE 3.9.0-old

開始日:

2012-09-14

期日:

進捗率:

予定工数:

説明

Overview (現象)¶

#3064 で実装された activity/post.json の images パラメーターは、images と複数形の名前であるにもかかわらず複数枚の画像を指定することが出来ない。そのため、images パラメーターが複数枚の画像を受け付けるように修正を施す。

Spec (仕様)¶

activity/post.json のアクションにて、images が配列であっても受け付けるように修正する。
また、activity/search.json のレスポンスに追加されている image_url, image_large_url は、アクティビティに複数枚の画像が添付されていることを考慮していないため、これも配列で返すように変更を行う。

関連するチケット 2 (0件未完了 — 2件完了)

操作

リンクをコピー

Youichi Kimura さんが約12年前に更新

ステータス を New（新規） から Pending Review（レビュー待ち） に変更
進捗率 を 0 から 50 に変更

更新履歴 8064b916465136cfdd8056cb23a012e60021145d で適用されました。

操作

リンクをコピー

Youichi Kimura さんが約11年前に更新

説明を更新 (差分)
ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更

#3064 の note-7 で指摘されている

API の応答は固定で1枚だけ画像のURLを出力するような実装ですが、複数枚投稿できるのであれば、複数枚出力した方が良くないでしょうか。

1枚だけ出力するような仕様に意図的にするのであれば、例えば一番新しい画像のURLを出力するといった対応にしたほうがよいのではないかと思います。

について、複数枚アップロードの実装を行った当チケットで対応する。

操作

リンクをコピー

Youichi Kimura さんが約11年前に更新

トラッカー を Bug（バグ） から Enhancement（機能追加・改善） に変更
題名を activity/post.json の images パラメータが複数枚の画像アップロードに対応していない から activity/post.json の images パラメータを複数枚の画像アップロードに対応させる に変更
説明を更新 (差分)

操作

リンクをコピー

Youichi Kimura さんが約11年前に更新

複数枚の画像に対応した activity/*.json のレスポンスは下記のような形式で実装します。

{
  "status": "success",
  "data": [
    {
      "id": "1058",
      "member": { ... },
      "body": "hogehoge",
      "body_html": "hogehoge",
      "uri": null,
      "source": "API",
      "source_uri": null,
      "images": [
        {
          "small_uri": "http://localhost/cache/img/gif/w48_h48/ac_1_cb7fd0a95e1e24ac16a9f81c088cfe9863fd15b8_gif.gif",
          "full_uri": "http://localhost/cache/img/gif/w_h/ac_1_cb7fd0a95e1e24ac16a9f81c088cfe9863fd15b8_gif.gif" 
        },
        {
          "small_uri": "http://localhost/cache/img/jpg/w48_h48/ac_1_75c975ed014d8e77c8d31619038b2a06f2edf9f9_jpg.jpg",
          "full_uri": "http://localhost/cache/img/jpg/w_h/ac_1_75c975ed014d8e77c8d31619038b2a06f2edf9f9_jpg.jpg" 
        }
      ],
      "created_at": "Thu, 12 Sep 2013 07:59:51 +0000" 
    }
  ]
}

操作

リンクをコピー

Youichi Kimura さんが約11年前に更新

ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更

更新履歴 b1c33881a08af31718adabe54c97713c23d99084 で適用されました。

操作

リンクをコピー

Shinichi Urabe さんが約11年前に更新

ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更

ActivityData モデル内に、画像 URL の一覧を配列形式で返すメソッドを追加したほうがよいかもしれないです。
- apps/pc_frontend/modules/default/templates/_activityRecord.php でも似た処理がありますので、共通化ができるかと思います。(メソッドに対して absolute を引数で指定できるようにしておいたほうがよいかもしれません) 共通化の対応する場合、別チケットになるかと思います。
full_uri, small_uri について、 full_uri という言葉から、full は uri に対する形容詞のように見えてしまうので絶対 URL をイメージしてしまいます。small_uri も絶対 URL ですので、 full_uri の名称が適切ではないように思います。名称は変えた方がよいかもしれません。(案を書いていますが、参考まで)

案¶

$array = array(
  'image_urls' => array(
    array(
      'full_size' => 'http://xxxxx/full.jpg',
      'small_size' => 'http://xxxxx/small.jpg',
    ),  
    array(
      'full_size' => 'http://xxxxx/full2.jpg',
      'small_size' => 'http://xxxxx/small2.jpg',
    )   
  )
);

操作

リンクをコピー

Youichi Kimura さんが約11年前に更新

参考メモ

apps/pc_frontend/modules/default/templates/_activityRecord.php の記述:

<?php if ($images[$i]->getFileId()): ?>
<?php echo op_image_tag_sf_image($images[$i]->getFile(), array('size' => '48x48')) ?>
<?php else: ?>
<?php echo op_image_tag($images[$i]->getUri(), array('width' => 48, 'height' => 48)) ?>
<?php endif; ?>

apps/api/lib/helper/opJsonApiHelper.php の記述:

    if (!is_null($activityImage->file_id))
    {
      $images[] = array(
        'small_uri' => sf_image_path($activityImage->getFile(), array('size' => '48x48'), true),
        'full_uri' => sf_image_path($activityImage->getFile(), array(), true),
      )
    }
    else
    {
      $images[] = array(
        'small_uri' => $activityImage->uri,
        'full_uri' => $activityImage->uri,
      )
    }

操作

リンクをコピー

Youichi Kimura さんが約11年前に更新

こんな感じのヘルパー関数を opActivityHelper.php に実装してみる

function op_activity_image_uri($activityImage, $options = array(), $absolute = false)
{
  if ($activityImage->relatedExists('File'))
  {
    return sf_image_path($activityImage->File, $options, $absolute);
  }
  else
  {
    return $activityImage->uri;
  }
}

function op_activity_image_tag($activityImage, $options = array())
{
  if ($activityImage->relatedExists('File'))
  {
    return op_image_tag_sf_image($activityImage->File, $options);
  }
  else
  {
    return op_image_tag($activityImage->uri, $options);
  }
}