Spage has uploaded a new change for review.
https://gerrit.wikimedia.org/r/56078
Change subject: Fix errors reported by phpcs in includes/HTMLForm.php
......................................................................
Fix errors reported by phpcs in includes/HTMLForm.php
I set property "$typeMappings" to private (no clients outside this class
in our codebase).
I removed property $jsAdded, it is unused in this class and throughout
the codebase and seems a leftover from 2008.
Change-Id: Id5668141835e22e3e1b043109e50d73ccd6b1229
Special:Preferences still works with this change.
---
M includes/HTMLForm.php
1 file changed, 1 insertion(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/78/56078/1
diff --git a/includes/HTMLForm.php b/includes/HTMLForm.php
index 4a527bb..816ea16 100644
--- a/includes/HTMLForm.php
+++ b/includes/HTMLForm.php
@@ -95,7 +95,7 @@
class HTMLForm extends ContextSource {
// A mapping of 'type' inputs onto standard HTMLFormField subclasses
- static $typeMappings = array(
+ private static $typeMappings = array(
'api' => 'HTMLApiField',
'text' => 'HTMLTextField',
'textarea' => 'HTMLTextAreaField',
@@ -1996,7 +1996,6 @@
* Select dropdown field, with an additional "other" textbox.
*/
class HTMLSelectOrOtherField extends HTMLTextField {
- static $jsAdded = false;
function __construct( $params ) {
if ( !in_array( 'other', $params['options'], true ) ) {
--
To view, visit https://gerrit.wikimedia.org/r/56078
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id5668141835e22e3e1b043109e50d73ccd6b1229
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Spage <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
