Fix AttributeError by accessing meta field when return_type is dict

[lqhuang]

I meet totally the same error to #1843

Traceback (most recent call last):
  File "${MY_PY_ENV}/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "${MY_PY_ENV}/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "${MY_WORKSPACE}/twitter_fetcher/fetch_tweet_timeline.py", line 174, in <module>
    main()
  File "${MY_WORKSPACE}/twitter_fetcher/fetch_tweet_timeline.py", line 84, in main
    for i, resp in enumerate(
  File "${MY_PY_ENV}/lib/python3.10/site-packages/tweepy/pagination.py", line 100, in __next__
    self.previous_token = response.meta.get("previous_token")
AttributeError: 'dict' object has no attribute 'meta'

My solution is to add an isinstance condition, it's very simple but useful. Any improvement or suggest for this PR is welcome.

Regards.

[lqhuang]

@Harmon758 Hi, Harmon. Could you please take a look to this PR. Our packages have some functions rely on dict as return_type with tweepy as upstream. If you have other ideas to implement or another consideration about program design, I'm really glad to adapt and help. Thanks to your efforts!

My regards.

[Harmon758]

With AsyncPaginator implemented now, a similar fix needs to be applied to it.
I can add it later if it's not part of this PR.

[lqhuang]

Here is a marker, previous update messages just show rebased records from the latest master branch.

[Harmon758]

You've rebased and duplicated 66 commits from the master branch onto your branch.
Either make a merge commit or directly rebase your commits onto an updated version of the branch.

[lqhuang]

@Harmon758 You can review it now, I have added similar patch to AsyncPaginator and a note part for both pagination classes. But I'm not very sure whether the document content would satisfy you, please check it. Some style refactors are affected by black.

You've rebased and duplicated 66 commits from the master branch onto your branch. Either make a merge commit or directly rebase your commits onto an updated version of the branch.

Yeah, I choose the later one that I actually only do rebase my commits onto the latest master. But it seems my commits are too elder to on top of history. I could rewrite these commits with forced push if you cannot rebase into master safely.

Regards.

[lqhuang]

Squash rebase is an excellent approach to merge

[Harmon758]

Again, you've rebased commits from the master branch to your branch, not the other way around.
I'm not going to review over 60 of my own commits changing 80+ files.

[lqhuang]

@Harmon758 I have fixed that. Sorry for the inconvenience.

[Harmon758]

AsyncPaginator is meant to be used with AsyncClient, not Client.
AsyncClient uses aiohttp instead of requests and does not ever return requests.Response.

[lqhuang]

Sorry, my faults, I forgot to check async http client.

[lqhuang]

Really thank you for your patience. 👍