api update correctly checks max length of notices

This commit is contained in:
Evan Prodromou 2009-08-21 08:27:21 -04:00
parent d1cc159a04
commit cb2184ed44

View File

@ -242,13 +242,14 @@ class TwitapistatusesAction extends TwitterapiAction
$status_shortened = common_shorten_links($status); $status_shortened = common_shorten_links($status);
if (mb_strlen($status_shortened) > 140) { if (Notice::contentTooLong($status_shortened)) {
// XXX: Twitter truncates anything over 140, flags the status // XXX: Twitter truncates anything over 140, flags the status
// as "truncated." Sending this error may screw up some clients // as "truncated." Sending this error may screw up some clients
// that assume Twitter will truncate for them. Should we just // that assume Twitter will truncate for them. Should we just
// truncate too? -- Zach // truncate too? -- Zach
$this->clientError(_('That\'s too long. Max notice size is 140 chars.'), $this->clientError(sprintf(_('That\'s too long. Max notice size is %d chars.'),
Notice::maxContent()),
$code = 406, $apidata['content-type']); $code = 406, $apidata['content-type']);
return; return;
} }