Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

UGC-4233 - Fixed multitude of problems with UsingData #7

Merged
merged 15 commits into from
Jun 2, 2023
Merged

Conversation

Michal-Chatlas1221
Copy link
Collaborator

During upgrade to 1.39 we did a major refactor of the extension, and it broke in multiple ways:

After spending some time trying to pinpoint the root causes and fix those problems I've decided to instead revert the codebase to it's pre-upgrade state and fix issues starting there, which proved significantly easier.

the diff that handles this part is folowing: a3f162e

@mszabo-wikia
Copy link
Contributor

Would be good to get the CI working (asking someone with admin perms to enable it?), it seems the current parser tests pass on this branch:

This is MediaWiki version 1.41.0-alpha (cc1bb28).

Running parser tests from "extensions/UsingData/tests/parser/parserTests.txt"...
Running test #data parser function [legacy]... PASSED
Running test #using parser function [legacy]... PASSED
Running test #using parser function with parameter override [legacy]... PASSED
Running test #using parser function with template override [legacy]... PASSED
Running test #using parser tag [legacy]... PASSED
Running test transcluded #using parser tag [legacy]... PASSED

Passed 6 of 6 tests (100%)... ALL TESTS PASSED!

Maybe we can add some more test cases that would have failed before?

@Michal-Chatlas1221
Copy link
Collaborator Author

Would be good to get the CI working (asking someone with admin perms to enable it?), it seems the current parser tests pass on this branch:

This is MediaWiki version 1.41.0-alpha (cc1bb28).

Running parser tests from "extensions/UsingData/tests/parser/parserTests.txt"...
Running test #data parser function [legacy]... PASSED
Running test #using parser function [legacy]... PASSED
Running test #using parser function with parameter override [legacy]... PASSED
Running test #using parser function with template override [legacy]... PASSED
Running test #using parser tag [legacy]... PASSED
Running test transcluded #using parser tag [legacy]... PASSED

Passed 6 of 6 tests (100%)... ALL TESTS PASSED!

Maybe we can add some more test cases that would have failed before?

Apparently I'm the admin, so I'll see what I can do

@Michal-Chatlas1221 Michal-Chatlas1221 force-pushed the UGC-4233 branch 9 times, most recently from e1d0040 to 2310e30 Compare May 31, 2023 09:04
@tmusial99 tmusial99 merged commit ed37d88 into master Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants