zendframework / ZendService_Twitter

PHP OOP wrapper for the Twitter API
BSD 3-Clause "New" or "Revised" License
22 stars 23 forks source link

Update the Direct Message text limit #40

Closed weierophinney closed 7 years ago

weierophinney commented 7 years ago

PRs #34 and #39 remove the direct message limit when creating a new direct message to a user. However, per https://dev.twitter.com/rest/reference/post/direct_messages/events/new, DM text can be no more than 10k characters in length.

This patch is based on #39, and will be rebased once that PR is merged. It modifies the limit, raising a separate exception if the number of characters is > 10000. Tests have been added for that method to verify the upper and lower bounds of message length.