Workaround for Facebook data store API behavior regression, fixes saving
authorBrion Vibber <brion@pobox.com>
Mon, 12 Oct 2009 22:36:17 +0000 (22:36 +0000)
committerBrion Vibber <brion@pobox.com>
Fri, 16 Oct 2009 21:49:46 +0000 (14:49 -0700)
commitd3bbf3ad24c27522694bfaed437897273069a1e8
tree1766f0963aca7fcb71f65ff6ec9cad56f9887e45
parentd1e70b4e37149e09b2189957e2a869b4014a05c5
Workaround for Facebook data store API behavior regression, fixes saving
of empty notice prefix text in facebook settings.

Filed bug upstream at http://bugs.developers.facebook.com/show_bug.cgi?id=7110

Per documentation, saving a pref value of "" or "0" will delete the pref key:
http://wiki.developers.facebook.com/index.php/Data.setUserPreference

which used to do what we want... Now Facebook throws back an error
"Parameter value is required" when we do this. Workaround appends a
space to empty string or "0" at save time, then trims the string when
we load it.

The input string was already trimmed at pref save time, so this won't
alter any user-visible behavior.

Thanks to ^demon in #mediawiki for pointing out the behavior regression
after testing the identi.ca Facebook app!
actions/facebooksettings.php
lib/facebookutil.php