huuanh1987 / facebook-java-api

Automatically exported from code.google.com/p/facebook-java-api
0 stars 0 forks source link

statusIncludeVerbs flag has no effect #149

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. with FacebookJsonRestClient call users_setStatus(String message)
2. facebook status will contain the word 'is'

What is the expected output? What do you see instead?

Walking through ExtensibleClient, users_setStatus(String message) calls
users_setStatus(message, false, false), passing false for clear and
statusIncludeVerbs. This should produce a status message with no verb ('is').

However, the way the parameters are constructed, they only add the flags if
their values are true: (1269-1274 in trunk)

if ( clear ) {
    params.add( newPair( "clear", "true" ) );
}
if ( statusIncludesVerb ) {
    params.add( newPair( "status_includes_verb", "true" ) );
}

I'm not sure if Facebook used to have defaults of false for these two
values, but it does not seem to use false anymore, which means that we
would need to convert the booleans to strings and pass them for both true
and false

Original issue reported on code.google.com by Chris.Sc...@gmail.com on 4 Dec 2008 at 3:16

GoogleCodeExporter commented 8 years ago
Never mind, I missed the subtlety of the 'status_includes_verbs' flag. Which 
means,
'my status has verbs, don't add then'

Original comment by Chris.Sc...@gmail.com on 8 Dec 2008 at 3:04

GoogleCodeExporter commented 8 years ago
cool. i guess i'll close this bug then. thanks for writing back.

Original comment by fern...@gmail.com on 8 Dec 2008 at 4:57