Skip to content
This repository has been archived by the owner on Jul 30, 2022. It is now read-only.

Commit

Permalink
Version bump, fix for suppress and restore warnings, requires in skin…
Browse files Browse the repository at this point in the history
….json, deletion of duplicated prepend in foreground.js
  • Loading branch information
Hutchy68 committed Jan 18, 2020
1 parent b0fa210 commit f3ed00f
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 7 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
## Version 2.3.0
* [compatibility, deprecated] Fix for deprecated wfSuppressWarnings and wfRestoreWarnings with new Wikimedia\...
* [bug, code] Remove duplicated prepend call in foreground.js
* [b/c break] Version minor bump, MW >= 1.34 now required
* [enhancement] Use of requires MW >= 1.34 in skin.json

## Version 2.2.4
* [bug] PR #352 Breaks Compatibility with MW < 1.34 - Rollback for re-release as v2.3.0
* [compatibility] Final Release v2.x.x requires MW <= 1.33
Expand Down
6 changes: 3 additions & 3 deletions Foreground.skin.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ class foregroundTemplate extends BaseTemplate {
public function execute() {
global $wgUser;
global $wgForegroundFeatures;
wfSuppressWarnings();
Wikimedia\suppressWarnings();
$this->html('headelement');
switch ($wgForegroundFeatures['enableTabs']) {
case true:
Expand Down Expand Up @@ -278,7 +278,7 @@ public function execute() {
</html>

<?php
wfRestoreWarnings();
Wikimedia\suppressWarnings();
}
}
?>
?>
2 changes: 0 additions & 2 deletions assets/scripts/foreground.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@

jQuery(document).ready(function() {

// Log errors
Expand Down Expand Up @@ -76,5 +75,4 @@ if ( jQuery( '#ca-addsection' ).length ) {
jQuery(window).load(function() {
jQuery('li#ca-ve-edit a').prepend('<div class="drop-icon"><i class="fa fa-pencil fa-fw"></i></div>')
jQuery('li#ca-viewsource a').prepend('<div class="drop-icon"><i class="fa fa-book fa-fw"></i></div>')
jQuery('li#ca-edit a').prepend('<div class="drop-icon"><i class="fa fa-pencil-square-o fa-fw"></i></div>')
});
7 changes: 5 additions & 2 deletions skin.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "Foreground",
"version": "2.2.4",
"version": "2.3.0",
"author": [
"Garrick Van Buren",
"Jamie Thingelstad",
Expand All @@ -14,6 +14,9 @@
"ValidSkinNames": {
"foreground": "Foreground"
},
"requires": {
"MediaWiki": ">= 1.34.0"
},
"MessagesDirs": {
"SkinForeground": [
"i18n"
Expand Down Expand Up @@ -72,4 +75,4 @@
},
"config": {},
"manifest_version": 1
}
}

0 comments on commit f3ed00f

Please sign in to comment.