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

Fix Link to "Special:Browse" no longer shown for Vector #4856

Merged
merged 4 commits into from Oct 9, 2020
Merged

Fix Link to "Special:Browse" no longer shown for Vector #4856

merged 4 commits into from Oct 9, 2020

Conversation

StarHeartHunt
Copy link
Contributor

No description provided.

@StarHeartHunt StarHeartHunt changed the title Fix Link to "Special:Browse" no longer shown for Vector Fix #4831 Link to "Special:Browse" no longer shown for Vector Oct 8, 2020
@StarHeartHunt StarHeartHunt changed the title Fix #4831 Link to "Special:Browse" no longer shown for Vector Link to "Special:Browse" no longer shown for Vector Oct 8, 2020
@StarHeartHunt StarHeartHunt changed the title Link to "Special:Browse" no longer shown for Vector Fix Link to "Special:Browse" no longer shown for Vector Oct 8, 2020
@kghbln
Copy link
Member

kghbln commented Oct 9, 2020

@JeroenDeDauw Could you have a peep...

* @see https://www.mediawiki.org/wiki/Manual:Hooks/SidebarBeforeOutput
*
* @license GNU GPL v2+
* @since 3.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please drop the since tags, they are incorrect

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dropped in the latest commit

@JeroenDeDauw
Copy link
Member

@StarHeartHunt did you test this? Against which MW version(s)?

It is a bit scary to merge this new code without any automated tests for it.

@StarHeartHunt
Copy link
Contributor Author

@StarHeartHunt did you test this? Against which MW version(s)?

It is a bit scary to merge this new code without any automated tests for it.

I have tested this on 1.35 & 1.31.10.I checked the behavior and found nothing wrong.Since I couldn't get the PHPUnit tests to run locally, I gave up to write a test script.☹️

@JeroenDeDauw JeroenDeDauw merged commit 0ea1116 into SemanticMediaWiki:master Oct 9, 2020
@JeroenDeDauw
Copy link
Member

Uhhhhh this broke the CI

There was 1 failure:

  1. SMW\Tests\MediaWiki\HooksTest::testCheckOnMissingHandlers

Missing a SidebarBeforeOutput test!

Failed asserting that an array has the key 'SidebarBeforeOutput'.

/home/travis/build/SemanticMediaWiki/mw/extensions/SemanticMediaWiki/tests/phpunit/Unit/MediaWiki/HooksTest.php:257

/home/travis/build/SemanticMediaWiki/mw/maintenance/doMaintenance.php:94

(from https://travis-ci.org/github/SemanticMediaWiki/SemanticMediaWiki/jobs/735989762)

@StarHeartHunt you could write it without having the tests run locally. Just push to a branch here on GitHub and see if it works on Travis.

@kghbln
Copy link
Member

kghbln commented Apr 21, 2021

I guess this is somehow breaking for SMW 3.2.x and MW 1.31.x in conjunction with the Medik skin 3.x, i.e. I either get the MediaWiki tools in the drowdown menu OR the link to Special:Browse. However since MW 1.31.x is on the way out in two months we should no longer worry. To get at least the MediaWiki stuff working again one has to set:

$smwgBrowseFeatures = ( $smwgBrowseFeatures & ~SMW_BROWSE_TLINK );

On MW 1.35.x with SWM 3.2.x and Medik 3.2.x everything is fluff. See sandbox.s-mw.o. Same for MW 1.31.x with SWM 3.1.x and Medik 3.2.x

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.

None yet

3 participants