[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add a test for list() support

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395760 ) Change subject: Add a test for list() support .. Add a test for list() support Change-Id: I24ecc104265fafe6179d8f3e42e16c2a2edf

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add test for array addition with SQL_NUMKEY

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395759 ) Change subject: Add test for array addition with SQL_NUMKEY .. Add test for array addition with SQL_NUMKEY Change-Id: Id539689bff92e484

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Some debugging fixes

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395755 ) Change subject: Some debugging fixes .. Some debugging fixes For issues emitted due to a hook, include the line number of the hook subs

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix various false positives found when testing with MW

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395761 ) Change subject: Fix various false positives found when testing with MW .. Fix various false positives found when testing with MW

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Move plugin entry points to root directory

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395763 ) Change subject: Move plugin entry points to root directory .. Move plugin entry points to root directory The plugin entry point

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Update README

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395764 ) Change subject: Update README .. Update README Change-Id: I43bb5e9617e94e7e249e9b78702cde708043b202 --- M README M src/TaintednessBaseV

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Minor fixes discovered during testing

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395758 ) Change subject: Minor fixes discovered during testing .. Minor fixes discovered during testing Change-Id: I5e0117cc41e6504537d0

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support installing via composer.

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395765 ) Change subject: Support installing via composer. .. Support installing via composer. Add scripts for easy running of test. Change-Id:

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix some false positives discovered while testing with MW

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395762 ) Change subject: Fix some false positives discovered while testing with MW .. Fix some false positives discovered while testing with MW

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support installing via composer.

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395765 ) Change subject: Support installing via composer. .. Support installing via composer. Add scripts for easy running of test. Cha

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix some false positives discovered while testing with MW

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395762 ) Change subject: Fix some false positives discovered while testing with MW .. Fix some false positives discovered while testing w

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Minor fixes to the eval case

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395756 ) Change subject: Minor fixes to the eval case .. Minor fixes to the eval case Include a doc comment about how setFuncTaint() can

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add test for array addition with SQL_NUMKEY

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395759 ) Change subject: Add test for array addition with SQL_NUMKEY .. Add test for array addition with SQL_NUMKEY Change-Id: Id539689b

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Some debugging fixes

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395755 ) Change subject: Some debugging fixes .. Some debugging fixes For issues emitted due to a hook, include the line number of the h

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Move plugin entry points to root directory

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395763 ) Change subject: Move plugin entry points to root directory .. Move plugin entry points to root directory The plugin entry points are wh

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Update README

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395764 ) Change subject: Update README .. Update README Change-Id: I43bb5e9617e94e7e249e9b78702cde708043b202 --- M README M src/Taintedn

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Update composer.json

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395769 ) Change subject: Update composer.json .. Update composer.json Change-Id: Ie5a7c5169bb01057d112d1655d670a8abd6252b5 --- M compose

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Update composer.json

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395769 ) Change subject: Update composer.json .. Update composer.json Change-Id: Ie5a7c5169bb01057d112d1655d670a8abd6252b5 --- M composer.json 1

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: placeholder-message for html form should be ->text() not ->p...

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/395910 ) Change subject: placeholder-message for html form should be ->text() not ->parse() .. placeholder-message for html form should be ->tex

[MediaWiki-commits] [Gerrit] mediawiki...MassAction[master]: Internationalize hardcoded strings from View Task

2017-12-06 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395817 ) Change subject: Internationalize hardcoded strings from View Task .. Internationalize hardcoded strings from View Task Bug: T96

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make README prettier

2017-12-07 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/395925 ) Change subject: Make README prettier .. Make README prettier Make the README a bit nicer introduction to the functionality prov

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add GPL license headers

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/396442 ) Change subject: Add GPL license headers .. Add GPL license headers Change-Id: Ie9106c80b23f23a393912a12c9368078bcdb3557 --- M GenericSe

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add GPL license headers

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396442 ) Change subject: Add GPL license headers .. Add GPL license headers Change-Id: Ie9106c80b23f23a393912a12c9368078bcdb3557 --- M G

[MediaWiki-commits] [Gerrit] mediawiki/core[wmf/1.31.0-wmf.11]: Revert replacing textarea in Special:Undelete with OOUI

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/396465 ) Change subject: Revert replacing textarea in Special:Undelete with OOUI .. Revert replacing textarea in Special:Undelete with OOUI OOUI

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add .gitreview

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396456 ) Change subject: Add .gitreview .. Add .gitreview For those using git-review tool. Change-Id: I2a5907b2cf5ab9627a8676176c6de4f8

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: build: update composer.json

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396455 ) Change subject: build: update composer.json .. build: update composer.json * Change empty whitespaces to tabs. * A Phabricator

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Follow-up on Ie9106c80

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396543 ) Change subject: Follow-up on Ie9106c80 .. Follow-up on Ie9106c80 Fix composer syntax error on L26 Change-Id: I5ed486efcdc195aa

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix test that didn't pass lint

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396559 ) Change subject: Fix test that didn't pass lint .. Fix test that didn't pass lint Change-Id: I925a24ceac8b5105eefbe556fcafd9afc4

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix test that didn't pass lint

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/396559 ) Change subject: Fix test that didn't pass lint .. Fix test that didn't pass lint Change-Id: I925a24ceac8b5105eefbe556fcafd9afc42bd87b -

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Rename to "mediawiki/phan-security-plugin"

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396557 ) Change subject: Rename to "mediawiki/phan-security-plugin" .. Rename to "mediawiki/phan-security-plugin" The mediawiki vendor i

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Version should be php 7.0 (7.1 not supported due to dependency)

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/396561 ) Change subject: Version should be php 7.0 (7.1 not supported due to dependency) .. Version should be php 7.0 (7.1 not supported due to d

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Version should be php 7.0 (7.1 not supported due to dependency)

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396561 ) Change subject: Version should be php 7.0 (7.1 not supported due to dependency) .. Version should be php 7.0 (7.1 not supported

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add a note about how it can't validate certain types of SQL

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/396563 ) Change subject: Add a note about how it can't validate certain types of SQL .. Add a note about how it can't validate certain types of S

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add a note about how it can't validate certain types of SQL

2017-12-08 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396563 ) Change subject: Add a note about how it can't validate certain types of SQL .. Add a note about how it can't validate certain ty

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Rename plugin to mediawiki/phan-taint-check-plugin

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397550 ) Change subject: Rename plugin to mediawiki/phan-taint-check-plugin .. Rename plugin to mediawiki/phan-taint-check-plugin [This shall be

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Disable progress bar in composer test, as ugly on jenkins

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397558 ) Change subject: Disable progress bar in composer test, as ugly on jenkins .. Disable progress bar in composer test, as ugly on jenkins

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add a test script for people whose php bin is not 7

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397559 ) Change subject: Add a test script for people whose php bin is not 7 .. Add a test script for people whose php bin is not 7 This is most

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Replace SecurityCheckPlugin:: with self:: where possible

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397561 ) Change subject: Replace SecurityCheckPlugin:: with self:: where possible .. Replace SecurityCheckPlugin:: with self:: where possible Ch

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add .gitattributes file

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397572 ) Change subject: Add .gitattributes file .. Add .gitattributes file Following guidelines at: https://www.mediawiki.org/wiki/Manual:Devel

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix some typos

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/396564 ) Change subject: Fix some typos .. Fix some typos Change-Id: Ic69098f3ad46c47c538cf30cdba2c215e90b1511 --- M GenericSecurityChec

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Finish rename to mediawiki/phan-taint-check-plugin

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397574 ) Change subject: Finish rename to mediawiki/phan-taint-check-plugin .. Finish rename to mediawiki/phan-taint-check-plugin This also adds

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Html escaping functions shouldn't clear non-html taint

2017-12-11 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/397587 ) Change subject: Html escaping functions shouldn't clear non-html taint .. Html escaping functions shouldn't clear non-html taint Change

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support recognizing $wgHooks/$_GLOBALS['wgHooks']

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394090 ) Change subject: Support recognizing $wgHooks/$_GLOBALS['wgHooks'] .. Support recognizing $wgHooks/$_GLOBALS['wgHooks'] Still does not y

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support recognizing $wgHooks/$_GLOBALS['wgHooks']

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394090 ) Change subject: Support recognizing $wgHooks/$_GLOBALS['wgHooks'] .. Support recognizing $wgHooks/$_GLOBALS['wgHooks'] Still do

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make more clear error messages, distinguishing different iss...

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394091 ) Change subject: Make more clear error messages, distinguishing different issue types .. Make more clear error messages, disting

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make more clear error messages, distinguishing different iss...

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394091 ) Change subject: Make more clear error messages, distinguishing different issue types .. Make more clear error messages, distinguishing

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Handle dispatching of hooks on Hooks::run()

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394093 ) Change subject: Handle dispatching of hooks on Hooks::run() .. Handle dispatching of hooks on Hooks::run() Slight negative to this is s

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support loading hook information from extension.json

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394092 ) Change subject: Support loading hook information from extension.json .. Support loading hook information from extension.json As of yet,

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix some minor issues

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394094 ) Change subject: Fix some minor issues .. Fix some minor issues Change-Id: I603b43de23e752915b1d56605050d5fc8962de60 --- M src/M

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support loading hook information from extension.json

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394092 ) Change subject: Support loading hook information from extension.json .. Support loading hook information from extension.json As

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix some minor issues

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394094 ) Change subject: Fix some minor issues .. Fix some minor issues Change-Id: I603b43de23e752915b1d56605050d5fc8962de60 --- M src/MWVisitor

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Handle dispatching of hooks on Hooks::run()

2017-11-29 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394093 ) Change subject: Handle dispatching of hooks on Hooks::run() .. Handle dispatching of hooks on Hooks::run() Slight negative to t

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make unit tests for extension.json always work

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394867 ) Change subject: Make unit tests for extension.json always work .. Make unit tests for extension.json always work previously it

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add support for IDatabase::select style arguments

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394870 ) Change subject: Add support for IDatabase::select style arguments .. Add support for IDatabase::select style arguments Change-I

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make error messages from hooks be in extension instead of core

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394866 ) Change subject: Make error messages from hooks be in extension instead of core .. Make error messages from hooks be in extension

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Avoid duplication in output

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394865 ) Change subject: Avoid duplication in output .. Avoid duplication in output This also fixes the json test Change-Id: Iaaed1b098

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix bug where non-local variables are treated like local

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394869 ) Change subject: Fix bug where non-local variables are treated like local .. Fix bug where non-local variables are treated like l

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add support for IDatabase::select style arguments

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394870 ) Change subject: Add support for IDatabase::select style arguments .. Add support for IDatabase::select style arguments Change-Id: Id52f

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add ARRAY_OK flag for functions that are safe with arrays

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394868 ) Change subject: Add ARRAY_OK flag for functions that are safe with arrays .. Add ARRAY_OK flag for functions that are safe with arrays

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add ARRAY_OK flag for functions that are safe with arrays

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/394868 ) Change subject: Add ARRAY_OK flag for functions that are safe with arrays .. Add ARRAY_OK flag for functions that are safe with

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make unit tests for extension.json always work

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394867 ) Change subject: Make unit tests for extension.json always work .. Make unit tests for extension.json always work previously it did not

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Make error messages from hooks be in extension instead of core

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394866 ) Change subject: Make error messages from hooks be in extension instead of core .. Make error messages from hooks be in extension instead

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Avoid duplication in output

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394865 ) Change subject: Avoid duplication in output .. Avoid duplication in output This also fixes the json test Change-Id: Iaaed1b0984552d1c7

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix bug where non-local variables are treated like local

2017-12-03 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/394869 ) Change subject: Fix bug where non-local variables are treated like local .. Fix bug where non-local variables are treated like local Pr

[MediaWiki-commits] [Gerrit] mediawiki...InputBox[master]: Have inputbox langconvert certain attributes

2017-11-20 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392568 ) Change subject: Have inputbox langconvert certain attributes .. Have inputbox langconvert certain attributes This approximates the beha

[MediaWiki-commits] [Gerrit] operations/mediawiki-config[master]: Enable Timeless everywhere

2017-11-20 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392576 ) Change subject: Enable Timeless everywhere .. Enable Timeless everywhere It has been tested in a few places, and there are requests at

[MediaWiki-commits] [Gerrit] mediawiki...InputBox[wmf/1.31.0-wmf.8]: Have inputbox langconvert certain attributes

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392745 ) Change subject: Have inputbox langconvert certain attributes .. Have inputbox langconvert certain attributes This approximates the beha

[MediaWiki-commits] [Gerrit] mediawiki...InputBox[REL1_29]: Have inputbox langconvert certain attributes

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392749 ) Change subject: Have inputbox langconvert certain attributes .. Have inputbox langconvert certain attributes This approximates the beha

[MediaWiki-commits] [Gerrit] mediawiki...InputBox[REL1_28]: Have inputbox langconvert certain attributes

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392750 ) Change subject: Have inputbox langconvert certain attributes .. Have inputbox langconvert certain attributes This approximates the beha

[MediaWiki-commits] [Gerrit] mediawiki...InputBox[REL1_27]: Have inputbox langconvert certain attributes

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392751 ) Change subject: Have inputbox langconvert certain attributes .. Have inputbox langconvert certain attributes This approximates the beha

[MediaWiki-commits] [Gerrit] mediawiki...Contributors[master]: Fix message missing parameter

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392758 ) Change subject: Fix message missing parameter .. Fix message missing parameter Change-Id: Id844fe905de6e25e35c431a5715b4a91d496e123 ---

[MediaWiki-commits] [Gerrit] mediawiki...Contributors[REL1_30]: makeSpecialUrlSubpage() expects parameter to not be url encoded

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392759 ) Change subject: makeSpecialUrlSubpage() expects parameter to not be url encoded .. makeSpecialUrlSubpage() expects parameter to not be u

[MediaWiki-commits] [Gerrit] mediawiki...Contributors[REL1_30]: makeSpecialUrlSubpage() expects parameter to not be url encoded

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/392759 ) Change subject: makeSpecialUrlSubpage() expects parameter to not be url encoded .. makeSpecialUrlSubpage() expects parameter to

[MediaWiki-commits] [Gerrit] mediawiki...Contributors[REL1_27]: makeSpecialUrlSubpage() expects parameter to not be url encoded

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/392760 ) Change subject: makeSpecialUrlSubpage() expects parameter to not be url encoded .. makeSpecialUrlSubpage() expects parameter to not be u

[MediaWiki-commits] [Gerrit] mediawiki...Contributors[REL1_27]: makeSpecialUrlSubpage() expects parameter to not be url encoded

2017-11-21 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/392760 ) Change subject: makeSpecialUrlSubpage() expects parameter to not be url encoded .. makeSpecialUrlSubpage() expects parameter to

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix remaining tests (mostly phpcs)

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393537 ) Change subject: Fix remaining tests (mostly phpcs) .. Fix remaining tests (mostly phpcs) Change-Id: Ic802e332ab0fbabc3434b308a4

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add taint for db related function. Fix handling of subclasses

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393539 ) Change subject: Add taint for db related function. Fix handling of subclasses .. Add taint for db related function. Fix handling

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Mention phan version requirements

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393538 ) Change subject: Mention phan version requirements .. Mention phan version requirements Change-Id: Id582792be158cc04112b03939019

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Support ParserFunctions, and start of work for hooks in general

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393540 ) Change subject: Support ParserFunctions, and start of work for hooks in general .. Support ParserFunctions, and start of work fo

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Fix remaining tests (mostly phpcs)

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/393537 ) Change subject: Fix remaining tests (mostly phpcs) .. Fix remaining tests (mostly phpcs) Change-Id: Ic802e332ab0fbabc3434b308a41353eaaf

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Add support for parser tag hooks

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393541 ) Change subject: Add support for parser tag hooks .. Add support for parser tag hooks Change-Id: Ie979f53ea92d2c17a40cc348e28faa

[MediaWiki-commits] [Gerrit] mediawiki...SecurityCheckPlugin[master]: Keep track of hook registrations

2017-11-26 Thread Brian Wolff (Code Review)
Brian Wolff has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/393542 ) Change subject: Keep track of hook registrations .. Keep track of hook registrations As of yet, it doesn't do anything with the

<    4   5   6   7   8   9