Project

General

Profile

Backport(バックポート) #2148

[api] /albums/@me/@self で取得したアルバムのサムネイルが未設定の場合 thumbnailUrl フィールドに間違った値が入る

Added by Youichi Kimura over 8 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2011-06-03
Due date:
% Done:

100%


Description

Overview (現象)

/albums/@me/@self でアルバムのコレクションを取得すると、サムネイルが未設定のアルバムに http://sns.example.com/cache/img/jpg/w180_h180/.jpg のようなURLが入ってしまう。

{
  "entry": [
    {
      "id": "1",
      "title": "test",
      "description": "aaaaa",
      "mediaItemCount": 2,
      "ownerId": "3",
      "thumbnailUrl": "http:\/\/sns.localhost\/sqlite\/cache\/img\/jpg\/w180_h180\/.jpg",
      "mediaType": "IMAGE" 
    }
  ],
  "startIndex":0,
  "totalResults":1,
  "itemsPerPage":10
}

Causes (原因)

lib/util/opJsonDbOpensocialService.class.php で以下のようにサムネイルの有無をチェックしている箇所があるが、サムネイルが未設定の場合 (album.file_id = null) は $object->getFile() メソッドの実行時に File オブジェクトのインスタンスが新規に生成されてしまうため条件式が偽にならずURLの生成が行われてしまう。

$result['thumbnailUrl'] = '';
if ($object->getFile())
{
  sfContext::getInstance()->getConfiguration()->loadHelpers(array('Asset', 'sfImage'));
  $result['thumbnailUrl'] = sf_image_path($object->getFile(), array('size' => '180x180'), true);
}

Way to fix (修正内容)

getFile() メソッドではなく getFileId() メソッドを使用してサムネイルの有無をチェックする。


Related issues

Related to opOpenSocialPlugin - Bug(バグ) #2145: [api] /albums/@me/@self で取得したアルバムのサムネイルが未設定の場合 thumbnailUrl フィールドに間違った値が入る Fixed(完了) 2011-06-03

Associated revisions

Revision 9ae56f5e (diff)
Added by Youichi Kimura over 8 years ago

fix weird thumbnailUrl of a album on RESTful API (fixes #2148, BP from #2145)

History

#1 Updated by Youichi Kimura over 8 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#2 Updated by Shogo Kawahara over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Fixed(完了)
  • % Done changed from 50 to 100

Also available in: Atom PDF