[ 
https://issues.apache.org/jira/browse/CAMEL-13320?focusedWorklogId=211960&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-211960
 ]

ASF GitHub Bot logged work on CAMEL-13320:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 12/Mar/19 19:18
            Start Date: 12/Mar/19 19:18
    Worklog Time Spent: 10m 
      Work Description: claudio4j commented on pull request #2820: CAMEL-13320 
- DirectMessageConsumerHandler.java pollConsume method ca…
URL: https://github.com/apache/camel/pull/2820
 
 
   …lls deprecated Twitter.getDirectMessages(Paging p)
   
   https://issues.apache.org/jira/browse/CAMEL-13320
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

            Worklog Id:     (was: 211960)
            Time Spent: 10m
    Remaining Estimate: 0h

> DirectMessageConsumerHandler.java [4] pollConsume method calls 
> Twitter.getDirectMessages(getLastIdPaging())
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-13320
>                 URL: https://issues.apache.org/jira/browse/CAMEL-13320
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-twitter
>            Reporter: Claudio Miranda
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> DirectMessageConsumerHandler.java [1] pollConsume method calls 
> Twitter.getDirectMessages(getLastIdPaging()), the method is deprecated [2] 
> and filters out the messages sent by the account [3]. The master version 
> removed the getDirectMessages(Paging p) method and doesn't filter the 
> messages. The fix is to call getDirectMessages(int count) instead.
> 1. 
> https://github.com/apache/camel/blob/master/components/camel-twitter/src/main/java/org/apache/camel/component/twitter/directmessage/DirectMessageConsumerHandler.java#L39
> 2. 
> https://github.com/Twitter4J/Twitter4J/blob/4.0.7/twitter4j-core/src/main/java/twitter4j/api/DirectMessagesResources.java#L53
> 3. 
> https://github.com/Twitter4J/Twitter4J/blob/4.0.7/twitter4j-core/src/main/java/twitter4j/TwitterImpl.java#L422



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to