Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Making query compatible with MW 1.26+ #36

Merged
merged 1 commit into from
Aug 5, 2015

Conversation

jdfoote
Copy link
Contributor

@jdfoote jdfoote commented Jul 17, 2015

According to https://www.mediawiki.org/wiki/API:Query#Backwards_compatibility_of_continue, in new versions of Mediawiki (including the version running currently on WP), the 'query-continue' parameter has been deprecated in favor of a simpler 'continue'. However, the old 'query-continue' can still be retrieved if the query parameter includes a 'rawcontinue' flag, which I added.

According to https://www.mediawiki.org/wiki/API:Query#Backwards_compatibility_of_continue, in new versions of Mediawiki (including the version running currently on WP), the 'query-continue' parameter has been deprecated in favor of a simpler 'continue'. However, the old 'query-continue' can still be retrieved if the query parameter includes a 'rawcontinue' flag, which I added.
halfak added a commit that referenced this pull request Aug 5, 2015
Making query compatible with MW 1.26+
@halfak halfak merged commit 846c36e into mediawiki-utilities:master Aug 5, 2015
@halfak
Copy link
Member

halfak commented Aug 5, 2015

Yikes. Thanks for the catch and sorry I missed the PR for so long. I'll get this released ASAP.

@jdfoote
Copy link
Contributor Author

jdfoote commented Aug 5, 2015

No worries. :)

FYI, I didn't look for other places in the code where this might be a
problem. I selfishly just fixed the one spot that I needed to get my
project working.

On Tue, Aug 4, 2015 at 8:54 PM, Aaron Halfaker notifications@github.com
wrote:

Yikes. Thanks for the catch and sorry I missed the PR for so long. I'll
get this released ASAP.


Reply to this email directly or view it on GitHub
#36 (comment)
.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants