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

Add the correction of bug #3 to the release #32

Closed
Korriged opened this issue Jun 16, 2010 · 2 comments
Closed

Add the correction of bug #3 to the release #32

Korriged opened this issue Jun 16, 2010 · 2 comments

Comments

@Korriged
Copy link

I have tested adagli's correction for bug #3 (http://github.com/facebook/facebook-android-sdk/issues/closed#issue/3/comment/259899)...
... and it works fine :). Thank you for this patch :).

I hope this correction will be incorporated very soon in the SDK.


BTW, when do you plan to make a first delivery?

@soneff
Copy link

soneff commented Jun 16, 2010

The code provided by adagli that you refer to does not address the original problem described in issue #3, which was "API Error Code: 3...". This error was corrected by a server-side change and the issue was closed.

Why do you need to incorporate the change suggested by adagli? I'm not sure what problem his code fixes. You shouldn't need to use URLs that start with https://api.facebook.com/method/..., so I don't believe any changes to the SDK are required.

For example you can make a wall post as follows:

Bundle parameters = new Bundle();
parameters.putString("method", "stream.publish");
String attachments = URLEncoder.encode("{"name":"Name=Title","href":"http://www.google.fr/\",\"caption\":\"Caption\",\"description\":\"Description\",\"media\":[{\"type\":\"image\",\"src\":\"http://www.kratiroff.com/logo-facebook.jpg\",\"href\":\"http://developers.facebook.com/\"}],\"properties\":{\"another link":{"text":"Facebook homepage","href":"http://www.facebook.com\"}}}");
parameters.putString("attachment", attachments);
mAsyncRunner.request(parameters, new WallPostRequestListener());

You shouldn't need to use https://api.facebook.com/method/stream.publish or need any changes to the SDK.

@Korriged
Copy link
Author

If Facebook corrected their bug and allow using ?method= that'sfine to me.
But I cannot test it since I am in vacations with no access to my office nor Internet...
Thank you for your help :).

rotorgames pushed a commit to rotorgames/facebook-android-sdk that referenced this issue Aug 2, 2022
- Added files to create nuget
- This fixes issue facebook#32
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants