Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flickr: Add JSon Output #3

Open
callmeal opened this issue Jan 12, 2012 · 3 comments
Open

Flickr: Add JSon Output #3

callmeal opened this issue Jan 12, 2012 · 3 comments

Comments

@callmeal
Copy link
Collaborator

... add JSon Response

@boncey
Copy link
Owner

boncey commented Aug 20, 2013

Am taking this out of the milestone as no work has been done on it and I'm a bit unclear on the intent of it.
Was the intention to replace the XML response parsing with JSON instead?

@Watcher24
Copy link

Actually i would like to have such functionality.
The way how i understand it: Give access to json response (instead of marshalizing into java objects like Photo or Photoset).
To use flickr4java as kind of proxy.
This way one could prepare json on server side which then could be passed to client via AJAX.
Such usage would avoid to expose API_key and API_secret on client side (if for some reasons it is required to fetch final data on the client).

I've already implemented a first working solution on a forked branch:
2 methods on on com.flickr4java.flickr.photos.PhotosInterface#PhotosInterface

  • public StringBuilder getInfoAsJson(String photoId, String secret, boolean noJsonCallback) throws FlickrException
  • public StringBuilder getSizesAsJson(String photoId, boolean noJsonCallback) throws FlickrException

1 method on on com.flickr4java.flickr.Transport

  • public abstract JSONResponse getJson(String path, Map<String, Object> parameters, String apiKey, String sharedSecret,boolean noJsonCallback) throws FlickrException;

(& interface and impl. for JSONResponse)

=> on forked branch https://github.com/Watcher24/Flickr4Java.git.

Let me know whether something like that could be added to the master branch somewhen.
Or how i should implement it different ... whatever.
Would be nice to get a feedback.
thnx in advance,
Christoph

@Watcher24
Copy link

Hi

I've added a pull request (#112) which contains a "first solution" how JSON could be fetched.
Is there a chance to get that or something similar into master in near future?

cheers,
Christoph

@ghost ghost mentioned this issue Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants