sst89 / as3flickrlib

Automatically exported from code.google.com/p/as3flickrlib
0 stars 0 forks source link

Missing farmId attribute in PhotoSet Class #14

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hello,

Besides Photo ID, Server ID and Secret etc. flickr also uses a Farm ID in
the URL of their photos. This issue has been fixed in the Photo class but
not the PhotoSet Class...

Necessary changes:
---------------------------
PhotoSet.as ( around line 66, add this line )
---------------------------
/**
 * The server farm id of the photo set
 *
 * @langversion ActionScript 3.0
 * @playerversion Flash 9.0
 * @tiptext
 */     
public var farmId:int;  
        public var farmId:int;

---------------------------
MethodGroupHelper.as ( in method parsePhotoSetList )
---------------------------

photoSet.farmId = parseInt( s.@farm.toString() );

This worked for me. Please release a new binary with the new changes so
everyone can profit.

Thanks
Daniel

Original issue reported on code.google.com by daniel.gasienica on 15 Aug 2007 at 10:04

GoogleCodeExporter commented 8 years ago
what's the farm attribut for?

Original comment by nekoupo...@gmail.com on 22 Aug 2007 at 10:21

GoogleCodeExporter commented 8 years ago
Check out this document to find out how flickr's URL are specified:
http://www.flickr.com/services/api/misc.urls.html

Example
http://farm{farm-id}.static.flickr.com/{server-id}/{id}_{secret}.jpg

Farm ID is used to build a correct URL for a flickr photo.

Regards
Daniel

Original comment by daniel.gasienica on 26 Aug 2007 at 6:32

GoogleCodeExporter commented 8 years ago
I ran into this issue and had to fix it myself here are the fixes, 

public function get farm():int {
            return _farm;
}

public function set farm( value:int ):void {
            _farm = value;
}

in Photo.as

and the in the MethodGroupHelper.as, 

photo.farm = parseInt(p.@farm); inside the parsePhotoSet function and also in 
the parsePhotoList function if used later.

I can upload the classes if required,

Regards,

Nayan Savla
nayansavla@gmail.com

Original comment by nsa...@gmail.com on 6 Jan 2008 at 12:45

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago

Original comment by spjwebster@gmail.com on 27 Jan 2008 at 3:16

GoogleCodeExporter commented 8 years ago
This was already fixed by mikechambers before I accepted this ticket. Marking 
as FIXED.

Original comment by spjwebster@gmail.com on 2 Feb 2008 at 1:14

GoogleCodeExporter commented 8 years ago
How is this fixed? I downloaded the latest version flickr-.87 and farm is still
nowhere to be found.

Original comment by flux.bla...@gmail.com on 3 Oct 2008 at 8:34

GoogleCodeExporter commented 8 years ago
I had the same experience as #3. I had to download and alter the source to get 
the farmID. It is neither in the source or swc hosted here. The issue should be 
reopened.

Original comment by vipes...@vipesoft.com on 2 Sep 2011 at 9:38